[alg] Slechtste programmeervoorbeelden deel 4

[Zsub]

Mwah, inderdaad even een check voor > 28, maar die had ik niet mee willen copypasten

Verder denk ik dat degene die dit geschreven heeft niet wist dat je prima even op die manier een string kan bouwen.

[Alex)]

MSDN: SPUtility.TestVss Method (Microsoft.SharePoint.Utilities) Prachtige documentatie van deze geweldig nuttige functie.

[Matis]

Jammer dat er niet staat: Returns true

[Alex)]

Matis schreef op woensdag 11 mei 2011 @ 13:44:
Jammer dat er niet staat: Returns true

Hij returnt ook gewoon true... even met Reflector bekeken:

C#:

public static bool TestVss()
{
    object[,] components = SPWriterServiceInstance.OnIdentify();
    SPWriterServiceInstance.OnPreBackup(components);
    SPWriterServiceInstance.OnPostRestore(components, true);
    return true;
}

Het ding lijkt iets van doen te hebben met de Volume Shadow Service (schaduwkopie)... maar wat het nou echt doet? Geen idee. Internet weet me verder ook bijna niks te vertellen over deze method.

[.Gertjan.]

Alex) schreef op woensdag 11 mei 2011 @ 13:39:
MSDN: SPUtility.TestVss Method (Microsoft.SharePoint.Utilities) Prachtige documentatie van deze geweldig nuttige functie.

Dat soort pagina's is bijna standaard voor X++/Axapta. Alle entiteiten, functies en datastructuren staan in MSDN, maar de meeste pagina's bevatten alleen een overzicht van de hiërarchie en een overzicht van de functies die erin zitten. Informatie wat iets dan doet ontbreek meestal volledig

[Haan]

Alex) schreef op woensdag 11 mei 2011 @ 13:54:
[...]

Hij returnt ook gewoon true... even met Reflector bekeken:

C#:

public static bool TestVss()
{
    object[,] components = SPWriterServiceInstance.OnIdentify();
    SPWriterServiceInstance.OnPreBackup(components);
    SPWriterServiceInstance.OnPostRestore(components, true);
    return true;
}

Het ding lijkt iets van doen te hebben met de Volume Shadow Service (schaduwkopie)... maar wat het nou echt doet? Geen idee. Internet weet me verder ook bijna niks te vertellen over deze method.

Is het niet (zoals de naam doet vermoeden) een test functie die ze vergeten zijn weg te halen?

[Manuel]

BM schreef op woensdag 11 mei 2011 @ 13:18:
Dat is niet helemaal hetzelfde? Voor dagen >28 word in je 1e voorbeeld 28.png gebruikt, in je 2e voorbeeld probeerd ie er een 29/30/31.png van te maken, toch?

Dan krijg je iets als:

C:

bier1.image = [UIImage imageNamed:[NSString stringWithFormat:@"%i.png", (days > 28 ? 28 : days)]];

offtopic:
Note: Geen ervaring met Objective-C, maar het zal wel mogelijk zijn als ik Wikipedia zo bekijk.

[Alex)]

Haan schreef op woensdag 11 mei 2011 @ 14:26:
[...]

Is het niet (zoals de naam doet vermoeden) een test functie die ze vergeten zijn weg te halen?

Zou kunnen, ik heb geen flauw idee. In SP2010 is hij wel gemarkeerd als Obsolete...

[PiepPiep]

Zsub schreef op woensdag 11 mei 2011 @ 13:13:
Klein stukje Objective C:

C:

if (days == 1){
        bier1.image = [UIImage imageNamed:@"1.png"];
    }
    if (days == 2){
        bier1.image = [UIImage imageNamed:@"2.png"];
    }
*knip*
    if (days == 27){
        bier1.image = [UIImage imageNamed:@"27.png"];
    }
    if (days == 28){
        bier1.image = [UIImage imageNamed:@"28.png"];
    }
    if (days > 28){
        bier1.image = [UIImage imageNamed:@"28.png"];
    }

Oplossing:

C:

bier1.image = [UIImage imageNamed:[NSString stringWithFormat:@"%i.png",days]];

Ik vraag me dat altijd af of die code met ctrl-c ctrl-v en het nummertje aanpassen gemaakt is of dat het echt elke keer opnieuw ingetikt is.

[Rutix]

PiepPiep schreef op donderdag 12 mei 2011 @ 12:22:
[...]

Ik vraag me dat altijd af of die code met ctrl-c ctrl-v en het nummertje aanpassen gemaakt is of dat het echt elke keer opnieuw ingetikt is.

Misschien is het wel gewoon gegenereerd Kan makkelijk als je een goede IDE met plugin hebt

[RayNbow]

Hier is (een reconstructie van) wat brakke code dat ik vanochtend geproduceerd heb:

Python:

for i in xrange(len(a)):
    a[i] *= 100.0
    a[i] /= a[-1]

En maar verbaasd staan kijken waarom het laatste element geen 100.0 was...

[PrisonerOfPain]

RayNbow schreef op donderdag 12 mei 2011 @ 18:18:
Hier is (een reconstructie van) wat brakke code dat ik vanochtend geproduceerd heb:

Python:

for i in xrange(len(a)):
    a[i] *= 100.0
    a[i] /= a[-1]

En maar verbaasd staan kijken waarom het laatste element geen 100.0 was...

Je mist een accolade

[Ryur]

PrisonerOfPain schreef op donderdag 12 mei 2011 @ 18:21:
[...]


Je mist een accolade

2 zelfs ^^
En 2x een puntcomma

[ Voor 5% gewijzigd door Ryur op 12-05-2011 18:31 ]

[RayNbow]

PrisonerOfPain schreef op donderdag 12 mei 2011 @ 18:21:
[...]


Je mist een accolade

GoTCoast schreef op donderdag 12 mei 2011 @ 18:30:
[...]

2 zelfs ^^
En 2x een puntcomma

Fixed:

Python:

for i in xrange(len(a)): #{
    a[i] *= 100.0 #;
    a[i] /= a[-1] #;
#}

[Matis]

Vandaag ook een beetje met python zitten prutsen.
Ik type achter iedere regel automagisch ; en dan vreemd opkijken waarom het niet werkt

[RayNbow]

Matis schreef op donderdag 12 mei 2011 @ 18:55:
Vandaag ook een beetje met python zitten prutsen.
Ik type achter iedere regel automagisch ; en dan vreemd opkijken waarom het niet werkt

Dat probleem heb ik niet... m'n IDE (PyDev) geeft rode kringeltjes als ik dat probeer.

[eBoR]

PiepPiep schreef op donderdag 12 mei 2011 @ 12:22:
[...]

Ik vraag me dat altijd af of die code met ctrl-c ctrl-v en het nummertje aanpassen gemaakt is of dat het echt elke keer opnieuw ingetikt is.

denk dat je alleen bij (days > 28) een probleem je hebt met jou functie.. of zie ik dat verkeerd?

[TweakBoy]

misschien is het de intentie om het zo te laten...
er bestaat geen bier dat speciaal voor de dagen 29,30 en 31 gebrouwd is... dus men zal in het ergste geval bier 4 dagen dezelfde soort bier nuttigen (van dag 28)

eBoR schreef op donderdag 12 mei 2011 @ 23:45:
[...]


denk dat je alleen bij (days > 28) een probleem je hebt met jou functie.. of zie ik dat verkeerd?

[Patriot]

eBoR schreef op donderdag 12 mei 2011 @ 23:45:
[...]


denk dat je alleen bij (days > 28) een probleem je hebt met jou functie.. of zie ik dat verkeerd?

Slechtste leesvoorbeeld deel 3245235

[.Gertjan.]

Matis schreef op donderdag 12 mei 2011 @ 18:55:
Vandaag ook een beetje met python zitten prutsen.
Ik type achter iedere regel automagisch ; en dan vreemd opkijken waarom het niet werkt

Heb dat vaak met VB.Net gehad. Daar leer je het verdomd lastig af om dat ding te typen... Voor je gevoel werk je namelijk gewoon in .NET (je functies enzo zijn bekend) en past automatisch de accolades en puntkomma's toe omdat je dat voor C# gewend bent... Heeft me geregeld sloten met errors opgeleverd

En dan ben je eenmaal aan de VB syntax gewend en ga je een plakje javascript schrijven, vergeet je daar spontaan de accolades en puntkomma's...

[Dragor]

Matis schreef op donderdag 12 mei 2011 @ 18:55:
Vandaag ook een beetje met python zitten prutsen.
Ik type achter iedere regel automagisch ; en dan vreemd opkijken waarom het niet werkt

Uhm... python negeert ; aan het einde toch gewoon?
Zolang de indenting en de rest van de syntax goed zijn.

[Wolfboy]

Dragor schreef op vrijdag 13 mei 2011 @ 08:11:
[...]


Uhm... python negeert ; aan het einde toch gewoon?
Zolang de indenting en de rest van de syntax goed zijn.

Het is geen negeren eigenlijk. Je kan gewoon meerdere statements achter elkaar plakken met ;

[MBV]

Javascript is bijna hetzelfde: je kan ; neerzetten als je dat leuk vind, hoeft niet in alle gevallen. Maar in sommige gevallen gaf het wel rare resultaten:

[.Gertjan.]

MBV schreef op vrijdag 13 mei 2011 @ 10:44:
Javascript is bijna hetzelfde: je kan ; neerzetten als je dat leuk vind, hoeft niet in alle gevallen. Maar in sommige gevallen gaf het wel rare resultaten:

Dat is wel bizar.... Zeker als het iet verplicht is, maar wel gekke dingen kan opleveren. Mooie manier om je code te obfuscaten

[BasieP]

MBV schreef op vrijdag 13 mei 2011 @ 10:44:
Javascript is bijna hetzelfde: je kan ; neerzetten als je dat leuk vind, hoeft niet in alle gevallen. Maar in sommige gevallen gaf het wel rare resultaten:

niet helemaal mee eens. bij javascript hoort ; gewoon. Als je het niet doet gaan de meeste browsers er redelijk goed mee om, maar dat wil nog niet zeggen dat het optioneel is...

[MBV]

http://mislav.uniqpath.com/2010/05/semicolons/

En als je het niet gelooft, pagina 25 en 26 van de standaard specificeren het: http://www.ecma-internati...iles/ECMA-ST/ECMA-262.pdf

Niet dat het leesbaarder wordt zonder ;, maar het het mag wel

[ Voor 14% gewijzigd door MBV op 13-05-2011 11:47 ]

[Makkelijk]

MBV schreef op vrijdag 13 mei 2011 @ 11:46:
http://mislav.uniqpath.com/2010/05/semicolons/

Maar deze meneer gelooft ook niet in minification omdat je ook gzip hebt dus ik vind het een slechte bron

[EddoH]

BasieP schreef op vrijdag 13 mei 2011 @ 11:41:
[...]

niet helemaal mee eens. bij javascript hoort ; gewoon. Als je het niet doet gaan de meeste browsers er redelijk goed mee om, maar dat wil nog niet zeggen dat het optioneel is...

http://aresemicolonsnecessaryinjavascript.com/

[.oisyn]

BasieP schreef op vrijdag 13 mei 2011 @ 11:41:
[...]

niet helemaal mee eens. bij javascript hoort ; gewoon. Als je het niet doet gaan de meeste browsers er redelijk goed mee om, maar dat wil nog niet zeggen dat het optioneel is...

Nonsens, newlines als end-of-statement delimiters zijn gewoon onderdeel van de taal.

.edit: euh ja anders lees ik de overige reacties ook eerst even. Maar ja, als ik zo'n onzin-statement zie moet ik gewoon reageren

[ Voor 17% gewijzigd door .oisyn op 13-05-2011 14:13 ]

[EddoH]

Je drang om z.s.m mensen te informeren is groot, een nobel streven

[Davio]

.oisyn schreef op vrijdag 13 mei 2011 @ 14:12:
[...]

Nonsens, newlines als end-of-statement delimiters zijn gewoon onderdeel van de taal.

.edit: euh ja anders lees ik de overige reacties ook eerst even. Maar ja, als ik zo'n onzin-statement zie moet ik gewoon reageren

Ach, bij de meeste programmeertalen heb je vaak een keuze tussen impliciet en expliciet programmeren, net wat je zelf wilt.

Je kunt altijd van een bepaalde default-werking uitgaan en je code zoveel mogelijk strippen van overbodige qualifiers, maar ik neig toch altijd naar de expliciete kant zodat ik in één oogopslag kan zien wat de bedoeling is en ; hoort daar wat mij betreft bij.

[raptorix]

EddoH schreef op vrijdag 13 mei 2011 @ 13:45:
[...]


http://aresemicolonsnecessaryinjavascript.com/

Voor high performance sites gaat het niet alleen om het aantal KB's maar vooral om het aantal requests

[EddoH]

raptorix schreef op vrijdag 13 mei 2011 @ 14:34:
[...]

Voor high performance sites gaat het niet alleen om het aantal KB's maar vooral om het aantal requests

uuh...so? Site gaat niet (alleen) over website performance, maar over of het neccesary is.

[Matis]

EddoH schreef op vrijdag 13 mei 2011 @ 13:45:
http://aresemicolonsnecessaryinjavascript.com/

tabs en spaties en naming convention ook niet, maar het maakt het soms wel een stuk leesbaarder

[Zsub]

Pfft, ik heb niet genoeg geslapen.

C:

return (NSInteger)INFINITY;

[EddoH]

Zsub schreef op vrijdag 13 mei 2011 @ 14:53:
Pfft, ik heb niet genoeg geslapen.

C:

return (NSInteger)INFINITY;

code:

typedef float NSInteger;

[ Voor 5% gewijzigd door EddoH op 13-05-2011 15:00 ]

[Jaap-Jan]

EddoH schreef op vrijdag 13 mei 2011 @ 14:58:
[...]

code:

typedef float NSInteger;

Syntactisch correct, semantisch niet.

[Sebazzz]

In de categorie "gevolg van slechte programmeervoorbeelden":


Ik kom het nog steeds veel te vaak tegen (o.a. op stagemotor bijvoorbeeld).

[Matis]

Sebazzz schreef op zaterdag 14 mei 2011 @ 01:11:
In de categorie "gevolg van slechte programmeervoorbeelden":
[afbeelding]

Ik kom het nog steeds veel te vaak tegen (o.a. op stagemotor bijvoorbeeld).

Ik snap er ook geen hout van; Zelfde bij de ING, het wachtwoord is niet hoofdlettergevoelig. De gebruikersnaam daarentegen wel

[Afvalzak]

Overactieve datamodelist ofzo ?

[YopY]

[nvm, verkeerde topic]

[ Voor 125% gewijzigd door YopY op 14-05-2011 18:05 ]

[EddoH]

omg... ik moet even een test php site in elkaar flansen. Nou ben ik niet zo bekend met PHP dus ik kijk even op php.net

Staan er dit soort juweeltjes tussen:

PHP:

This is a function to count the number of files of a directory

<?php
function countFiles($dir){
    $files = array();
    $directory = opendir($dir);
    while($item = readdir($directory)){
    // We filter the elements that we don't want to appear ".", ".." and ".svn"
         if(($item != ".") && ($item != "..") && ($item != ".svn") ){
              $files[] = $item;
         }
    }
    $numFiles = count($files);
    return $numFiles;
}
?>

[TJHeuvel]

Over het algemeen vind ik PHP goed gedocumenteerd, maar je moet inderdaad niet de comments lezen.
Maar wat is er nou zo gigantisch fout aan het script? Die $numFiles is inderdaad wat overbodig, en het excluden zou wat minder omslachtig kunnen maar het script werkt en is toch best duidelijk?

[ Voor 4% gewijzigd door TJHeuvel op 25-05-2011 09:57 ]

[EddoH]

- loop stopt als een filename "false" heet
- een countfiles fucntie die voor jou gaat bepalen welke files er wel of niet meegenomen moeten worden (".svn?")
- dir handle wordt neit geclosed
- onnodig gecopy van filenames

Vond er nogal veel wtf's inzitten voor zo'n simpel stukje code...

[TJHeuvel]

- Nee dat doet hij niet, "false" evalueert naar true.
- Ja, hij exclude inderdaad een hidden file. Ik weet het niet zeker, maar zou .svn voorkomen op Linux? Daar worden files die met een punt beginnen gehide toch? Dan zou het een patch zijn om het ook laten werken op Windows.
- Valide punt, dat is niet netjes.
- Het is inderdaad ook mogelijk om gewoon een tellertje bij te houden, al zou het verschil minimaal zijn.

Kortom, het is inderdaad niet bijzonder netjes, maar het niveau is wel hoger dan het gemiddelde script in de comments

[EddoH]

CyCloneNL schreef op woensdag 25 mei 2011 @ 10:10:
- Nee dat doet hij niet, "false" evalueert naar true.

eehm...que? echt waar? Maar "0" evalueert toch wel naar false?

CyCloneNL schreef op woensdag 25 mei 2011 @ 10:10:
- Ja, hij exclude inderdaad een hidden file. Ik weet het niet zeker, maar zou .svn voorkomen op Linux? Daar worden files die met een punt beginnen gehide toch? Dan zou het een patch zijn om het ook laten werken op Windows.

en andere hidden files dan, die geen "svn" heten?...

Kortom, het is inderdaad niet bijzonder netjes, maar het niveau is wel hoger dan het gemiddelde script in de comments

..wauw, dan kijk ik iig daar niet meer naar

[ Voor 31% gewijzigd door EddoH op 25-05-2011 10:18 ]

[TJHeuvel]

Ja, zoals gedocumenteerd.
Beetje raar inderdaad, maar gelukkig staat het gewoon ergens opgeschreven.

Anders dan bij bijvoorbeeld Javascript, waar je de keuze hebt om de language specificaties door te nemen, waarbij je 3 dagen bezig bent om de inhoudsopgave te lezen of browser-specifieke development pages te gebruiken welke soms informatie bevatten over alleen hun implementatie.

[ Voor 46% gewijzigd door TJHeuvel op 25-05-2011 10:19 ]

[BarôZZa]

Dat is toch een vrij logisch gevolg van het feit dat PHP automagisch numerieke strings naar integers converteert?
"0" -> 0 -> false

Daarnet nog een hele lelijke oplossing gebouwd voor een probleem. Situatie: wordpress icm met spaghetticode, een verschrikkelijke combinatie. Ergens was er een loopje met een array die wat posts moest bevatten. Werkte op alle pagina's, behalve op de indexpagina waar de array altijd leeg bleef. Van alles geprobeerd, maar geen idee waar het aan lag. Dus uiteindelijk ervoor gekozen om de array weg te schrijven naar een bestand indien de array niet leeg is en op elke pagina de inhoud van het bestandje te tonen . Werkt prima!

[.oisyn]

CyCloneNL schreef op woensdag 25 mei 2011 @ 10:10:
- Nee dat doet hij niet, "false" evalueert naar true.

Maar dat haalt z'n punt verder niet onderuit dat de loop stopt bij bepaalde filenames. Niet bij "false" idd, maar "0" is ook een geldige filename.

Ja, hij exclude inderdaad een hidden file. Ik weet het niet zeker, maar zou .svn voorkomen op Linux? Daar worden files die met een punt beginnen gehide toch?

Als je gewoon een standaard 'ls' vanuit de command prompt opvraagt ja. Net zoals een normale 'dir' in Windows ook geen hidden gevlagde files laat zien. Maar dat betekent niet meteen dat ze ook niet geënumereerd worden bij dergelijke API functies. En dan nog, waarom alleen .svn, en niet .htaccess oid, of generieker, elke filename die begint met een punt (pak je "." en ".." ook meteen mee)?

[ Voor 51% gewijzigd door .oisyn op 25-05-2011 23:09 ]

[dev10]

Sommige websites...

Vanavond wilde ik inloggen op een website, maar daar kreeg ik een foutmelding waar de complete connection-string naar de database in stond. De locatie waar PHPMyAdmin stond was vrij gemakkelijk, en binnen vijf minuten zat ik in de database.

Wat zou een database met 10000 emailadressen waard zijn?

Natuurlijk heb ik de eigenaar van de website op de hoogte gesteld. Ik hoop dat ze het serieus oppakken, al betwijfel ik of ze beseffen wat dt precies inhoudt.

[FragFrog]

EddoH schreef op woensdag 25 mei 2011 @ 10:00:
- dir handle wordt neit geclosed

Mmm, valt op zich wel mee. Het is een PHP request wat waarschijnlijk een tiental miliseconden daarna toch klaar is waarna die dir handle automatisch geclosed wordt. Dan kun je het handmatig doen en de gebruiker erop laten wachten, maar een echt voordeel is het niet.

PHP vereist qua garbage collecting een iets andere mindset dan veel andere talen. Juist niet zo snel mogelijk je eigen rotsooi opruimen, maar juist zo snel mogelijk je content eruit poepen en daarna eventueel gaan opruimen - maar liever nog gewoon het request beindigen waarna het voor je gedaan wordt. Al was het alleen maar omdat het geheugen waarschijnlijk toch pas weer wordt vrijgegeven nadat het script allang klaar is. Daar zijn wat uitzonderingen op (lange requests, of exclusive resources als file handles) maar doorgaans is het veel minder not-done dan bij andere talen.

Zo heb ik ook wel mensen gezien die voor elke SQL query een connectie opzetten en daarna mysql_close aanroepen, imho een grotere WTF

[ Voor 6% gewijzigd door FragFrog op 26-05-2011 23:28 ]

[ajakkes]

.oisyn schreef op woensdag 25 mei 2011 @ 23:05:
[...]

Maar dat haalt z'n punt verder niet onderuit dat de loop stopt bij bepaalde filenames. Niet bij "false" idd, maar "0" is ook een geldige filename.


[...]

Als je gewoon een standaard 'ls' vanuit de command prompt opvraagt ja. Net zoals een normale 'dir' in Windows ook geen hidden gevlagde files laat zien. Maar dat betekent niet meteen dat ze ook niet geënumereerd worden bij dergelijke API functies. En dan nog, waarom alleen .svn, en niet .htaccess oid, of generieker, elke filename die begint met een punt (pak je "." en ".." ook meteen mee)?

Het is slechts een voorbeeld. Er wordt dus laten zien wat je zou kunnen doen en ik kan me heel goed voorstellen dat lezers van het script een "oh ja" moment beleven op het moment dat ze zien staan ".svn" niet meetellen en denken oh daar maak ik van wat ik wil. Er staat ook duidelijk in de comments dat .svn niet meegeteld worden. Voor de functie zou het netter zijn als .svn wordt meegegeven als 2e variabele. Maar daar wordt het script wel iets on-overzichtelijker van.

Dat bij file "0" de functie stopt met tellen is wel een bug. Waar stond het script?

close $dir schijnt ook een beetje overbodig te zijn en het kopieren van bestandsnamen zie ik ook niet echt een probleem in.

Dus qua WTF vind ik het nog al meevallen.

[Janoz]

De grootste WTF is nog wel dat de functie 'countFiles' heet terwijl hij ook directories telt.

[kenneth]

In Unix, everything's a file

[EddoH]

ajakkes schreef op maandag 30 mei 2011 @ 07:53:
[...]


Het is slechts een voorbeeld. Er wordt dus laten zien wat je zou kunnen doen en ik kan me heel goed voorstellen dat lezers van het script een "oh ja" moment beleven op het moment dat ze zien staan ".svn" niet meetellen en denken oh daar maak ik van wat ik wil. Er staat ook duidelijk in de comments dat .svn niet meegeteld worden. Voor de functie zou het netter zijn als .svn wordt meegegeven als 2e variabele. Maar daar wordt het script wel iets on-overzichtelijker van.

Dat bij file "0" de functie stopt met tellen is wel een bug. Waar stond het script?

close $dir schijnt ook een beetje overbodig te zijn en het kopieren van bestandsnamen zie ik ook niet echt een probleem in.

Dus qua WTF vind ik het nog al meevallen.

Sorry maar al die punten die aangehaald zijn bij elkaar maken het gewoon een rete-slecht stuk code. Vooral als het op een manual-actige pagina staat waar je verwacht dat er mensen komen die nog niet alles gelijk doorhebben.
Het laat m.i gewoon zien hoe je niet hoort te programmeren. Als we dit soort stukken code op m'n werk zouden produceren dan wordt het geen leuk funtioneringsgesprek in ieder geval.

[Apache]

dev10 schreef op donderdag 26 mei 2011 @ 21:12:
Sommige websites...

Vanavond wilde ik inloggen op een website, maar daar kreeg ik een foutmelding waar de complete connection-string naar de database in stond. De locatie waar PHPMyAdmin stond was vrij gemakkelijk, en binnen vijf minuten zat ik in de database.

Wat zou een database met 10000 emailadressen waard zijn?

Natuurlijk heb ik de eigenaar van de website op de hoogte gesteld. Ik hoop dat ze het serieus oppakken, al betwijfel ik of ze beseffen wat dt precies inhoudt.

Aan je e-mail al een lijst van de data hangen, dan heeft hij het misschien wat sneller door

[YakuzA]

Apache schreef op maandag 30 mei 2011 @ 11:12:
[...]


Aan je e-mail al een lijst van de data hangen, dan heeft hij het misschien wat sneller door

of iedereen uit de DB cc'en

[console]

dev10 schreef op donderdag 26 mei 2011 @ 21:12:
Sommige websites...

Vanavond wilde ik inloggen op een website, maar daar kreeg ik een foutmelding waar de complete connection-string naar de database in stond. De locatie waar PHPMyAdmin stond was vrij gemakkelijk, en binnen vijf minuten zat ik in de database.

Wat zou een database met 10000 emailadressen waard zijn?

Natuurlijk heb ik de eigenaar van de website op de hoogte gesteld. Ik hoop dat ze het serieus oppakken, al betwijfel ik of ze beseffen wat dt precies inhoudt.

Hehe, in het weekend als ik mij verveel ga ik altijd op zoek naar dit 'soort' gevalletjes. Afgelopen zondag 13 site's gevonden die gebruik maken van PDO en tja.. als je dit niet goed afvangt krijg je idd een leuke connection string te zien waarin de 'user/pass' combinatie instaat.

En dan is het simpel of even 'http://www.site.nl/phpmyadmin/' of een ander bekend adres waar phpmyadmin zou kunnen staan en je bent binnen.

En soms krijg je aardige grote site's (Media gigant, Boekenhandel, Groothandel, Community's bv) voor geschoteld waar je met een biertje in het ene hand en andere hand je muis zo de database kan doorzoeken...

[RetroTycoon]

console schreef op maandag 30 mei 2011 @ 11:30:
[...]


Hehe, in het weekend als ik mij verveel ga ik altijd op zoek naar dit 'soort' gevalletjes. Afgelopen zondag 13 site's gevonden die gebruik maken van PDO en tja.. als je dit niet goed afvangt krijg je idd een leuke connection string te zien waarin de 'user/pass' combinatie instaat.

En dan is het simpel of even 'http://www.site.nl/phpmyadmin/' of een ander bekend adres waar phpmyadmin zou kunnen staan en je bent binnen.

En soms krijg je aardige grote site's (Media gigant, Boekenhandel, Groothandel, Community's bv) voor geschoteld waar je met een biertje in het ene hand en andere hand je muis zo de database kan doorzoeken...

Mmm, dan zou mijn website ook hier misschien een lek hebben. Wat moet je precies afvangen?

edit: iig is het lek, bij invalide gegevens gooit hij zo de wachtwoorden op straat

[ Voor 4% gewijzigd door RetroTycoon op 30-05-2011 11:55 ]

[haneev]

Geen errors aan site bezoekers laten zien.
Dus error_reporting(0) e.d.

Laatst zat ik wat rond te kijken in wat oude scripts. Ongeveer 8 jaar terug. Krijg je dit:

PHP:

mysql_query("INSERT INTO `gast` (`naam`,`text`,`ip`) VALUES ('{$_POST['titel']}','{$_POST['info']}','{$_SERVER['REMOTE_ADDR']}')");

Hier gaan mijn nekharen helemaal van overeind staan! Wat een bagger.

[RetroTycoon]

haneev schreef op maandag 30 mei 2011 @ 12:00:
Geen errors aan site bezoekers laten zien.
Dus error_reporting(0) e.d.

Laatst zat ik wat rond te kijken in wat oude scripts. Ongeveer 8 jaar terug. Krijg je dit:

PHP:

mysql_query("INSERT INTO `gast` (`naam`,`text`,`ip`) VALUES ('{$_POST['titel']}','{$_POST['info']}','{$_SERVER['REMOTE_ADDR']}')");

Hier gaan mijn nekharen helemaal van overeind staan! Wat een bagger.

Had ik standaard al uitstaan, maar heb nu voor de zekerheid ook even de connectiecode gewrapt in try-catch-blokken

[Caelorum]

console schreef op maandag 30 mei 2011 @ 11:30:
[...]


Hehe, in het weekend als ik mij verveel ga ik altijd op zoek naar dit 'soort' gevalletjes. Afgelopen zondag 13 site's gevonden die gebruik maken van PDO en tja.. als je dit niet goed afvangt krijg je idd een leuke connection string te zien waarin de 'user/pass' combinatie instaat.

En dan is het simpel of even 'http://www.site.nl/phpmyadmin/' of een ander bekend adres waar phpmyadmin zou kunnen staan en je bent binnen.

En soms krijg je aardige grote site's (Media gigant, Boekenhandel, Groothandel, Community's bv) voor geschoteld waar je met een biertje in het ene hand en andere hand je muis zo de database kan doorzoeken...

Evil modus aan: Ah gratis tv's! ^^

[Sebazzz]

Matis schreef op zaterdag 14 mei 2011 @ 09:52:
[...]

Ik snap er ook geen hout van; Zelfde bij de ING, het wachtwoord is niet hoofdlettergevoelig. De gebruikersnaam daarentegen wel

Mijn wachtwoord is te complex voor MijnING. Wachtwoorden zijn bij MijnING trouwens ook niet hoofdlettergevoelig.

[BarôZZa]

ING is sowieso mooi. Ik heb ooit eens half slaapdronken drie keer m'n verkeerde wachtwoord ingevuld. Vloekend zag ik dat het account geblokkeerd was en er een nieuw wachtwoord per post werd opgestuurd. Toen kwam ik erachter dat ik de verkeerde username had ingevuld.

Je kan dus zo willekeurige mensen een paar dagen van het internetbankieren afhouden.

[ Voor 14% gewijzigd door BarôZZa op 30-05-2011 14:25 ]

[Caelorum]

BarôZZa schreef op maandag 30 mei 2011 @ 14:24:
ING is sowieso mooi. Ik heb ooit eens half slaapdronken drie keer m'n verkeerde wachtwoord ingevuld. Vloekend zag ik dat het account geblokkeerd was en er een nieuw wachtwoord per post werd opgestuurd. Toen kwam ik erachter dat ik de verkeerde username had ingevuld.

Je kan dus zo willekeurige mensen een paar dagen van het internetbankieren afhouden.

OMG, dat is wel een heel erg. Waarom blokkeren ze niet gewoon voor een uur en sturen ze een smsje met een waarschuwing dat iemand heeft geprobeerd in te loggen? Ze zijn toch zo van de smsjes bij de ING?

[Woy]

Sebazzz schreef op maandag 30 mei 2011 @ 12:31:
[...]

Mijn wachtwoord is te complex voor MijnING. Wachtwoorden zijn bij MijnING trouwens ook niet hoofdlettergevoelig.

Het wachtwoord is ook niet belangrijk bij ING. Je gebruikersnaam, daar moet je zuinig mee zijn . Immers is je gebruikersnaam wel hoofdletter gevoelig, en kun je met je gebruikersnaam + telefoon + nog wat persoonlijke gegevens ( Die meestal makkelijk te achterhalen zijn ) ook gewoon het wachtwoord resetten.

Ik vind het SMS systeem van de ING echt super handig, maar ik snap echt niet wie het username/password beleid daarbij bedacht heeft.

[.oisyn]

Ja maar het voldoet aan de normen die gesteld zijn door de toezichthouders. Dus.

[ Voor 13% gewijzigd door .oisyn op 30-05-2011 15:38 ]

[Aloys]

Misschien moeten we dan maar geen kritiek hebben op de ING maar op de waardeloze eisen die de Nederlandse Bank toezichthouders stellen.

Weer zo'n ninja edit hierboven .

Feature request: geef niet alleen bij nieuwe meldingen, maar ook bij edit's de mooie "nieuwe berichten" balk .

[ Voor 45% gewijzigd door Aloys op 30-05-2011 15:43 ]

[Avalaxy]

Of een timer inbouwen zodat oisyn z'n bericht pas na 5 minuten mag versturen zodat hij hem eerst nog een paar keer goed kan lezen

[.oisyn]

Editen is king

[ Voor 67% gewijzigd door .oisyn op 30-05-2011 15:45 ]

[Aloys]

Edit dat eens eventjes heel gauw!

[PrisonerOfPain]

.oisyn schreef op maandag 30 mei 2011 @ 15:45:
Jullie zijn sukkels

[.oisyn]

Damn, te laat

[.oisyn]

Anyway, dit is wat ING mij mailde na navraag

Graag geven wij u een duidelijke uitleg waarom wij gekozen hebben voor het heraanvragen van het wachtwoord via mobiele telefoon.

De afgelopen jaren is gebleken dat veel klanten de huidige procedure voor het heraanvragen van het wachtwoord als onprettig ervaren. Als gevolg hiervan heeft de ING gezocht naar een meer klantvriendelijke oplossing. Hierbij heeft de ING een afweging gemaakt tussen veiligheid en gebruiksgemak.

Er is een oplossing gevonden waarbij het gebruiksgemak sterk is verbeterd en de veiligheid naar onze mening gewaarborgd blijft. De nieuwe procedure voldoet bovendien aan de richtlijnen die toezichthouders hebben opgesteld ten aanzien van internetbankieren.

Ook heeft ING een team van veiligheidsexperts in huis die zich dagelijks bezig houdt met de veiligheid van internetbankieren en voortdurend aanpassingen doet aan onze systemen. Natuurlijk volgen we alle ontwikkelingen met betrekking tot de veiligheid van internet en mobiele telefonie nauwlettend en blijven continue investeren in de veiligheid.

U als klant kunt ook zelf maatregelen treffen om veilig te bankieren. Tip die ING aan klanten geeft is om een gebruikersnaam en wachtwoord te kiezen die minder voor de hand liggen. Ook moet je deze inlogcodes nooit met andere delen. Als u de gebruikersnaam voor Mijn ING wilt bewaren, raden we aan dit alleen te doen op uw eigen computer.

Wij vertrouwen erop dat mensen zorgvuldig omgaan met persoonlijke bezittingen, zoals de Betaalpas, identiteitsbewijzen en de mobiele telefoon.

Om misbruik te kunnen maken van Mijn ING heeft iemand uw gebruikersnaam, geboortedatum, mobiele telefoon, Betaalpas en vervaldatum gelijktijdig nodig.
Mocht onverhoopt een dergelijke situatie zich toch voordoen, dan is het van belang dat u zo snel mogelijk contact opneemt met de Alarmlijn. Uw account van Mijn ING wordt in dat geval direct geblokkeerd. U kunt de Alarmlijn 24 uur per dag bereiken via 058 212 6000.

Wij willen tot slot benadrukken dat we de mogelijkheid van het frauduleus aanvragen van een wachtwoord via SMS zeer onwaarschijnlijk achten. Het nieuwe proces waarbij je per sms een nieuw wachtwoord kan opvragen is een veilig proces. Veiligheidsexperts van ING hebben hier uitvoerig onderzoek naar gedaan.

[Spiked]

Registratie voor iDeal van de Rabobank:
FOUT: Het nieuwe wachtwoord bevat ongeldige tekens.
Wat daar het idee achter is..?

[Woy]

.oisyn schreef op maandag 30 mei 2011 @ 15:45:
Editen is king

Daar is natuurlijk het knopje voor uitgevonden

.oisyn schreef op maandag 30 mei 2011 @ 15:47:
Anyway, dit is wat ING mij mailde na navraag

[...]

Exact dezelfde reactie als die ik gehad heb

[Aloys]

Om misbruik te kunnen maken van Mijn ING heeft iemand uw gebruikersnaam, geboortedatum, mobiele telefoon, Betaalpas en vervaldatum gelijktijdig nodig.

Het nieuwe proces waarbij je per sms een nieuw wachtwoord kan opvragen is een veilig proces. Veiligheidsexperts van ING hebben hier uitvoerig onderzoek naar gedaan.

Juist!

Lijkt mij erg makkelijk als je collega even zijn portemonnee en mobieltje op tafel legt (zit niet lekker in de broekzak) en dan eventjes ergens heen moet. Lijkt me niet zo'n moeilijk dus .

[.oisyn]

Of als je een handtas rooft oid, daar zitten ook alle gegevens in. Een gebruikersnaam laat zich doorgaans wel raden.

Aan de andere kant, de meeste mensen nemen dan weer niet de moeite om hun vooropgegeven ING gebruikersnaam te wijzigen, en die is redelijk random.

[ Voor 39% gewijzigd door .oisyn op 30-05-2011 15:53 ]

[Aloys]

Misschien een idee om de handtas van zo'n veiligheidsinspecteur van de ING te stelen?

[ Voor 3% gewijzigd door Aloys op 30-05-2011 15:53 ]

[Woy]

.oisyn schreef op maandag 30 mei 2011 @ 15:52:
Een gebruikersnaam laat zich doorgaans wel raden.

Zeker omdat de default username voorletter.achternaam is/was, ze zelf de optie bieden om hem op de pc op te slaan ( Wat gewoon clear-text in een cookie gebeurt, waar elke sysadmin hem weer af kan halen ) en hij gewoon mee te lezen is van het scherm als je meekijkt.

[ Voor 11% gewijzigd door Woy op 30-05-2011 15:57 ]

[PrisonerOfPain]

Woy schreef op maandag 30 mei 2011 @ 15:50:
[...]

Daar is natuurlijk het [afbeelding] knopje voor uitgevonden

Mod-superpowers, of wat?

[CoolGamer]

PrisonerOfPain schreef op maandag 30 mei 2011 @ 15:55:
[...]


Mod-superpowers, of wat?

Ja, dat is alleen voor modjes. Alleen bij een TS die gedeeld wordt kan je als mede-auteur er ook gebruik van maken.

[Remus]

Woy schreef op maandag 30 mei 2011 @ 15:54:
[...]

Zeker omdat de default username voorletter.achternaam is/was, ze zelf de optie bieden om hem op de pc op te slaan ( Wat gewoon clear-text in een cookie gebeurt, waar elke sysadmin hem weer af kan halen ) en hij gewoon mee te lezen is van het scherm als je meekijkt.

In het verleden was de username overigens giro-rekeningnummer + 3 random karakters (alfanumeriek).

[YopY]

Remus schreef op woensdag 01 juni 2011 @ 10:53:
[...]

In het verleden was de username overigens giro-rekeningnummer + 3 random karakters (alfanumeriek).

Spannend, een password van 3 karakters .

[Olaf van der Spek]

YopY schreef op woensdag 01 juni 2011 @ 21:45:
[...]


Spannend, een password van 3 karakters .

Er was toch ook nog een wachtwoord?

[Patriot]

Olaf van der Spek schreef op woensdag 01 juni 2011 @ 23:22:
[...]

Er was toch ook nog een wachtwoord?

Dat is het wachtwoord, het girorekeningnummer (maar als je de bankpas heb, dan heb je die al) plus drie tekens. Maar dan ben je dus al een heel eind.

[FragFrog]

Patriot schreef op donderdag 02 juni 2011 @ 01:42:
Dat is het wachtwoord, het girorekeningnummer (maar als je de bankpas heb, dan heb je die al) plus drie tekens. Maar dan ben je dus al een heel eind.

Nouja, dan moet je er wel vanuit gaan dat iemands gebruikersnaam nog die originele nummer + 3 tekens is. Bovendien kun je maar een beperkt aantal keren proberen voor de account geblokkeerd wordt, dus brute-forcen is ook niet echt een optie.

Mocht het een dief wel lukken je telefoon + bankpas te stelen en op een of andere manier je username te raden en een wachtwoord reset uit te voeren is het alsnog vrij simpel om je geld terug te krijgen: er is immers precies bekend welke transacties gedaan zijn door de dief (die na de reset). Bovendien moet die maar net een manier hebben om zijn identiteit te verbergen, ook niet bepaald triviaal. Denk dat het in de praktijk niet vaak genoeg voorkomt om echt een probleem te zijn

[Crazy D]

Patriot schreef op donderdag 02 juni 2011 @ 01:42:
Dat is het wachtwoord, het girorekeningnummer (maar als je de bankpas heb, dan heb je die al) plus drie tekens. Maar dan ben je dus al een heel eind.

Volgens mij was in de inlognaam het gironr met 3 random letters, het wachtwoord stond daar volgens mij los. Dat was het iig in de tijd dat ik internet bankieren kreeg van meneer de Postbank. Het default wachtwoord weet ik ondertussen niet meer, heb al eeuwen een eigen wachtwoord

[Patriot]

Crazy D schreef op donderdag 02 juni 2011 @ 11:54:
[...]

Volgens mij was in de inlognaam het gironr met 3 random letters, het wachtwoord stond daar volgens mij los. Dat was het iig in de tijd dat ik internet bankieren kreeg van meneer de Postbank. Het default wachtwoord weet ik ondertussen niet meer, heb al eeuwen een eigen wachtwoord

Ik had geloof ik een beetje last van de spreekwoordelijke stront in mijn ogen, want er klopte weinig van wat ik zei Het was inderdaad de gebruikersnaam en niet het wachtwoord.

[PiepPiep]

Een mooie die ik waarschijnlijk zelf gemaakt heb (kan ook collega zijn maar is een tijdje terug) in een klein stukje code die blijkbaar niet vaak gebruikt wordt.

C#:

DateTime.Now.ToString("yyyy-mm-dd");

spoiler:

Oplossing:
DateTime.Now.ToString("yyyy-MM-dd");

[YopY]

1. Date formatting is hoe dan ook kud.
2. Fout / bug had snel verholpen kunnen worden met een unit testje.
3. Fout / bug had zich nooit voorgedaan als er eerst een testje geschreven was (assertEquals ("08-06-2011", functie(new Date(vandaag));)

[ Voor 44% gewijzigd door YopY op 08-06-2011 14:56 ]

[kenneth]

Ohjajoh, waarom denk je dat hij het in dit topic plaatst

[Kwastie]

Waarschijnlijk lezen een aantal mensen hier wel The Daily WTF, maar omdat dit voorbeeld zo extreem "goed" (slecht) is neem ik hem even over van: http://thedailywtf.com/Articles/Divine-by-Zero.aspx

PHP:

<?
/* this is my function to conveniently divide even when things are zero */
function safely_divideByZero($numerator,$denominator)
{
    /* check to see if the denominator == "php" - remember, 0 == "php" */
    if ( $denominator == "php" )
    {
        /*  if the denominator == "php" we can safely set the value of the denominator to "php"  */
        $denominator = "php";

        /* now check to see if the denominator == TRUE */
    if ( $denominator == TRUE )
        {
            /* if the denominator == TRUE we can set the value of the denominator to TRUE  */
            $denominator = TRUE;

        // now check to see if the denominator == 1 or -1
        if ( $denominator== 1 || $denominator == -1 )
        {
        /* this part is clever, we don't know which is the right answer, so we can use a for loop to randomize */
        for ( $n = 0; $n == 0 ; )
        {
            /* we don't want zero as the answer (back to square one if it is!) 
            so we randomize from 0 to 2, but the for loop throws out results
            that == 0 and makes it try again */
            $n = rand(0,2);
        }
                
        if ( $n == 1 )              
        {
            /* if the answer == 1, then we set the denominator to 1, somewhat obvious here */
            $denominator = $n;
        }
        else
        {
            /*otherwise the denominator must be -1 by process of elimination */
            $denominator = -1;
        }  
            }
        }                               
    }

    /* we return as the result a simple division, except now we can divide by zero without error */
    $result = $numerator/$denominator;  

    return $result;      
}
?>

Het eerste wat in mij opkwam toen ik dit las was: "WTF, waarom zou je willen delen door 0?"


Een van de reacties:

For reference, here is what happens when it is executed...

safely_divideByZero(25,0);
25

safely_divideByZero(25,0);
-25

safely_divideByZero(25,0);
-25

safely_divideByZero(25,0);
-25

safely_divideByZero(25,0);
-25

safely_divideByZero(25,2);
12.5

safely_divideByZero(25,-2);
-12.5

safely_divideByZero(25,"arse");
PHP Warning: Division by zero in /home/dave/argh.php on line 44

safely_divideByZero(25,"php");
-25

[ Voor 12% gewijzigd door Kwastie op 08-06-2011 15:17 ]

[BarôZZa]

Mensen die oplossingen gaan geven voor wtf's vind ik ook aardig wtf.

[Tsunami]

Vind deze reactie daarop ook wel mooi:

User: The application's not working.
Developer: Yes, but sometimes it's working really well. You'll notice that these occurrences average out to mostly working, which is exactly what you were expecting. No need to thank me, just doing my job.

Best developer excuse ever.

[Onbekend]

Wij hebben ook een stukje software (geprogrammeerd door derden) die delen door 0 letterlijk als "onbepaald" beschouwt.
Bij een som als 29 / 0 is het antwoord altijd 0, 1 of 29. Wij hebben zelf nog steeds geen idee wanneer welk antwoord wordt gegeven. Wij delen natuurlijk ook niet expres door 0, maar verwachten niet zonder een warning/error/exception een getal terug om verder mee te rekenen.....

[Marientjuh]

Letterlijk uit een oud systeem:

PHP:

<?php
$seminarid = strip_tags($seminarid);

if(preg_match('/[^0-9_]/',$seminarid))
    $seminarid = 1;

$rescheck = @mysql_db_query("$dbname","select * from seminars where id='$seminarid'");
?>

[ Voor 5% gewijzigd door Marientjuh op 09-06-2011 11:18 ]

[YakuzA]

YopY schreef op woensdag 08 juni 2011 @ 14:55:
1. Date formatting is hoe dan ook kud.
2. Fout / bug had snel verholpen kunnen worden met een unit testje.
3. Fout / bug had zich nooit voorgedaan als er eerst een testje geschreven was (assertEquals ("08-06-2011", functie(new Date(vandaag));)

en vandaag zou die unit test weer falen... Goede oplossing

[Caelorum]

YakuzA schreef op donderdag 09 juni 2011 @ 13:42:
[...]

en vandaag zou die unit test weer falen... Goede oplossing

alle data in een lijstje zetten, dan datadriven unittest aanmaken en test markeren als geslaagd als één van de data asserts slaagt.
*Caelorum voelt zich nu vies*

[YopY]

YakuzA schreef op donderdag 09 juni 2011 @ 13:42:
[...]

en vandaag zou die unit test weer falen... Goede oplossing

was een voorbeeld, jwz. Zelf deden we het door een long in een date te stoppen, maar ook dat is niet ideaal. Een datum in code weergeven op een korte manier is lastig. Door het als string te doen zit je weer met locale instellingen, bijvoorbeeld.