Veľky projekt

Snacker · Napísal **Snacker**: 22.08.2009 11:09

to ti urcite nebude stacit, vacsinu vstupov budes musiet osetrit na mieru

Ďuri · Napísal **Ďuri**: 22.08.2009 11:15

Flety píše:

predpokladám, že na ošetrenie by mala stačiť stenleyho funkcia

To mozes na ten velky projekt rovno zabudnut. Vsetko, co robi ta funkcia, je v podstate htmlspecialchars. Ak robis s databazou, ovela dolezitejsie bude osetrenie backslashov a apostrofov (tie apostrofy sa daju vyriesit tiez pomocou htmlspecialchars, ale treba pridat 2.parameter).

stenley · Napísal **stenley**: 22.08.2009 11:41

ta funkcia vznikla preto, aby nebolo treba osetrovat kazdu jednu premennu zvlast, teda aby sa tato cinnost zjednodusila a zautomatizovala... ak si vsimnes, uz v jej nadpise je uvedene, ze sa jedna o zakladne osetrenie a teda je potrebne, aby si si ju prisposobil, tj pridal dalsie osetrenia vstupov, ktore povazujes za nevyhnutne v tvojom projekte...

Snacker · Napísal **Snacker**: 22.08.2009 11:56

Na stenleyho funkciu sa nespoliehaj (ani na ziadnu inu), ktoru najdes na nete a je ku nej napisane ze osetrenie vsupov. Sam musis pochopit ake ti hrozia rizika (SQL injction, XSS, nulovy bajt atd...) a na zaklade toho konat.
Ak mas napr. 3x radio kde mas value napr. S, M, L tak by si mal spravit funkciu(metodu v class) na oserenie, ktora takyto vstup ocakava a vracia TRUE or FALSE. Teda robit veci na mieru a nedoverovat uzivatelovi

stenley · Napísal **stenley**: 22.08.2009 12:29

zas az tak namieru by som to nerobil, nie je totiz efektivne, aby si na kazdy vstup mal zvlast metodu/funkciu (viem, ze si to tak asi nemyslel, ale nedalo mi)... staci jedna, ale poriadne prepracovana, pre zaciatok by jej stacilo predavat hodnotu vstupu, pole dovolenych hodnot (ak su konkretne stanovene) pripadne regularny vyraz , triedu vstupu (meno,cislo,email,sql hodnota...), osetrenie nad ramec zakladneho osetrenia a pod...

GoodWill · Napísal **GoodWill**: 22.08.2009 18:54

Podla mna uplne staci nieco taketo v kombinacii s prepared statement:
http://www.phpclasses.org/browse/file/9402.html

Tominator · Napísal **Tominator**: 22.08.2009 21:05

Ok takže skúsim sa vyjadriť hoci nerobím vo veľkých firmách s väčšími projektami som sa stretol.

Čítal som, že každý kód, aby bol dobre čítateľný a prehľadný by mal byť na jednu obrazovku. S časti s týmto súhlasím teda - tvoj kód má byť jednoduchý, prehľadný, ľahko udržiavateľný, bezpečný.

Ako to dostiahnuť. Tuším GoodWill tu na druhej strane vravel ako dokumentovať. Súhlasím iba s časti. V radách sa trošičku odvolávam na knihu Java 6, od tvorcov Java. V podstate sú to rady ktoré nájdeš vo všetkých väčších projektoch, a ja som na ne (bohužiaľ) prišiel sám

Premenné a poznámky
Používať vhodné premenné. Premenné typu: $u, $g, $a nepripadajú do úvahy. Názov premennej má byť stručný ale výstižný. Pri použití dvoch slov sa každé ďalšie slovo začína veľkým písmenom. Pri používaní konštant sa používajú veľké písmená, amiesto medzier sú potržníky:

Kód:

$meno = "Denis";
$pocetPrvkov = mysql_...;
$MAX_PRVKOV = 6;

Mne sa ešte osvedčilo používať "predpony", čím sa zprehľadnilo zopár vecí:

Kód:

$q_user = $DB -> Query("SELECT * FROM user"); //query
$a_user = $DB -> FetchArray($q_user); //array

Tým, že budeš mať takto zvolené názvy premien, metód, funkcií vytváraš samodokumentáciu. Určite nedokumentuj každý krok, lebo to má kontraproduktívny účinok:

Kód:

$prveCislo = 1; // priradil som hodnotu pre prvé číslo výpočtu
$druheCislo = 3; // priradil som hodnotu pre druhé číslo výpočtu
if($prveCislo > 0 && $druheCislo > 0) { // ak rátame s reálnymi číslami
$vysledok = $prveCislo * $druheCislo; // výsledok je násobok oboch čísel
echo $vysledok; //vypisem výsledok
} //koniec podmienky
else { // ak nerátame s reálnymi číslami
echo "Nezadali ste reálne čísla"; //vypíšeme chybovú hlášku
} // koniec else podmienky

Dokumentuj hlavne dôležité časti aplikácie, hlavne aby si rýchlo našiel čo hľadáš a aby si sa rýchlo dostal do obrazu. Teda ak zvolíš vhodné názvy netreba ti robiť dokumentáciu načo tá-daná funkcia slúži: zmensiObrazok(), vypisAdresar(), pagination(), safety() a pod. Ak je trieda príliš obsiahla tak dokumentovať hlavne na začiatku kde zhrnieš čo robí, na čo slúži, čo obsahuje. V kóde iba jednotlivé úseky oddeliť. Ja som za to aby bolo v kóde čo najmenej komentárov, a čo najviac kvalitného kódu

Pokiaľ sa dá každej drobnosti venuj jeden riadok, teda nepchaj 7 vecí do jedneho riadku. Taktiež ak sa dá používaj skrátené formy:

Kód:

//NIE
$prveCislo=1;$druheCislo=3;if($prveCislo>0 && $druheCislo>0)$vysledok=$prveCislo*$druheCislo;else $vysledok="Zlé čísla";echo $vysledok;

//ANO
$prveCislo = 1;
$druheCislo = 3;
$vysledok = ($prveCislo > 0 && $druheCislo > 0) ? $prveCislo * $druheCislo : "Zlé čísla";
echo $vysledok;

Bezpečnosť
Odporúčam používať stenleyho funkciu. Slúži na základnú bezpečnosť. To znamená, že tie najväčie muchy to odstráni a teba stačí tu danú časť ošetriť podľa tej danej situácie. Uprednostňovať whitelist pred blacklistom, kde to je možné. Snažiť sa vyhnúť, zložitým nezmyselným veciam, pretože tam práve vznikajú chyby.

Taktiež ak sa Ti nejaká časť kódu opakuje v kóde/och viac krát odporúčam vytvoriť funkciu. Takteiž ako GoodWill dobre povedal, vytvoriť modularitu.

Na záver bezpečnosti by som odporúčal prečítať si nejakú dobrú knihu, osobne odporúčam: Zraniteľný kód od Cpress. Dočítaš sa v nej veľa dobrých rád a odporúčaní.

Ešte odporúčam urobiť možno aj smiešne veci, ale veľmi to ocenníš:
Písať si na paier, čo má daná vec robiť, ako sa má správať čo ma poskytovať. napr. rozanalyzovať celý projekt. Spočiatku som to robil v hlave ale zistil som, že je to oničom. Ak si spíšeš zoznam, tak potom môžeš ísť krok-po-kroku, čo máš a čo ešte nie. Teraz som našiel na oko dobre vyzerajúci článok od smashingmagazine kde to pravdepodobne vysvetľujú:
http://www.smashingmagazine.com/2009/08 ... ductivity/ (čítal som nadpisi a páčil sa mi

)

Nakoniec by som podotkol, aby si nezabudol spracovávať chyby, výnimky, a dôležité, aby si oddeloval vrstvy: logiku od designu (, + niekto vraví databázu od logiky[zatiaľ sa mi to v reálne nepodarilo

]).

Viac v tomto momente na srdci nemám. Ak ma niečo napadne doplním

rooobertek · Napísal **rooobertek**: 23.08.2009 18:34

Tominator, na prvej strane som spomínal coding standards pre zend. Pozri si to, tam je riešené všetko vrátane komentárov a dokumentácie.

Tominator · Napísal **Tominator**: 23.08.2009 18:45

Je to na spomínané na druhej strane

Zbežne som to preletel, ale mne to netreba. Tým nechcem povedať, že vedome odmietam písať pekný zrozumiteľný kód, ale myslím si, že pravidlá, ktoré používam ja, sú priateľné, a v niektorých prípadoch zhodné s tými zenďackými, o čom svedčí aj môj príspevok nad tvojím

Teda aj môj kód je pekný a zruzumiteľný, len strikne nedodržiavam zenďacke pravidlá

// Rozhodne tým však neopovrhujem, a môžem z toho čerpať "inšpiráciu"

Veľky projekt

Podobné témy