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!
Todoli: Simple Django To-Do Listen App
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'])
* 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'])
Danke schon mal!
Alle, die mal testen wollen: http://www.todoli.orzanna.de/todolists/
Erledigt* Keine Datenbanken, pyc, oder __pycache__ Ordner einchecken
Wird gemacht* Tests schreiben
EbensoPEP-008 anschauen
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.Sachen wie https://github.com/orschiro/todoli/blob ... ews.py#L32 lieber über einen Update query erledigen oder .save(update_fields=['status'])
Alle, die mal testen wollen: http://www.todoli.orzanna.de/todolists/
- jens
- Python-Forum Veteran
- Beiträge: 8502
- Registriert: Dienstag 10. August 2004, 09:40
- Wohnort: duisburg
- Kontaktdaten:
könntest für Kommentare das comment contrib nehmen: https://docs.djangoproject.com/en/1.5/r ... /comments/
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: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.
Code: Alles auswählen
def todo_change(request, action, slug):
action = True if action=='done' else False
...
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 SinnBlackJack 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.