Vind ik nu niet echt heel slecht. Sure, het 2e is korter, maar zodra die checks iets complexer worden is je 1e voorbeel imho stukken leesbaarder
Xbox
Even the dark has a silver lining | I'm all you can imagine times infinity, times three
1
| someButton.Enabled = someBoolean ? true : false; |
Doet me denken aan mijn eerste stukjes code:
Sommigen kunnen het if-elsen ook overdrijven, zo kwam ik hier in de frontend het volgende eens tegen:
C#:
Dat is dan weer wat overdreven, naar mijn idee.
Overigens zie ik het hier bij return statements ook vaak terugkomen, ook bij korte condities. Zelf heb ik de voorkeur voor de tweede optie zonder if-else, maar dat is persoonlijk.
in VB5 (wat ik puur uit hobby en interesse deed).1
2
3
4
5
6
7
8
| If ((myVar = 5) = True) Then myButton.Visible = True End If 'Of bijvoorbeeld If(myButton.Visible = True) Then 'Switch button visibility myButton.Visible = False End If |
). Gelukkig is het allemaal goed gekomen na een paar jaartjes oefenen en een informatica opleiding 1
| someButton.Enabled = someBoolean ?? false; //If null just use false as value |
The #1 programmer excuse for legitimately slacking off: "My code's compiling"
Firesphere: Sommige mensen verdienen gewoon een High Five. In the Face. With a chair.
Als ik aan complexere expressies denk, denk ik eerder aan Introduce Explaining Variable, niet of ik wel of geen if moet gebruiken.:
[...]
Vind ik nu niet echt heel slecht. Sure, het 2e is korter, maar zodra die checks iets complexer worden is je 1e voorbeel imho stukken leesbaarder
Ipsa Scientia Potestas Est
NNID: ShinNoNoir
Dat is ook wel weer waar. Anders moet je waarschijnlijk een hele lap comments boven je return gooien om uit te leggen wat code doet. Goede code verklaart zichzelf en met een explaining variable kun je het inderdaad duidelijker maken.
The #1 programmer excuse for legitimately slacking off: "My code's compiling"
Firesphere: Sommige mensen verdienen gewoon een High Five. In the Face. With a chair.
De site hoort bij o.a. dit boek. Niet alles wat in het boek staat staat ook op de site. Het boek is prima leesbaar, zelfs al gebruikt het een oude versie Java om voorbeelden te bespreken.:
Van die site had ik nog nooit gehoord, eens snuffelen of er meer leuke dingen op staan
Ipsa Scientia Potestas Est
NNID: ShinNoNoir
Kijk zo steek ik nog eens wat op:
[...]
De site hoort bij o.a. dit boek. Niet alles wat in het boek staat staat ook op de site. Het boek is prima leesbaar, zelfs al gebruikt het een oude versie Java om voorbeelden te bespreken.
* RayNbow heeft het boek ooit met korting op de kop getikt bij Donner *).
. Ik heb een vergelijkbaar boek gelezen "Code Complete 2", staan er in dit boek veel vernieuwende dingen of is het zonde van het geld als ik al CC2 in mijn kast heb staan.The #1 programmer excuse for legitimately slacking off: "My code's compiling"
Firesphere: Sommige mensen verdienen gewoon een High Five. In the Face. With a chair.
Ik heb Code Complete 2 niet gelezen, dus ik kan daar helaas geen uitspraak over doen.:
Kijk zo steek ik nog eens wat op
Ipsa Scientia Potestas Est
NNID: ShinNoNoir
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
Beide boeken zijn aanraders, ik heb ze beiden en sluiten elkaar zeker niet uit.:
[...]
Kijk zo steek ik nog eens wat op
Vond Code Complete wel een vrij duidelijk boek met leuke/goede voorbeelden. Niet heel veel van opgestoken, want bij veel dingen had ik vooral: "ja, duh.." en pas ik al toe in mijn dagelijkse development, maar het is altijd handig om voorbeelden te hebben als je bijvoorbeeld met juniors zou werken.
[ Voor 5% gewijzigd door Michali op 02-04-2010 13:43 ]
Je bedoelt: What was & thinking!
How much can you compute with the "ultimate laptop" with 1 kg of mass and 1 liter of volume? Answer: not more than 10^51 operations per second on not more than 10^32 bits.
1
2
3
4
| string m_strServerAddress; string m_strUsername; string m_strPassword; string m_strDomain; |
1
2
| // update record updateRecord(); |
1
2
3
4
5
6
7
8
| if (value.Equals("false")) { boolean.Value = bool.Parse("false"); } else { boolean.Value = bool.Parse("true"); } |
Kater? Eerst water, de rest komt later
If money talks then I'm a mime
If time is money then I'm out of time
Ach, zulke naamgeving is niet echt een slecht programmeervoorbeeld. Er worden alleen conventies gebruikt die alweer gedateerd zijn.:
Foutje kan iedereen maken, dat maakt het niet meteen een slecht programmeervoorbeeld
Nee, neem dan een koppeling die een kleine vier jaar geleden is gebouwd en waar ik deze week een aanpassing aan moest maken
Staat vol met dit soort dingen:
C#:
[..]
Geniaal, nu de uitwerkingen voor een sixtupliton :
If money talks then I'm a mime
If time is money then I'm out of time
Losse pols::
[...]
Geniaal, nu de uitwerkingen voor een sixtupliton :
1
2
3
4
5
6
7
8
9
10
11
12
13
14
| class Sixuptilon<T> implements Ningleton<T> { List<T> instances = new ArrayList<T>(6); int currentinstance = 0; public T getInstance() { if (currentinstance++ > 6) currentinstance = 0; T instance = instances.get(currentinstance); if (instance == null) { instance = new T(); // zal wel niet werken, ben lui. instances.put(currentinstance, instance); } return instance; } } |
Dat zo iemand dan ook niet bij zichzelf denkt: "Hmm, als het nu groter is dan 150...? D'r moet een handigere manier zijn dan 100x copy-pasten...."
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
Dat lijkt me al veel te complex als je gewoon geen idee hebt. Zo'n antwoord lijkt me echt een last resort omdat hij het echt niet wist, in de hoop nog wat puntjes bij elkaar te sprokkelen (hopelijk niet gelukt, maar je weet maar nooit...):
Idd, de meer typisch studenten oplossing zou zijn om er een string van te maken en dan te kijken of de achterste digit een 2, 4, 6, 8 of 0 is
Wie ziet de bug:
[...]
Je kunt het toch in een loopje zetten?
code:
for (int n = 0; n < Integer.MaxValue; n += 2) { if (willekeurigeArray[i] == n) return "even"; } return "oneven";
(en dan bedoel ik niet de Integer.MAX_VALUE typefout.)
[ Voor 8% gewijzigd door CoolGamer op 19-04-2010 19:31 ]
¸.·´¯`·.¸.·´¯`·.¸><(((º>¸.·´¯`·.¸><(((º>¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸<º)))><¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸
Verwijderd
1
| printf("0\t1\n1\t2\n2\t4\n enz enz.."); |
:
[...]
Wie ziet de bug
1
2
| if (willekeurigearray[i] < 0) crash(); |
[ Voor 27% gewijzigd door Verwijderd op 19-04-2010 19:31 ]
Echt wel.:
Niet eens een infinite loop.
¸.·´¯`·.¸.·´¯`·.¸><(((º>¸.·´¯`·.¸><(((º>¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸<º)))><¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸
Verwijderd
Ik ging er even van uit dat Integer.MaxValue een correcte waarde was, maar zo niet dan loopt hij daar gelijk al stuk op?
Als het een correcte waarde was, dan was het geen infinite loop, aangezien de "max value" niet infinite is.
Integer.MaxValue is 2147483647, is gewoon een correcte waarde.:
[...]
Ik ging er even van uit dat Integer.MaxValue een correcte waarde was, maar zo niet dan loopt hij daar gelijk al stuk op?
¸.·´¯`·.¸.·´¯`·.¸><(((º>¸.·´¯`·.¸><(((º>¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸<º)))><¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸
Waar komt die i vandaan:
[...]
Wie ziet de bug
1
2
3
4
5
6
7
8
| for (int n = 0; n < Integer.MaxValue; n += 2) { for(int i = 0; i < willekeurigeArray.Length; i++) { if (willekeurigeArray[i] == n){ return "even"; } return "oneven"; |
Verwijderd
Volgens mij moet het nooit iets zijn wat daar op lijkt.:
[...]
Waar komt die i vandaan
moet het dan niet zijn:
Java:
overflows daargelaten
[ Voor 51% gewijzigd door bobo1on1 op 19-04-2010 21:10 ]
Impedance, a measure of opposition to time-varying electric current in an electric circuit.
Not to be confused with impotence.
en daarbij weet ik ook niet uit m'n hoofd of de maxvalue even of oneven is
Ik hoop maar dat max int een oneven getal is, anders stort het universum in.:
Hoe zit het dan met negatieve getallen?
Integer.MaxValue is trouwens ook een even getal en die wordt ook niet gecontroleerd.
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
Kan zo snel geen taal verzinnen die een overflow waarde heeft voor gehele getallen. Voor floating point getallen doen de talen het standaard wel.:
Hebben ze dan niet 1 getal gereserveerd om overflows te checken? zal vast wel een taal zijn die dat doet
Probeer het eens uit en probeer dan eens het antwoord te verklaren voor jezelf.:
Hoe zit het dan met negatieve getallen?
Zo lang duurt de loop nou ook weer niet.
¸.·´¯`·.¸.·´¯`·.¸><(((º>¸.·´¯`·.¸><(((º>¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸<º)))><¸.·´¯`·.¸.·´¯`·.¸.·´¯`·.¸
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
1
2
3
| $ip = $_SERVER['REMOTE_ADDR']; //ip adres van gebruiker $id = $_COOKIE['PHPSESSID']; //huidige session id $agent = $_SERVER['HTTP_USER_AGENT']; |
En het lek is dan dat die user-agent niet werd escaped? Anders zie ik het probleem niet zo...:
Net even een WTF-je tegengekomen van m'n voorganger.
[...]
Stom dat ik deze niet eerder gezien heb, want het is een fijn lek zo.
Regeren is vooruitschuiven
1
2
3
| public static object Nothing(this object obj) { return obj; } |
[Te koop: 3D printers] [Website] Agile tools: [Return: retrospectives] [Pokertime: planning poker]
Verwijderd
1
2
3
4
5
6
| <ul class="table"> <li class="header"><span class="colA">A</span> ... enz ... <span class="colZ">Z</span></li> <li><span class="colA">A</span> ... enz ... <span class="colZ">Z</span></li> <li><span class="colA">A</span> ... enz ... <span class="colZ">Z</span></li> <li><span class="colA">A</span> ... enz ... <span class="colZ">Z</span></li> </ul> |
[Te koop: 3D printers] [Website] Agile tools: [Return: retrospectives] [Pokertime: planning poker]
Verwijderd
Haha ja, het viel me nog mee dat dat niet gebruikt werd.:
Net zoals die mensen die div's en spans in elkaar stacken en display: table-cell meegeven
Verwijderd
Het kan nog erger! In het zelfde project als de vorige post, sterker nog; op dezelfde pagina, wordt er nog iets moois gebruikt... Ik zag het in eerste instantie over het hoofd omdat het er op 't eerste gezicht heel normaal uit ziet ten opzichte van de eerder gepost code...:
Net zoals die mensen die div's en spans in elkaar stacken en display: table-cell meegeven
1
2
3
4
5
6
7
8
9
10
11
12
13
| <table> <caption>titeltje</caption> <tr> <td class="a">a</td> <td class="b">b</td> <td class="c">c</td> </tr> <tr> <td class="a">a</td> <td class="b">b</td> <td class="c">c</td> </tr> </table> |
Dat was ook mijn eerste gedachte, maar niks is in dit project wat het lijkt, ook in dit geval:1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
| tr { display: inline; float: left; height: 20px; width: 200px; } td.a { display: inline; float: left; height: 20px; width: 50px; background: url('tableRow.jpg'); } td.b { display: inline; float: left; height: 20px; width: 50px; background: url('tableRow.jpg'); } td.c { display: inline; float: left; height: 20px; width: 100px; background: url('tableRow.jpg'); } |
Ohja, de background is een border, waardoor het aanpassen van de breedtes nóg makkelijker gemaakt wordt...
Verwijderd
Ik heb die even zelf snel in html gegoten omdat ik me niet kon voorstellen wat daar visueel uitkwam:
[...]
.... wat moet ik daar nou nog over zeggen?
Mis ik iets geniaals?
Verwijderd
Ja, het ziet er gewoon uit als een normale table, zowel op de pagina als in de html...:
[...]
Ik heb die even zelf snel in html gegoten omdat ik me niet kon voorstellen wat daar visueel uitkwam
Het erge is dat hij geen rekening hield met de dikte van zijn border, anders had het resultaat er als een doodnormale tabel uitgezien...of was dit juist met opzet? God knows...
Ik heb al vele "nuttige" smerige oplossingen gezien, maar ik kan me hier gewoon geen enkel nut voor verzinnen, wie wel? 
Ik hoop nog steeds op een logische verklaring, dat ik iets over het hoofd zie...
De eerste WTF was dat foutmeldingen naar een label op het scherm van de gebruiker werden weggeschreven. De tweede WTF was dat alle schermen automatisch gesloten worden na submitten, dus je hebt nooit de kans om überhaupt een foutmelding te lezen 
Kater? Eerst water, de rest komt later
Ik vind de volgende constructie altijd wat netter met zelf beheerde caches:
1
2
3
4
5
6
7
8
9
10
| // controleer of object in cache staat, zo niet adden if (!getEntityCache().contains(primaryKey)) { EntityDomain entityDomain = getEntityDomain(primaryKey.getClass().getName()); Object entity = entityDomain.find(primaryKey, context); getEntityCache().addEntityToCache(entity); } // return object in cache return getEntityCache.getEntityFromCache(primaryKey); } |
(los van de (waarschijnlijk) wenselijke log warning)
tubby.nl - Artes Moriendi - q1 - bf1942 - WoT - pubg - LinkedIN
Gebrekkige kwaliteitscontrole, slechts gebruikers die het geheel aan de voorkant 'testen' door er even doorheen te klikken... d'r zijn genoeg redenen te bedenken. Meestal is het gewoon luiheid of gierigheid.
Eensch, alhoewel je hierbij wel altijd die cache aanroept (als je echt kritisch op performance bent, alhoewel dit bij een map eigenlijk marginaal is).quote: Tubby
Ik haat wiskunde, maar je hebt gelijkquote: Tubby
. 1
| this.questionAmount = Math.max(Math.min(qamount, MAX_QUESTION_AMOUNT), NO_QUESTION_AMOUNT); |
Waarom eigelijk?
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
| out SomeObjectToReturn result = null; if(find(..., ..., out result)) { // doe iets met result } else { // niets gevonden } // of SomeObjectToReturn result = find(..., ...); if(null != result) { // doe iets met result } else { // niets gevonden } |
Going for adventure, lots of sun and a convertible! | GMT-8
[ Voor 6% gewijzigd door Haan op 29-04-2010 08:44 ]
Kater? Eerst water, de rest komt later
1
2
3
4
5
| $aProducts = array_keys($aBasket); foreach ($aProducts as &$sKey) { $sKey = (int)$sKey; // haal alle extra info uit de key } unset($sKey); |
Maar krijg je dan niet dat je in je returnObject (indien die 'gevuld' moet worden met gegevens zoals ik denk dat je bedoelt) ook logica in moet vullen dat aangeeft dat hij 'leeg' is? (dwz nullobject of een variant daarop)
De null controle,quote: Snake
. Op laatst ga je ook echt alles op null controleren, en dat wordt oplaatst nogal storend - vindt ik tenminste. Als je ervanuit kunt gaan dat een methode nooit null teruggeeft, maar bijvoorbeeld een lege lijst, kun je minder defensief programmeren en met minder code toe. Zie ook:1
2
3
4
5
6
| List<SomeObject> result = getSomeObject(param); if (result != null) { for (SomeObject o : result) { // iets } } |
1
2
3
| for (SomeObject o : getSomeObject(param) { // iets } |
1
2
3
4
5
6
7
| function List<SomeObject> getSomeObject(int param) { List<SomeObject> result = Collections.<SomeObject>emptyList(); if (param > 0) { result = someDao.findById(param); } return result; } |
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
. In beide gevallen kan je drive geformatteerd worden . De code doet dan ook vermoeden dat colorcode altijd tussen 0 en 8 ligt.
[ Voor 22% gewijzigd door .oisyn op 29-04-2010 12:25 ]
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
Inderdaad doe het dan zo
1
2
3
4
5
| if ((redc+shade) > 1.0f) { float scalecolor = 1.0f/(redc+shade); redc*=scalecolor; } |
1
2
3
4
5
6
7
| bluec += shade; if (bluec > 1.0f) { float scalecolor = 1.0f / bluec; bluec*=scalecolor; } glColor3f(redc,greenc,bluec); |
[ Voor 25% gewijzigd door Woy op 06-05-2010 15:04 ]
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
Tja, een string literal is nou eenmaal impliciet converteerbaar naar char*, die warning is dus stuk
Give a man a game and he'll have fun for a day. Teach a man to make games and he'll never have fun again.
staat ook nog mooi in hun verkiezingsprogramma:
[ Voor 32% gewijzigd door korgakos op 07-05-2010 13:23 ]
Verwijderd
Een beetje student had het pas 8 Juni 23:59 opgeleverd:
Tsja. Het past wel bij studenten: dingen opleveren die eigenlijk nog helemaal niet af zijn
( Of liever nog later )
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
Zolang het maar minimaal een 5,6 oplevert ja
Niet dat ik daarom minder onvoldoendes haalde dan anderen hoor [ Voor 37% gewijzigd door RobIII op 07-05-2010 13:22 ]
There are only two hard problems in distributed systems: 2. Exactly-once delivery 1. Guaranteed order of messages 2. Exactly-once delivery.
Je eigen tweaker.me redirect
Over mij
Dat wordt ondersteund ja
En volgens mij kan 't in VBA ook door de juiste lokale te setten "ergens" als ik me niet vergis.
[ Voor 20% gewijzigd door RobIII op 07-05-2010 14:12 ]
There are only two hard problems in distributed systems: 2. Exactly-once delivery 1. Guaranteed order of messages 2. Exactly-once delivery.
Je eigen tweaker.me redirect
Over mij
Datatypes datatypes.. Je moet niet zo in hokjes denken joh!
IF IF = THEN THEN THEN = ELSE ELSE ELSE = IF;
Dit topic is gesloten.
:strip_exif()/u/34748/ferrari_avatar.gif?f=community){kind=avatar}
:strip_icc():strip_exif()/u/104670/66407.jpg?f=community)
:strip_icc():strip_exif()/u/59957/got-icon.jpg?f=community){kind=icon}
:strip_icc():strip_exif()/u/190884/dug.jpg?f=community)
:strip_exif()/u/240696/Darth%2520Stewie.gif?f=community)
/u/81985/crop566b00b43645a.png?f=community)
:strip_icc():strip_exif()/u/12461/crop65b195553d948.jpg?f=community)
:strip_icc():strip_exif()/u/126096/crop580e4f8566fec_cropped.jpeg?f=community)
:strip_icc():strip_exif()/u/85602/ElvisSpace_112.jpg?f=community)
:strip_icc():strip_exif()/u/206968/325134.jpg?f=community)
:strip_icc():strip_exif()/u/77332/crop5e54ec42433bc_cropped.jpeg?f=community)
:strip_exif()/u/242341/crop55d72c2ca29b1.gif?f=community)
/u/145301/mo.png?f=community)
/u/31525/boblight2-icon.png?f=community){kind=icon}
:strip_icc():strip_exif()/u/6143/rincewind.jpg?f=community)
:strip_icc():strip_exif()/u/141741/crop5dd6aa7923c3f_cropped.jpeg?f=community)
/u/24411/img.png?f=community)
:strip_exif()/u/138622/kikker.gif?f=community)
/u/189240/av70.png?f=community)
/u/313216/crop679f937b7f4c1_cropped.png?f=community)
/u/117810/cupcake2.png?f=community)
:strip_exif()/u/26425/got1.gif?f=community)
:strip_icc():strip_exif()/u/148505/31bb5b263ef71bb4d5751e9a89035105.jpeg.jpg?f=community)
:strip_icc():strip_exif()/u/43473/crop5e12eeae6a054_cropped.jpeg?f=community)
/u/301488/avadude2.png?f=community)
/u/84769/crop5f8b3cbb73ded_cropped.png?f=community)
/u/296460/beaker_s.png?f=community)
:strip_icc():strip_exif()/u/43400/avatar.jpeg?f=community){kind=avatar}
:strip_exif()/u/261638/Happy2.gif?f=community)
:strip_icc():strip_exif()/u/47583/ollie.jpg?f=community)
:strip_exif()/u/185685/walking_boxes_s.gif?f=community)
