Autor Zpráva
Schooby
Profil *
Zdravim, napsal jsem takovej jednoduchej script pro takovou kartotéku přes objekty. Zajímalo by mě, jaký dělám chyby. Celý script je na adrese http://fousac.cz/cd.htm
Schooby
Profil
Jo a ještě jsem tam zapoměl napsat ty tabulky:
Typ: ID, Nazev
Prostor: ID, Nazev
Medium: ID, Typ, Nazev, Pozice
Hugo
Profil
Nemám čas to procházet, ale doporučil bych ti nemíchat PHP a HTML. Lepší je využít některý šablonovací systém. Protože měnit třídu jen proto, že jsi změnil uspořádání webu je blbost.
Schooby
Profil
Právě proto jsem dělal třídu Tlumočník, která pracuje jako mezičlen mezi ostatními třídami a uživatelem...
joejoe
Profil *
http://cz2.php.net/manual/en/function.highlight-string.php
joejoe
Profil *
nekontrolujes si vysledky operaci
doporucuju konstrukuci:
if (!mysql_query($query)) { throw new Exception(mysql_error()); }
medden
Profil *
1) Prosím ťa, daj to do nejakého <pre> aby sa to dalo čítať. Najlepšie ešte použiť aj ten highlight_string(), lepšie sa potom čítajú kódy.
2) getNazev, getProstor, ... si snáď robíš srandu? Keď už to chceš mať česky, tak to maj celé česky, a nie tento podivný mix
3) Treba dávať výstižné mená funkcií a tiež tried. Ak by si chcel niečo seriózne vytvoriť (čo predpokladám, že áno) tak veci ako Zhulenec nikdy nepoužiješ. Väčšina ľudí nemá kryštálovú guľu aby vedeli čo tá trieda zhruba bude robiť. Museli by sa prácne pozrieť do kódu.
4) Nepochopil som ani čo si vlastne chcel vytvoriť. Skús tam dať nejaký pokec a vysvetliť to. Mimochodom, najlepšie sa veci naučíš praxou, takže skús si napísať nejaký zmysluplný projekt, tam sa toho naučíš najviac, a nie s prepáčením takéto hlúposti. A navyše ak to bude zaujimavé, tak to môžeš zverejniť (napr. ako nejakú knižnicu apod.)

Skús si ešte pozrieť niečo o návrhových vzoroch, dosť sa ti to zíde. Ostatné príde časom :)
Držím palce.
ronnie
Profil
Nejdříve by nebylo špatné zaměřit se na štábní kulturu...za pár dní bude o tom článek na Intervalu. Do té doby si prostuduj http://framework.zend.com/manual/en/coding-standard.html
Toto téma je uzamčeno. Odpověď nelze zaslat.