Seite 1 von 1
Todoli: Simple Django To-Do Listen App
Verfasst: Dienstag 23. Juli 2013, 13:43
von orschiro
Hallo zusammen,
als kleine Uebung meiner sehr bescheidenen Python-Kenntnisse habe ich eine ganz simple To-Do Listen App in Python / Django geschrieben.
Link: Todoli
Bisher ist die App noch nicht online zu testen. Daher kann ich euch leider nur den Code anbieten. Koennt ihn aber gerne runterladen und lokal testen.
Ich wuerde mich freuen, wenn der ein oder andere Django-Juenger mal drueber sehen kann und seinen Kommentar geben kann.
Denn ich weiss, dass vieles verbessert werden kann.
Danke!
Re: Todoli: Simple Django To-Do Listen App
Verfasst: Mittwoch 24. Juli 2013, 15:49
von apollo13
So auf die Schnelle:
* Keine Datenbanken, pyc, oder __pycache__ Ordner einchecken
* Tests schreiben
* PEP-008 anschauen
* Sachen wie
https://github.com/orschiro/todoli/blob ... ews.py#L32 lieber über einen Update query erledigen oder .save(update_fields=['status'])
Re: Todoli: Simple Django To-Do Listen App
Verfasst: Donnerstag 25. Juli 2013, 06:17
von orschiro
Danke schon mal!
* Keine Datenbanken, pyc, oder __pycache__ Ordner einchecken
Erledigt
* Tests schreiben
Wird gemacht
PEP-008 anschauen
Ebenso
Gut. Was haeltst du denn generell von meinen todo_* views? Ich habe das Gefuehl, das sei nicht elegant geloest, und koennte vielleicht mit einem Generic Class Based View gemacht werden. Bloss habe ich da nicht so richtig raus, wie ich die anwende.
Alle, die mal testen wollen:
http://www.todoli.orzanna.de/todolists/
Re: Todoli: Simple Django To-Do Listen App
Verfasst: Donnerstag 25. Juli 2013, 08:30
von jens
Re: Todoli: Simple Django To-Do Listen App
Verfasst: Sonntag 28. Juli 2013, 09:03
von apollo13
orschiro hat geschrieben:
Gut. Was haeltst du denn generell von meinen todo_* views? Ich habe das Gefuehl, das sei nicht elegant geloest, und koennte vielleicht mit einem Generic Class Based View gemacht werden. Bloss habe ich da nicht so richtig raus, wie ich die anwende.
Tendentiell würde sich ein DetailView (
https://docs.djangoproject.com/en/dev/r ... detailview) anbieten, bei dem du POST überschreibst, übrigens liefert
int(request.POST['todo_pk']) einen ServerError sobald
todo_pk kein Integer ist. Ich würde auch undo/done zu einem View zusammenfassen ala:
Code: Alles auswählen
def todo_change(request, action, slug):
action = True if action=='done' else False
...
Re: Todoli: Simple Django To-Do Listen App
Verfasst: Sonntag 28. Juli 2013, 20:46
von BlackJack
Das ``if``/``else`` ist unnötig, mit ``action = action == 'done'`` hätte man das selbe Ergebnis. Wobei hier unschön ist, dass der Name an derart unterschiedliche Typen gebunden wird.
Re: Todoli: Simple Django To-Do Listen App
Verfasst: Sonntag 28. Juli 2013, 21:38
von apollo13
BlackJack hat geschrieben:Das ``if``/``else`` ist unnötig, mit ``action = action == 'done'`` hätte man das selbe Ergebnis. Wobei hier unschön ist, dass der Name an derart unterschiedliche Typen gebunden wird.
Ups, als ich das schrieb machte es noch mehr Sinn
