[tutorial] Kartka z kalendarza PHP+Smarty Opcje

[JoShiMa]

Robiłam ostatnio w jednym z moich skryptów robiłam kalendarium w postaci kartki z kalendarza z zaznaczonymi wydarzeniami. Postanowiłam zrobić z tego tutorial. Chciałabym, żebyście napisali co myślicie na jego temat i czy coś należałoby poprawić (zagmatwane, niejasne sformułowania, skróty myślowe, mało maślane itp).

Ten post edytował JoShiMa 2.07.2008, 10:21:53

[nospor]

Tresci oceniac nie będę - brak czasu, ale wytkne ci pare bledow (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

- _GET['miesiac'] - hmmm, kod nie ma prawa dzialac
- pozatym skoro bierzesz cos z url i to cos ma byc liczba, to rob jeszcze rzutowanie (int), szczegolnie gdy to idzie do bazy
- "tekst" - uczmy ludzi pisac dobrze 'tekst' - jak nie ma zmiennych to nie uzywajmy ""
- "SELECT ID,data
FROM tabela_wydarzenia
WHERE data BETWEEN '".$od."' ...
skoro juz uzywam "" to korzystajmy z jego dobrodziejstw:
"SELECT ID,data
FROM tabela_wydarzenia
WHERE data BETWEEN '$od'....

- $szablon->assign('kalendarz', $dane); - $dane to dosc spora tablica. warto wiec przekazac ją przez referencje: $szablon->assign_by_ref('kalendarz', $dane);

- aplikacja na stronie co dalas nie dziala. Gdy klikam na kolejny miesiac/rok to dostaje 404

[JoShiMa]

- _GET['miesiac'] - hmmm, kod nie ma prawa dzialac

Upssss. Masz rację (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

- pozatym skoro bierzesz cos z url i to cos ma byc liczba, to rob jeszcze rzutowanie (int), szczegolnie gdy to idzie do bazy

Poza elegancją jakie to ma znaczenie?

- "tekst" - uczmy ludzi pisac dobrze 'tekst' - jak nie ma zmiennych to nie uzywajmy ""

Rozumiem, że to tez elegancja?

- "SELECT ID,data
FROM tabela_wydarzenia
WHERE data BETWEEN '".$od."' ...
skoro juz uzywam "" to korzystajmy z jego dobrodziejstw:
"SELECT ID,data
FROM tabela_wydarzenia
WHERE data BETWEEN '$od'....

Słusznie. Przerabiałam ten kod z bardziej skomplikowanego, gdzie tak się nie dało.

- $szablon->assign('kalendarz', $dane); - $dane to dosc spora tablica. warto wiec przekazac ją przez referencje: $szablon->assign_by_ref('kalendarz', $dane);

Zainteresuję sie tym

- aplikacja na stronie co dalas nie dziala. Gdy klikam na kolejny miesiac/rok to dostaje 404

Dziwne, bo mi działa (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) Napiszesz mi jaki link dostajesz?