wat is daar zo natuurlijk aan?:
Even een 'kleine' code uit mijn systeem gehaald om even snel te laten zien hoe ik code
PHP:
Ik doe btw nooit HTML in me PHP, daarvoor gebruik ik natuurlijk templates.
"Real software engineers work from 9 to 5, because that is the way the job is described in the formal spec. Working late would feel like using an undocumented external procedure."
Alvast mijn excuses iig als het wat onsamenhangend overkomt, aangezien ik nogal gek aan het worden ben van de hoeveelheid discrete wiskunde dat ik vandaag door m'n strot heb gestouwd 
, en als je dan toch de afweging moet maken, kies dan voor onderhoudbaarheid; overcapaciteit voor performance kun je binnen 3 seconde aanschaffen itt code opnieuw van scratch opbouwen dat je ongetwijfeld veel tijd, moeite en tranen zal kosten. En ja, tijd is geld... 
, dus de afweging zou duidelijk moeten zijn . Goede documentatie draagt bij aan onderhoudbaarheid en dit is een GROOT voordeel als je met meerdere programmeurs aan een project werkt; hanteer hierbij allen dezelfde conventie (docstyle) zodat eenduidigheid bewerkstelligd kan worden. Duidelijke pre- en postcondities formuleren kan complexiteit ook tegengaan. Oh ja en voordat ik het vergeet, applicaties zijn zelden echt 'AF', hou rekening met de toekomst en kies onderhoudbaarheid; computerkracht wordt over tijd al steeds groter.. Ik hoop dat iig dat het bruikbaar voor iemand kan zijn
the-blueprints.com - The largest free blueprint collection on the internet: 50000+ drawings.
Ik ben volledig bekend met OO binnen Delphi, waar ik bijna alles in classes gooi, echter ik gebruik tegenwoordig ook PHP en daar zie ik nog steeds niet het nut van classes. Ik snap nog niet hoe OO je kan helpen in een stateless situatie als een html-pagina. Na het uitpoepen van een pagina zijn v.w.b. de server volgens mij weer alle objecten vernietigd.:
Mijn god zeg, 'k kan 't niet meer aanzien, 'k moet me even mengen in de discussie en OO standpunten (voor zover ik heb gelezen van genoil) ondersteunen.
[...]
Een systeem bestaat over het algemeen (afhankelijk van de vereiste functionaliteit) uit minstens 3 lagen:
1. User Interface (Presentatielaag: Raampjes, weetikveel - Client)
2. Model (Codelaag: Classen etc.. - Server)
3. Data Management (Datalaag: Databasefile systeem)
Verwijderd
(ff snel iets uit de losse pols neergekrabbeld):1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
| <?php /* * Project: Bla die bla * Author: Michiel */ class template { var $template; function template () { # Doe iets echo "blaat!"; } function actie ( $blaat ) { # Woei echo $blaat; } } $cTemplate = new template(); switch( $_GET['actie'] ) { case "een": $cTemplate->actie('dat is een'); break; case "twee": default: $cTemplate->actie('en de rest'); break; } ?> |
[ Voor 16% gewijzigd door Verwijderd op 26-10-2004 01:25 ]
Allereerste wat me opvalt: keyword uppercasing
, niet doen 
en als je het toch doet, doe het dan overal, Class, If, Else etc..Doe dan echt stoer:
1
2
3
| <?php ($gotDesc == 'leet') ? $userIsGotter = true : $userIsGotter = false; ?> |
[ Voor 107% gewijzigd door prototype op 26-10-2004 02:03 ]
Wannabe:
Doe dan echt stoer:
PHP:
Korter is per definitie niet duidelijker, bah.
1
| $userIsGotter = ($gotDesc == 'leet') ? true : false; |
1
| $userIsGotter = $gotDesc == 'leet'; |
"Beauty is the ultimate defence against complexity." David Gelernter
Ik wilde zo dicht mogelijk bij het oorspronkelijke voorbeeld blijven:
[...]
Wannabe
PHP:
Alleen is hier een if/else onnodig:
PHP:
[ Voor 6% gewijzigd door prototype op 26-10-2004 02:15 ]
Doe het dan direct goed:
[...]
Wannabe
PHP:
Alleen is hier een if/else onnodig:
PHP:
1
| $userIsGotter = ($gotDesc == 'leet'); |
Dat zou ik iets anders doen:
1
2
3
4
5
6
7
8
9
10
| <? switch($_GET['actie']) { case "een": $cTemplate->actie('dat is een'); break; case "twee": default: $cTemplate->actie('en de rest'); } ?> |
1
| if ($_GET['actie'] == "een") $cTemplate->actie(foo) else $cTemplate->actie(bar); |
1
2
3
4
| /**********************\ * Hier staat een klein * * stukje commentaar! * \**********************/ |
Ik prefereer een blokje commentaar voor elke functie en verder alleen commentaar bij ifs en elses:1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
| <?php /* * This function checks whether this is a new visitor or not and * takes care of the validation of the sessionvars and the * pagerequest. */ function checkVisitor() { if (isSet($_SESSION["page"])) { // not a new visitor validateSessionvars(); } else { // new visitor! setDefaultSessionvars(); } validatePagerequest(); } ?> |
[ Voor 36% gewijzigd door Bergen op 26-10-2004 04:17 . Reden: add-on ]
Ik zorg er over het algemeen voor dat de code in een notepad venstertje past op 1024, als het daar niet te breed voor is vind ik het wel best eigenlijk.
"Real software engineers work from 9 to 5, because that is the way the job is described in the formal spec. Working late would feel like using an undocumented external procedure."
Op deze manier verschuif je het 'probleem' met globals eigenlijk alleen, door een nieuwe functie toe te voegen welke de waarde uit de string haalt, maar dus nog steeds direct uit een globale variable. Ikzelf ben eigenlijk nog nooit een situatie tegengekomen waarbij globals de enigste mogelijke oplossing zouden bieden. Globals kan je vaak eenvoudig wegwerken door de variable als parameter aan de functie mee te geven.
1
2
3
4
5
6
7
8
| $string = 'wat/een/heerlijke/kaas'; $arr = explode('/', $string); functie doe_iets() { global $arr; if ($arr[3] == 'kaas') echo 'lekker!'; } |
1
2
3
4
5
6
7
8
9
| $string = 'wat/een/heerlijke/kaas'; $arr = explode('/', $string); functie doe_iets($array) { if ($array[3] == 'kaas') echo 'lekker!'; } doe_iets($arr); |
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
| $string = 'wat/een/heerlijke/kaas'; function splitStringToValue($string, $index) { $arr = explode('/', $string); return $arr[$index]; } function isKaasString($string) { if (arr($string, 3) == 'kaas') echo 'lekker!'; } isKaasString($string); ?> |
1. Waarom niet? Ik ben altijd in voor commentaar, maar geef dan wel een reden.:
[...]
Allereerste wat me opvalt: keyword uppercasing
Ten tweede, zoals ik je code even 123 snel lees, zie ik helemaal geen behoefte aan globals, daar je deze als instantievariabelen kunt declareren binnen je classe en deze bv kunt setten in je constructor.
[...]
Hehe, ik moet toegeven na even wat bladeren door wat van mijn code: ik ben niet erg consistent. Ik ga over op total camelcase, lijkt me wat netter.[ Voor 19% gewijzigd door Cavorka op 26-10-2004 20:04 ]
the-blueprints.com - The largest free blueprint collection on the internet: 50000+ drawings.
1) Je geef zelf al de reden; inconsistent, en dat kan resulteren tot ambiguiteit.:
[...]
1. Waarom niet? Ik ben altijd in voor commentaar, maar geef dan wel een reden.
2. Als je files include in een function, krijgen die dus scope van de functie. Al je classes die dus aanmaakt in dat bestand krijgen ook die scope. Als je dus er iets uit wilt halen, of in wilt stoppen, kan dat (bijna) niet anders dan met globals. Je zou het eventueel mee kunnen zenden als parameters van de constructor, maar daar wordt ikzelf ook niet vrolijk van. Het maakt het in mijn ogen niet duidelijker, overzichtelijker, of wat dan ook.
[ Voor 3% gewijzigd door prototype op 26-10-2004 20:19 ]
Als Je Alle Keyword Met Een Hoofdletter Schrijft, Leest 'T Ook Veel Kutter, Net Als In Deze Reply.:
[...]
1. Waarom niet? Ik ben altijd in voor commentaar, maar geef dan wel een reden.
"Real software engineers work from 9 to 5, because that is the way the job is described in the formal spec. Working late would feel like using an undocumented external procedure."
Daar gaat het nog niet eens fout.
1
2
3
4
5
6
| $bla = 1; {$blaat{$bla}} = 'Eén'; $bla = 2; {$blaat{$bla}} = 'Twee'; echo "$blaat$bla"; |
[ Voor 10% gewijzigd door Mithrandir op 26-10-2004 20:57 ]
Please guys, I'm trying to troll here.
If I failed to offend you in this post, please leave a message. I will address you in next post.
leuk voorbeeld, de "fout" is echter niet wat jij zegt, maar meer dat je totaal verkeerde variabelenamen gebruikt
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
), kan de ballen van kloppen (niet getest nog, met notepad geschreven 
), maar dat doet er even niet aan toe Het gaat even om de notatie van pre/postcondities in docstyle.1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
| <?php /** * Math class; dummy placeholder * @author Ninh Bui (ninh@phusion.nl) * @version 0.0.0.1 */ class Math { //TODO: Write math (super)class. } /** * Probability class * @author Ninh Bui (ninh@phusion.nl) * @version $Revision 1.3.3.7 $ */ class Probability extends Math { /** * Computes possible combinations; order omitted, reoccurence not allowed * @require $n > 0 && $r >= 0 * @param $n Size of collection * @param $r Size of selection * @return $result == $n! / (($n - $r)! * $r!) * @ensure $result > 0 */ function nCr($n, $r) { return ($this->nPr($n, $r) / $this->factorial($r)); } /** * Computes possible permutations; order of importance, reoccurence not allowed * @require $n > 0 && $r >= 0 * @param $n Size of collection * @param $r Size of selection * @return $result == $n! / ($n - $r)! * @ensure $result > 0 */ function nPr($n, $r) { return ($this->factorial($n) / $this->factorial($n - $r)); } /** * Computes the factorial of given $n; recursively * @require $n >= 0 * @param $n positive integer * @return $result == $n * ($n - 1) * ($n - 2) ... * ($n - ($n - 2)) * 1 * @ensure $result >= 1 */ function factorial($n) { $n = (int) abs($n); switch ($n) { case 0: case 1: return 1; break; default: return ($n * $this->factorial ($n - 1)); break; } } } ?> |
[ Voor 7% gewijzigd door prototype op 27-10-2004 02:41 ]
Gege, kwestie van de juiste IDE gebruiken die comment/methodblocks kan expanden en collapsen
(eclipse bv)
[ Voor 11% gewijzigd door prototype op 27-10-2004 03:21 ]
Het hangt ook van de editor af... Vi laat 't bijvoorbeeld prima zien:
[ Voor 17% gewijzigd door Bergen op 27-10-2004 03:16 ]
ehm, je stuurt die dbclass toch gewoon mee naar die functie als die het nodig heeft?
1
2
3
4
5
6
7
8
9
10
11
| //veel comments enzo function blog($acDatabase) { $this->cDatabase = $acDatabase; return TRUE; }//blog() (constructor) //weer veel comments, 'n variable die de query heeft en dan: $rSelect_blog = $this->cDatabase->query($qSelect_blog); |
[ Voor 33% gewijzigd door blizt op 27-10-2004 22:57 ]
United we stand, and divided we fall
Ik weet niet precies hoe PHP dit aanpakt, maar je zou het dus kunnen oplossen door in je classe een referentie naar een object te bewerkstelligen; bij het instantieren van een Blog object, geef je dus in de constructor aanroep meteen je database object mee.:
Hm, leg 'ns uit ...
Ik weet wat 'n constructor is hoor, maar hoe pak ik dat nu in de praktijk aan?
Wat ik nu van je begrijp is dat ik de variable die de class bevat (bv. $cDatabase) als parameter doorgeef aan de functie....
Hm, misschien wel 'n idee, ja ... Tijdens het typen van deze post zie ik eigenlijk al meer in dat constructors daar misschien wel eens voor bedoelt zouden kunnen zijn
[edit]
Nu dit gedaan:
PHP:
Ben ik zo op de goede weg of niet?
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
| <?php class Blog { var $db; function Blog ($db) { $this->setDb($db); } function setDb($db) { $this->db = $db; } function getArticle($id) { return $this->db->query(''); } } ?> |
[ Voor 4% gewijzigd door prototype op 27-10-2004 23:32 ]
.United we stand, and divided we fall
Verwijderd
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
| /* Blaat functie, zoek de bom */ function blaat( $foo ) { #En hier wat onzin switch ($foo) { case "kablam": echo "Bom gevonden!!!"; break; default: echo "Er is niets aan het handje"; break; // Ja die laatste break er ook bij, rare tik misschien } } |
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
| /* Blaat functie, zoek de bom */ function blaat( $foo ) { #En hier wat onzin switch ($foo) { case "kablam": echo "Bom gevonden!!!"; break; default: echo "Er is niets aan het handje"; break; // Ja die laatste break er ook bij, rare tik misschien } if (1) { } elseif (2) { // Die elseif iets meer ruimte geven omdat daar nog // wat extra tekst in staat } else { // De else wel op zun kortst } } # Variabelen doe ik soms $zoongeveer of $zo_ongeveer # (net als de PHP builders ben ik er ook nog niet uit) |
, en met een kleine reso kun je alles haarfijn op elkaar goed lezen. Lettergrootte (mocht je die kunnen veranderen) speelt misschien een rol, etc etc.
Oh? Wat is er dan niet netjes aan queries(getters) en commando's(setters)?:
[...]
Getters en setters zijn uberhaupt niet netjes. Maargoed dat is een hele andere discussie
[ Voor 10% gewijzigd door prototype op 28-10-2004 00:31 ]
Het is een eigenschap, daarbij doet de setter in dit geval niet meer als dat je het rechtstreeks zou doen. Dus waarom zou je?:
[...]
Oh? Wat is er dan niet netjes aan queries(getters) en commando's(setters)?
Je plaatst zo'n reply, dus verklaar je nader, please
[ Voor 4% gewijzigd door PrisonerOfPain op 28-10-2004 00:43 ]
Ligt eraan hoe je je query/commando method access toekent, indien je binnen je classe een commando/query private declareert die een private instancevar kan veranderen/waarde van teruggeven dan kan je prima deze setter binnen de classe gebruiken. Evt overload je de commando/query voor een evt andere signatuur (bv een public of protected access) indien dat wenselijk is. Ik zie het probleem dus niet
Het punt is dat je een method gebruikt op een plaats waar je met een gewone variabele, of __set()/__get() kan voldoen. Waarom zou je dan een getter of setter methode 'willen' gebruiken?
PHP ook PHP5 ondersteunt geen overloading op die manier. Overloading in PHP zijn methodss als __get, __set en __call. Nou kun je met __call de boel faken, maar dan gebruik je alsnog een method.Evt overload je de commando/query voor een evt andere signatuur (bv een public of protected access) indien dat wenselijk is. Ik zie het probleem dus niet
Mogelijk wil je in de toekomst extra functionaliteit toevoegen, en niet zomaar bruutweg de waarde veranderen danwel opvragen 'as is'. Mogelijk moet het resultaat van bv een query geformateerd worden. Ik begrijp opzich nog niet wat je bedoelt met __set()/__get(), maar then again, ben ook niet echt into PHP
Ergo PHP zuigt
[ Voor 9% gewijzigd door prototype op 28-10-2004 00:57 ]
Daardoor mis je dus vrijwel het hele punt:
Ik begrijp opzich nog niet wat je bedoelt met __set()/__get(), maar then again, ben ook niet echt into PHP
Ik zie dat je een .Net-ter bent dus;1
2
3
4
5
6
7
8
9
10
11
| public string Property
{
get
{
return myProperty
}
set
{
myProperty = value;
}
} |
"vervang heks"?
[ Voor 19% gewijzigd door Bergen op 28-10-2004 02:21 ]
Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz
Hey Isaac...let's go shuffleboard on the Lido - deck...my site koli-man => MOEHA on X-Box laaaiiiff
1
| if ($leeftijd >= 0) |
[ Voor 57% gewijzigd door Bosmonster op 28-10-2004 15:18 ]
Euh... zoiets dan?:
code:
Ow.. mooie check... alles (strings/booleans) evalueert zo'n beetje naar >= 0
1
| if (isset($leeftijd) && is_numeric('0'.$leeftijd) && $leeftijd >= 0) |
[ Voor 33% gewijzigd door Bergen op 28-10-2004 21:14 ]
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
| <?php class Pruts { private $myLeeftijd; public function __set ($Var, $Value) { switch ($Var) { case 'Leeftijd': { if ($Value >= 0) { $this->myLeeftijd = $Value; } else { throw new SignedValueException(); } }break; } return; } } class SignedValueException extends Exception {} |
)
Zie ik daar nou een sql injection exploit bug?:
Even een 'kleine' code uit mijn systeem gehaald om even snel te laten zien hoe ik code
PHP:
Ik doe btw nooit HTML in me PHP, daarvoor gebruik ik natuurlijk templates.
Ik vind het wel meevallen
komt simpelweg omdat de variabele in de class voorrang heeft op de __get en __set functies. (Note dat class::$blaat hier dezelfde naam heeft als die in je __get en __set()).1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
| <?php class testclass { private $myblaat; //Variabele declareren doe we ook niet meer? function __get($var) { if($var = 'blaat') { echo "__get blaat: '{$this->myblaat}'\r\n"; } } function __set($var,$val) { if($var = 'blaat') { echo "__set blaat to: '$val'\r\n"; $this->myblaat = $val; } } } $t = new testclass; echo "echoing t->blaat: '" . $t->blaat . "'\r\n"; $t->blaat = 'Pom'; echo "echoing t->blaat: '" . $t->blaat . "'\r\n"; $t->blaat = 'Woei'; echo "echoing t->blaat: '" . $t->blaat . "'\r\n"; ?> |
dit evalueert altijd naar true
1
2
3
4
| if($var == 'blaat') { echo "__get blaat: '".$this->blaat."'\r\n"; } |
die punt zit er zo hard in, hoewel ik met echo wel vaker probeer om die komma te gebruiken, is eigenlijk ook beter wat het hoeft idd geen concatenatie plaats te vinden, het is immers gewoon output en dan is het idd onzinnig om eerst alles samen te voegen en dan pas naar de client te sturen
Dan nog vind ik het ranzig. Het werkt totaal anders dan je (ik) zou verwachten.:
[...]
Ik vind het wel meevallen
Mijn oplossing; variabelen prefixen met my, of iets anders.
[...]
Als je in jou voorbeeld $blaat eerst private had gemaakt had je trouwens een foutmelding gekregen dat je de prive eigenschap $blaat niet aan mag
. Maar NOG vind ik het een vage implementatie. Waarom ze zich niet gewoon aan de 'standaard' oplossing houden is me een raadsel. Maar in plaats van daar commentaar op te leveren had ik liever een inhoudelijke reply gezien [ Voor 12% gewijzigd door Mithrandir op 29-10-2004 16:28 ]
PHP is uberhaupt een taal waar je weinig 'standaard' dingen van kunt verwachten:
[...]
Dan nog vind ik het ranzig. Het werkt totaal anders dan je (ik) zou verwachten.
En prefixen is helemaal bah, dan kun je het beter nog in een array zetten
. Persoonlijk vind ik de C# manier ook een stuk netter. Maar er valt mee te leven. Probeer trouwens hetzelfde voorbeeld als wat jij postte eens naar C# te porten. Je compiler zal dan een foutmelding geven als:Dat PHP deze foutmelding niet kan geven ligt aan de implementatie, je weet van te voren namelijk niet welke variabelen er allemaal afgevangen zullen worden door __get en __set.
Okee, maar dan zou je natuurlijk wel dit hebben::
[...]
PHP is uberhaupt een taal waar je weinig 'standaard' dingen van kunt verwachten
[...]
Dat PHP deze foutmelding niet kan geven ligt aan de implementatie, je weet van te voren namelijk niet welke variabelen er allemaal afgevangen zullen worden door __get en __set.
In dit geval vind ik zelf trouwens variabelen in een array zetten weer viesjes, omdat de variabelen niet allemaal dezelfde betekenis hebben.
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
| class MyClass { private int x; public int X { get { return x; } set { x = value; } } } |
[ Voor 87% gewijzigd door PrisonerOfPain op 29-10-2004 19:08 ]
1
2
3
4
5
6
7
8
9
| // Voorbeeld commentaar // Tweede regel van het commentaar function voorbeeld() { global $variabele; if ($dit_topic == 'goed') { echo 'ik lever ook commentaar!'; } } |
1
2
3
4
| $output.= "<table> <tr><td colspan=\"2\">dit is cel 1</td></tr> <tr><td>cel 2</td><td>cel 3</td></tr> </table>"; |
1
| // Commentaar |
Verwijderd
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
| <!-- +---------+-------------------------------+ | | |\ | author | Arthur Dam || | date | 5/10/2004 || | client | http://gathering.tweakers.net || | contact | Jan de Baas || | | copyrighted 2003-2004(c) || | | all rights reserved || | | || +---------+-----------------------------=-+| | | || | script | LeetnessChecker || | | version 2.1 || | | || +---------+-------------------------------+| ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯ --> <? require_once 'config.php'; require_once 'functions.php'; session_start(); $banaan = new banana(); connectdB(); if ($vVar1 == bUser) { // print dat je een blaataap bent! print 'in ben een blaataap'; $iets = $banaan->vUser; } else { // print dat je ownt! print 'ik own bijna'; ?> |
zoals uLogin, vVar, dbServ ofzo).[ Voor 42% gewijzigd door Verwijderd op 29-10-2004 20:00 ]
Verwijderd
echo naar print omdat 't sneller blijkt te zijn plus wat kleine dingen die ik nooit gebruik, van include naar require_once zodat ik het sneller doorheb als ik domme dingen doe
[ Voor 7% gewijzigd door Verwijderd op 29-10-2004 20:02 ]
Zoals Prisoner al zegt, return print 'n waarde. Is dus (voor zover ik weet) trager dan echo. Ik heb er wel 'ns benchmarks van gezien, zal ze zo even opzoeken als ik er tijd voor heb.:
[...]
echo naar print omdat 't sneller blijkt te zijn, van include naar require_once zodat ik het sneller doorheb als ik domme dingen doe
United we stand, and divided we fall
Dit zie je niet zo vaak. Waarom indent je niet bij je method body ?
Bracket was gewoon een vergeet gevalletje. Verder ben ik nog beginnend java'er, zo wordt het ons op school geleerd.:
[...]
Dit zie je niet zo vaak. Waarom indent je niet bij je method body ?
Ik vind het niet echt overzichtelijk omdat het niet duidelijk aangeeft waar een methode begint of eindigt. Misschien dat je daarom vergeten bent somGetallen() met een } af te sluiten
[ Voor 14% gewijzigd door Copyman op 29-10-2004 20:43 ]
Zeer belangrijke informatie: Inventaris
Toch vind ik het niet zo netjes, de ene keer ident je anders dan de ander, bij functies e.d. ident je 'm niet, maar bij ifs wel... en bij de ene regel gaat de { naar de volgende, bij de ander niet
|>
) beschreven hoe je zou moeten coderen lijkt me:
"Ik heb zo veel soep gegeten, dat kan een mens niet aan. Ik heb zo veel soep gegeten, kan bijna niet meer staan. Ik zat daar maar te slurpen achter die grote kop en als ik bijna klaar was, dan schepten ze weer op!" (Hans Teeuwen)
1
2
3
4
5
6
| <? if($var == 'blaat') { echo "__get blaat: '", $this->blaat, "'\r\n"; } ?> |
1
2
3
4
| <? if($var == 'blaat') echo "__get blaat: '", $this->blaat, "'\r\n"; ?> |
Ok het helpt wel bij grote scripts net als mijn site die vele mb's aan code is, maar het is stom om bijv het volgende te doen:)1
2
3
4
5
6
| login.php <? /* * Login pagina */ ?> |
[ Voor 53% gewijzigd door Rex op 29-10-2004 23:27 ]
Rex
omdat je als je die brackets wel neer zet direct overzicht hebt over welke code waarbij hoort, en daarnaast is de code makkelijker uit te breiden en hoef je niet steeds overal brackets bij te zetten. Maar volgens mij heb ik dit al een keer genoemd in dit topic...
ehm, right, lees over een x aantal maanden die code nogmaals, of vraag iemand anders om die code eens door te nemen:
[Aanvulling]
• Ik gebruik zo min mogelijk commentaar. (Waarom zou je commentaar geven als je je scripts door en door kent?
PHP:
Ik bedoel, doh! zelfs me oma ziet dat het de login pagina is, door alleen al naar de naam van de file te kijken.
[ Voor 28% gewijzigd door Erkens op 29-10-2004 23:29 ]
Ok, enigzins heb je wel gelijk. Voor sommige mensen zou gelden dat ze commentaar nodig hebben.[...]
ehm, right, lees over een x aantal maanden die code nogmaals, of vraag iemand anders om die code eens door te nemen
Rex
vroeg of laat kom je jezelf wel tegen:
[...]
Ok, enigzins heb je wel gelijk. Voor sommige mensen zou gelden dat ze commentaar nodig hebben.
In mijn geval niet, omdat mijn code alleen maar door 2 andere personen aangepast mag worden, en niemand anders mag het bekijken. Ook is het zo dat we dagelijks aan het coderen zijn en dat we al zo lang met elkaar coderen dat wij met één woord elkaar begrijpen. Dus ok, ik heb niet echt rekening gehouden met de "normale tweaker"
alles is even wennen, maar na een if/while/etc typ ik automatisch die brackets zodat het nooit fout gaat, ieder zijn eigen style natuurlijk. Maar waarom zouden die brakcets onnodig zijn? ze geven namelijk duidelijk aan welke code waarbij hoort, dat kan je van jouw manier niet zeggen, je kan er snel overheen lezen namelijkToch vind ik wel dat onnodige brackets gewoon eruit kunnen. Als je later een statement erbij doet, dan zie je zeker wel dat het een bracket mist. (Tenminste in mijn geval wel, maar ik hanteer deze "techniek" al jaren, dus ik zie meteen dat iets geen brackets heeft en die dus nodig heeft)
Het is dus gewoon even wennen, na een tijdje gaat het vanzelf
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
| /** * @author B-Man * @date 30-10-2004 * @version 4.0 * @copyright 2004 */ /* Import libraries */ CFW::lib('auth'); CFW::lib('db'); CFW::lib('form'); CFW::lib('requestcache'); CFW::lib('page'); /* Verify & handle request */ if(!RequestCache::setup('client', 'mode', array('mode', 'id'))) { Request::validate(array( 'mode' => 'req;in[add/edit]', 'id' => 'req[+mode:edit];numeric[pos]' )); RequestCache::cache(); } /* Import variables we need in the script */ $mode = Request::getSafe('mode', 'string'); $id = Request::getSafe('id', 'integer'); /* Export variables we need for output */ Page::export(array( 'mode' => &$mode )); |
). Maar mocht het ooit nodig zijn, dan kan het wel. (Zo gaat er nu een andere programmeur aan de slag me een stel componenten van me, en die kon ik in een minuut of 15 laten zien hoe e.e.a. werkt, en kon ook zo documentatie genereren).[ Voor 8% gewijzigd door B-Man op 30-10-2004 01:13 ]
:strip_icc():strip_exif()/u/43473/crop5e12eeae6a054_cropped.jpeg?f=community)
:strip_icc():strip_exif()/u/50076/dosto_60.jpg?f=community)
/u/63661/Leen2.JPG?f=community)
:strip_exif()/u/28900/sharingan.gif?f=community)
:strip_exif()/u/82642/porsche-993-gt2.gif?f=community)
/u/34216/avatar-60x60.png?f=community){kind=avatar}
:strip_icc():strip_exif()/u/2626/nice_eye.jpg?f=community)
:strip_icc():strip_exif()/u/18164/elmo.jpg?f=community)
:strip_exif()/u/18996/GoT_avatar.gif?f=community){kind=avatar}
/u/23440/c.png?f=community)
/u/20796/crop64ea633835d95_cropped.png?f=community)
:strip_icc():strip_exif()/u/32690/icon.jpg?f=community){kind=icon}
:strip_icc():strip_exif()/u/12461/crop65b195553d948.jpg?f=community)
:strip_icc():strip_exif()/u/77332/crop5e54ec42433bc_cropped.jpeg?f=community)
:strip_exif()/u/73955/foxavatar-60.gif?f=community){kind=avatar}
:strip_exif()/u/22119/ch.gif?f=community)
:strip_icc():strip_exif()/u/76420/isaac.jpg?f=community)
:strip_exif()/u/27216/bosmonster_dropdown.gif?f=community)
:strip_icc():strip_exif()/u/35767/images.jpg?f=community)
:strip_exif()/u/11775/SoulTaker.gif?f=community)
:strip_icc():strip_exif()/u/2332/lektroluv.jpg?f=community)
/u/19875/deadmau5-got.png?f=community)
:strip_exif()/u/3398/rh.gif?f=community)
:strip_exif()/u/33970/hans.gif?f=community)
:strip_icc():strip_exif()/u/92327/Image1.jpg?f=community)
:strip_icc():strip_exif()/u/23850/aquarius.jpg?f=community)