[alg] Slechtste programmeervoorbeelden deel 4

[Alex)]

What the fuck?

[YopY]

Michali schreef op maandag 23 augustus 2010 @ 09:25:
Mijn maandagochtend begint goed:

PHP:

// Den_Haag naar Den Haag omzetten
function ReplaceDenHaag($plaats)
{
    if ($plaats == "Den_Haag") {
        $newplaats = "Den Haag";
    } else {
        $newplaats = $plaats;
    }

    return $newplaats;
}

[plaag]Wat, dat je apart een functie schrijft om Den_Haag om te zetten naar Den Haag of de overbodige else?[/plaag]

Waarschijnlijk kun je een lijstje / array maken van input die je vervangen moet (en hun vervangers) en dan alle plaatsen in één keer replacen. Als dat van toepassing is natuurlijk.

[Orion84]

Die else is niet overbodig, als je die functie per ongeluk aanroept met Rotterdam, dan geeft ie gewoon weer Rotterdam terug, in plaats van Den Haag, dat is in mijn ogen nog wel nuttig te noemen.

De enige WTF is inderdaad dat je een functie maakt voor zo'n specifiek geval, maar zonder de context is het ook lastig te zeggen hoe WTF dat is. Het zou vooral een WTF zijn als er ook een ReplaceDenBosch en een ReplaceDenHelder etc. zouden zijn in plaats van een meer generieke oplossing

[ Voor 10% gewijzigd door Orion84 op 23-08-2010 10:47 ]

[creator1988]

Orion84 schreef op maandag 23 augustus 2010 @ 10:46:
Die else is niet overbodig, als je die functie per ongeluk aanroept met Rotterdam, dan geeft ie gewoon weer Rotterdam terug, in plaats van Den Haag, dat is in mijn ogen nog wel nuttig te noemen.

De enige WTF is inderdaad dat je een functie maakt voor zo'n specifiek geval, maar zonder de context is het ook lastig te zeggen hoe WTF dat is. Het zou vooral een WTF zijn als er ook een ReplaceDenBosch en een ReplaceDenHelder etc. zouden zijn in plaats van een meer generieke oplossing

Wij hebben ook plaatsvarianten, en onze test hiervoor is 'asd', dat naar Amsterdam moet worden omgezet. Werkt niet, dus issue aangemaakt, opgelost etc. Krijg ik de melding dat 'den haag' niet meer werkt (moet 's Gravenhage worden). Staat echt op het allerlaagste lowlevel niveau, bij het parsen van de URL:

code:

if(url.Contains("asd")) Criteria.Add(Plaats, "Amsterdam");

argh!

[TJHeuvel]

Die else kan overbodig zijn:

PHP:

function ReplaceDenHaag(&$plaats)
{
    if($plaats == "Den_Haag")
        $plaats = "Den Haag";
}

Uiteraard is die functie gewoon brak. Misschien om wat data van een oud systeem naar een nieuw systeem te importeren?

[ Voor 28% gewijzigd door TJHeuvel op 23-08-2010 11:21 ]

[ZaZ]

CyCloneNL schreef op maandag 23 augustus 2010 @ 11:21:
Die else kan overbodig zijn:

PHP:

function ReplaceDenHaag(&$plaats)
{
    if($plaats == "Den_Haag")
        $plaats = "Den Haag";
}

Uiteraard is die functie gewoon brak. Misschien om wat data van een oud systeem naar een nieuw systeem te importeren?

great, dan heb je een functie die niets teruggeeft. of heeft php geen procedures?

[CoolGamer]

ZaZ schreef op maandag 23 augustus 2010 @ 11:27:
[...]

great, dan heb je een functie die niets teruggeeft.

$plaats wordt doorgegeven als reference. Zo wordt de variabele direct bewerkt en heb je geen return meer nodig.

[TJHeuvel]

Zoals TheCoolGamer al zei, heb ik stiekem er een reference van gemaakt dmv het & symbool.

[ZaZ]

Ja dat zie ik ook wel.
Maar een functie die niets teruggeeft is lelijk, want dan is het geen functie nietwaar?

edit:

Ah, zal wel weer zo'n bijzonder dingetje van php zijn.
Geen void, procedure, sub of whatever.

[ Voor 34% gewijzigd door ZaZ op 23-08-2010 11:46 ]

[Noxious]

CyCloneNL schreef op maandag 23 augustus 2010 @ 11:21:
Die else kan overbodig zijn:

PHP:

function ReplaceDenHaag(&$plaats)
{
    if($plaats == "Den_Haag")
        $plaats = "Den Haag";
}

Uiteraard is die functie gewoon brak. Misschien om wat data van een oud systeem naar een nieuw systeem te importeren?

PHP:

function ReplaceDenHaag($plaats){
    return str_replace("Den_Haag", "Den Haag", $plaats);
}

Alhoewel het vast sneller kan, is dit imo netter
Al kun je dan imo beter op de locatie waar het nodig is direct die str_replace uitvoeren.

[ Voor 8% gewijzigd door Noxious op 23-08-2010 11:48 ]

[kwaakvaak_v2]

ik zou eerder een mooie lookup tabel maken waar je alles door heen haalt

code:

$in = array("denhaag",denbosch","denhelder")
$out = array("'s-Gravenhage","'s-Hertogenbosch","Den Helder")
return str_ireplace($in,$out,$string);

Net zo makkelijk, en scheelt je een hoop functies voor elke plaats

Oh en niet case sensitive uitaard, want stel dat iemand denHaag schrijft

[Noxious]

Als je er meerdere hebt wel ja, maar misschien hebben ze alleen locaties in Amsterdam, Rotterdam en Den Haag oid en komen andere plaatsen niet voor.

Dan kun je nog beter een str_replace("_", " ", $plaats) doen uiteraard.

[TJHeuvel]

Noxious schreef op maandag 23 augustus 2010 @ 11:48:

PHP:

function ReplaceDenHaag($plaats){
    return str_replace("Den_Haag", "Den Haag", $plaats);
}

Alhoewel het vast sneller kan, is dit imo netter
Al kun je dan imo beter op de locatie waar het nodig is direct die str_replace uitvoeren.

Dat doet niet hetzelfde, de functie moet de string 'Den_Haag' omzetten naar 'Den Haag'.
Bij 'Den_Haag' gaat bij jou alles goed, maar bij 'Ik woon in Den_Haag' niet, want die zou hij niet moeten aanpassen

[Noxious]

CyCloneNL schreef op maandag 23 augustus 2010 @ 12:01:
[...]

Dat doet niet hetzelfde, de functie moet de string 'Den_Haag' omzetten naar 'Den Haag'.
Bij 'Den_Haag' gaat bij jou alles goed, maar bij 'Ik woon in Den_Haag' niet, want die zou hij niet moeten aanpassen

Da's helemaal waar.

[Voutloos]

CyCloneNL schreef op maandag 23 augustus 2010 @ 11:34:
Zoals TheCoolGamer al zei, heb ik stiekem er een reference van gemaakt dmv het & symbool.

Output parameters in PHP zijn serieus not done. Je kan returnen wat je wilt, zelfs verschillende types en of complexere structuren dus vermijd aub references hiervoor. Als je hele codestyle zo is, is dat wmb een grotere wtf dan de originele functie.

[TJHeuvel]

Ik ben niet zo van de references, maar waarom zou het anders zijn om een reference te maken of een waarde te returnen? Dit kan toch ook van alles zijn.

[mithras]

CyCloneNL schreef op maandag 23 augustus 2010 @ 12:09:
Ik ben niet zo van de references, maar waarom zou het anders zijn om een reference te maken of een waarde te returnen? Dit kan toch ook van alles zijn.

PHP:

<?php
function foo (&$bar)
{
    $bar = 'baz';
}

$bar = 'bat';
var_dump(foo($bar));    // null

oepsie

[Orion84]

kwaakvaak_v2 schreef op maandag 23 augustus 2010 @ 11:53:
ik zou eerder een mooie lookup tabel maken waar je alles door heen haalt

code:

$in = array("denhaag",denbosch","denhelder")
$out = array("'s-Gravenhage","'s-Hertogenbosch","Den Helder")
return str_ireplace($in,$out,$string);

Net zo makkelijk, en scheelt je een hoop functies voor elke plaats

Oh en niet case sensitive uitaard, want stel dat iemand denHaag schrijft

maar dan toch hopelijk op een elegantere manier dan met twee losse arrays die je zo in je code definieert, want dat schreeuwt imho om het maken van foutjes in de volgorde etc. op het moment dat je later plaatsen gaat toevoegen/verwijderen etc.

[Erkens]

Orion84 schreef op maandag 23 augustus 2010 @ 12:35:
[...]

maar dan toch hopelijk op een elegantere manier dan met twee losse arrays die je zo in je code definieert, want dat schreeuwt imho om het maken van foutjes in de volgorde etc. op het moment dat je later plaatsen gaat toevoegen/verwijderen etc.

Inderdaad, dan liever zo doen (als je het zo wilt doen natuurlijk ) is tenminste iets meer overzichtelijker

PHP:

$stuff = array(
    "denhaag"   =>"'s-Gravenhage",
    "denbosch"  =>"'s-Hertogenbosch",
    "denhelder" =>"Den Helder"
);
return str_ireplace(array_keys($stuff),array_values($stuff),$string);

[Icekiller2k6]

Noxious schreef op maandag 23 augustus 2010 @ 11:59:
Als je er meerdere hebt wel ja, maar misschien hebben ze alleen locaties in Amsterdam, Rotterdam en Den Haag oid en komen andere plaatsen niet voor.

Dan kun je nog beter een str_replace("_", " ", $plaats) doen uiteraard.

en str_replace werkt ook op arrays normaal.;

[TJHeuvel]

mithras schreef op maandag 23 augustus 2010 @ 12:18:
[...]

PHP:

<?php
function foo (&$bar)
{
    $bar = 'baz';
}

$bar = 'bat';
var_dump(foo($bar));    // null

oepsie

Dat is best logisch, var_dump dumpt wat foo returned.

Wat ik wil zeggen, is dat je in PHP nooit weet wat een functie met je variable doet, of returned. En dit dus niet specifiek iets met references is.

[kwaakvaak_v2]

Erkens schreef op maandag 23 augustus 2010 @ 12:42:
[...]

Inderdaad, dan liever zo doen (als je het zo wilt doen natuurlijk ) is tenminste iets meer overzichtelijker

PHP:

$stuff = array(
    "denhaag"   =>"'s-Gravenhage",
    "denbosch"  =>"'s-Hertogenbosch",
    "denhelder" =>"Den Helder"
);
return str_ireplace(array_keys($stuff),array_values($stuff),$string);

Waarbij ik dan mag hopen dat dit stukje niet in je main loop zit en vaak wordt aangeroepen. Want array_keys, en array_values zijn relatief trage operaties in PHP. Beter is het dan om het direct goed op te slaan in je database.

Wat een speculaties weer he... Waarschijnlijk wordt deze code 1 keer per 100 jaar aangeroepen ofzo

[Sebazzz]

kwaakvaak_v2 schreef op maandag 23 augustus 2010 @ 14:12:
[...]

Wat een speculaties weer he... Waarschijnlijk wordt deze code 1 keer per 100 jaar aangeroepen ofzo

Als programmeur hoor je imo perfectionistisch te zijn.

[Michali]

kwaakvaak_v2 schreef op maandag 23 augustus 2010 @ 14:12:
Wat een speculaties weer he... Waarschijnlijk wordt deze code 1 keer per 100 jaar aangeroepen ofzo

Sterker nog, de code wordt helemaal nooit meer aangeroepen. Hij werd volgens mij gebruikt om een waarde om te zetten naar een waarde voor weergave. Ik denk dat alleen Den Haag met een underscore gescheiden werd en alle andere plaatsen maar uit 1 woord bestonden.

[Woy]

Sebazzz schreef op maandag 23 augustus 2010 @ 14:14:
[...]

Als programmeur hoor je imo perfectionistisch te zijn.

Maar vooral pragmatisch

[kenneth]

Woy schreef op maandag 23 augustus 2010 @ 14:31:
[...]

Maar vooral pragmatisch

Alleen als dat tot een beter product leidt

[ZaZ]

Er is zoveel verschrikkelijke code ontstaan door iets wat maar 'even' hoefde te werken, of alleen in 1 bepaald geval.
Wanneer je je laat verleiden tot code schrijven die lelijk en/of heel inefficient, kun je er donder op zeggen dat vroeg of laat een aantal van die stukken toch een eigen leven gaan leiden.

Voor je het weet ben je workarounds aan het maken.

[kenneth]

Het is inderdaad een kunst om het midden te vinden tussen academisch correcte code en "het werkt, wat zeur je?"...

Perfectionistisch is wat mij betreft geen goede insteek als het gaat om betaald programmeren. Factoren als tijd en geld spelen nu eenmaal een rol.

The perfect is the enemy of the good.

[Erkens]

kwaakvaak_v2 schreef op maandag 23 augustus 2010 @ 14:12:
Beter is het dan om het direct goed op te slaan in je database.

True, maar helaas moet je vaak de troep van andere opruimen of is een dergelijke aanpassing niet mogelijk binnen een bepaald budget/bepaalde tijd.
En traag? Ach, dan laten we toch gewoon snellere server kopen

[ZaZ]

Ja ok, dat is het andere uiterste.
Een ex collega had zoiets. Die kreeg een opdracht en soms was het gewoon iets wat je in een dag in elkaar draait.
Die ging dan eerst zitten tekenen en alles uitdenken en kwam tot 30 keer terug op zijn ontwerp omdat het nog meer robuust en strakker kon. Dan kwam ie meteen met nieuwe standaarden zodat alle tools nog beter op elkaar zouden aansluiten en die moesten dan ook weer veranderen etc.
Een echte overdesigner dus...

Maar iedereen die aan betaald programmeren doet heeft te maken gehad met tools die bijv eerst alleen maar effe een database moest importeren, dus dat het verschrikkelijke code is moet niet uitmaken.
Daarna moest er toch ook nog dit gebeuren en als ie dit en dat zou doen, zou het ook handig zijn.
Voor je het weet is het een draak van een tool.

En het is helemaal erg als in je core app dat soort dingen gebeuren.

[Guldan]

kenneth schreef op maandag 23 augustus 2010 @ 15:08:
Het is inderdaad een kunst om het midden te vinden tussen academisch correcte code en "het werkt, wat zeur je?"...

Perfectionistisch is wat mij betreft geen goede insteek als het gaat om betaald programmeren. Factoren als tijd en geld spelen nu eenmaal een rol.

The perfect is the enemy of the good.

Deze staat op een murphy poster die ik thuis heb. Is heel toepasselijk als ontwikkelaar:

There is never enough time to do it right. But there's always enough time to do it again

[mithras]

^^ 9/10 keer schrijf ik code zo goed als het kan in de gestelde termijn + begroting. Meestal is het wel netjes, maar meestal zie ik ook bij een revisit dat het toch beter had gekund. Zo itereer je telkens verder tot perfecte code

[kenneth]

mithras schreef op maandag 23 augustus 2010 @ 15:51:
^^ 9/10 keer schrijf ik code zo goed als het kan in de gestelde termijn + begroting. Meestal is het wel netjes, maar meestal zie ik ook bij een revisit dat het toch beter had gekund. Zo itereer je telkens verder tot perfecte code

Ik las:

Zo irriteer je telkens verder tot perfecte code

Maar dat zal wel komen door de code die ik heb moeten onderhouden.

[Michali]

mithras schreef op maandag 23 augustus 2010 @ 15:51:
^^ 9/10 keer schrijf ik code zo goed als het kan in de gestelde termijn + begroting. Meestal is het wel netjes, maar meestal zie ik ook bij een revisit dat het toch beter had gekund. Zo itereer je telkens verder tot perfecte code

Ja dat doe ik ook wel. Als ik code terugzie en ik snap het even niet helemaal goed meer, dan probeer ik wat ik dan leer er in te verwerken zodat het duidelijker wordt.

[Erkens]

Gelukkig dat ik vaak nog even snel een regeltje commentaar bij zulke hack-code plaats (en al is het vaak alleen maar een ticket-nummer) anders ben je echt nog uren bezig om te bedenken waaorm je die code zo hebt geschreven

[Davio]

ZaZ schreef op maandag 23 augustus 2010 @ 15:21:
Ja ok, dat is het andere uiterste.
Een ex collega had zoiets. Die kreeg een opdracht en soms was het gewoon iets wat je in een dag in elkaar draait.
Die ging dan eerst zitten tekenen en alles uitdenken en kwam tot 30 keer terug op zijn ontwerp omdat het nog meer robuust en strakker kon. Dan kwam ie meteen met nieuwe standaarden zodat alle tools nog beter op elkaar zouden aansluiten en die moesten dan ook weer veranderen etc.
Een echte overdesigner dus...

Maar iedereen die aan betaald programmeren doet heeft te maken gehad met tools die bijv eerst alleen maar effe een database moest importeren, dus dat het verschrikkelijke code is moet niet uitmaken.
Daarna moest er toch ook nog dit gebeuren en als ie dit en dat zou doen, zou het ook handig zijn.
Voor je het weet is het een draak van een tool.

En het is helemaal erg als in je core app dat soort dingen gebeuren.

Ach, ik heb dat wel afgeleerd in een 'corporate environment'. Soms heeft een klant er meer aan dat iets binnen 2 weken werkt dan dat het over 10 jaar eventueel met één aanpassing heel makkelijk uitgebreid kan worden.

Het is stiekem ook beter om dat niet te doen, natuurlijk, anders heb je snel geen werk meer.

[SPee]

En je kunt ook doorslaan en alles zo uitbreidbaar mogelijk maken.
Ook gehad met een applicatie, Er werd veel gebruik gemaakt van een Map met key/value velden.
De mogelijke keys werden in een aparte tabel bijgehouden, maar nergens stond gedocumenteerd welke keys bij elkaar hoorden. Dus bij nieuwe records, werden er vaak keys vergeten/verkeerd ingevuld of teveel ingevuld. En het ging nog met de hand ook (SQL)
Dus je moest echt tig keer testen, voordat je het goed werkend had. Met een simpel datamodel had je dat probleem niet.

Had dus soms zin om de bedenker ervan te slaan en niet alleen voor die applicatie.

[Davio]

Ja, ik heb ook wel eens iets gemaakt om menustructuren in een database te onderhouden waarna er nooit een item werd toegevoegd; maar ja, dat was niet mijn probleem.

[Michali]

Davio schreef op maandag 23 augustus 2010 @ 17:54:
Het is stiekem ook beter om dat niet te doen, natuurlijk, anders heb je snel geen werk meer.

Je moet het juist laten lijken dat het veel werk is en lekker op GoT rondhangen natuurlijk.

[YopY]

kenneth schreef op maandag 23 augustus 2010 @ 15:08:
Het is inderdaad een kunst om het midden te vinden tussen academisch correcte code en "het werkt, wat zeur je?"...

Perfectionistisch is wat mij betreft geen goede insteek als het gaat om betaald programmeren. Factoren als tijd en geld spelen nu eenmaal een rol.

Niet helemaal eens. Ik vindt dat je als programmeur minstens één keer een programma of onderdeel gemaakt moet hebben dat volgens de hedendaagse maatstaven 'perfect' zoveel mogelijk benadert - goede documentatie, tests, design patterns waar ze logisch zijn en iets toevoegen, en een heel rijtje andere termen die je uit de betere / standaardprogrammerwerken haalt.

Zo zul je die, misschien onderbewust, in toekomstige projecten ook toepassen waar mogelijk - en omdat je er bekend mee bent zal het op de korte duur nauwelijks extra tijd kosten (je poept zo een unit testje uit, ipv het geheel te compilen en handmatig erdoorheen te klikken (bijvoorbeeld)), en zich op de lange duur terugbetalen.

In theorie natuurlijk, ik heb het nooit goed toe kunnen passen ivm tijdgebrek en gebrek aan goeie projecten .

Eigenlijk is het stiekem best wel erg dat het enige project waar ik echt tevreden over ben mijn eerste stageopdracht van ~2.5 jaar geleden is. 20 weken alleen op een project zonder ook maar enige randvoorwaarden kunnen storten die zelfs het stagebedrijf zelf niet goed heeft kunnen definieren . Toch een 9 op gekregen toen.

[ Voor 13% gewijzigd door YopY op 23-08-2010 20:51 ]

[kenneth]

Eens Zoiets zat ik ook al te denken --een soort basisniveau-- maar wist het even niet onder woorden te brengen, dat is jou wel gelukt

[RobIII]

Orion84 schreef op maandag 23 augustus 2010 @ 10:46:
Die else is niet overbodig, als je die functie per ongeluk aanroept met Rotterdam, dan geeft ie gewoon weer Rotterdam terug, in plaats van Den Haag, dat is in mijn ogen nog wel nuttig te noemen.

PHP:

<?php
// Den_Haag naar Den Haag omzetten
function ReplaceDenHaag($plaats)
{
    if ($plaats == "Den_Haag")
        return "Den Haag";
    return $plaats;
}
?>

Los van het feit dat de functie nogal kansloos is; als je een if-zonder-else wilt gebruiken is dit de way to go IMHO.

[ Voor 34% gewijzigd door RobIII op 23-08-2010 21:00 ]

[Ulster Seedling]

RobIII schreef op maandag 23 augustus 2010 @ 20:59:
[...]

PHP:

<?php
// Den_Haag naar Den Haag omzetten
function ReplaceDenHaag($plaats)
{
    if ($plaats == "Den_Haag")
        return "Den Haag";
    return $plaats;
}
?>

Los van het feit dat de functie nogal kansloos is; als je een if-zonder-else wilt gebruiken is dit de way to go IMHO.

Of zo:

PHP:

<?php
// Den_Haag naar Den Haag omzetten
function ReplaceDenHaag($plaats)
{
    return ($plaats == "Den_Haag") ? "Den Haag" : $plaats;
}
?>

[ZaZ]

Ja met een ternary operator is gewoon een if statement. Zo kunnen we er nog wel 10 verzinnen denk ik. We kunnen het ook omdraaien natuurlijk

PHP:

if ($plaats != "Den_Haag") 
        return $plaats;
    return "Den Haag";

En dan nog alle andere mogelijke combinaties

[Freeaqingme]

Ik snapte niet waarom als ik in method A iets weghaalde, function B niet meer werkte, code is van ongeveer een jaar geleden. Zojuist extraheer ik daar het volgende uit:

PHP:

<?php
function B($options = array()) {
    $out = $this->A($options);
    $options = $out['options'];

    // rest van method hier
}

function A($options) {
    foreach($options as $key => $value) {
        // Hier 25 regels code

        $options[$key] = $value; // Deze lijn veranderde ik, en gek genoeg werkte function B niet meer

        // Hier nog eens 25 regels
    }

    return ('options' => $options, 'out' => $out);
}

[ Voor 1% gewijzigd door Freeaqingme op 24-08-2010 01:49 . Reden: Kleurtjes toegeovegd aan code ]

[Caelorum]

freakingme schreef op dinsdag 24 augustus 2010 @ 01:47:
Ik snapte niet waarom als ik in method A iets weghaalde, function B niet meer werkte, code is van ongeveer een jaar geleden. Zojuist extraheer ik daar het volgende uit: [...]

^^ Met mijn relatief beperkte ervaring met php begrijp ik er niks van. Iemand zin om het uit te leggen? Lijkt me alsof iemand niet weet hoe die met verwijzingen kan werken, maar verder kom ik niet.

[TJHeuvel]

Misschien omdat functie B afhankelijk is van functie A? Die $out zal wel geen options key gehad hebben.

[!null]

Ik zie ook niet meteen wat er mis gaat, maar het moge duidelijk zijn dat er niet al te intelligente keuzes zijn gemaakt omtrent de naamgeving
Zo heb je snel bugs te pakken, of onbegrip van jezelf of anderen die aan deze code moeten werken.

[basvd]

PHP:

foreach($options as $key => $value) { 
   $options[$key] = $value; // Deze lijn veranderde ik, en gek genoeg werkte function B niet meer 
}

Dat is toch logisch, dan overschrijf je de initiële waardes van de array waar je doorheen loopt?!?

PHP:

return ('options' => $options, 'out' => $out);

Dit lijkt mij trouwens ook niet te werken...

[ Voor 6% gewijzigd door basvd op 24-08-2010 09:07 ]

[YopY]

PHP:

/**
 * @param $plaats - Must be "Den_Haag". If nothing is passed. "Den_Haag" will be used.
 * @return "Den Haag". 
 */
function replaceDenHaag($plaats = "Den_Haag") {
   return "Den Haag";
}

Design-by-contract, . De aanroeper controleert zelf maar of 'ie Den_Haag of Pietje_Puk naar de functie stuurt.

[MBV]

SPee schreef op maandag 23 augustus 2010 @ 18:04:
En je kunt ook doorslaan en alles zo uitbreidbaar mogelijk maken.
[knip]

Om er dan achter te komen dat je je eigen RDBMS hebt geschreven "Kijk, je hoeft geen letter meer in PHP te programmeren!" Dat is dan volledig vervangen door je eigen taal die verstopt zit in de database ofzo. Iets te vaak gezien. Maak het liever 100% niet-uitbreidbaar en niet-flexibel, met een goede architectuur, dan 100% uitbreidbaar.

[Alex)]

En dan krijg je dus deze gedrochten?

[boe2]

YopY schreef op dinsdag 24 augustus 2010 @ 09:42:

PHP:

/**
 * @param $plaats - Must be "Den_Haag". If nothing is passed. "Den_Haag" will be used.
 * @return "Den Haag". 
 */
function replaceDenHaag($plaats = "Den_Haag") {
   return "Den Haag";
}

Design-by-contract, . De aanroeper controleert zelf maar of 'ie Den_Haag of Pietje_Puk naar de functie stuurt.

We kunnen nog een stap verder optimaliseren in dat geval

PHP:

<?php 
function replaceDenHaag() { 
   return "Den Haag"; 
} 
?>

[YopY]

Boeboe schreef op dinsdag 24 augustus 2010 @ 10:32:
[...]


We kunnen nog een stap verder optimaliseren in dat geval

PHP:

<?php 
function replaceDenHaag() { 
   return "Den Haag"; 
} 
?>

Je moet er nog wel een verhaal bijdoen over self-documenting code hé? En dan kan ik zeggen "Je functienaam is incorrect". Je zou het dan een naam "returnDenHaag()" of "giveMeTheStringThatSaysDenHaagWithASpaceInBetween()" moeten geven.

...waarom bestond deze functie ook maar weer?

[!null]

YopY schreef op dinsdag 24 augustus 2010 @ 09:42:

PHP:

/**
 * @param $plaats - Must be "Den_Haag". If nothing is passed. "Den_Haag" will be used.
 * @return "Den Haag". 
 */
function replaceDenHaag($plaats = "Den_Haag") {
   return "Den Haag";
}

Design-by-contract, . De aanroeper controleert zelf maar of 'ie Den_Haag of Pietje_Puk naar de functie stuurt.

Dit vind ik toch wel een van de mooiere, echt geweldig. Waarschijnlijk had de functie ooit nog nut, maar is dat nut eruit gesloopt en de rest blijven staan

[CodeCaster]

Alle Den Haaglol ten spijt kwam ik vanmorgen weer een pareltje tegen. Ik ben pertinent tegen geneste if's. Het lijkt wel of men hier alleen positieve waarden aan een if wil geven, en "exit is een soort goto dus kun je beter niet gebruiken". Ik heb dus if's van een keer of tien diep gezien...

Dat levert dus zulke code op:

Delphi:

procedure BepaaldeProcedure;
begin
   if (someQuery.recordcount > 0) then
   begin
      if (testBestandsnaamExtensie) then
      begin
         try
          // doe wat leuks met een zootje variabelen
         except
            on E: exception do
            begin
               // Log error, geef melding
            end;
         end;

         try
          // Doe magic-achtige dingen
         except
         end;
      end
      else
      begin
         if (eenVoorwaarde) then
         begin
            // Voer wat magic uit;
         end
         else
         begin
            andereVoorwaarde := true;
            // Verander wat anders
         end;

         if (andereVoorwaarde) then
         begin

            if (specifiekeVoorwaarde) then
            begin
              // Doe wat extra voorbereidend werk
            end;

            if (weerAndereVoorwaarde) then
            begin
              // Doe de uiteindelijke magic
            end
            else
            begin
              // Doe nog wat leuks wat op magic lijkt
            end;
         end
         else
         begin
            // Toon error dat variabele niet Ok is
         end;
      end;
   end
   else
   begin
      // Toon melding dat er geen records zijn
   end;
end;

En daar heb ik meer dan 100 regels code weggeknipt. Totaal niet te volgen, hoewel dat ook aan gebrek aan functionaliteit van de IDE kan liggen.

Ik zou het zo doen, quick & dirty:

Delphi:

procedure BepaaldeProcedure;
begin
  if (someQuery.recordcount = 0) then
  begin
   // Toon melding dat er geen records zijn
   exit;
  end;

  if (testBestandsnaamExtensie) then
  begin
     try
      // doe wat leuks met een zootje variabelen
     except
        on E: exception do
        begin
           // Log error, geef melding
        end;
     end;

     try
        // Doe magic-achtige dingen
     except
     end;
     exit;
  end
  
  // Dit wordt alleen bij andere extensies gedaan
  if (eenVoorwaarde) then
  begin
    // Voer wat magic uit;
    // Toon error dat variabele niet Ok is
    exit;
  end;
  
  // Verander wat anders
  
  if (specifiekeVoorwaarde) then
  begin
  // Doe wat extra voorbereidend werk
  end;

  if (weerAndereVoorwaarde) then
  begin
    // Doe de uiteindelijke magic
  end
  else
  begin
    // Doe nog wat leuks wat op magic lijkt
  end;
end;

Iemand?

[MBV]

Alex) schreef op dinsdag 24 augustus 2010 @ 10:17:
En dan krijg je dus deze gedrochten?

Mooi voorbeeld van het Inner Platform Effect.
Ik dacht meer aan Soft Coding

Ik kan alleen dat plaatje van de grafische rule-engine niet meer vinden

[ Voor 8% gewijzigd door MBV op 24-08-2010 12:51 ]

[ajakkes]

code:

$res = mysql_query("SELECT username FROM loginusers WHERE username='".$username."' AND username != '".$username."'");

Om te kijken of de gebruikersnaam al bestaat.

[CodeCaster]

Zet er dan meteen neer "WHERE False = 'FileNotFound'"

[.oisyn]

Da's een quantum mechanische database. Er wordt gezocht naar een rij die in de superpositie staat waarbij de kolom username zowel wel als niet overeenkomt met $username. Lijkt me geen slecht programmeervoorbeeld.

[ Voor 11% gewijzigd door .oisyn op 25-08-2010 16:48 ]

[Woy]

.oisyn schreef op woensdag 25 augustus 2010 @ 16:48:
Da's een quantum mechanische database. Er wordt gezocht naar een rij die in de superpositie staat waarbij de kolom username zowel wel als niet overeenkomt met $username. Lijkt me geen slecht programmeervoorbeeld.

Alleen loop je dan meteen tegen het probleem aan dat je de staat van de database veranderd door hem te observeren

[Sebazzz]

Woy schreef op woensdag 25 augustus 2010 @ 16:55:
[...]

Alleen loop je dan meteen tegen het probleem aan dat je de staat van de database veranderd door hem te observeren

Dus MySQL is een quantummechanische database?

[Lye]

Sebazzz schreef op woensdag 25 augustus 2010 @ 17:03:
[...]

Dus MySQL is een quantummechanische database?

Af en toe, als de stand van de zon, maan en venus zo staan dat de qm-bit van mysql op 2 staat

[Vinnienerd]

Lye schreef op woensdag 25 augustus 2010 @ 20:16:
[...]


Af en toe, als de stand van de zon, maan en venus zo staan dat de qm-bit van mysql op 2 staat

Best handig, met quantum entanglement hoef je replication niet meer aan te zetten. Je hebt zelfs geen netwerkverbinding meer nodig

[.oisyn]

Het jammere is alleen dat als je het ene entanglede deeltje uitleest dat het andere dan ook z'n superpositie verliest

[ajakkes]

Ik heb nu al een tijdje geprobeerd om de qm-bit zelf op twee te krijgen maar dat lukt niet. Kan iemand hier mee helpen of moet ik hier een nieuw topic voor openen.
Ik zal toch moeten testen of de functie wel werkt als de zon, de maan en venus in de goede stand staan.

[Hydra]

ajakkes schreef op donderdag 26 augustus 2010 @ 07:30:
Ik heb nu al een tijdje geprobeerd om de qm-bit zelf op twee te krijgen maar dat lukt niet. Kan iemand hier mee helpen of moet ik hier een nieuw topic voor openen.
Ik zal toch moeten testen of de functie wel werkt als de zon, de maan en venus in de goede stand staan.

Het is makkelijker als je in de buurt van de LHC woont.

[R3veNG]

Hydra schreef op donderdag 26 augustus 2010 @ 14:20:
[...]


Het is makkelijker als je in de buurt van de LHC woont.

Vergeet dan meteen de vogel-laat-stokbrood-vallen routine niet te unittesten..

[Woy]

Laten we nu verder maar weer on-topic gaan

[BarôZZa]

Ik kwam net deze tegen (beetje aangepast). getNumberOfDays haalt het aantal dagen van die maand op.

JavaScript:

  function bla(){
    LastMonthYear='201107';
    var month = parseInt(LastMonthYear.substr(4,2),10);
    var year = parseInt(LastMonthYear.substr(0,4),10);

    var noOfDays = getNumberOfDays(month, year);
    var lastDate = '' + LastMonthYear + setLengthTwo(noOfDays);
  }

  //converts days and months to two characters (e.g. 8 -> 08)
  function setLengthTwo(string)
  {
    var temp = ''+string;
    if (temp.length >= 2) {return temp;}
    else if (temp.length < 2) {temp = '0'+temp; return temp;}
  }

en nog eentje:

JavaScript:

amount = Math.round(amount+0.49);  //round up to integer

Werkt allemaal prima, daar niet van . Behalve als amount 0.001 zou zijn, maar dat is gelukkig nooit het geval.

[ Voor 14% gewijzigd door BarôZZa op 31-08-2010 04:37 ]

[TJHeuvel]

code:

var lastDate = '' + LastMonthYear + setLengthTwo(noOfDays);

Hehe, string casting

[SysRq]

CodeCaster schreef op dinsdag 24 augustus 2010 @ 12:26:
Alle Den Haaglol ten spijt kwam ik vanmorgen weer een pareltje tegen. Ik ben pertinent tegen geneste if's. Het lijkt wel of men hier alleen positieve waarden aan een if wil geven, en "exit is een soort goto dus kun je beter niet gebruiken". Ik heb dus if's van een keer of tien diep gezien...

Dat levert dus zulke code op:
[...]
En daar heb ik meer dan 100 regels code weggeknipt. Totaal niet te volgen, hoewel dat ook aan gebrek aan functionaliteit van de IDE kan liggen.

Ik zou het zo doen, quick & dirty:

Delphi:

[...]

     try
        // Doe magic-achtige dingen
     except
     end;
     exit;
  end
[...]

Iemand?

Een try-except-end zonder code in de except?? Iew ..

Als je het al doet, dan met een heleboel commentaar waarom het _echt_ noodzakelijk is..

[Laurens-R]

Zoiets kan gewoon haast niet nodig zijn, omdat je dan feitelijk logic-by-exception (ik verzin hem terplekke; excuus als hier een officielere term voor bestaat) aan het implementeren bent. Exceptions zijn tenslotte exceptional. Exceptions zijn wat mij betreft een uiterst redmiddel.

[CodeCaster]

Ik heb het stuk code niet meer, geen idee ook meer waar het vandaan kwam (nogal veel code voorbij zien komen de afgelopen week), maar volgens mij was het een parse van een string naar een int, waarbij de int eerst op 0 was gezet. Dat kan natuurlijk ook in de except, maar ik vond het de moeite niet waard om te verplaatsen.

[Killemov]

CodeCaster schreef op dinsdag 24 augustus 2010 @ 12:26:
Ik ben pertinent tegen geneste if's. Het lijkt wel of men hier alleen positieve waarden aan een if wil geven, en "exit is een soort goto dus kun je beter niet gebruiken". Ik heb dus if's van een keer of tien diep gezien...

In Java-land lopen ook wel puristen rond die vinden dat je een return in een method alleen aan het eind moet zetten. Vervolgens is de opvatting als je meer dan 3 control-structures moet nesten dat je dan beter een stuk code moet refactoren.

De regel-van-de-duim voor mij is: "Minder code is beter." Dat geldt ook voor het "oppervlak" dat een stuk code inneemt.

[Janoz]

Mijn 'duimregel' is voornamelijk "minder code in een methode is beter".

Nikss maakt je code leesbaarder wanneer je een deel van je code extract naar een aparte private method, mits je een duidelijke naam voor die methode kiest. Mensen die vinden dat er een maximum lengte is voor methodenamen moeten daarentegen standrechtelijk worden geëxecuteerd.

[BarôZZa]

CyCloneNL schreef op dinsdag 31 augustus 2010 @ 09:48:
[...]


Hehe, string casting

Yes, maar de grap zat hem vooral in het feit dat setLengthTwo(noOfDays) aangeroepen wordt. Ik ben nog geen maand van minder dan 10 dagen tegengekomen

[Pathogen]

BarôZZa schreef op dinsdag 31 augustus 2010 @ 13:23:
[...]

Yes, maar de grap zat hem vooral in het feit dat setLengthTwo(noOfDays) aangeroepen wordt. Ik ben nog geen maand van minder dan 10 dagen tegengekomen

Nee, maar wel de dagen 1 t/m 9. Nou is dat natuurlijk prima op te lossen met een display maskje ergens, maar goed.

[BarôZZa]

Er wordt een functie aangeroepen die de lengte van de maand geeft, zoals ik al zei.
Dagen 1 tm 9 zijn irrelevant.

[SysRq]

Laurens-R schreef op dinsdag 31 augustus 2010 @ 11:09:
Zoiets kan gewoon haast niet nodig zijn, omdat je dan feitelijk logic-by-exception (ik verzin hem terplekke; excuus als hier een officielere term voor bestaat) aan het implementeren bent. Exceptions zijn tenslotte exceptional. Exceptions zijn wat mij betreft een uiterst redmiddel.

Ik heb het een enkele keer wel eens gehad op het moment dat er een onjuiste exceptie gegeven werd in third-party code, waarvan wij de broncode niet hadden.

CodeCaster schreef op dinsdag 31 augustus 2010 @ 11:14:
Ik heb het stuk code niet meer, geen idee ook meer waar het vandaan kwam (nogal veel code voorbij zien komen de afgelopen week), maar volgens mij was het een parse van een string naar een int, waarbij de int eerst op 0 was gezet. Dat kan natuurlijk ook in de except, maar ik vond het de moeite niet waard om te verplaatsen.

Psst: StrToIntDef()

[Janoz]

BarôZZa schreef op dinsdag 31 augustus 2010 @ 13:35:
Er wordt een functie aangeroepen die de lengte van de maand geeft, zoals ik al zei.
Dagen 1 tm 9 zijn irrelevant.

mwah.. De grootste WTF in die code vind ik meer dat eerst de 0 er af gesloopt wordt door de parseInt om hem gelijk daarna weer toe te voegen, maar dat kan ook bij het aanpassen van het voorbeeld ontstaan zijn.

Laurens-R schreef op dinsdag 31 augustus 2010 @ 11:09:
Exceptions zijn wat mij betreft een uiterst redmiddel.

Er is een heel groot grijs gebied vanaf 'afwijken van de happyflow' tot aan 'computercrash'.

Exceptions zijn een ideale manier om aanroepende code op de hoogte te brengen van een probleem wat lokaal niet afgehandeld kan worden.

[ Voor 31% gewijzigd door Janoz op 31-08-2010 13:53 ]

[BarôZZa]

Janoz schreef op dinsdag 31 augustus 2010 @ 13:49:
[...]

mwah.. De grootste WTF in die code vind ik meer dat eerst de 0 er af gesloopt wordt door de parseInt om hem gelijk daarna weer toe te voegen, maar dat kan ook bij het aanpassen van het voorbeeld ontstaan zijn.

Er wordt een functie getNumberOfDays aangeroepen met die integers, daar is dat voor.

Voor de rest een hoop copy pasta. Is trouwens van een zeer groot bedrijf.

[G70boX]

Nu het toch over datums gaat....

PHP:

for($i = 1; $i <= 31; $i++)
{
    $month = date("m", strtotime($values['startDate']));
    $year = date("Y", strtotime($values['startDate']));
    $date = $year.$month.$i;

    if ( ($date >= $values['startDate']) && ($date <= $values['endDate']) )
    {
       $rd = substr($date,0,4)."-".substr($date,4,2)."-".substr($date,6,2)."T00:00:00";                       
       doeIets();
    }
}

Versimpeld voorbeeld van een systeem dat ik moet onderhouden en het staat vol met dit soort dingen .

[Janoz]

Ah, ik kijk krom. Is het 1 of andere functie die de datum in juliaanse notatie omzet oid?

[YopY]

G70boX schreef op dinsdag 31 augustus 2010 @ 15:44:
Nu het toch over datums gaat....

PHP:

// hax

Versimpeld voorbeeld van een systeem dat ik moet onderhouden en het staat vol met dit soort dingen .

Dat, en ook meer van dat soort dingen, lijkt me echt een gevalletje gebrek aan bekendheid (of beschikbaarheid) met datum-API's, standaard (binnen PHP of JS zelf) of als externe bibliotheek. NIH syndroom, maar ook wel The Developer's Dilemma (alhoewel het in dezen een beginnende variant daarvan lijkt te zijn).

(Samenvatting voor degenen die niet op TDWTF durven kijken: programmeurs hebben vaak een hekel aan saaie code en willen graag (te) complexe dingen maken (ofwel: de bestaansreden van TDWTF))

(Edit: URL fixxed, TDWTF moet geen uitroeptekens of komma's in zijn URLs gebruiken. Thanks nescafe!)

[ Voor 5% gewijzigd door YopY op 31-08-2010 16:51 ]

[Salandur]

G70boX schreef op dinsdag 31 augustus 2010 @ 15:44:
Nu het toch over datums gaat....

PHP:

for($i = 1; $i <= 31; $i++)
{
    $month = date("m", strtotime($values['startDate']));
    $year = date("Y", strtotime($values['startDate']));
    $date = $year.$month.$i;

    if ( ($date >= $values['startDate']) && ($date <= $values['endDate']) )
    {
       $rd = substr($date,0,4)."-".substr($date,4,2)."-".substr($date,6,2)."T00:00:00";                       
       doeIets();
    }
}

Versimpeld voorbeeld van een systeem dat ik moet onderhouden en het staat vol met dit soort dingen .

Jaja, wat gebeurd er als de start & enddate in 2 verschillende maanden liggen...

[Grijze Vos]

@Yopy, je linkje is stuk:

http://thedailywtf.com/Ar...-Least,-It-Ought-To-.aspx

[YopY]

Laurens-R schreef op dinsdag 31 augustus 2010 @ 11:09:
Zoiets kan gewoon haast niet nodig zijn, omdat je dan feitelijk logic-by-exception (ik verzin hem terplekke; excuus als hier een officielere term voor bestaat) aan het implementeren bent. Exceptions zijn tenslotte exceptional. Exceptions zijn wat mij betreft een uiterst redmiddel.

Ja, alhoewel het, tenminste in Java en tenminste volgens bepaalde bronnen die er vandaag dingen over ge'blogd' hebben qua performance niet eens zoveel uit maakt.

Maar als het niet zoveel uit maakt moet je gewoon programmeren zodat het logisch is.

(hij geeft overigens ook wel een elegante oplossing voor het 'if null return X' ding - maak een nullobject die X teruggeeft. Als programmeur moet je er dan natuurlijk wel voor zorgen dat je nooit en te nimmer dingen op null initialiseert of null teruggeeft - een methode die ik tegenwoordig zoveel mogelijk probeer toe te passen, ook al programmeer ik niet zoveel)

[Verwijderd]

Ik wil zo weinig mogelijk van mezelf kunnen posten, maar dit is wel heel erg... Bij een hickup van de database server, werd (tot 5 min. geleden) de db username en password weergeven in de PDO Exception.

Heb er maar 1 woord voor:
Epic fail..

[Michali]

Is dat een fout van jezelf of van PDO? Waarom gebeurde dat dan en hoe heb je het opgelost?

[YopY]

Verwijderd schreef op vrijdag 03 september 2010 @ 03:11:
Ik wil zo weinig mogelijk van mezelf kunnen posten, maar dit is wel heel erg... Bij een hickup van de database server, werd (tot 5 min. geleden) de db username en password weergeven in de PDO Exception.

Heb er maar 1 woord voor:
Epic fail..

Dat zijn typisch (wigwams) van die dingen die een developer doet tijdens development, en dan vooral als hij nog niet helemaal goed weet wat er gebeurt, maar die vergeten worden en gewoon zo gelaten worden.

[Verwijderd]

Michali schreef op vrijdag 03 september 2010 @ 09:25:
Is dat een fout van jezelf of van PDO? Waarom gebeurde dat dan en hoe heb je het opgelost?

Van mezelf, doordat hij geen verbinding kon maken met de database gooide hij een Exception die ik niet gevangen heb , en daar staat dan dus gebruikersnaam en wachtwoord als plaintext in. Nu wordt het wel afgevangen en gelogd.. En wachtwoord veranderd voor zekerheid.

[Janoz]

Nou, dan is de grootste WTF dus echt niet van jezelf. Degene die de credentials in de message zet is degene die gekielhaald moet worden.

[Laurens-R]

* zucht * ik wil niet vervelend zijn, maar soms vraag je je af wat voor hobbyisten er in het bedrijfsleven rondlopen als het aankomt op programmeren.

Ik kwam zojuist gewoon een functie tegen die een willekeurige folder van je harde schijf inleest. Ik kreeg alleen een jeukend gevoel toen ik las in de specificatie dat hij niet verder dan 3 niveau's diep in de onderliggende folderstructuur kon 'scannen'. Kijk ik in de code; kom ik het volgende tegen... (in psuedocode):

code:

folderCount = GetSubFolderCount()
if folderCount > 0 then
    foreach (subfolder in Folders)
        //scancode hier...
        folderCount = GetSubFolderCount()
        if folderCount > 0 then
            foreach (subfolder in Folders)
            //scancode hier...
                folderCount = GetSubFolderCount()
                if folderCount > 0 then
                    foreach (subfolder in Folders)
...etc

you get the picture....

... recursie is voor velen blijkbaar een moeilijk concept, maar ik denk niet dat je dan in een bedrijf software moet gaan ontwikkelen

[ Voor 6% gewijzigd door Laurens-R op 03-09-2010 10:06 ]

[Haan]

Dat is wel echt om te janken ja

[Verwijderd]

@Laurens-R: Hoe heb je het opgelost, de loop nog 10x gekopieerd? 13 niveau's zal wel genoeg zijn, zo niet gewoon nog iets vaker kopiëren.

[Laurens-R]

Nadat ik met een aantal collega's heb staan huilen vanwege al het codeleed in de betreffende applicatie (dit was niet de enige WTF), heb ik de functie maar even herschreven ... in 8 regels code (was 72).

... ander leuk voorbeeldje in diezelfde app was de import functionaliteit van data in de SQL Server database. Dat ging niet echt op een manier van regel voor regel inlezen en een stored procedure gebruiken... zelfs een INSERT statement was al te veel gevraagd... Want wat doe je als je in 1 keer data wilt dumpen in een tabel... (houd je vast):

- start een batch script.
- remove all constraints van een tabel (anders gaat het import proces natuurlijk 'zeuren')
- plemp de data direct in die tabel via een command line tool
- alle constraints weer aanzetten
- hoop dat data voldoet aan de constraints.
- start de hoofdapplicatie
- start met bidden.

[CodeCaster]

Janoz schreef op vrijdag 03 september 2010 @ 09:38:
Nou, dan is de grootste WTF dus echt niet van jezelf. Degene die de credentials in de message zet is degene die gekielhaald moet worden.

Niet in de message, in de stack trace.

code:

exception 'PDOException' with message 'SQLSTATE[HY000] [2005] Unknown MySQL server host 'pietjepuk' (1)' in /var/www/test/Database.php:21
Stack trace:
#0 /var/www/test/Database.php(21): PDO->__construct('mysql:dbname=te...', 'user', 'password')
#1 /var/www/test/index.php(52): Database->Connect()
#2 {main}

Ik weet niet of ik dat een WTF moet vinden, lijkt me eerder by design. Je moet de stacktrace gewoon nooit buiten debug laten outputten.

[Noxious]

Verwijderd schreef op vrijdag 03 september 2010 @ 09:31:
[...]

Van mezelf, doordat hij geen verbinding kon maken met de database gooide hij een Exception die ik niet gevangen heb , en daar staat dan dus gebruikersnaam en wachtwoord als plaintext in. Nu wordt het wel afgevangen en gelogd.. En wachtwoord veranderd voor zekerheid.

Ach... Mijn werkgever vind security maar tijdverspilling en zowat alles hier heeft generieke passwords, geen passwords, guest accounts of default passwords...

Ja ook de databases van het kloksysteem, urenregistratie, hrm-pakket en moar

[Alex)]

CodeCaster schreef op vrijdag 03 september 2010 @ 10:40:
[...]

Niet in de message, in de stack trace.

code:

exception 'PDOException' with message 'SQLSTATE[HY000] [2005] Unknown MySQL server host 'pietjepuk' (1)' in /var/www/test/Database.php:21
Stack trace:
#0 /var/www/test/Database.php(21): PDO->__construct('mysql:dbname=te...', 'user', 'password')
#1 /var/www/test/index.php(52): Database->Connect()
#2 {main}

Ik weet niet of ik dat een WTF moet vinden, lijkt me eerder by design. Je moet de stacktrace gewoon nooit buiten debug laten outputten.

Je moet niet de waardes van de variabelen laten outputten omdat je nooit weet wie die dingen leest. C# toont je het type en de naam van de parameter, maar niet de waarde.