[alg] Slechtste programmeervoorbeelden deel 4

[Haan]

Ik wist niet dat de CSS wel netjes geminified werd, dat verandert de zaak natuurlijk.

[Sorcerer8472]

Worden Peppie en Kokkie ook netjes minified?

[ID-College]

In het kader van slecht programmeren (toen ik net begon met PHP), de index.php zag er destijds zo uit:

PHP:

if($Adjust=="DeleteCL")
    {
        mysql_query("DELETE FROM CableLocation WHERE LocationID = '".$LocationID."'", $db) or die (mysql_error());
        $Redirect = "CL";
    }
if($Adjust=="DeleteCList")
    {
        mysql_query("DELETE FROM CableList WHERE CableID = '".$CableID."'", $db) or die (mysql_error());
        $Redirect = "CList";    
    }
if($Adjust=="DeleteCM")
    {
        mysql_query("DELETE FROM CableModule WHERE ModuleID = '".$ModuleID."'", $db) or die (mysql_error());
        $Redirect = "Module";
    }
if($Adjust=="DeleteCT")
    {
        mysql_query("DELETE FROM CableType WHERE TypeID = '".$TypeID."'", $db) or die (mysql_error());
        $Redirect = "CT";
    }
if($Adjust=="DeleteCP")
    {
        mysql_query("DELETE FROM CablePin WHERE PinID = '".$PinID."'", $db) or die (mysql_error());
        $Redirect = "CP";
    }
if($Adjust=="DeleteTS")
    mysql_query("DELETE FROM TableSchema WHERE SchemaID = '".$SchemaID."'", $db) or die (mysql_error());

if($Adjust=="DeleteObject")
    {
        mysql_query("DELETE FROM CableObject WHERE ObjectID = '".$ObjectID."'", $db) or die (mysql_error());
        $Redirect = "NO";
    }
    if( ($Adjust == "ChangeCList" OR $NieuwLijst == "OK" OR $NieuwCList == "Nieuw" OR $WijzigLijst == "OK" ) )
        {
            include("addnewclist.php");
            $WindowInUse = true;
        }
    if( ($WijzigLijst == "Annuleren" OR $ZoekenCList == "Zoeken" OR $Redirect == "CList" OR $Adjust == "CheckKable" OR $WijzigCList == "Annuleren" OR $SType != "") AND $Schema != "Schema" AND $NieuwCList != "Nieuw" AND $NieuwLijst != "OK" AND $CM != "CM" AND $Adjust != "ChangeCList" AND $WijzigLijst != "OK" AND $Adjust != "Checkimage")
        {
            include("cablelist.php");
            $WindowInUse = true;
        }
    if( ($NieuwObject == "Nieuw" OR $NieuwCO == "OK" OR $WijzigCO == "OK" OR $Adjust == "ChangeObject") )
        {
            include("addnewobject.php");
            $WindowInUse = true;
        }
    if( ($Redirect == "NO" OR $Adjust == "CheckObjects" OR $NieuwCO == "Annuleren" OR $WijzigCO == "Annuleren" OR $Action == "SorteerNaamCODESC" OR $Action == "SorteerNaamCOASC" OR $ZoekenCO == "Zoeken") )
        {
            $QResultObject = "SELECT * FROM TableSchema, CableObject WHERE CableObject.SchemaID = TableSchema.SchemaID AND CableObject.SchemaID = '".$SchemaID."' AND (CableObject.Naam LIKE '%".$Searchstr."%') ORDER BY CableObject.Naam";
                if($Action == "SorteerNaamCODESC")
                    $QResultObject .= " DESC";
            $RResultObject = mysql_query($QResultObject, $db) or die (mysql_error());
            include("newobject.php");
            $WindowInUse = true;
        }
    if( ($Adjust == "ChangeCL" OR $NieuwCL == "OK" OR $WijzigCL == "OK" OR $NieuwCL == "Nieuw") )
        {
            include("addnewcl.php");
            $WindowInUse = true;
        }
    if( ($CL == "CL" OR $Action == "SorteerNaamCLDESC" OR $Action == "SorteerNaamCLASC" OR $ZoekenCL == "Zoeken" OR $Redirect == "CL" OR $NieuwCL == "Annuleren" OR $WijzigCL == "Annuleren") )
        {
            $QueryCL = "SELECT * FROM CableLocation WHERE Naam LIKE '%".$Searchstr."%' ORDER BY Naam";
                if($Action == "SorteerNaamCLDESC")
                    $QueryCL .= " DESC";
            $ResultCL = mysql_query($QueryCL, $db) or die (mysql_error());
            include("newcl.php");
            $WindowInUse = true;
        }
    if( ($Redirect == "Module" OR $CM == "CM" OR $WijzigCM == "Annuleren" OR $NieuwCM == "Annuleren" OR $Action == "SorteerNaamCMASC" OR $Action == "SorteerNaamCMDESC" OR $ZoekenCM == "Zoeken" OR $NieuwCat == "Annuleren" OR $Actionn == "SortCategoryDESC" OR $Actionn == "SortCategoryASC") )
        {
            $QueryCM = "SELECT * FROM CableModule, CableModuleCat WHERE CableModule.CatID = CableModuleCat.CatID AND (CableModule.Naam LIKE '%".$Searchstr."%')";
                if($Actionn == "SortCategoryDESC") 
                    $QueryCM .= " ORDER BY CableModuleCat.Category DESC";
                elseif($Actionn == "SortCategoryASC")
                    $QueryCM .= " ORDER BY CableModuleCat.Category";
                elseif($Action == "SorteerNaamCMDESC")
                    $QueryCM .= " ORDER BY CableModule.Naam DESC";
                else
                    $QueryCM .= " ORDER BY CableModule.Naam";
            $ResultCM = mysql_query($QueryCM, $db) or die (mysql_error());
            include("newcm.php");
            $WindowInUse = true;
        }
    if( ($NieuwCM == "Nieuw" OR $NieuwCM == "OK" OR $Adjust == "ChangeCM" OR $WijzigCM == "OK") )
        {
            include("addnewcm.php");
            $WindowInUse = true;
        }
    if( ($NieuwCat == "Nieuw / Wijzig cat" OR $NieuwCat == "OK" OR $WijzigCat == "OK" OR $CableModuleCat != "") AND $NieuwCat != "Annuleren")
        {
            include("addnewccat.php");
            $WindowInUse = true;
        }
    if( ($WijzigCP == "Annuleren" OR $NieuwCP =="Annuleren" OR $Adjust == "CheckPin" OR $Action == "SorteerNaamCPASC" OR $Action == "SorteerNaamCPDESC" OR $ZoekenCP == "Zoeken" OR $Redirect == "CP") )
        {
            $QueryCP = "SELECT * FROM CableModule, CablePin WHERE CablePin.ModuleID = CableModule.ModuleID AND CablePin.ModuleID = '".$ModuleID."' AND (CablePin.Naam LIKE '%".$Searchstr."%') ORDER BY CablePin.Naam";
                if($Action == "SorteerNaamCPDESC")
                    $QueryCP .= " DESC";
            $ResultCP = mysql_query($QueryCP, $db) or die (mysql_error());
            include("newcp.php");
            $WindowInUse = true;
        }
    if( ($NieuwCP == "Nieuw" OR $NieuwCP == "OK" OR $Adjust == "ChangeCP" OR $WijzigCP == "OK") )
        {
            include("addnewcp.php");
            $WindowInUse = true;
        }
    if( ($Redirect == "CT" OR $CT == "CT" OR $NieuwCT=="Annuleren" OR $WijzigCT=="Annuleren" OR $Action == "SorteerNaamCTDESC" OR $Action == "SorteerNaamCTASC" OR $ZoekenCT == "Zoeken") )
        {
            $QueryCT = "SELECT * FROM CableType WHERE Naam LIKE '%".$Searchstr."%' ORDER BY Naam";
                if($Action == "SorteerNaamCTDESC")
                    $QueryCT .= " DESC";
            $ResultCT = mysql_query($QueryCT, $db) or die (mysql_error());
            include("newct.php");
            $WindowInUse = true;
        }
    if( ($NieuwCT=="Nieuw" OR $NieuwCT=="OK" OR $Adjust=="ChangeCT" OR $WijzigCT=="OK") )
        {
            include("addnewct.php");
            $WindowInUse = true;
        }
    if( ($Adjust=="ChangeTS" OR $WijzigTS=="OK" OR $NieuwTS=="Nieuw" OR $NieuwTS=="OK") )
        {
            include("addnew.php");
            $WindowInUse = true;
        }
    if( (!$WindowInUse OR $NieuwTS == "Annuleren" OR $Schema == "Schema" OR $Redirect == "Schema") )
    { */....................
..............
............
.............
etc...*/
}

Slecht? Anyone?

[Avalaxy]

Chique hoor

[R4gnax]

Keeper schreef op dinsdag 27 maart 2012 @ 22:36:
Hehe, de speciale IE stylesheets zijn ook wel leuk

code:

/* och lieve mensen, ondersteunt IE eindelijk border-radius, werkt het weer niet samen met gradients.
   huilt u even met ons mee?
   Aan de andere kant, wie dit bestand leest heeft een IE browser, en die huilt de hele dag al...
   toch? */

http://www.nrc.nl/css/ie9.css

Wow, kansloos. Het werkt inderdaad niet als je nog als een halve zool vasthoudt aan -ms-filter met het gradient transform filter, maar als je nou gewoon een SVG background maakt en in die SVG je gradient definieert; werkt perfect. Microsoft heeft er zelfs een tooltje voor gehost op hun eigen testdrive site. (Dat tooltje output de SVG ook nog eens minified en base64-encoded in een data URI voor je. Hoe makkelijk wil je het hebben?)

[hostname]

R4gnax schreef op woensdag 28 maart 2012 @ 20:58:
[...]

Wow, kansloos. Het werkt inderdaad niet als je nog als een halve zool vasthoudt aan -ms-filter met het gradient transform filter, maar als je nou gewoon een SVG background maakt en in die SVG je gradient definieert; werkt perfect. Microsoft heeft er zelfs een tooltje voor gehost op hun eigen testdrive site. (Dat tooltje output de SVG ook nog eens minified en base64-encoded in een data URI voor je. Hoe makkelijk wil je het hebben?)

Dat ze gewoon de standaard gradients en border radius in CSS3 ondersteunen? Leuk dat het via-via met een SVG en een tooltje wel kan, maar kunnen ze de tijd die ze besteden aan het maken van zo'n tooltje niet veel beter besteden aan gewoon de CSS standaard supporten in IE?

[AW_Bos]

Sorcerer8472 schreef op woensdag 28 maart 2012 @ 12:40:
Worden Peppie en Kokkie ook netjes minified?

Er blijft niks van het duo over....

[xiffy]

Haan schreef op woensdag 28 maart 2012 @ 09:25:
Ik vind het eerlijk gezegd niet heel professioneel. Prima als je zoiets in je eigen website stopt, maar in opdracht van een klant vind ik het niet zo netjes.

Grote voordeel is, dit IS onze website. Mijn collega's en ik werken fulltime bij NRC en maken de websites. En nee, we verliezen niet onze baan over onze gein.

[Firesphere]

xiffy schreef op woensdag 28 maart 2012 @ 23:17:
[...]

Grote voordeel is, dit IS onze website. Mijn collega's en ik werken fulltime bij NRC en maken de websites. En nee, we verliezen niet onze baan over onze gein.

Comeback of the year?

Ach, ik heb het ook... speciaal voor IE7 heb ik vaak divs met de class "FuckIE" of "FYIE7"

[Avalaxy]

xiffy schreef op woensdag 28 maart 2012 @ 23:17:
[...]

Grote voordeel is, dit IS onze website. Mijn collega's en ik werken fulltime bij NRC en maken de websites. En nee, we verliezen niet onze baan over onze gein.

Hey leuk, een NRC developer Hoe kom je nu hier terecht? (in dit topic bedoel ik)

[ Voor 3% gewijzigd door Avalaxy op 28-03-2012 23:24 ]

[xiffy]

Ach ja, we kwamen onze open-dir vandaag via twitter op het spoor, de rest is geschiedenis.

[Firesphere]

xiffy schreef op woensdag 28 maart 2012 @ 23:26:
Ach ja, we kwamen onze open-dir vandaag via twitter op het spoor, de rest is geschiedenis.

Ach ja, de class/id's in de HTML zeiden al genoeg, die kende ik inderdaad al.

Wiep is inderdaad een schatje hoor!

Enneuh... devver die NU PAS registreert op t.net?

---

In het kader van slechte programmeurs...

Adobe wil 9% van de opbrengst van goedlopende flash-games.

Ja, loosely translated, maar toch...

[ Voor 25% gewijzigd door Firesphere op 28-03-2012 23:41 ]

[cellcote]

Firesphere schreef op woensdag 28 maart 2012 @ 23:31:
Enneuh... devver die NU PAS registreert op t.net?

Registratiedatum is anders toch al in 2000 hoor

[Firesphere]

cellcote schreef op woensdag 28 maart 2012 @ 23:44:
[...]


Registratiedatum is anders toch al in 2000 hoor

* Firesphere is burned en druipt af met z'n 8 jaar lurken en dan pas registreren

Maar wel meer posts!

[Alex)]

xiffy schreef op woensdag 28 maart 2012 @ 23:17:
[...]

Grote voordeel is, dit IS onze website. Mijn collega's en ik werken fulltime bij NRC en maken de websites. En nee, we verliezen niet onze baan over onze gein.

Het zijn echt geweldige comments. Ga vooral door

[xiffy]

Dank allen voor de positieve commentaren op onze css, dat was bij het live-gaan van deze site wel anders. (Toen waren we nog niet ge-minified, en maakte de gemiddelde marketeer er een sport van onze Jeugd van Tegenwoordig class-names af te kraken omdat zij het niet begrepen en het dus onprofessioneel was)
@Firesphere hehe jij wint met veel meer posts.
Tjonge jonge, alweer 11 jaar geregistreerd ...

[Firesphere]

xiffy schreef op donderdag 29 maart 2012 @ 00:09:
Dank allen voor de positieve commentaren op onze css, dat was bij het live-gaan van deze site wel anders. (Toen waren we nog niet ge-minified, en maakte de gemiddelde marketeer er een sport van onze Jeugd van Tegenwoordig class-names af te kraken omdat zij het niet begrepen en het dus onprofessioneel was)
@Firesphere hehe jij wint met veel meer posts.
Tjonge jonge, alweer 11 jaar geregistreerd ...

Meeste devvers kunnen dergelijk commentaar etc. wel waarderen gelukkig

Ben al lang blij dat de meeste dingen die ik bouw, ook bij mij blijven... want rants als deze (in de php) zijn soms vaker regel dan uitzondering als ik oudere "voor mijn tijd" code bekijk:

PHP:

/**
 * WTF? Welke malloot heeft dit bedacht? Nogal zinloze functie he?
 * Kan iemand alstublieft uitleggen waarom hier dit wordt gedaan? Het is allemaal al bekend!
 * Ik vind het wel prima, zoek het maar uit. Zie nieuwe class "ActualWorkingDatacollector"
 * Mocht de originele schrijver van de komende functies dit lezen:
 * Zoek ff ander werk wil je? Dit schiet niet op.
 */

[xiffy]

Gelukkig zitten wij naast elkaar en kunnen we het gewoon zeggen. Of we noteren het in de commit bericht.

[Firesphere]

xiffy schreef op donderdag 29 maart 2012 @ 00:22:
Gelukkig zitten wij naast elkaar en kunnen we het gewoon zeggen. Of we noteren het in de commit bericht.

Flamewar tijd?
Git, HG of SVN?

Ik ben al tijden bezig om alles van m'n werk naar mijn eigen private git-repo's te verhuizen... is de overstap makkelijker...

[xiffy]

Git tegenwoordig, SVN 2 jaar geleden. Darcs op een thuissysteem, gewoon omdat het kan
En CVS bij mijn vorige bazen. Nu zou ik voor Git kiezen als ik ergens wat moet introduceren.
[update]
dus eigenlijk ben ik een omnivoor die nog nooit met HG heeft gewerkt
[/update]

[ Voor 22% gewijzigd door xiffy op 29-03-2012 00:37 ]

[Firesphere]

xiffy schreef op donderdag 29 maart 2012 @ 00:26:
Git tegenwoordig, SVN 2 jaar geleden. Darcs op een thuissysteem, gewoon omdat het kan
En CVS bij mijn vorige bazen. Nu zou ik voor Git kiezen als ik ergens wat moet introduceren.
[update]
dus eigenlijk ben ik een omnivoor die nog nooit met HG heeft gewerkt
[/update]

HG is een variatie op Git, heb er ook geen ervaring mee, maar het is ongeveer hetzelfde dacht ik (correct me if I'm wrong!)

Ach, alles beter dan CVS of SVN toch?

Hebben jullie een local GIT repo of via github o.i.d.?

[Down]

Git, SVN, HG.... Pff, ik zit hier met sourcesafe. Ben benieuwd welke bestanden ik morgen weer 'uitgecheckt' heb.

[xiffy]

@Down lol
Onze repo's leven op unfuddle, tracker systeempje, svn en git repo's en we wonen daar alweer een paar jaar. Toen was bestond github denk ik nog niet eens, en we zaten zelf nog op SVN. HG en Git lijken inderdaad veel op elkaar. Waarbij HG syntax weer veel op SVN lijkt.
Ik ben geen version-control ninja, maar kan ermee doen wat ik moet doen, code pushen en productie up to date houden terwijl we ergens anders aan coole nieuwe features werken.

[Firesphere]

Down schreef op donderdag 29 maart 2012 @ 00:42:
Git, SVN, HG.... Pff, ik zit hier met sourcesafe. Ben benieuwd welke bestanden ik morgen weer 'uitgecheckt' heb.

De kracht van Git en HG is vooral "updates" lokaal houden, terwijl je aan je eigen versie werkt. Dit kan je makkelijk mergen en geen conflicts-gezeik omdat het "line-by-line" gaat met herkenning van "ow, dit is nieuw, dit bestaat al 10 lines verder"

sourcesafe ken ik niet, en volgens mij... moet ik daar blij mee zijn?

Version control is easy, just learn the syntax

[Kwastie]

Firesphere schreef op donderdag 29 maart 2012 @ 00:54:
[...]
sourcesafe ken ik niet, en volgens mij... moet ik daar blij mee zijn?

Sourcesafe is een ongelofelijk kl**** pakket, gemaakt door Microsoft. Het communiceert bijv. dmv. SMB (Ja een netwerkshare ). Voordat je ergens in gaat werken moet je files globaal locken. Maar als je wilt compilen mogen files niet gelocked zijn, maar soms ook weer wel

anyway leesvoer: http://www.highprogrammer.com/alan/windev/sourcesafe.html

[ Voor 10% gewijzigd door Kwastie op 29-03-2012 01:15 ]

[kenneth]

Geen serieuze professional gebruikt toch nog VSS? Down, klaag je werkgever aan, echt

[Dragor]

kenneth schreef op donderdag 29 maart 2012 @ 08:22:
Geen serieuze professional gebruikt toch nog VSS? Down, klaag je werkgever aan, echt

Tijd voor een ARBO-dienst voor software pakketten!

[alienfruit]

Firesphere schreef op woensdag 28 maart 2012 @ 23:31:
Adobe wil 9% van de opbrengst van goedlopende flash-games.

Alleen voor webgames. Het geld niet voor AIR based applicaties . 9% Angry Birds op Facebook is natuurlijk wel leuk voor Adobe. Maar goed, ik weet echt niet meer wat ze aan het doen zijn bij Adobe. Stelletje idioten

[Styxxy]

Kwastie schreef op donderdag 29 maart 2012 @ 01:11:
[...]
Sourcesafe is een ongelofelijk kl**** pakket, gemaakt door Microsoft.

Gekocht en verder ontwikkeld door Microsoft .

[Down]

kenneth schreef op donderdag 29 maart 2012 @ 08:22:
Geen serieuze professional gebruikt toch nog VSS? Down, klaag je werkgever aan, echt

Geloof me, dit probeer ik hier ook duidelijk te maken. Maar er waait hier nu eindelijk een nieuwe wind, er wordt gekeken naar TFS.

Ik word ZO chagrijnig van Sourcesafe. Scheitpakket claimt ook dat ik bepaalde bestanden heb uitgecheckt terwijl het niet zo is. Echt ongelooflijk. Laten we het maar niet hebben over de mergecapaciteiten. Of dat een rename van een file direct op de server wordt doorgevoerd waardoor de buildserver over z'n nek gaat

[Macros]

Ik heb nooit gebruik gemaakt van SourceSafe, maar ik vind alle verhalen erover wel enorm vermakelijk. Het design is gewoon zo fout dat je het alleen je ergste vijanden toewenst. Ook grappig is dat ze het bij MS zelf nooit gebruikt hebben!

[urk_forever]

Kom net deze tegen in de code van onze applicatie:

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.Date.AddHours(now.Hour).AddMinutes(now.Minute).AddSeconds(now.Second);

Ik snap er niet echt iets van? De ontwikkelaar die het gemaakt had snapte het ook niet.

[roboreaper]

urk_forever schreef op vrijdag 30 maart 2012 @ 10:39:
Kom net deze tegen in de code van onze applicatie:

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.Date.AddHours(now.Hour).AddMinutes(now.Minute).AddSeconds(now.Second);

Ik snap er niet echt iets van? De ontwikkelaar die het gemaakt had snapte het ook niet.

hij vraagt tijd op en daarnaast nog een var met de zelfde tijd erin? huh?
oh wacht.. hij gebruikt die regel om datum uit date time te splitsen... tja...

[ Voor 7% gewijzigd door roboreaper op 30-03-2012 10:45 ]

[Amras]

kenneth schreef op donderdag 29 maart 2012 @ 08:22:
Geen serieuze professional gebruikt toch nog VSS? Down, klaag je werkgever aan, echt

Bij m'n huidige klant wordt voor alle projecten VSS gebruikt, ook al staat het invoeren van TFS als source control systeem al een jaar of drie 'op de drempel'.

Bij m'n vorige klant werd in het project waaraan ik werkte ook VSS gebruikt. In totaal werk ik dus nu al 4 jaar onafgebroken met die meuk en weet ik eigenlijk niet beter.

[Styxxy]

Amras schreef op vrijdag 30 maart 2012 @ 10:47:
[...]
Bij m'n vorige klant werd in het project waaraan ik werkte ook VSS gebruikt. In totaal werk ik dus nu al 4 jaar onafgebroken met die meuk en weet ik eigenlijk niet beter.

Brainwash . Laat de ontwikkelaars lang genoeg met iets werken en ze kennen niet meer beter . Ik ben blij dat ik met TFS mag werken.

[urk_forever]

roboreaper schreef op vrijdag 30 maart 2012 @ 10:44:
[...]


hij vraagt tijd op en daarnaast nog een var met de zelfde tijd erin? huh?
oh wacht.. hij gebruikt die regel om datum uit date time te splitsen... tja...

Precies, mijn eerste reactie was ook

[.oisyn]

roboreaper schreef op vrijdag 30 maart 2012 @ 10:44:
hij vraagt tijd op en daarnaast nog een var met de zelfde tijd erin? huh?
oh wacht.. hij gebruikt die regel om datum uit date time te splitsen... tja...

Hij pakt "now", daar het datumgedeelte van, en daar telt ie vervolgens het aantal uren, minuten en seconden van "now" erbij op.

Als je het mij vraagt probeert ie gewoon af te ronden op hele seconden. Niet eens zo heel raar, maar ik zou het aantal ticks er gewoon vanaf hebben getrokken:

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.AddTicks(-now.Ticks);

[urk_forever]

.oisyn schreef op vrijdag 30 maart 2012 @ 11:13:
[...]

Hij pakt "now", daar het datumgedeelte van, en daar telt ie vervolgens het aantal uren, minuten en seconden van "now" erbij op.

Als je het mij vraagt probeert ie gewoon af te ronden op hele seconden. Niet eens zo heel raar, maar ik zou het aantal ticks er gewoon vanaf hebben getrokken:

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.AddTicks(-now.Ticks);

Zou kunnen, hij wist het zelf niet meer.

[Grijze Vos]

.oisyn schreef op vrijdag 30 maart 2012 @ 11:13:
[...]

Hij pakt "now", daar het datumgedeelte van, en daar telt ie vervolgens het aantal uren, minuten en seconden van "now" erbij op.

Als je het mij vraagt probeert ie gewoon af te ronden op hele seconden. Niet eens zo heel raar, maar ik zou het aantal ticks er gewoon vanaf hebben getrokken:

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.AddTicks(-now.Ticks);

Dat is absoluut niet equivalent met de code die gepost is.

[.oisyn]

Als je er dan ook even bijzegt waarom ipv gewoon maar een losse opmerking te posten...

.edit: oh ik zie het al, Ticks is het totaal aantal ticks, niet het aantal ticks binnen een seconde. Tja, dan lijkt me de code zoals ie was ook gewoon het handigst. Je kan het totaal aantal ticks nemen en dat afronden op een veelvoud van 10.000.000, maar erg veel duidelijker wordt het daar ook niet van.

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.AddTicks(-(now.Ticks % 10000000));

Wel suf zeg, DateTime is dus preciezer dan dan z'n afzonderlijke componenten.

[ Voor 82% gewijzigd door .oisyn op 30-03-2012 11:37 ]

[TJHeuvel]

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.Date.AddHours(now.Hour);

Misschien ben ik simpel, maar dit zorgt er toch voor dat het om 5 uur 'sochtends 10 uur word? Hij telt gewoon het aantal uur bij de huidige tijd op.

[ Voor 8% gewijzigd door TJHeuvel op 30-03-2012 11:35 ]

[.oisyn]

Nee, er staat now.Date.AddHours()...

Als now 2012-03-30 11:23:53 is, dan is het resultaat van now.Date gelijk aan 2012-03-30 0:00:00

[ Voor 56% gewijzigd door .oisyn op 30-03-2012 11:38 ]

[Amras]

TJHeuvel schreef op vrijdag 30 maart 2012 @ 11:35:

C#:

DateTime now = DateTime.Now;
DateTime dateTime = now.Date.AddHours(now.Hour).AddMinutes(now.Minute).AddSeconds(now.Second);

Misschien ben ik simpel, maar dit zorgt er toch voor dat het om 5 uur 'sochtends 10 uur word? Hij telt gewoon het aantal uur bij de huidige tijd op.

Nee, hij telt het aantal uren, minuten en seconden op bij de huidige date.

... spuit11

[roboreaper]

Amras schreef op vrijdag 30 maart 2012 @ 11:36:
[...]

Nee, hij telt het aantal uren, minuten en seconden op bij de huidige date.

... spuit11

nope iedereen heeft het fout

ff snel in VS die code toegepast
er veranderd absoluut niks

ik print now en Datetime allebij de zelfde notatie en tijd dus now = 30-3-2012 11:52:33 dan is Date time dat ook.



fail code....

[ Voor 6% gewijzigd door roboreaper op 30-03-2012 11:58 ]

[Amras]

roboreaper schreef op vrijdag 30 maart 2012 @ 11:52:
[...]


nope iedereen heeft het fout

ff snel in VS die code toegepast

er veranderd absoluut niks

ik rpint now en Datetime allebij de zelfde notatie en tijd dus now = 30-3-2012 11:52:33 dan is Date time dat ook.

fail code....

Ook het aantal milliseconden geprint?

[PiepPiep]

code:

DateTime dateTime = now.Date.AddHours(now.Hour).AddMinutes(now.Minute).AddSeconds(now.Second);

Dit kan volgens mij ook hele foute antwoorden geven.
Als tussen het aanroepen van AddHours en AddMinutes de tijd net verspringt van 11:59 naar 12:00 dan krijg je 11 uur 0 minuten.

[Amras]

PiepPiep schreef op vrijdag 30 maart 2012 @ 12:04:

code:

DateTime dateTime = now.Date.AddHours(now.Hour).AddMinutes(now.Minute).AddSeconds(now.Second);

Dit kan volgens mij ook hele foute antwoorden geven.
Als tussen het aanroepen van AddHours en AddMinutes de tijd net verspringt van 11:59 naar 12:00 dan krijg je 11 uur 0 minuten.

Daarom wordt de huidige datum/tijd ook in een variabele gestopt, zodat de tijd niet kan verspringen.

[.oisyn]

roboreaper schreef op vrijdag 30 maart 2012 @ 11:52:
er veranderdt absoluut niks

Misschien moet je voortaan alle reacties even duidelijk lezen

fail code....

fail post....

[ Voor 61% gewijzigd door .oisyn op 30-03-2012 12:41 ]

[PiepPiep]

Amras schreef op vrijdag 30 maart 2012 @ 12:36:
[...]

Daarom wordt de huidige datum/tijd ook in een variabele gestopt, zodat de tijd niet kan verspringen.

Ah, lezen is ook een vak apart
Maar ik heb het dus wel eens ooit ergens gezien dat er DateTime.Now meerdere keren op een regel werd gebruikt. Dan zou het dus fout kunnen gaan.

[Vedett.]

Ik denk dat het op zoiets neerkomt:

C#:

var now = DateTime.Now;
now = now.AddMilliseconds(-1*now.Millisecond);

[Grijze Vos]

.oisyn schreef op vrijdag 30 maart 2012 @ 11:29:
Als je er dan ook even bijzegt waarom ipv gewoon maar een losse opmerking te posten...

Hehe, sorry. Gewoonte van me om in prg niet alles voor te kauwen voor mensen, zodat GoT geen code snackbar wordt. Die instelling is een beetje overbodig in dit topic uiteraard.

[.oisyn]

Vedett. schreef op vrijdag 30 maart 2012 @ 13:03:
Ik denk dat het op zoiets neerkomt:

C#:

var now = DateTime.Now;
now = now.AddMilliseconds(-1*now.Millisecond);

Ik heb al gezegd waarom dat niet klopt. Een DateTime is preciezer dan een milliseconde (10.000 keer zo precies)

En waarom -1 * x ipv gewoon -x

[ Voor 4% gewijzigd door .oisyn op 30-03-2012 14:51 ]

[BertS]

Bij de Kamer van Koophandel moet je jaarlijks je wachtwoord wijzigen. Geef je een nieuw wachtwoord in, krijg je het volgende:

quote: KvK

Fout opgetreden bij het wijzigen van het wachtwoord (FOUT3007).

Probleem:

Er is een fout opgetreden bij het wijzigen van uw wachtwoord. Deze fout is ontstaan omdat u zich niet heeft gehouden aan één van de vereisten van het nieuwe wachtwoord of omdat u een verkeerde toegangscode heeft ingevuld.

Vereisten:
U dient een niet eerder gebruikt wachtwoord in te voeren.
De toegangscode heeft 8 tekens (deze begint altijd met WWW).
U dient alle velden in te vullen.
Het wachtwoord moet minimaal 6 en maximaal 10 tekens hebben.
Het wachtwoord ter controle moet gelijk zijn aan het nieuwe wachtwoord.
Het wachtwoord mag geen twee aangrenzende cijfers bevatten en ook niet beginnen met een cijfer.
Het wachtwoord mag geen punten, komma's en dergelijke bevatten.

Oplossing:
Wij adviseren u om bovenstaande tekst goed te lezen en uw wachtwoord OPNIEUW te wijzigen.
Als dit niet werkt bel dan onze klantenservice (Tel. [phone]) onder vermelding van uw toegangscode en bovenstaande melding (FOUT3007).

[sopsop]

Lijkt mij geen slecht programmeervoorbeeld, eerder slechte - danwel belachelijke - functionele specificaties.

[Haan]

sopsop schreef op woensdag 18 april 2012 @ 08:50:
Lijkt mij geen slecht programmeervoorbeeld, eerder slechte - danwel belachelijke - functionele specificaties.

Als er een max. wachtwoord lengte is, weet je eigenlijk al hoe laat het is, dan wordt je wachtwoord zeer waarschijnlijk plain-text opgeslagen, waarom zou je anders die eis hebben..
Dat kan je slecht programmeren noemen, of slecht databaseontwerp

[Sebazzz]

Plain-text of (hopelijk) ge-encrypt. Bij encryptie is de lengte ook te voorspellen. Neemt niet weg dat het gigantisch stom is. Maar in de ideale wereld kon je ook bij een website passphrase invoeren in plaats van een wachtwoord met een bepaalde complexiteit.

[azerty]

BertS schreef op woensdag 18 april 2012 @ 08:01:
Bij de Kamer van Koophandel moet je jaarlijks je wachtwoord wijzigen. Geef je een nieuw wachtwoord in, krijg je het volgende:

[...]

Tjah, hoe veel keer ik zo'n beperkingen ook al niet tegengekomen ben, of bij wachtwoord recovery mijn oude wachtwoord in plaintext teruggekregen heb (wat het vermoeden doet rijzen dat er geen hashing op toegepast wordt, versleuteling kan dan natuurlijk nog altijd, maar goed, het schept geen vertrouwen)... Om van te huilen.

Bijvoorbeeld de vdab:

In uw mail merkte u ook dat dat wanneer u zich registreerde op de hoofdsite (ik vermoed www.vdab.be) een wachtwoord van 30 tekens niet geaccepteerd wordt. Het paswoord dat u zelf kan kiezen op de website van VDAB kan maximaal 25 tekens lang zijn.

25 is op zich al beter, maar waarom toch weer die limiet?

[KnoppenSpook]

Ja waarom een limiet. Je gaat het ww toch niet plain in je DB opslaan?
En een hash heeft toch altijd dezelfde lengte. Maakt niet uit hoe lang je ww is toch?

[BertS]

Sebazzz schreef op woensdag 18 april 2012 @ 09:42:
Plain-text of (hopelijk) ge-encrypt. Bij encryptie is de lengte ook te voorspellen. Neemt niet weg dat het gigantisch stom is. Maar in de ideale wereld kon je ook bij een website passphrase invoeren in plaats van een wachtwoord met een bepaalde complexiteit.

Dat ja, wist dat ik het ergens had gezien en wilde het hier noemen.

[Avalaxy]

Wat betreft password: ik heb in MVC3 op m'n password field een validation attribute om de string length te checken, maar daarbij moet je per sé een max length opgeven. Passwords zijn bij mij dan ook minimaal 8, maximaal 255 tekens. Lijkt me best redelijk.

Wachtwoorden zelf worden met SHA-2 of SHA-3 opgeslagen.

[Keiichi]

Avalaxy schreef op woensdag 18 april 2012 @ 20:21:
Wat betreft password: ik heb in MVC3 op m'n password field een validation attribute om de string length te checken, maar daarbij moet je per sé een max length opgeven. Passwords zijn bij mij dan ook minimaal 8, maximaal 255 tekens. Lijkt me best redelijk.

Wachtwoorden zelf worden met SHA-2 of SHA-3 opgeslagen.

SHA-3? Ik lees er wel iets over, maar is het ook al ergens imgeplementeerd?

Verder kun je altijd een extra salt toevoegen en meerdere rounds hashen waardoor brutoforcen nog moeilijker word.

[Avalaxy]

Keiichi schreef op woensdag 18 april 2012 @ 20:25:
[...]


SHA-3? Ik lees er wel iets over, maar is het ook al ergens imgeplementeerd?

Verder kun je altijd een extra salt toevoegen en meerdere rounds hashen waardoor brutoforcen nog moeilijker word.

Excusez moi, ik dacht dat SHA-384 en SHA-512 onder SHA-3 vallen maar dat blijkt SHA-2 te zijn. Die dus

[Matis]

.oisyn schreef op vrijdag 30 maart 2012 @ 14:51:
En waarom -1 * x ipv gewoon -x

Ik schrijf ook altijd -1 * x ipv -x. -x kan een typo zijn, als je -1 * x schrijft. Dan heb je er iig over nagedacht

[Verwijderd]

En als je -x schrijft heb je er niet over nagedacht? Vind het een vreemde aanname.

[Matis]

Verwijderd schreef op woensdag 18 april 2012 @ 22:38:
En als je -x schrijft heb je er niet over nagedacht? Vind het een vreemde aanname.

Dat zeg ik niet. Ik zei dat -x een typo kan zijn. -1*x absoluut niet. Dan heb je echt de intentie om de negatieve waarde te geven. Dus heb je er over nagedacht, opzettelijk. En dus geen ongeluk.
De uitkomst is natuurlijk hetzelfde.

[ Voor 4% gewijzigd door Matis op 18-04-2012 22:53 ]

[RayNbow]

Tsk, iedereen weet toch dat je 0-x moet opschrijven i.p.v. -1*x? Aftrekken kost minder rekentijd dan vermenigvuldigen!

[YakuzA]

dan kun je ook -1*-1*-1*x neerzetten, dan weet je absoluut, absoluut, absoluut zeker dat het opzettelijk is
Dan heb je echt de intentie om de negatieve waarde te geven. Dus heb je er over nagedacht, opzettelijk. En dus geen ongeluk.

[Haan]

De kans dat -x een typo is, lijkt me vrij klein.. De '-' zit op zo'n plek op je toetsenbord (iig bij de standaard US-int. indeling) dat je die echt niet per ongeluk intypt

[Verwijderd]

Plus lijkt me eerder dat je dan deze fout maakt:
-1*-x

Wat gelijk is aan 1*x.

[.oisyn]

Ik vind het ook maar onzin. Wanneer kun je in hemelsnaam -x als typo typen . Volgens die logica kun je geen enkele code meer vertrouwen, ook -1*x niet, want wellicht had het juist 1*x moeten zijn, wat je immers typte omdat gewoon 'x' een typo had kunnen zijn.

Veel belangrijker, als ik persoonlijk -1*x lees dan is de intentie voor mij niet meteen duidelijk, integenstelling tot -x waarvan ik meteen snap wat ermee bedoelt wordt. Als je dan toch het rare typo-argument aan wilt houden, typ dan -(x)

[ Voor 87% gewijzigd door .oisyn op 19-04-2012 09:44 ]

[YopY]

C#:

var now = DateTime.Now; 
now = now.AddMilliseconds(-now.Millisecond); // ja dit is opzettelijk, niet twijfelen, gij kleingelovige!

[ Voor 29% gewijzigd door YopY op 19-04-2012 12:47 ]

[.oisyn]

Wel jammer dat die oplossing niet klopt. Maar ja, de comment zegt dat het opzettelijk is en men niet mag twijfelen

[-Sander1981-]

C#:

DateTime now = DateTime.Now;
DateTime dateTime = new DateTime(now.Year, now.Month, now.Day, now.Hour, now.Minute, now.Second);

Is het niet handiger om de daarvoor bedoelde constructor te gebruiken?

[Verwijderd]

Need to turn a unix timestamp into a date string? No problem:

PHP:

if ($seconds/60 >=1){
    $minutes=floor($seconds/60);
    if ($minutes/60 >= 1){ # Hours
        $hours=floor($minutes/60);
        if ($hours/24 >= 1){ #days
            $days=floor($hours/24);
            if ($days/7 >=1){ #weeks
                 $weeks=floor($days/7);
                 if ($weeks>=2) $return="$weeks Weeks";
                 else $return="$weeks Week";
            } #end of weeks
            $days=$days-(floor($days/7))*7;
            if ($weeks>=1 && $days >=1) $return.=", ";
            if ($days >=2) $return.=" $days days";
            if ($days ==1) $return.=" $days day";
        } #end of days
        $hours=$hours-(floor($hours/24))*24;
        if ($days>=1 && $hours >=1 && ($weeks<1)) $return.=", ";
        if ($hours >=2 && ($weeks<1)) $return.=" $hours hours";
        if ($hours ==1 && ($weeks<1)) $return.=" $hours hour";
    } #end of Hours
    $minutes=$minutes-(floor($minutes/60))*60;
    if ($hours>=1 && $minutes >=1 && ($days<1&&$weeks<1)) $return.=", ";
    if ($minutes >=2 && ($days<1&&$weeks<1)) $return.=" $minutes minutes";
    if ($minutes ==1 && ($days<1&&$weeks<1)) $return.=" $minutes minute";
} #end of minutes
$seconds=$integer-(floor($integer/60))*60;
if ($minutes>=1 && $seconds >=1 && ($hours<1&&$days<1&&$weeks<1)) $return.=", ";
if ($seconds >=2 && ($hours<1&&$days<1&&$weeks<1)) $return.=" $seconds seconds";
if ($seconds ==1 && ($hours<1&&$days<1&&$weeks<1)) $return.=" $seconds second";

return ltrim($return);

[Verwijderd]

WTF

Sorry, meer kan ik er niet op zeggen.

[MBV]

Ik heb laatst ook een stuk of 20 strcat statements vervangen door 1 sprintf. Was een stukje penny-wise pound-foolish code: misschien wel micro-optimalisaties door sprintf te vermijden, maar op andere plekken echt minder handig. Wat denk je van deze constructie?

C:

//header
#if (LOGGING==LOW || LOGGING==MED || LOGGING==HIGH)
MY_LOGGING(TEXT) MyLogging(text);
#else
MY_LOGGING(TEXT)
#endif

//implementatie
MyLogging(const char* text)
{
#if MY_LOGGING==MED
//10 strcat statements
#endif
}

Macro's om logging aan/uit te zetten snap ik, en dat je de method-call wilt voorkomen in productiecode ook wel een beetje, maar dit is wel een beetje bijzonder

[ Voor 13% gewijzigd door MBV op 20-04-2012 23:08 ]

[CMG]

in .NET kan je ook gewoon .Date property van je DateTime gebruiken...

[Petervanakelyen]

Op school zojuist verwezen naar een website om oefeningen op te doen. De code ziet er als volgt uit:

code:

var ans = new Array;
var done = new Array;
var score = 0;
ans[1] = "d";
ans[2] = "b";
ans[3] = "a";
ans[4] = "b";
ans[5] = "c";

Blijkbaar hebben ze zelfs rekening gehouden met gebruikers die naar de source kijken :

code:

function NextLevel () {
if (score > 40) {
alert("Cheater!");
}

[Davio]

Ooooh, try2hack!

[.oisyn]

MBV schreef op vrijdag 20 april 2012 @ 23:08:
Macro's om logging aan/uit te zetten snap ik, en dat je de method-call wilt voorkomen in productiecode ook wel een beetje, maar dit is wel een beetje bijzonder

Die macro faalt sowieso

C:

if (condition)
    MyLogging("bla")
DoSomethingImportant();

Haal de ; weg uit de macro zodat bovestaande code ongeldig wordt, en maak van de andere definitie een do;while(0) zodat het een ; vereist

[MBV]

I know. Gelukkig heeft dat ding in jaren niet aangestaan, dus loop je dat risico niet Zorgwekkender is dat dit de standaard-stijl is van onze leverancier. Het is een van de includes die bij een model-compiler wordt meegeleverd. Gelukkig hebben ze in de nieuwe versie de standaard-macro's niet voorzien van ;, dus die moet je altijd neerzetten. En natuurlijk komt in de gegenereerde code jouw voorbeeld nooit voor, maar dat maakt het niet echt veel beter ben ik bang.

[Serpie]

Petervanakelyen schreef op woensdag 25 april 2012 @ 08:53:
Op school zojuist verwezen naar een website om oefeningen op te doen. De code ziet er als volgt uit:

code:

var ans = new Array;
var done = new Array;
var score = 0;
ans[1] = "d";
ans[2] = "b";
ans[3] = "a";
ans[4] = "b";
ans[5] = "c";

Blijkbaar hebben ze zelfs rekening gehouden met gebruikers die naar de source kijken :

code:

function NextLevel () {
if (score > 40) {
alert("Cheater!");
}

Blijkbaar hebben ze het niet zelf geschreven, maar gewoon copy paste:
http://www.robynoro.info/...cellaneous/user-quiz.html

[.oisyn]

Pareltje van mezelf. Zojuist ingetypt, maar ik post het maar hier voor m'n eigen gemoedsrust

Vaak wil ik zoiets:

code:

if (conditie)
{
    doe iets;
    if (andere conditie gebaseerd op 'doe iets')
        goto elseCase
}
else
{
elseCase:
    doe iets anders;
}

Kun je wel gestructureerd oplossen. Bijvoorbeeld door van doe iets een functie te maken die de juiste conditie terugstuurt zodat de eerste if gewoon if (conditie && functieDoeIets()) kan zijn, maar dat wordt wat lastiger als er veel afhankelijke lokale state is. Een andere oplossing is met extra lokale booleans, maar dat wordt er over het algemeen ook niet heel veel leesbaarder op:

code:

bool ok = conditie;
if (ok)
{
    doe iets;
    if (andere conditie gebaseerd op 'doe iets')
        ok = false;
}

if (!ok)
   doe iets anders;

Werkt prima, maar imho niet heel erg duideljik dat de 'else' case een fall through is.

Mijn oplossing:

code:

switch(conditie)
{
case true:
    doe iets;
    if (!andere conditie gebaseerd op 'doe iets')
        break;
    // fall through
case false:
    doe iets anders;
}

[Verwijderd]

Huh?

code:

bool doeIets()
{
    //iets doen en controleren

   return (conditie gebaseerd op iets doen);
}

void functieVanVorigePoster()
{
     if(!(conditie && doeIets()))
     {
            //Doe iets anders
     }
}

[ Voor 16% gewijzigd door Verwijderd op 25-04-2012 15:30 ]

[.oisyn]

Die oplossing behandel ik toch ook gewoon?
"maar dat wordt wat lastiger als er veel afhankelijke lokale state is"

[Verwijderd]

Daar hebben ze parameters voor uitgevonden

State -> stateObject, doorgeven aan doeIets().

[.oisyn]

Wat gewoon irritant is. Zit je in C++ weer dingen via een pointer te accessen omdat ze nou eenmaal zijn geconstruct op de stack van de aanroepende functie, etc. Bottom line is, ik kwam zelf al met die oplossing, maar ik heb er in dit geval bewust niet voor gekozen.

[ Voor 101% gewijzigd door .oisyn op 25-04-2012 15:57 ]

[Firesphere]

Even een voorbeeldje van mezelf. Een bepaalde query ging soms mis, als er een product was weggegooit. Het moest gefixt worden, maar
"wel zo snel mogelijk, want we krijgen er niet voor betaald"
Oplossing,:

PHP:

try{
    $this->DB->getObject('product', $ID):
} catch(exception $e){ 
    // ja doei;
}

* Firesphere schaamt zich eigenlijk stiekem er voor...

[HeSitated]

Vandaag weer een pareltje tegen gekomen van hoe iets simpels ingewikkeld te maken valt....

C#:

private SearchFilterCompany _filter = null;
public SearchFilterCompany SearchFilter
{
    get
    {
        if (_filter == null) _filter = GetSearchFilter();
        return _filter;
    }
    set { SetSearchFilter(value); }
}

private void SetSearchFilter( SearchFilterCompany filter )
{
    HttpContext.Current.Session[SessionKeySearchFilter] = filter as SearchFilterCompany;
}

private SearchFilterCompany GetSearchFilter()
{
    SearchFilterCompany zoekFilter = null;

    if (HttpContext.Current.Session[SessionKeySearchFilter] != null)
    {
        zoekFilter = HttpContext.Current.Session[SessionKeySearchFilter] as SearchFilterCompany;
    }
    return zoekFilter;
}

Vooral dit stuk:

C#:

    if (HttpContext.Current.Session[SessionKeySearchFilter] != null)
    {
        zoekFilter = HttpContext.Current.Session[SessionKeySearchFilter] as SearchFilterCompany;
    }

Briljant toch?

Als je dan ook nog eens bedenkt dat dit in de DAL-library zat....

[Grijze Vos]

Moet iemand met java ervaring zijn. Anders ga je niet je properties nog eens wrappen in functies.

[farmertjes]

Ik geloof niet dat ik "Vooral dit stuk" begrijp. Als je een niet bestaande key opvraagt uit de current session wordt er toch een exception gethrowed ... z'n check voor null is dan toch goed? of bedoelde je de overbodige cast naar SearchFilterCompany?

[Struikrover]

Firesphere schreef op woensdag 25 april 2012 @ 16:38:
Even een voorbeeldje van mezelf. Een bepaalde query ging soms mis, als er een product was weggegooit. Het moest gefixt worden, maar
"wel zo snel mogelijk, want we krijgen er niet voor betaald"
Oplossing,:

PHP:

try{
    $this->DB->getObject('product', $ID):
} catch(excoption $e){ 
    // ja doei;
}

* Firesphere schaamt zich eigenlijk stiekem er voor...

Wat is een excoption?

[CodeCaster]

-

[ Voor 100% gewijzigd door CodeCaster op 25-04-2012 17:02 . Reden: Dan kan ik geen code lezen, laat maar ]

[martijnve]

Ik ben geen C#-developer maar volgensmij is het naast onnodig ingewikkeld ook fout. Als ik de property eerst get dan set en dan nog een keer get krijg je de oude waarde (van voor dat je hem overschreven had).

farmertjes schreef op woensdag 25 april 2012 @ 16:59:
Ik geloof niet dat ik "Vooral dit stuk" begrijp. Als je een niet bestaande key opvraagt uit de current session wordt er toch een exception gethrowed ... z'n check voor null is dan toch goed? of bedoelde je de overbodige cast naar SearchFilterCompany?

Tenzij ik een of ander C# detail mis bedoelt hij dit:
Hij set eerst zijn temp op null, dan zet hij de nieuwe waarde er in mits deze geen null is en returned dat.

C#:

    SearchFilterCompany zoekFilter = null; 

    if (HttpContext.Current.Session[SessionKeySearchFilter] != null) 
    { 
        zoekFilter = HttpContext.Current.Session[SessionKeySearchFilter] as SearchFilterCompany; 
    } 
    return zoekFilter; 

is dus equivalent aan

C#:

return HttpContext.Current.Session[SessionKeySearchFilter];

[ Voor 69% gewijzigd door martijnve op 25-04-2012 17:08 ]

[farmertjes]

martijnve schreef op woensdag 25 april 2012 @ 17:02:
[/code]
is dus equivalent aan

C#:

return HttpContext.Current.Session[SessionKeySearchFilter];

Volgens mij krijg je in C# een exception als je niet eerst checkt op null, dus is zijn code wel goed.

De code erboven klopt ook gewoon, dat is de gewoon de property syntax voor C#.

[martijnve]

Tenzij ik wat magie mis wordt er na de eerste opvraging van SearchFilter nooit meer een assignment aan _filter gedaan?

[ Voor 21% gewijzigd door martijnve op 25-04-2012 17:19 ]

[farmertjes]

martijnve schreef op woensdag 25 april 2012 @ 17:17:
Tenzij ik wat magie mis wordt er behove bij de eerste get nergens een assignment aan _filter gedaan?

Ik zie net dat ik degene ben die de magie mist

Je hebt helemaal gelijk ...

[CodeCaster]

farmertjes schreef op woensdag 25 april 2012 @ 17:15:
Volgens mij krijg je in C# een exception als je niet eerst checkt op null, dus is zijn code wel goed.

Nee, je krijgt een exception als je een methode uitvoert of property opvraagt op een null. Je kunt prima een null returnen of assignen.

[MatHack]

farmertjes schreef op woensdag 25 april 2012 @ 17:19:
[...]


Ik zie net dat ik degene ben die de magie mist

Je hebt helemaal gelijk ...

_filter wordt toch echt gevuld met het resultaat van de methode GetSearchFilter().
Ik vind de "as SearchFilterCompany" in de SetSearchFilter() toch echt het mooiste staaltje onzin.

[ Voor 14% gewijzigd door MatHack op 25-04-2012 17:24 ]

[martijnve]

Alleen bij de eerste opvraging, als je die property set gebeurt het niet.