[alg] Slechtste programmeervoorbeelden deel 4

[engelbertus]

PiepPiep schreef op woensdag 06 juni 2012 @ 08:56:
[...]

Dan vind ik correct als password veel slimmer.
Als er dan iemand anders onder jouw naam probeerd in te loggen krijgen ze de melding dat het password correct is maar komen ze er niet in.
Dwaalspoor!

En om jou te slim af te zijn heb ik mijn wachtwoord voor tweakers zojuist veranderd in "hij-fietst".

Dat zou je nog verkeerd typen als ik je vertel wat het wachtwoord is.

edit:
spelfoutje

[ Voor 3% gewijzigd door engelbertus op 10-06-2012 15:58 ]

[Verwijderd]

http://www.reddit.com/r/p...q84/mysql_error_handling/

This is one of mine, from when I took a database course. I didn't see any error throwing mechanism in mysql, so I created a table,

SQL:

create table errors(
    errorID bigint primary key not null,
    noSuchModel bigint unique not null,
    modelNameInUse bigint unique not null,
    modelAlreadySpecialized bigint unique not null,
    noModelOrEffect bigint unique not null
)

Then

SQL:

insert into errors values(0,0,0,0,0);
insert into errors values(1,1,1,1,1);

Okay, so now the errors table is spring-loaded.
Here's the killer: When you want to throw an error, update the table:

SQL:

if( <model not found> ) then
    update errors set noModelOrEffect = 'could not find that model.';

Now, mysql is perfectly okay with inserting a string into a bigint column (it casts it to 0, FYI), but the trick here is that it will cast the string to 0 and then insert it in both entries in the table, violating the unique constraint. This, finally, causes an error that mySQL doesn't swallow, and it makes it back up to the application. Where the actual error handling exists.

[Zsub]

CodeCaster schreef op vrijdag 01 juni 2012 @ 13:56:



[...]

Ah, something. Bedankt.

Na de afgelopen weken vrij intensief met PHP gewerkt te hebben: de docs zijn nogal af en toe.

[curvemod]

[quote]gerrymeistah schreef op zaterdag 09 juni 2012 @ 13:18:
Kwam zojuist dit tegen:

PHP:

list($invalidtimes, $incuficienttimeleft) = mysqli_fetch_array(mysqli_query($conn, "
   SELECT 
      TIME_TO_SEC('$starttime') >= TIME_TO_SEC('$endtime'), 
      $stats_xml->timeleft < (TIME_TO_SEC('$endtime') - TIME_TO_SEC('$starttime')) / 60"
   )
);

Ach ja, de gemiddelde date constructie in PHP word je sowieso niet vrolijk van. Zal eens kijken of ik uit mijn huidige project wat leuke voorbeelden kan halen, dat is bijna nog erger dan dit. Terwijl er een prachtige DateTime API is

[Avalaxy]

jhuiting schreef op maandag 11 juni 2012 @ 07:30:

de gemiddelde date constructie in PHP wordt je sowieso niet vrolijk van.

Van sommige taalconstructies ook niet

[ Voor 18% gewijzigd door Avalaxy op 11-06-2012 11:56 ]

[alienfruit]

PHP:

$arrCc = trimsplit(',', $objTemplate->recipient_cc);
foreach ((array)$arrCC as $email)

Argh! Stomme fout... duurde maar een uurtje (50minuten vorige week en 5min vandaag) om te zien

[PiepPiep]

Nice!

Ik had ooit een collega die in vb6 option explicit uitzetten.
Zat er nog 1 ding in vb6 dat (als hij aanstond) mooi was, zet ze het uit

Ik heb zelf ooit een halve dag zitten zoeken op iets als dit :

C:

for (int i = 0; i < 100; i++);
{
    eenofanderiets();
}

En maar zoeken waar het maar 1 keer ipv 100 keer uitgevoerd werd.

[curvemod]

Avalaxy schreef op maandag 11 juni 2012 @ 11:56:
[...]
Van sommige taalconstructies ook niet

Oops

[Alex)]

Dev 1: "Oei, een race condition. Hoe gaan we dat nu oplossen?"
Dev 2: "Oh, daar weet ik wel wat op!"

C#:

for (int tellertje = 1; tellertje <= 60000; ++tellertje)
{

}

[daaan]

Oh boy

PHP:

<?smarty if $smarty.server.REQUEST_URI|strstr:"="|substr:2:5 == 'datum' ?>
        <?smarty $smarty.get.datum|date_format:"%A %e %B" ?><?smarty if $smarty.get.datum|date_format:"%Y" != $smarty.now|date_format:"%Y" ?> <?smarty $smarty.get.datum|date_format:"%Y" ?><?smarty /if ?>
    <?smarty elseif $smarty.server.REQUEST_URI|strstr:"="|substr:2:5 == 'month' ?>
        <?smarty $smarty.server.REQUEST_URI|strstr:"="|substr:1:2|replace:"10":"Oktober"|replace:"11":"November"|replace:"12":"December"|replace:"01":"Januari"|replace:"02":"Februari"|replace:"03":"Maart"|replace:"04":"April"|replace:"05":"Mei"|replace:"06":"Juni"|replace:"07":"Juli"|replace:"08":"Augustus"|replace:"09":"September"|replace:"&":"" ?><?smarty if $smarty.server.REQUEST_URI|strstr:"&"|substr:1:9|replace:"year=":"" != $smarty.now|date_format:"%Y" ?> <?smarty $smarty.server.REQUEST_URI|strstr:"&"|substr:1:9|replace:"year=":"" ?><?smarty /if ?>
    <?smarty elseif $smarty.server.REQUEST_URI|strstr:"="|substr:1:24 == 'zoeken&evenementmenuitem' ?>
        <?smarty $smarty.server.REQUEST_URI|strstr:"="|substr:26|replace:"_":" "|replace:"&":" &raquo; "|replace:"=":" "|replace:"menuitem":""|replace:"productsoortnaam":""|replace:"organisatiesoortnaam":""|replace:"trefwoord":""|replace:"woonplaats":""|replace:"month":""|replace:"01":"Januari"|replace:"02":"Februari"|replace:"03":"Maart"|replace:"04":"April"|replace:"05":"Mei"|replace:"06":"Juni"|replace:"07":"Juli"|replace:"08":"Augustus"|replace:"09":"September"|replace:"10":"Oktober"|replace:"11":"November"|replace:"12":"December" ?>
    <?smarty elseif $smarty.server.REQUEST_URI|strstr:"="|substr:1:15 == 'zoeken&menuitem' ?>
        <?smarty $smarty.server.REQUEST_URI|strstr:"="|substr:17|replace:"_":" "|replace:"&":" &raquo; "|replace:"=":" "|replace:"menuitem":""|replace:"productsoortnaam":""|replace:"organisatiesoortnaam":""|replace:"trefwoord":""|replace:"woonplaats":""|replace:"month":""|replace:"01":"Januari"|replace:"02":"Februari"|replace:"03":"Maart"|replace:"04":"April"|replace:"05":"Mei"|replace:"06":"Juni"|replace:"07":"Juli"|replace:"08":"Augustus"|replace:"09":"September"|replace:"10":"Oktober"|replace:"11":"November"|replace:"12":"December" ?>
    <?smarty elseif $selected ?>
        <?smarty $smarty.server.REQUEST_URI|strstr:"="|substr:1|replace:"datum":""|replace:"_":" "|replace:"&":" &raquo; "|replace:"=":" "|replace:" q ":" "|replace:"%20":"+"|replace:"menuitem":""|replace:"productsoortnaam":""|replace:"organisatiesoortnaam":""|replace:"trefwoord":""|replace:"woonplaats":""|replace:"month=01":"Januari"|replace:"month=02":"Februari"|replace:"month=03":"Maart"|replace:"month=04":"April"|replace:"month=05":"Mei"|replace:"month=06":"Juni"|replace:"month=07":"Juli"|replace:"month=08":"Augustus"|replace:"month=09":"September"|replace:"month=10":"Oktober"|replace:"month=11":"November"|replace:"month=12":"December" ?>
    <?smarty elseif $smarty.server.REQUEST_URI|strstr:"="|substr:1:12 == 'zoeken&datum' ?>
        <?smarty $smarty.server.REQUEST_URI|strstr:"="|substr:14|date_format:"%A %e %B" ?><?smarty if $smarty.server.REQUEST_URI|strstr:"="|substr:1|date_format:"%Y" != $smarty.now|date_format:"%Y" ?> <?smarty $smarty.server.REQUEST_URI|strstr:"="|substr:1|date_format:"%Y" ?><?smarty /if ?>
    <?smarty else ?>
        <?smarty $smarty.server.REQUEST_URI|strstr:"="|substr:1|replace:"_":" "|replace:"%20":" "|replace:"+":" "|replace:"xapian":""|replace:"&q=":""|replace:"beurzenenmarkten":"beurzen en markten"|replace:"discussiebijeenkomstenofthemabijeenkomsten":"discussiebijeenkomsten of themabijeenkomsten"|replace:"opendagen":"open dagen"|replace:"routesentochten":"routes en tochten"|replace:"dansfeestenenpartys":"dansfeesten en partys" ?>
    <?smarty /if ?>

[HuHu]

Hier stopte ik met lezen:

PHP:

<?smarty

[Afvalzak]

daaan schreef op dinsdag 12 juni 2012 @ 10:05:
Oh boy

Man oh man, na 1 regel moet je toch denken dat het beter kan

[ Voor 4% gewijzigd door Afvalzak op 12-06-2012 10:36 ]

[Patriot]

PiepPiep schreef op maandag 11 juni 2012 @ 15:18:
Ik heb zelf ooit een halve dag zitten zoeken op iets als dit :

C:

for (int i = 0; i < 100; i++);
{
    eenofanderiets();
}

En maar zoeken waar het maar 1 keer ipv 100 keer uitgevoerd werd.

Pfoehhh, die is ook wel érg sneaky hoor, ik moest echt 3 keer kijken..

[HeSitated]

Patriot schreef op dinsdag 12 juni 2012 @ 10:44:
Pfoehhh, die is ook wel érg sneaky hoor, ik moest echt 3 keer kijken..

Ik heb vroeger zoiets gehad met mijn allereerste asp-project waar ik een heel klein stukje javascript gebruikte....

Oude DOS-editor en dan 1 verrekte punt-komma.....

[bwerg]

Patriot schreef op dinsdag 12 juni 2012 @ 10:44:
[...]


Pfoehhh, die is ook wel érg sneaky hoor, ik moest echt 3 keer kijken..

En die IDE's maar lekker geen warning geven, je zoekt 't maar uit...

(Ja, natuurlijk, het is gewoon toegestane syntax, maar zo'n puntkomma gevolgd door een block in accolades is best vaak een fout, en een enorme bitch van een fout, dat ik d'r best even van op de hoogte gestelds wil worden)

[EddoH]

Daar is natuurlijk niets fout aan

Of wil je dat de compiler elke keer gaat zeuren als je dergelijk block maakt ivm met local scope?

[Ram0n]

Een assignment in een if-statement is ook niet fout, maar een beetje IDE geeft daar wel een waarschuwing. Het lijkt me dan ook helemaal niet gek om een waarschuwing te tonen bij de combinatie van een for-loop, een puntkomma en een block in accolades.

[Verwijderd]

In C# geeft Visual Studio een Warning als je dat doet "Possible mistaken empty statement".

[ Voor 5% gewijzigd door Verwijderd op 12-06-2012 15:58 ]

[MBV]

GCC kan er ook een foutmelding voor geven, als ik het me goed herinner.

[ZaPPZion]

Ja, daarom compile ik altijd met de hoogst mogelijke warnings aan. Dan kan ik altijd een bepaald type warning nog excluden als ik denk dat ik daar geen fouten in maak.

[EddoH]

Maar dan kun je weer warnings niet als errors laten beschouwen door de compiler, behalve als je ze weer exclude. Ach ja, zo blijf je bezig..

[bwerg]

EddoH schreef op dinsdag 12 juni 2012 @ 15:43:
Daar is natuurlijk niets fout aan

Zoals ik al zeg: als dat getypt wordt, is het best vaak een fout (om niet te zeggen, ik kan me de laatste keer niet herinneren dat ik een for-loop met lege statement wilde laten volgen door een losstaand block).

Ja, natuurlijk is het niet per definitie fout, en iets als "field is not used" is ontwerptechnisch wel een fout. Maar een IDE hoort een programmeur het leven gemakkelijker te maken, en dat kan door dit kreng van een fout te detecteren, want de praktijk leert dat je die puntkomma pas na lang staren ziet.

@onder: en *poef* daar gaat je kijkplezier.

[ Voor 33% gewijzigd door bwerg op 12-06-2012 19:38 ]

[Mavamaarten]

Het is niet echt een codevoorbeeld maar het kan ervoor door

Want een GUI interface in Visual Basic gaat je een IP adres opleveren.

[ Voor 24% gewijzigd door Mavamaarten op 12-06-2012 19:35 ]

[Verwijderd]

PiepPiep schreef op maandag 11 juni 2012 @ 15:18:
Nice!

Ik had ooit een collega die in vb6 option explicit uitzetten.
Zat er nog 1 ding in vb6 dat (als hij aanstond) mooi was, zet ze het uit

Ik heb zelf ooit een halve dag zitten zoeken op iets als dit :

C:

for (int i = 0; i < 100; i++);
{
    eenofanderiets();
}

En maar zoeken waar het maar 1 keer ipv 100 keer uitgevoerd werd.

Wow, daar kan je heeeel lang naar zoeken. Ik kwam er ook pas achter nadat ik een van onderstaande comments las ...

[Verwijderd]

Mavamaarten schreef op dinsdag 12 juni 2012 @ 19:34:
Het is niet echt een codevoorbeeld maar het kan ervoor door
[video]

Want een GUI interface in Visual Basic gaat je een IP adres opleveren.

Een gevalletje à la

[Caelorum]

Verwijderd schreef op dinsdag 12 juni 2012 @ 19:46:
[...] Wow, daar kan je heeeel lang naar zoeken. Ik kwam er ook pas achter nadat ik een van onderstaande comments las ...

Hij viel mij wel op, maar heb dat dan ook zelf een keer meegemaakt. Ik gok dat je vanaf nu veel meer let op waar de puntkomma's staan

[Verwijderd]

Mavamaarten schreef op dinsdag 12 juni 2012 @ 19:34:
Het is niet echt een codevoorbeeld maar het kan ervoor door
[video]

Want een GUI interface in Visual Basic gaat je een IP adres opleveren.

Visual Basic:

    Private Sub Form1_Load(sender As System.Object, e As System.EventArgs) Handles MyBase.Load
        Dim ipAdressen As IPAddress() = Dns.GetHostAddresses(Dns.GetHostName)

        ListBox1.Items.AddRange(ipAdressen)

    End Sub

Zo wel

[Verwijderd]

AAAAAAH
Eeeen .NET taal

[Verwijderd]

Visual Basic is al een tijdje een .NET taal

[Verwijderd]

Geef mij maar gewoon low-level assembly
Maar GTK+ met C++ vindt ik fijner werken dan mono + c++ .net/c#(wat een rotzooi)

[Verwijderd]

Tja, dat hangt af van wat je programmeert.

[Caelorum]

Verwijderd schreef op dinsdag 12 juni 2012 @ 21:24:
[..] Maar GTK+ met C++ vindt ik fijner werken dan mono + c++ .net/c#(wat een rotzooi)

Tja of je gebruikt gewoon mono/C# + GTK#

[Avalaxy]

Verwijderd schreef op dinsdag 12 juni 2012 @ 21:24:
mono + c++ .net/c#(wat een rotzooi)

Want?

[Verwijderd]

Verwijderd schreef op dinsdag 12 juni 2012 @ 21:24:
Geef mij maar gewoon low-level assembly
Maar GTK+ met C++ vindt ik fijner werken dan mono + c++ .net/c#(wat een rotzooi)

Mooie en informatieve onderbouwing heb je ook

[.oisyn]

Lemmingen storten zich nou eenmaal graag in de afgrond

[Avalaxy]

Verwijderd schreef op dinsdag 12 juni 2012 @ 23:08:
[...]

Mooie en informatieve onderbouwing heb je ook

Geboortedatum 5 februari 1997
Opleiding Voortgezet Onderwijs

Komt nog wel

[roy-t]

Damn die programmeurs worden ook steeds jonger. Toen ik 15 was kon ik nog geen GTK+ of C++, toen deed ik alles in VB6

[YopY]

VB? C++? zzt-oop, bitch.

[Janoz]

Toen ik 15 was klooide ik met TurboPascal. GTK bestond nog niet eens..

[creator1988]

Janoz schreef op woensdag 13 juni 2012 @ 11:28:
Toen ik 15 was klooide ik met TurboPascal. GTK bestond nog niet eens..

Ik deed nog old skool ASP en VB6. Pas in 2005 naar .NET overgestapt.

[.oisyn]

Janoz schreef op woensdag 13 juni 2012 @ 11:28:
Toen ik 15 was klooide ik met TurboPascal. GTK bestond nog niet eens..

* .oisyn too . Een jaar later overgestapt op C though (Turbo C).

[Verwijderd]

Op mijn 15e was ik bezig met VBA
Daarna met VB.NET begonnen

[Davio]

Ik ben begonnen met in QBasic bij Snake de snelheid en lengte van de slang aanpassen, jong genie van 8 jaar dat ik was. Ik heb zelf toen ook mijn eigen boter, kaas en eieren gemaakt, daar kijk ik toch wel met verbazing naar terug.

* Davio gaat even 18 jaar terug in de tijd om zichzelf een schouderklopje te geven

[RayNbow]

Visual Basic op m'n 11e

(Fuck, da's lang geleden...)

[Mavamaarten]

Haha, op mijn 10 jaar begonnen met VB6. Best vroeg als je er zo naar kijkt

[Verwijderd]

DOS Batch op mn 9e, VB6 op mn 12e. VB.NET op mn 18e (nu tien jaar geleden), C# op mn 22ste. Verder Java op mn 20ste, en C++ op mn 24e.

Verder ook nog Ruby, PHP, JS, HTML, maar daarvan weet ik niet precies wanneer.
Objective-C op mn 26ste nog vergeten

[ Voor 96% gewijzigd door Verwijderd op 13-06-2012 15:54 ]

[Ventieldopje]

MSX-DOS op mijn ... 8e ... 9e?

* Ventieldopje heeft nu een ontzettende hekel aan een onleesbare taal genaamd VB (en aftreksels als turbo-basic, power basic etc.)

[ Voor 70% gewijzigd door Ventieldopje op 13-06-2012 15:53 ]

[Zeebonk]

Assembly in m'n moeders buik.

[Ventieldopje]

Zeebonk schreef op woensdag 13 juni 2012 @ 15:53:
Assembly in m'n moeders buik.

Denk eerder dat je moeder de 'assembly' deed

[Xesxen]

Zij was de compiler

[Afvalzak]

Davio schreef op woensdag 13 juni 2012 @ 15:46:
Ik ben begonnen met in QBasic bij Snake de snelheid en lengte van de slang aanpassen, jong genie van 8 jaar dat ik was. Ik heb zelf toen ook mijn eigen boter, kaas en eieren gemaakt, daar kijk ik toch wel met verbazing naar terug.

* Davio gaat even 18 jaar terug in de tijd om zichzelf een schouderklopje te geven

Hier hetzelfde, Lingo gemaakt en samen met een paar vrienden (die helemaal niets met programmeren doen tegenwoordig) ook een soort Snake gemaakt maar dat was toen ik 13/14 was HTML al wel wat eerder mee gekloot
.
Wil die QBasic code echt niet meer zien

[epic_gram]

Ik begon toen ik 12 was met shellscripts. twee of drie jaar later met C++ begonnen en later dingen als javascript. php, css en html. Ik kan het niet zo goed, enige talen die ik wel redelijk onder de knie heb is C en assembly. Maar ook daar moet ik nog een hoop mee leren.

[Davio]

Goto baby.

Lingo, ik vraag me af met wat voor woordenlijst.
Vast niet met...

[jip_86]

Xesxen schreef op woensdag 13 juni 2012 @ 15:54:
Zij was de compiler

En op 20 weken word je standaard even aan de debugger gehangen

[epic_gram]

jip_86 schreef op woensdag 13 juni 2012 @ 16:01:
[...]

En op 20 weken word je standaard even aan de debugger gehangen

En na 9 maanden naar de loader gestuurd.

[Afvalzak]

Davio schreef op woensdag 13 juni 2012 @ 15:59:
Goto baby.

Lingo, ik vraag me af met wat voor woordenlijst.
Vast niet met...
[afbeelding]

Haha nee het waren toen nog niet zoveel letters;)

[jip_86]

O, laatst ook nog gezien. Strings comparen met = ipv equals. Dat hoort hier ook wel toch
@.oisyn: ging om C#

[ Voor 10% gewijzigd door jip_86 op 13-06-2012 23:52 ]

[.oisyn]

Kan gewoon in sommige talen, zoals VB.

[dcm360]

Niets mis mee inderdaad, gewoon strings vergelijken met =, <, > en <>

[YakuzA]

jip_86 schreef op woensdag 13 juni 2012 @ 23:44:
O, laatst ook nog gezien. Strings comparen met = ipv equals. Dat hoort hier ook wel toch
@.oisyn: ging om C#

voor zover ik weet gaat strings comparen met == niet goed in java en php.
.NET gaat dat wel goed

Dus die fail is afhankelijk van je progtaal

[Verwijderd]

YakuzA schreef op woensdag 13 juni 2012 @ 23:56:
[...]

voor zover ik weet gaat strings comparen met == niet goed in java en php.
.NET gaat dat wel goed

Dus die fail is afhankelijk van je progtaal

In Java7 was dat volgens mij opgelost. Blijkbaar toch niet.

PHP zit op dat gebied wel erg raar in elkaar. = == en ===. Is er iemand die me trouwens nog even kan vertellen wat het verschil is tussen die laatste 2?

[Xudonax]

== controleert op waarde, dus in PHP kan dan bij mijn weten de integer 1 en de string "1" gelijk zijn, === checkt ook op type.

1 == "1" kan dus best waar zijn in PHP, maar 1 === "1" is niet waar, want int != string

[jip_86]

msdn.microsoft.com/en-us/library/cc165449.aspx
Daar word gebruik van == en != toch afgeraden.

[Caelorum]

jip_86 schreef op donderdag 14 juni 2012 @ 00:04:
msdn.microsoft.com/en-us/library/cc165449.aspx
Daar word gebruik van == en != toch afgeraden.

Hier is wel een mooi artikel waarom het wordt afgeraden: http://blogs.msdn.com/b/c...-should-i-use-equals.aspx
Niet zozeer dus omdat het niet goed is, maar meer omdat je het risico loopt dat het fout kan gaan en je dus echt 100% zeker moet zijn dat er aan bepaalde eisen wordt voldaan.

[Verwijderd]

Xudonax schreef op donderdag 14 juni 2012 @ 00:01:
== controleert op waarde, dus in PHP kan dan bij mijn weten de integer 1 en de string "1" gelijk zijn, === checkt ook op type.

1 == "1" kan dus best waar zijn in PHP, maar 1 === "1" is niet waar, want int != string

Thanks.

[YopY]

Ook in Javascript heb je == en ===; waarschijnlijk, net zoals bij PHP, wordt als je de == operator gebruikt een cast van het ene naar het andere datatype geprobeerd. Dan krijg je leuke vaagheden zoals:

JavaScript:

",,," == Array(4) // is true

dit komt omdat Array (4) naar string geconverteerd wordt dmv Array.toString, en daar komt ",,," uit.

[Davio]

Caelorum schreef op donderdag 14 juni 2012 @ 00:14:
[...]

Hier is wel een mooi artikel waarom het wordt afgeraden: http://blogs.msdn.com/b/c...-should-i-use-equals.aspx
Niet zozeer dus omdat het niet goed is, maar meer omdat je het risico loopt dat het fout kan gaan en je dus echt 100% zeker moet zijn dat er aan bepaalde eisen wordt voldaan.

Ik gebruik het ook alleen voor strings, want daar werkt het prima mee en ik vind het ook vrij natuurlijk.
Voor overige objecten gebruik ik gewoon Equals.

De context is ook belangrijk. Zo gebruik ik die "==" bijvoorbeeld eerder als ik een key opvraag dan wanneer ik iemands naam vergelijk.

C#:

const string MyConst = "SomeSortOfKey";

if (key == MyConst) { doeDit(); }

[Verwijderd]

Het gaat eigenlijk alleen fout als je twee char arrays naar twee objecten gaat casten en die gaat vergelijken. En dat is natuurlijk ook niet vreemd.

C#:

string a = new string(char[] { 'a', 'b' });
string b = new string(char[] { 'a', 'b'});

a == b = true.
(object)a == (object)b = false
(object)"ab" == (object)"ab" = true

Dus als je zelfs al een string als object wil hebben (parameter bv) dan kan je deze altijd nog naar string casten en dan vergelijken.

[jip_86]

Hmm, dat valt me eigenlijk nog mee. Is me altijd bijgebleven dat dit echt niet kon. Maar dat zal dan nog wel stammen uit de tijd dat we met Java begonnen op school

[cpf_]

try {
logger.debug("Some simple static string");
} catch (Exception e) {
logger.error("Error occured");
}

Jup jup... Waarschijnlijk historisch gegroeid (mag ik hopen...), maar de persoon die de oorspronkelijke code dan naar dit refactorde verdient toch wel een facepalm...

[.oisyn]

Verwijderd schreef op donderdag 14 juni 2012 @ 09:43:
Het gaat eigenlijk alleen fout als je twee char arrays naar twee objecten gaat casten en die gaat vergelijken.

Die char arrays hebben er niets mee te maken. Waar het om gaat is dat als je objecten comparet, je alleen hun reference vergelijkt. Als je 2x een new string doet, dan zijn de referenties dus per definitie ongelijk. Als je de referenties pakt van string literals, dan is de kans groot dat de compiler die gemerged heeft (maar dat hoeft niet!)

[HeSitated]

Caelorum schreef op donderdag 14 juni 2012 @ 00:14:
Hier is wel een mooi artikel waarom het wordt afgeraden: http://blogs.msdn.com/b/c...-should-i-use-equals.aspx
Niet zozeer dus omdat het niet goed is, maar meer omdat je het risico loopt dat het fout kan gaan en je dus echt 100% zeker moet zijn dat er aan bepaalde eisen wordt voldaan.

Ik vind dit wel weer een mooi voorbeeld van spijkers-op-laag-water-zoeken-discussie's...

In alle gevallen dat je twee als string gedefinieerde variabelen met elkaar vergelijkt, dan kun je gewoon == gebruiken, want zoals in de blog beschreven, het is een stuk leesbaarder.

In de gevallen waar je niet zeker weet of je met strings van doen hebt (object s, var t en dynamic x), zou je ".Equals" kunnen gebruiken.

Echter daar ben ik zeker in VB.Net een paar goed mee op m'n bek gegaan (zijn we toch weer terug bij topic ), dus ik prefereer dan toch echt liever de volgende werkwijze:

• Test of beide variabelen null zijn, zo ja return true
• Test of een van de beide null is en de ander niet, zo ja return false
• Gebruik bij beide de ToString() en doe dan een compare.

Het lijkt overdone, maar het is voor een ieder volstrekt duidelijk wat de intentie is.

[RayNbow]

HeSitated schreef op donderdag 14 juni 2012 @ 15:59:
In de gevallen waar je niet zeker weet of je met strings van doen hebt (object s, var t en dynamic x), zou je ".Equals" kunnen gebruiken.

Het type van variabele t is gewoon statisch en bekend?

[Davio]

cpf_ schreef op donderdag 14 juni 2012 @ 11:26:
try {
logger.debug("Some simple static string");
} catch (Exception e) {
logger.error("Error occured");
}

Jup jup... Waarschijnlijk historisch gegroeid (mag ik hopen...), maar de persoon die de oorspronkelijke code dan naar dit refactorde verdient toch wel een facepalm...

Helemaal omdat 'ie én een algemene Exception vangt (zelden de bedoeling) én niks met de variable e doet, doe het dan zo als je dat echt nodig hebt:

C#:

try { logger.debug("..."); }
catch // Hier leeg laten
{
  logger.error("Error");
}

.oisyn schreef op donderdag 14 juni 2012 @ 11:43:
[...]

Die char arrays hebben er niets mee te maken. Waar het om gaat is dat als je objecten comparet, je alleen hun reference vergelijkt. Als je 2x een new string doet, dan zijn de referenties dus per definitie ongelijk. Als je de referenties pakt van string literals, dan is de kans groot dat de compiler die gemerged heeft (maar dat hoeft niet!)

Dat is niet helemaal waar, hij internt die char arrays toch naar dezelfde reference. Ik vermoed dat tenminste omdat bij mijn code hier toch true uitkomt:

C#:

string a = new string(new char[] {'a'});
string b = new string(new char[] {'a'});
        
Console.WriteLine (a==b);

Daarbij geeft 'ie alleen false als hij er niet meer achter kan komen of beide objecten strings zijn.

[ Voor 40% gewijzigd door Davio op 14-06-2012 18:13 ]

[RayNbow]

[b]Davio schreef op donderdag 14 juni 2012 @ 18:06:Ik vermoed dat tenminste omdat bij mijn code hier toch true uitkomt:

C#:

string a = new string(new char[] {'a'});
string b = new string(new char[] {'a'});
        
Console.WriteLine (a==b);

Daar komt true uit omdat het statische type van a namelijk string is. De compiler pakt dus de statische string.operator==() methode.

[ Voor 7% gewijzigd door RayNbow op 14-06-2012 19:24 ]

[HeSitated]

RayNbow schreef op donderdag 14 juni 2012 @ 17:00:
Het type van variabele t is gewoon statisch en bekend?

Je hebt gelijk.

Op het moment dat, bij bijv. een third-party dll, het type wijzigt, krijg je wel een foutmelding en/of waarschuwing...

[Alex)]

Maar dan moet je die 3^e partij meppen omdat ze zonder erover te communiceren hun API wijzigen.

[HeSitated]

Alex) schreef op donderdag 14 juni 2012 @ 23:33:
Maar dan moet je die 3^e partij meppen omdat ze zonder erover te communiceren hun API wijzigen.

Hmm, dan had ik toch al heel wat mensen het ziekenhuis mogen meppen....

Zelfs grote partijen lieten bij een van mijn vorige opdrachtgevers zien maling te hebben aan afspraken uit het verleden...

[.oisyn]

Davio schreef op donderdag 14 juni 2012 @ 18:06:
Dat is niet helemaal waar, hij internt die char arrays toch naar dezelfde reference.

Dat boeit dus niet. Je vergelijkt de string reference, niet de interne references van string.

C#:

string s = "abc";
(object)new string(s) == (object)new string(s)

geeft ook false, ookal zijn de intene references exact identiek.

[Korben]

HeSitated schreef op donderdag 14 juni 2012 @ 15:59:
[...]
In alle gevallen dat je twee als string gedefinieerde variabelen met elkaar vergelijkt, dan kun je gewoon == gebruiken, want zoals in de blog beschreven, het is een stuk leesbaarder.

Tenzij je wil case-insensitive wilt vergelijken bijvoorbeeld.

Echter daar ben ik zeker in VB.Net een paar goed mee op m'n bek gegaan (zijn we toch weer terug bij topic ), dus ik prefereer dan toch echt liever de volgende werkwijze:

• Test of beide variabelen null zijn, zo ja return true
• Test of een van de beide null is en de ander niet, zo ja return false
• Gebruik bij beide de ToString() en doe dan een compare.

Het lijkt overdone, maar het is voor een ieder volstrekt duidelijk wat de intentie is.

Het ís overdone. Bij strings een ToString() aanroepen? Dit is nóg makkelijker, correct, en laat nog beter weten wat de intentie is:

C#:

bool equal = StringComparer.InvariantCultureIgnoreCase.Equals(a, b)

Of je nou null of een daadwerkelijke string meegeeft, dit gaat altijd goed.

[LEDfan]

Gisteren een eerste stukje OO PHP geschreven:

PHP:

require("login.php");
$login = new login;
$login->login();

En login.php :

PHP:

class login
{
    public function login()
    {
            echo 'Functie_login':
    }
}

Dit gaf 2 keer Functie_login op het scherm:

Wijzigen van de naam van de class of van de functie werkte uiteraard.

[Verwijderd]

LEDfan schreef op dinsdag 19 juni 2012 @ 12:29:
Gisteren een eerste stukje OO PHP geschreven:

PHP:

require("login.php");
$login = new login;
$login->login();

En login.php :

PHP:

class login
{
    public function login()
    {
            echo 'Functie_login':
    }
}

Dit gaf 2 keer Functie_login op het scherm:

Wijzigen van de naam van de class of van de functie werkte uiteraard.

Dat is toch logisch? Als ik me niet vergis gaat je function login() zich gedragen als constructor.

[LEDfan]

Ja, dat klopt, ligt niet aan PHP, maar aan mijn code, daarom slecht programmeer voorbeeld .

[Afvalzak]

LEDfan schreef op dinsdag 19 juni 2012 @ 12:37:
Ja, dat klopt, ligt niet aan PHP, maar aan mijn code, daarom slecht programmeer voorbeeld .

Inmiddels niet meer. Gelukkig gebruik ik altijd al __construct()

http://php.net/manual/en/language.oop5.decon.php

Nvm gaat alleen om namespaced klasen

[ Voor 17% gewijzigd door Afvalzak op 19-06-2012 13:09 ]

[Ethnocentrix]

Een 'login' class is op zichzelf al een slecht programmeervoorbeeld.

Om even te quoten:

NMe schreef op dinsdag 05 oktober 2010 @ 15:16:
[...]

Je maakt niet een "inlogclass". Je maakt een SecurityContext-class die methods heeft voor inloggen, uitloggen, checken van rechten ongeacht of je al dan niet ingelogd bent, en die bovendien een method biedt voor persistentie gedurende de sessie. Daarnaast zou je daar een parentcontext aan kunnen hangen zodat je in de securitycontext van Pietje kan werken met jouw eigen context als parent, oftewel: impersonatie.

Dit kun je allemaal ook zonder OOP wel schrijven maar dan wordt het meteen een heel stuk onoverzichtelijker en lastiger te onderhouden.

[Afvalzak]

Ethnocentrix schreef op dinsdag 19 juni 2012 @ 15:18:
Een 'login' class is op zichzelf al een slecht programmeervoorbeeld.

Om even te quoten:

[...]

Heel leuk maar als dit zijn eerste stukje OOP PHP is wat neem ik aan voor zichzelf is zou ik me daar niet te druk over maken.
Neemt niet weg dat dit inderdaad wat mij betreft de beste methode is.

[whoami]

HeSitated schreef op donderdag 14 juni 2012 @ 15:59:
[...]

Ik vind dit wel weer een mooi voorbeeld van spijkers-op-laag-water-zoeken-discussie's...

In alle gevallen dat je twee als string gedefinieerde variabelen met elkaar vergelijkt, dan kun je gewoon == gebruiken, want zoals in de blog beschreven, het is een stuk leesbaarder.

Wat als je case-insensitive wilt comparen ?
Om strings te comparen, gebruik ik altijd StringComparer.

/wat Korben dus ook al zei .... * whoami moet regelmatiger meelezen.

[ Voor 13% gewijzigd door whoami op 19-06-2012 15:24 ]

[Verwijderd]

Als ik al strings vergelijk (doe ik niet vaak, meestal gebruik ik enums) bekijk ik per voorkomen wat er gewenst is. Ik ga niet blind een stringcomparer gebruiken als == voldoet voor de vereisten.

Strings vergelijken doe ik enkel na user input, en zelfs dan gebruik ik nog vaak een enum.

[ Voor 19% gewijzigd door Verwijderd op 19-06-2012 15:33 ]

[Alex)]

String vergelijken doe ik eigenlijk altijd met String.Equals(a, b, StringComparison) waarbij ik kies voor wat van toepassing is in mijn situatie.

[PiepPiep]

Ik heb weer een mooie.
Mijn sql commando deed niets. Gewoon niets. Zelfs geen foutmelding.

code:

_db.DataUitvoeren(command.ToString());

Returnt dit dus een object uit een class die een functie Uitvoeren heeft, dus code moest zijn,

code:

_db.DataUitvoeren(command.ToString()).Uitvoeren();

[Janoz]

afvalzak schreef op dinsdag 19 juni 2012 @ 15:19:
[...]


Heel leuk maar als dit zijn eerste stukje OOP PHP is wat neem ik aan voor zichzelf is zou ik me daar niet te druk over maken.
Neemt niet weg dat dit inderdaad wat mij betreft de beste methode is.

Euhm, juist wel. OOP eerste eis is dat je ook object georienteerd gaat denken. Als je dan met een inlog object aankomt is de eerste stap al misgegaan. Als je bij de eerste stap al de verkeerde kant op gaat heb je een heel lange weg te gaan.....

[Davio]

Toch hoeven we er niet zo streng op te letten.

Laat hem lekker een Inlog en Uitlog-klasse maken en er zelf achterkomen dat het handig is om die samen te voegen.

[BasieP]

Davio schreef op vrijdag 22 juni 2012 @ 12:16:
Toch hoeven we er niet zo streng op te letten.

Laat hem lekker een Inlog en Uitlog-klasse maken en er zelf achterkomen dat het handig is om die samen te voegen.

eens
een mens leert door fouten niet door alles in 1x goed te doen omdat iemand dat tegen hem zegt

[LEDfan]

Haha, ik heb weer maar eens wat gepost.... OO staat in men boek dat ik pas heb gekregen dus dat komt eraan. Maar mijn idee was één grote class te maken genaamd login, en daar dan functies in om te regristreren, inloggen, beveiliging aan te zetten (lees: beveiligd deel), uitloggen, profiel wijzigen. Misschien is de naam van de class wat misleidend? Ik had het stukje ook eerder gemaakt om eens te testen hoe dat OO werkt, dan direct te gaan gebruiken in men applicaties.

EDIT: anders had ik het nooit in een class gegooid en het gehouden zoals nu, met allemaal lossen bestanden ergens gedownload...

[ Voor 12% gewijzigd door LEDfan op 22-06-2012 12:30 . Reden: Toevoeging ]

[Alex)]

Dan is de naam verkeerd... kun je beter kiezen voor een naam met Security of Authentication erin.

[LEDfan]

Idd, namen kiezen van functies en variables vind ik niet gemakkelijk.

[Alex)]

Gelukkig is daar tooling voor

[Afvalzak]

LEDfan schreef op vrijdag 22 juni 2012 @ 12:54:
Idd, namen kiezen van functies en variables vind ik niet gemakkelijk.

Dan raad ik je aan het boek Clean Code te lezen. Het komt er eigenlijk op neer dat je je functies / variabellen gewoon beschrijft zoals ze zijn.

Dus een functie die checkt of je ingelogd bent:

isCurrentUserLoggedIn()

en om in te loggen:
isUserAuthenticationCorrect() o.i.d.

[LEDfan]

Oei nog een boek, als ik echt goed wil worden moet ik er nog 5 kopen, de eerste heb ik gelukkig gekregen van men ouders.

Meestal heb ik problemen met men namen doordat ik Engels en Nederlands in de code door elkaar begin te gebruiken. Ik zou eigenlijk alles naar het Engels moeten omzetten. (Nieuwe code ga ik sowieso doen, en een groot deel van de code moet toch nog naar PDO vertaalt worden, dus kan ik evengoed naar Engels omzetten).

[jip_86]

Je bepaalt natuurlijk je eigen codestandaard als je dat voor jezelf doet, dan staat er ook nergens dat het alleen engels mag zijn. Het is wel zo netjes om niet Nederlands en Engels door elkaar te gebruiken

Maar als je met een Login klasse ook kan uitloggen dan dekt de naam de lading niet helemaal. Dat kun je voor zijn door een goed ontwerp te maken, maar kun je ook nog bijstellen als later blijkt dat de naam toch niet helemaal correct blijkt te zijn.