Instantretro, app para subir y modificar fotos. Webapp en appengine

[cvander]

Estaba charlando de InstantRetro, un proyecto de Arturo Jamaica con @razpeitia vía chat y me pareció que sus observaciones al revisar el código pueden ayudar a varios que estamos aprendiendo python observando el código de otros.

https://github.com/ajamaica/InstantRetro

Esta app sirve para subir y modificar fotografías con efectos retro (lo que hizo famoso a Instagram) y tuvo muy buena acogida en su app para Chrome.

La aplicación en acción está aquí:
http://www.instantretro.com/

[razpeitia]

Una recomendación es leer y seguir la guía de estilo.

PD: Hay otras cosas que me gustaría comentar pero no puedo hacerlo en publico.

[ajamaica]

Yo personalmente (siendo mi código) arreglaría lo siguiente :

1. Guardar Img en Google Storage no en la DB?
2. Guardaria tamaños de las imagenes en pixeles
3. Usaria otra clase de llave en lugar de usar los hashes
4. Usaria otra clase de llave en lugar de usar los hashes
5. Separaría los datos de la WSGIApplication a un Url.py
6. Agregaría mas estilo
7. Hay unos problemas con los indexes (por no haberlos creado desde un inicio)

[dual3nigma]

tendrias que tener más cuidado con tu "sensitive data" como facebook_app_id, facebook_app_secret.

Cosas como estas O.o???:

Código Python:

query = db.GqlQuery("SELECT * FROM Foto ORDER BY date DESC LIMIT 0, 50")
    fotos = list()
    c=0
    for q in query:
        if q.shared:
            fotos.append(q)
            c=c+1
        if c==5:
            break
 

mejor asi

Código Python:

fotos= db.GqlQuery("SELECT * FROM Foto WHERE shared = TRUE ORDER BY date DESC LIMIT 5 ")
 

o asi

Código Python:

consulta = Fotos.all()
consulta.filter("shared =", True)
consulta.order("-date")
fotos = consulta.fetch(5)
 

por otro lado, que tal si defines una variable global para evitar este tipo de cosas, requerda que asi es más fácil que los que descarguen tu código puedan hacer pruebas

Código Python:

urllib.quote('http://www.instantretro.com/view/'+str(foto.key()))
 

tambien en esta parte:

Código Javascript:

var woo_settings = {idle_timeout:'300000', domain:'instantretro.com'};
 

[ajamaica]

Código Python:

fotos= db.GqlQuery("SELECT * FROM Foto WHERE shared = TRUE ORDER BY date DESC LIMIT 5 ")
 

Esto esta es la mejor solución (no es que no sepa GQL) el problema es que los indexes no estaban así desde el principio y al tratar de re escribirlos google sigue "Building". El campo de shared se agrego muy a la fuerza despues. Parece que google no cambiará proximamente mis indexes y no me permite hacer este tipo de consultas esta el momento. Recuerda que una base de datos como Big Table no puede ordenar o sortear sin tener indexes validos. Es una práctica horrible pero fue mi solución temporal

[dual3nigma]

fair enough, entonces por lo menos podrías ponerlo de esta manera:

Código Python:

query = db.GqlQuery("SELECT * FROM Foto ORDER BY date DESC LIMIT 0, 50")
fotos = [q for q in query if q.shared][:5]
 

o asi:

Código Python:

query = db.GqlQuery("SELECT * FROM Foto ORDER BY date DESC LIMIT 0, 50")
fotos = filter(lambda q: q.shared, query)[:5]