[alg] Slechtste programmeervoorbeelden deel 4

[CMG]

Hydra schreef op donderdag 16 augustus 2012 @ 15:21:
[...]
Je cast een integer naar een decimal om daar Math.floor op aan te roepen. En je begrijpt niet wat hier misgaat?
[...]

en weer heb je gelijk; had niet eens gezien dat het .Days was ipv .TotalDays. Zo past het i.d.d. mooi in dit draadje van slechte voorbeelden.

*knip*

[ Voor 39% gewijzigd door MueR op 16-08-2012 16:23 ]

[ValHallASW]

CMG schreef op donderdag 16 augustus 2012 @ 15:37:
en weer heb je gelijk; had niet eens gezien dat het .Days was ipv .TotalDays. Zo past het i.d.d. mooi in dit draadje van slechte voorbeelden.

Dat is nog steeds niet hét probleem met je code. Hoeveel dagen zitten er immers in een jaar?


..oh, wacht, dat hád je door, en toch verdedig je je code omdat het maar 'een kleine afwijking' is... vertel dat eens aan Paus Gregorius de 13e ;-)

*knip*

De reactie van Hydra ging niet zozeer over het voorbeeld an sich, maar ook vooral over hoe je op zijn/haar commentaar reageerde.

[ Voor 66% gewijzigd door MueR op 16-08-2012 16:23 ]

[RaZ]

ValHallASW schreef op donderdag 16 augustus 2012 @ 15:44:
[...]

Dat is nog steeds niet hét probleem met je code. Hoeveel dagen zitten er immers in een jaar?

Wil je het aantal weten van een normaal jaar, of een schrikkeljaar?

[Korben]

CMG schreef op donderdag 16 augustus 2012 @ 15:37:
[...]


En bedankt. Het ging me er om dat je niet wilt in zien dat een stukje dat ik 6 jaar geleden snel geschreven heb om ff mijn leeftijd te laten zien op een prive website weinig te maken heeft met hoe ik een zakelijke app zou programmeren.

Behalve dat je op je website er bij zet 'kijk eens hoe goed ik ben, ik kan op de verkeerde manier het verschil tussen twee data berekenen'.

[GateKeaper]

Misschien heb je hier wat aan. Staan wel creatieve oplossingen tussen:
http://stackoverflow.com/...ulating-age-from-birthday



*link met dank aan pedorus

[ Voor 9% gewijzigd door GateKeaper op 16-08-2012 16:19 ]

[pedorus]

offtopic:
Mooie copy

edit:
had mijn post inderdaad eigenlijk op de verkeerde plek gezet, dus geen bezwaar tegen kopiëren. Enkel wilde CMG ook even informeren over het andere draadje, iets met niet achter de rug om enzo

[ Voor 70% gewijzigd door pedorus op 16-08-2012 16:22 ]

[GateKeaper]

pedorus schreef op donderdag 16 augustus 2012 @ 16:15:

offtopic:
Mooie copy

offtopic:
Had jou hier nog niet eerder gezien, en wou de mensen hier deze tip niet onthouden. Kennis is macht?

[MueR]

Modbreak:

Het over en weer schoppen mag weer ophouden. Ik heb er geen zin in.

[CMG]

Heb nu iig genoeg motivatie gehad om het aan te passen

Heb er van gemaakt:

C#:

    DateTime Today = DateTime.Today, BirthDate = new DateTime(1983, 4, 18);
    int AgeInYears = Today.Year - BirthDate.Year;
    if (BirthDate.DayOfYear > Today.DayOfYear)
    {
        AgeInYears--;
    }

Geinspireerd op http://stackoverflow.com/a/1404/458991, maar vond direct DayOfYear vergelijken mooier dan .AddYears(-AgeInYears);

[pedorus]

Schitterend, dan zit die schrikkelbug er nog steeds in... (Naast de "This is wrong!" comment)

Mijn oplossing was trouwens pedorus in "De Devschuur Coffee Corner - Iteratie 2"

[CMG]

pedorus schreef op donderdag 16 augustus 2012 @ 17:25:
Schitterend, dan zit die schrikkelbug er nog steeds in... (Naast de "This is wrong!" comment)

Mijn oplossing was trouwens pedorus in "De Devschuur Coffee Corner - Iteratie 2"

Nee hoor, dit werkt prima.

Als je dit even test in een nieuwe C# Console App, zul je zien dat het wel klopt (geeft gewoon 8 weer):

C#:

            DateTime BirthDate = new DateTime(2000, 2, 29);
            DateTime Today = new DateTime(2009, 2, 28);
            int AgeInYears = Today.Year - BirthDate.Year;
            if (BirthDate.DayOfYear > Today.DayOfYear)
                AgeInYears--;
            Console.WriteLine(AgeInYears);

[ Voor 32% gewijzigd door CMG op 16-08-2012 17:33 ]

[pedorus]

CMG schreef op donderdag 16 augustus 2012 @ 17:31:
[...]

Nee hoor, dit werkt prima.

Als je dit even test in een nieuwe C# Console App, zul je zien dat het wel klopt (geeft gewoon 8 weer):

C#:

            DateTime BirthDate = new DateTime(2000, 2, 29);
            DateTime Today = new DateTime(2009, 2, 28);
            int AgeInYears = Today.Year - BirthDate.Year;
            if (BirthDate.DayOfYear > Today.DayOfYear)
                AgeInYears--;
            Console.WriteLine(AgeInYears);

Surely? Zoek in dit draadje nog eens op "Volgens deze code ben je op " voor wat testdata...

[.oisyn]

Wat pedorus bedoelt te zeggen () is dat DayOfYear geen goede referentie is omdat die schuift afhankelijk van of het een schrikkeljaar is of niet. Als je op 1 maart in een schrikkeljaar bent geboren dan is dat de 61e dag van het jaar. Als het dan een keer 1 maart is zonder schrikkeljaar dan is het slechts de 60e dag, en dan denkt jouw code dus dat er een jaar afgetrokken moet worden. Je code klopt alleen als Birthdate en Today beide een schrikkeljaar of geen schrikkeljaar zijn.

Je moet dus echt datums (zonder jaar) gaan vergelijken.

[ Voor 16% gewijzigd door .oisyn op 16-08-2012 20:03 ]

[Jeldert]

Kwam ik vanmiddag tegen... (query heb ik even weggehaald)

Visual Basic:

' .... deze regel !! .....
'oSQL.execute("insert into ...")                
'componentID = oSQL.getLastID()

Zo van: ok, de regel is belangrijk? Maar staat in commentaar... Na overleg maar het hele stuk verwijderd

[PiepPiep]

You should never tell someone to "go to hell".

There are always better control structures than "go to".

[L-VIS]

Ik vind dit een creatieve manier om twee weken in tijd terug te gaan.

C#:

DateTime.Now.AddHours(-24 * 14)

[ Voor 3% gewijzigd door L-VIS op 05-09-2012 14:31 ]

[spleethoven]

Lijkt me nogtans een goede manieer te zijn...

[Haan]

spleethoven schreef op donderdag 06 september 2012 @ 09:56:
Lijkt me nogtans een goede manieer te zijn...

.. ware het niet dat er ook een AddDays method is

[Caelorum]

Er was hier laatst toch al een mooie discussie over? Een dag had toch niet altijd precies 24uur?

[L-VIS]

Caelorum schreef op donderdag 06 september 2012 @ 11:08:
Er was hier laatst toch al een mooie discussie over? Een dag had toch niet altijd precies 24uur?

Maakt niet veel uit, want onder water leveren ze beide hetzelfde resultaat. Ze roepen allebei de

C#:

 private DateTime Add(double value, int scale); 

aan. Waarbij de scale van AddDays precies 24 keer zo groot is.

[daaan]

Dit kwam ik in code van een ex collega tegen (snippets):

PHP:

private $daysOfMonths=array(31,28,31,30,31,30,31,31,30,31,30,31);
private $whichQuarterYear=array(1,1,1,2,2,2,3,3,3,4,4,4);
private $quearterPeriodes=array(array('begin'=>1,'end'=>3),array('begin'=>4,'end'=>6),array('begin'=>7,'end'=>9),array('begin'=>10,'end'=>12));

public function getPeriodBeginEnd($get) {
    if((int)$get['year'] % 4 == 0){
        $this->daysOfMonths[1]=29;
     }else{
        $this->daysOfMonths[1]=28;       
     }
    $datumBegin=strtotime($get['year']."-".$get['month'].'-01');
    $datumEnd=strtotime($get['year']."-".$get['month'].'-'.$this->daysOfMonths[(int)$get['month']-1]);
    return array('begin'=>$datumBegin,'end'=>$datumEnd);
}

$gewensteKwartaalGrenzen=$this->quearterPeriodes[(int)($this->whichQuarterYear[(int)date("m")-1])-1];
$gewenstePeriodeBegin=array('month'=>$gewensteKwartaalGrenzen['begin'],'year'=>date("Y"));
$gewenstePeriodeEnd=array('month'=>$gewensteKwartaalGrenzen['end'],'year'=>date("Y"));
$periodeStart=$this->getPeriodBeginEnd($gewenstePeriodeBegin);
$periodeEnd=$this->getPeriodBeginEnd($gewenstePeriodeEnd);

[Caelorum]

Afgezien van die code.. Arg als je dan Engels doet doe het dan goed. En aub blijf zoveel mogelijk in 1 taal werken. Dat er bepaalde begrippen niet kunnen of zouden moeten/mogen worden vertaald is niet erg, maar "quearterPeriodes" en "getPeriodBeginEnd($gewenstePeriodeBegin);"

[daaan]

Ja hou op. Ik ben nog wekelijks aan het puinruimen. Scheelt alweer dat in dit voorbeeld geen Turkse variabelenamen staan...

[azerty]

PHP:

if((int)$get['year'] % 4 == 0){

Dit al gecorrigeerd? xD

[Haan]

(voor de ASP.NET webforms mensen)

C#:

public void Text_ServerValidate(object source, ServerValidateEventArgs args)
{
    if (string.IsNullOrWhiteSpace(args.Value))
    {
        args.IsValid = false;
    }
}

icm :

ASP.NET:

<asp:CustomValidator runat="server" ID="valLastName" ClientValidationFunction="textValidation"
ControlToValidate="tbLastname" Display="Dynamic" ErrorMessage="Dit veld is verplicht"
EnableClientScript="true" SetFocusOnError="true" ValidateEmptyText="true" ValidationGroup="SubmitAction"  OnServerValidate="Text_ServerValidate" />

en

JavaScript:

function textValidation(sender, args)
{
    if (jQuery.trim(args.Value) == '')
    {
        args.IsValid = false;
    }
}

[Alex)]

Daar bestaat toch gewoon een RequiredFieldValidator voor?

[Haan]

Daarom post ik het ook hier

[kenneth]

Hebben het begin en eind van een tijdsinterval dezelfde dag en maand?

C#:

if (new DateTime(1, tijdsinterval.MinPeriode.Month, tijdsinterval.MinPeriode.Day) ==
    new DateTime(1, tijdsinterval.MaxPeriode.Month, tijdsinterval.MaxPeriode.Day))

[HeSitated]

Vandaag ook weer een mooie gezien, vrij geinterpreteerd:

C#:

var p = list.Where( x => (x.Name == null ? 1 == 1 : x.Name == "Test"));

Iemand die perse ? : wilde gebruiken......

[roy-t]

HeSitated schreef op vrijdag 07 september 2012 @ 21:48:
Vandaag ook weer een mooie gezien, vrij geinterpreteerd:

C#:

var p = list.Where( x => (x.Name == null ? 1 == 1 : x.Name == "Test"));

Iemand die perse ? : wilde gebruiken......

Haha dat vind ik wel een leuke. Toch nog best lastig parsen zo'n terniary in een linq statement. Maar eigenlijk had het dus ook dit kunnen zijn? (niet al te helder vandaag)

C#:

var p = list.Where( x => (x.Name == null || x.Name == "Test"));

[PdeBie]

Haan schreef op donderdag 06 september 2012 @ 13:58:
(voor de ASP.NET webforms mensen)

C#:

public void Text_ServerValidate(object source, ServerValidateEventArgs args)
{
    if (string.IsNullOrWhiteSpace(args.Value))
    {
        args.IsValid = false;
    }
}

icm :

ASP.NET:

<asp:CustomValidator runat="server" ID="valLastName" ClientValidationFunction="textValidation"
ControlToValidate="tbLastname" Display="Dynamic" ErrorMessage="Dit veld is verplicht"
EnableClientScript="true" SetFocusOnError="true" ValidateEmptyText="true" ValidationGroup="SubmitAction"  OnServerValidate="Text_ServerValidate" />

en

JavaScript:

function textValidation(sender, args)
{
    if (jQuery.trim(args.Value) == '')
    {
        args.IsValid = false;
    }
}

Misschien was die customValidator nog niet af en moest er nog heel wat na komen?
(al zou ik het persoonlijk dan alsnog met een aparte requiredFieldValidator doen.)

[Haan]

Sanity check:

C#:

if (DateTime.Compare(eindDate, nextDate) < 0) { ..}

is toch gewoon hetzelfde als

C#:

if (eindDate < nextDate) {..}

(we negeren even de variabele namen.. )

Het commentaar erboven zegt namelijk

loopen tot datum < einddatum

, wat zou suggereren dat de check juist andersom zou moeten zijn.

pdebie schreef op maandag 10 september 2012 @ 14:41:
[...]


Misschien was die customValidator nog niet af en moest er nog heel wat na komen?
(al zou ik het persoonlijk dan alsnog met een aparte requiredFieldValidator doen.)

Hè toevallig dat ik net weer post hier
Er is geen enkele aanwijzing dat de customvalidator nog uitgebreid zou gaan worden, dus ga er maar vanuit dat die code volledig 'by design' zo is gemaakt

[ Voor 66% gewijzigd door Haan op 10-09-2012 14:54 ]

[PdeBie]

Haan schreef op maandag 10 september 2012 @ 14:48:
[...]

Hè toevallig dat ik net weer post hier
Er is geen enkele aanwijzing dat de customvalidator nog uitgebreid zou gaan worden, dus ga er maar vanuit dat die code volledig 'by design' zo is gemaakt

ok, dan is het een uitermate goed voorbeeld voor hier

[Davio]

Haan schreef op maandag 10 september 2012 @ 14:48:
Sanity check:

C#:

if (DateTime.Compare(eindDate, nextDate) < 0) { ..}

is toch gewoon hetzelfde als

C#:

if (eindDate < nextDate) {..}

(we negeren even de variabele namen.. )

Het commentaar erboven zegt namelijk

[...]
, wat zou suggereren dat de check juist andersom zou moeten zijn.


[...]

Hè toevallig dat ik net weer post hier
Er is geen enkele aanwijzing dat de customvalidator nog uitgebreid zou gaan worden, dus ga er maar vanuit dat die code volledig 'by design' zo is gemaakt

Compare geeft 1 als links groter is dan rechts en -1 als rechts groter is dan links.
En 0 als ze gelijk zijn.

Het is dus

C#:

if (eindDate > nextDate) {..}

[.oisyn]

Euh, nee. Het is kleiner dan 0 als links juist kleiner is dan rechts. Daardoor kun je ook dit doen:

C#:

int Compare(int a, int b)
{
    return a-b;
}

En dat geeft meteen een handig ezelsbruggetje:

C#:

if (a < b) // ...

// kun je ook lezen als:

if (a - b < 0) // ...

// en dus

if (Compare(a,b) < 0) // ...

[ Voor 37% gewijzigd door .oisyn op 11-09-2012 14:43 ]

[Verwijderd]

Afgesplitst van De dingen die je af en toe tegenkomt in programmatuur...

---

... van anderen.

count($data = array_filter($data,create_function('$a','return !is_array($a);'))) && $dbh->Query(sprintf("UPDATE `%s` SET <SET> WHERE `ID` = %d",$table,$ID),$data,$debug);

Iedereen die geen hoofdpijn heeft na het uitvogelen van bovenstaande regel maakt kans op een koekje.

[ Voor 15% gewijzigd door RobIII op 15-09-2012 15:12 ]

[Grannd]

Je was mij al kwijt bij de $

[Snow_King]

Ik ga niet door voor het koekje... Het enige wat er in me op kwam: wtf?

[Feanathiel]

Verwijderd schreef op zaterdag 15 september 2012 @ 13:28:
[mbr]Afgesplitst van De dingen die je af en toe tegenkomt in programmatuur...[/]

---

... van anderen.

count($data = array_filter($data,create_function('$a','return !is_array($a);'))) && $dbh->Query(sprintf("UPDATE `%s` SET <SET> WHERE `ID` = %d",$table,$ID),$data,$debug);

Iedereen die geen hoofdpijn heeft na het uitvogelen van bovenstaande regel maakt kans op een koekje.

De query wordt aangeroepen voor alle niet-array waarden (anonymous-filter) van $data, maar alleen als $data niet-array elementen bevat (lazy-evaluation). Met een beetje formatting kom je een heel eind, maar dit moet je gewoon niet willen in de codebase.

[Grijze Vos]

Die programmeur prutser denkt vast dat zn baas moet afrekenen per loc oid.

Ik snap niet waarom mensen zo graag 5-6 commands op een regel proberen te frotten om maar geen if statements te hoeven maken. If statements zijn vaak zoveel leesbaarder. Zelfde geldt voor ternary operator hell, met 3-4 genest, Awesome.

[xzaz]

Grijze Vos schreef op maandag 17 september 2012 @ 11:18:
Die programmeur prutser denkt vast dat zn baas moet afrekenen per loc oid.

Ik snap niet waarom mensen zo graag 5-6 commands op een regel proberen te frotten om maar geen if statements te hoeven maken. If statements zijn vaak zoveel leesbaarder. Zelfde geldt voor ternary operator hell, met 3-4 genest, Awesome.

Omdat men vaak het idee heeft dat iets sneller en makkelijker kan? Onder het mom van; if else is zo 2000.

[MBV]

Als iets er kleiner uitziet, zal het vast ook sneller werken. Totdat je iets over O(n^2) etc leert is dat een impliciete aanname...

[Davio]

Misschien wel omdat er een automatische refactor overheen wordt gehaald?

[Janoz]

xzaz schreef op maandag 17 september 2012 @ 11:30:
[...]

Omdat men vaak het idee heeft dat iets sneller en makkelijker kan? Onder het mom van; if else is zo 2000.

Sterker nog, vaak leeft het onterechte idee dat iets wat korte code oplevert dus ook wel sneller zou zijn

[Davio]

Leesbare code > micro-optimalisatie.

[Solopher]

Oke, ik moest net even naar een site kijken omdat er SQL injectie mogelijk was, dat was helaas niet het enige.

PHP:

if ($soort == "klas") {
    $el = "class";
} else {
    if ($soort == "lokaal") {
        $el = "room";
    } else {
        if ($soort == "docent") {
            $el = "teacher";
        }
    }
}

En $soort was natuurlijk een $_GET variable, waardoor als iemand soort = bla' deed, $el leeg was en vervolgens de query ook niet werkte. Overigens was dit nog niet eens het probleem met de SQL injectie. Dat kwam later met $jaar = $_GET['jaar'] en $maand = $_GET['maand'] en vervolgens in de query gewoon $maand en $jaar gebruiken. Heerlijk

[.oisyn]

Sowieso, schrijf dan gewoon "else if" ipv al die extra nesting

[Alex)]

Waarom niet gewoon een switch?

[.oisyn]

Of een array lookup, maar that's not really the point.

[azerty]

oh dear, dat is waarschijnlijk ook de laatste keer dat ik een pizza online bestel xD

[kenneth]

vEn vDie vVoorvoegsels vDan, vZal vLekker vLezen vAls vDeveloper

[YopY]

wsitedesign schreef op dinsdag 18 september 2012 @ 12:30:
oh dear, dat is waarschijnlijk ook de laatste keer dat ik een pizza online bestel xD

[afbeelding]

Bestelde je de fameuze "Pizza' OR 1 = 1 --"?

[azerty]

YopY schreef op dinsdag 18 september 2012 @ 12:39:
[...]


Bestelde je de fameuze "Pizza' OR 1 = 1 --"?

Zelfs nog niet, alhoewel het mij nu wel interessanter begint te lijken.

Dit krijg je als doodnormale klant te zien als je de bestelling wilt plaatsen (definitief) -_-

[RedHat]

He ziet wel vaker dat er prefixes gebruikt worden. (om daar op in te haken). Zo kunnen ze voor hun gevoel visitors(v), users(u), distributors(d) bijvoorbeeld uit elkaar houden.

Of het fout is, mwah. Er zijn betere manieren.

Ik kan ook zo 1-2-3 geen SQL injectie ontdekken, dus misschien wordt er wel netjes ge-escaped? Een query displayen vind ik dan wel weer een fuckup.

Edit: zag de 'a niet staan. Wel typisch dat dit überhaupt nog kan / bestaat op het net

[ Voor 35% gewijzigd door RedHat op 18-09-2012 13:01 ]

[boe2]

aangezien daar wel ergens een injectiemogelijkheid MOET verborgen zijn: Waarom klooien met de server als je zo ook gratis pizza kan bestellen?

[azerty]

Ik heb niks uitgeprobeerd, maar gewoon het feit dat de query al op de pagina verschijnt is een teken dat er ergens iets mis loopt

[Alex)]

C#:

internal static void HandleException(Exception ex)
{
  throw ex;
}

Oh you silly SharePoint

Ja, ik begrijp dat er wellicht #if DEBUG ... #endif in Microsoft's sourcecode staat, maar dat zie ik natuurlijk niet

[Styxxy]

Alex) schreef op dinsdag 18 september 2012 @ 15:58:

C#:

internal static void HandleException(Exception ex)
{
  throw ex;
}

Oh you silly SharePoint

Ja, ik begrijp dat er wellicht #if DEBUG ... #endif in Microsoft's sourcecode staat, maar dat zie ik natuurlijk niet

Wij hadden ergens een methode "ThrowException" die gewoon een "Throw ex" deed. En dat was net een stuk dat ik eens moest debuggen (zat een error in, duh). Leuk, krijg je een exception zonder bruikbare stacktrace én de exception was van het vaagste niveau. Uiteindelijk op alle aanroepen van "ThrowException" een breakpoint gezet .

[BasieP]

Styxxy schreef op dinsdag 18 september 2012 @ 18:02:
[...]


Wij hadden ergens een methode "ThrowException" die gewoon een "Throw ex" deed. En dat was net een stuk dat ik eens moest debuggen (zat een error in, duh). Leuk, krijg je een exception zonder bruikbare stacktrace én de exception was van het vaagste niveau. Uiteindelijk op alle aanroepen van "ThrowException" een breakpoint gezet .

en iemand een schop onder z'n kont cursus gegeven?

[roy-t]

Als je een exception door gooit met

code:

throw ex;

krijg je toch juist wel een bruikbare stacktrace (via InnerException). Of werd er een nieuwe lege exception gemaakt?

[Firesphere]

Met je ex gooien levert zelden iets bruikbaars op.
Behalve dat het weg is...

[Styxxy]

roy-t schreef op dinsdag 18 september 2012 @ 20:24:
Als je een exception door gooit met

code:

throw ex;

krijg je toch juist wel een bruikbare stacktrace (via InnerException). Of werd er een nieuwe lege exception gemaakt?

InnerException is null en stacktrace is foetsie (stacktrace zal tussen zijn tot en met waar je throw ex hebt gedaan en niet verder, waar de exception origineel was opgetreden).

[BasieP]

Wat wel kan is zoiets:

C#:

try {
   ...
} 
catch (Exception ex)
{
    log(ex);
    throw;
}

waarbij je de exception doorgooit zonder dat hij gemodificeerd is

[ Voor 24% gewijzigd door BasieP op 18-09-2012 20:52 ]

[Zeebonk]

Firesphere schreef op dinsdag 18 september 2012 @ 20:25:
Met je ex gooien levert zelden iets bruikbaars op.
Behalve dat het ze weg boos is...

[Firesphere]

Alsnog is ze weg

Daarnaast heb ik het even expres onzijdig gehouden

[Zeebonk]

Hehe ik wist het niet zeker of het wel of niet als grap was bedoeld. Het is natuurlijk allemaal onder voorwaarde dat ze nergens meer wordt gecatched

[Soultaker]

wsitedesign schreef op dinsdag 18 september 2012 @ 12:30:
oh dear, dat is waarschijnlijk ook de laatste keer dat ik een pizza online bestel xD
[afbeelding]

Ah ja, ik zie al wat het probleem is: 't is door Belgen gecoded.

[Verwijderd]

Zeebonk schreef op dinsdag 18 september 2012 @ 22:41:
Hehe ik wist het niet zeker of het wel of niet als grap was bedoeld. Het is natuurlijk allemaal onder voorwaarde dat ze nergens meer wordt gecatched

De Just-In-Time debugger/catcher?

[Styxxy]

BasieP schreef op dinsdag 18 september 2012 @ 20:51:
Wat wel kan is zoiets:
[...]waarbij je de exception doorgooit zonder dat hij gemodificeerd is

Dat is ook de policy bij ons, als je catcht en je wilt dat hij door wordt gegooid (maar je wilt er eerst iets mee doen, stel aan de hand van ErrorCode kijken of dat je een bepaalde handeling kan doen ofzo...) dan doe je gewoon "Throw". Maar ergens was het op een punt mis gegaan .

Soultaker schreef op dinsdag 18 september 2012 @ 23:02:
[...]
Ah ja, ik zie al wat het probleem is: 't is door Belgen gecoded.

[azerty]

Soultaker schreef op dinsdag 18 september 2012 @ 23:02:
[...]

Ah ja, ik zie al wat het probleem is: 't is door Belgen gecoded.

Ben ik blij dat Parijs (http://www.dominospizza.be/nl/credits.html) sinds nu bij België hoort xD

[Firesphere]

Had ik deze al eens gepost?

PHP:

$rand = md5(rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999));

[Davio]

Klant heeft een probleem.
Klant stuurt code.
* Davio ziet een parameter uit een get request direct in een nieuwe URL gepropt worden
* Davio vertelt klant: Je hebt wel meer problemen.

[.oisyn]

Firesphere schreef op woensdag 19 september 2012 @ 15:54:
Had ik deze al eens gepost?

PHP:

$rand = md5(rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999));

D'r zit bijzonder veel entropie in die hash ja

[jochem4207]

Firesphere schreef op woensdag 19 september 2012 @ 15:54:
Had ik deze al eens gepost?

PHP:

$rand = md5(rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999));

Kan iemand me deze even uitleggen?

Ik ben zelf net een tijdje bezig met PHP , ik heb zelf ook een hash maar dan op andere manier.
Ik zit op ict opleiding , maar school vind asp.net veel belangrijker.

Misschien dat dat verklaart waarom ik niet zie wat er fout gaat?

[Verwijderd]

Firesphere schreef op woensdag 19 september 2012 @ 15:54:
Had ik deze al eens gepost?

PHP:

$rand = md5(rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999) . rand(100000000, 999999999));

Hartstikke mooie random hash toch? Misschien voor extra zekerheid nog een paar keer door md5() heen halen.

[Feanathiel]

jochem4207 schreef op donderdag 20 september 2012 @ 20:35:
[...]


Kan iemand me deze even uitleggen?

Ik ben zelf net een tijdje bezig met PHP , ik heb zelf ook een hash maar dan op andere manier.
Ik zit op ict opleiding , maar school vind asp.net veel belangrijker.

Misschien dat dat verklaart waarom ik niet zie wat er fout gaat?

Je plakt hier strings aan elkaar vast. Elk karakter van deze string bestaat slechts uit 10 verschillende waarden (0-9). Er zijn 256 mogelijke waarden voor een enkel karakter, dus in feite gooi je dus 96% van je keyspace per karakter weg. Mooier zou zijn om de volledige 256-byte range te gebruiken, maar goed, dan heb je eigenlijk geen md5 meer nodig. Een paar keer md5 erover heen maakt dit alleen maar erger.

* verbeter of vul aan waar nodig

[Verwijderd]

Feanathiel schreef op donderdag 20 september 2012 @ 20:49:
[...]


Een paar keer md5 erover heen maakt dit alleen maar erger.

Indien doelend op mijn eerdere post; was toch wel duidelijk dat dat sarcasm was?

[jochem4207]

Feanathiel schreef op donderdag 20 september 2012 @ 20:49:
[...]


Je plakt hier strings aan elkaar vast. Elk karakter van deze string bestaat slechts uit 10 verschillende waarden (0-9). Er zijn 256 mogelijke waarden voor een enkel karakter, dus in feite gooi je dus 96% van je keyspace per karakter weg. Mooier zou zijn om de volledige 256-byte range te gebruiken, maar goed, dan heb je eigenlijk geen md5 meer nodig. Een paar keer md5 erover heen maakt dit alleen maar erger.

* verbeter of vul aan waar nodig

Dankjewel! , Dat maakt het stuk helderder

[Feanathiel]

Verwijderd schreef op donderdag 20 september 2012 @ 21:05:
[...]

Indien doelend op mijn eerdere post; was toch wel duidelijk dat dat sarcasm was?

Uiteraard. Even een korte toelichting kan geen kwaad als je 'm niet direct ziet.

[.oisyn]

Er gaat in principe niets fout, het is alleen een beetje onzinnig om een random hash op die manier te berekenen. PHP kennende is de seed waarmee de interne staat van rand() geïnitialiseerd wordt gebaseerd op de huidige tijd in microseconden en het process id en bovendien maar een 32 bits int. Wat rand() verder retourneert is volledig voorspelbaar, dus er zijn maar een 'paar' hashes mogelijk (orde van grootte van 2³²).

Feanathiel schreef op donderdag 20 september 2012 @ 20:49:
Je plakt hier strings aan elkaar vast. Elk karakter van deze string bestaat slechts uit 10 verschillende waarden (0-9). Er zijn 256 mogelijke waarden voor een enkel karakter, dus in feite gooi je dus 96% van je keyspace per karakter weg.

Dit is nonsens. Hoewel het niet echt een efficiente manier is, is het absoluut niet zo dat je 96% van je keyspace weggooit. De string kent namelijk nog altijd wel 900.000.000⁴ verschillende mogelijkheden (als de RNG écht random was), oftewel ~2¹¹⁸. Een MD5 hash heeft 2¹²⁸ mogelijkheden, dus eigenlijk mis je maar 10 bits aan entropie.

Feitelijk gooi je dus 99,9% van je keyspace weg , maar eigenlijk is dat getal niet zo heel relevant. Wat relevant is is hoeveel je overhoudt. 2¹¹⁸ is wmb afdoende om brute forcing tegen te gaan . Nou alleen nog een goede RNG gebruiken. Goede opties zijn lezen van /dev/urandom (werkt niet onder Windows) of iets als http://php.net/manual/en/...l-random-pseudo-bytes.php. En dan heb je md5() ook niet meer nodig, dan bouw je gewoon je string direct van die random bits.

[ Voor 20% gewijzigd door .oisyn op 20-09-2012 21:41 ]

[RobIII]

nvm. .Oisyn was me voor en ik rekende er een tikkie langs (ik kwam op 10 bits entropie "over" i.p.v. "te kort" fwiw)

[ Voor 165% gewijzigd door RobIII op 20-09-2012 21:39 ]

[Feanathiel]

.oisyn schreef op donderdag 20 september 2012 @ 21:24:
[...]

Interessant. In principe kun je dus eigenlijk nooit aan op een RNG die niet vanuit een externe bron komt (environment noise), of zijn daar andere middelen voor? Verder bedankt voor de toelichting. Ik had het vermoeden dat de keyspace steeds verder afnam, maar deze wordt blijkbaar weer aangevuld aan de hand van de lengte die gebruikt is.

Weer een voorbeeld voor in het archief waarom er zomaar niet aan cryptografie gewerkt moet worden zonder kennis.

[YopY]

Feanathiel schreef op donderdag 20 september 2012 @ 21:44:Weer een voorbeeld voor in het archief waarom er zomaar niet aan cryptografie gewerkt moet worden zonder kennis.

Dit. Oplossingen zoals die zijn een typisch geval van klok/klepel, als in, 'Goh ja random nummers hoe meer random hoe randomer!'

[HuHu]

YopY schreef op donderdag 20 september 2012 @ 23:07:
[...]


Dit. Oplossingen zoals die zijn een typisch geval van klok/klepel, als in, 'Goh ja random nummers hoe meer random hoe randomer!'

"hoe meer random, hoe meer ran-dommer"

[AtleX]

Weer een pareltje gevonden:

JavaScript:

// determine if user may edit <redacted>
var editAllowed = false;

Goed voor de veiligheid ja. Variabele op True gezet en ik kon los.

[Merethil]

AtleX schreef op vrijdag 21 september 2012 @ 11:08:
Weer een pareltje gevonden:

JavaScript:

// determine if user may edit <redacted>
var editAllowed = false;

Goed voor de veiligheid ja. Variabele op True gezet en ik kon los.

Javascript is toch ook veilig? Een client kan daar nóóit bij! ...Oh wacht...

[Verwijderd]

Wat je met een goede Javascript debugger (Opera DragonFly, FireBug, etc) al niet kan bereiken op veel sites

[Glashelder]

Ik vind niet echt dat het in dit topic thuishoort zonder te weten of er wel een server-side controle aanwezig was.

[PatrickH89]

AtleX schreef op vrijdag 21 september 2012 @ 11:08:
Variabele op True gezet en ik kon los.

Lijkt me dat dat genoeg zegt

[eagle00789]

Raar gedoe met Highstock...

Heb je een json string met waardes, wordt de grafiek netjes getekend.
Heb je een json string met waardes met null waardes, wordt de grafiek NIET getekend, maar is de data met mouseover wel gewoon zichtbaar.
Check http://energy.decomputeur.nl/index.php maar eens, log in mep meppel als gebruikersnaam/wachtwoord, kies voor dag en kies de datum 2012-09-15........

[Glashelder]

PatrickH89 schreef op vrijdag 21 september 2012 @ 12:45:
[...]


Lijkt me dat dat genoeg zegt

zit wat in ja..

[L-VIS]

Declaratiemail van de menzis:

€ 50,70

Ik heb ze even het volgende artikel gestuurd: http://www.joelonsoftware.com/articles/Unicode.html

@kenneth:
Problems

[ Voor 60% gewijzigd door L-VIS op 21-09-2012 16:20 ]

[kenneth]

En voor jou: http://www.spatiegebruik.nl/regels.html

[Jegorex]

[qless]

Omg, geen wildcard afgevangen?

[azerty]

Of gewoon letterlijk proberen iemand met emailadres "*@*.*" uit te schrijven... Uit veiligheidsoverwegingen zou je bijvoorbeeld gewoon altijd een bevestiging kunnen tonen, om te vermijden dat emailadressen op die manier uitlekken (misschien vergezocht, dat wel )

[Avalaxy]

Ik denk niet dat het echt als een regex wordt geïnterpreteerd maar gewoon als een string...

[dcm360]

Ik zou zelf ook %@%.% nog even geprobeerd hebben. Als die zou werken, is het voor de beheerders duimen dat er een backup is.

[_Peter2_]

Avalaxy schreef op zondag 23 september 2012 @ 13:26:
Ik denk niet dat het echt als een regex wordt geïnterpreteerd maar gewoon als een string...

code:

*@*.*

is ook geen regexp (eerder wildcards zoals gebruikt in directory listings)

Bedenk me net dat MS ook wel * heeft gebruikt als wildcard in SQL-queries. (Volgens mij was dat in Access)

[ Voor 15% gewijzigd door _Peter2_ op 24-09-2012 09:00 . Reden: * als wildcard in MS Datbases toegevoegd ]

[boe2]

me dunkt dat ze geen LIKE statement gaan gebruiken om de email parameter te bepalen, dus dat zal nogal meevallen denk ik