:strip_icc():strip_exif()/u/12461/crop65b195553d948.jpg?f=community)
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.
:strip_icc():strip_exif()/u/35767/images.jpg?f=community)
Java is echt zo'n Apple-taal: eerst roepen dat je dat helemaal niet nodig hebt en dat je zelfs beter bent omdat je het niet hebt, en het een jaar later dan met veel bombarie presenteren als nieuwe feature 
Anoniem: 516179
Meningen en feiten zijn iets heel anders. We hebben met z'n allen afgesproken een bepaalde golflengte van licht groen te noemen en aantoonbaar (meetbaar) reflecteert gras (onder wit licht) licht met de golflengte die valt in het gebied wat we groen noemen. Dat is dus een feit.:
[...]
Ik word zo moe van die meningen
Dat staat nogal ver af van een mening over een codestijl, waar voor zo ver ik weet geen universele afspraken vergelijkbaar met de afspraken rondom kleuren van zijn. Daarnaast is je reactie een voorbeeld van een "Argumentum ad hominem" waarbij je de originele stelling voorbij gaat en in plaats daarvan mijn overige uitspraken aanvalt.
Volgens mij heb ik dat wel gedaan? Ik heb in ieder geval geen tegenargument gezien buiten dat het onzin zou zijn en een achterlijke smiley. Plus een voorbeeld wat niet direct een realistische situatie is en dus geen goed voorbeeld van wat ik bedoel. Let ook op dat ik zeg "waarschijnlijk", er zijn dus prima situaties te bedenken waarbij de code wel goed is en whitespace de enige oplossing is. In de praktijk zal dat echter niet zo snel het geval zijn.
Als je een hele lijst van variabele declaraties met toekenningen hebt dan is het logisch dat je denkt: Pfoe das niet echt leesbaar zo.
Mijn mening is dat je dan niet moet gaan proberen met whitespace het meer leesbaar te maken, maar in plaats daarvan moet kijken naar de structuur van het geheel om te voorkomen dat je een hele lijst variabele declaraties met toekenningen hebt.
Oftewel: Onleesbare code is vaak op een ander niveau niet goed, dat proberen op te lossen met whitespace pakt niet de oorzaak aan, maar slechts een symptoom.
Daarnaast is het qua onderhoud niet echt handig, als je over de grens van spaties van je voorganger heen gaat moeten alle regels worden aangepast.
In de woorden van Robert C Martin met betrekking tot horizontale uitlijning: "If I have long lists that need to be aligned, the problem is the length of the lists, not the lack of alignment."
[ Voor 29% gewijzigd door Anoniem: 516179 op 18-09-2013 09:22 ]
/u/513893/Kat.png?f=community)
Dat niet alleen, als je het hebt over geel dan kan je nog steeds de kleur geel zien terwijl de golflengten totaal niet overeenkomen met de kleur geel. Kan bijv. ook een menging van rood en groen zijn, beide substantieel verschillend van geel, maar heeft dezelfde invloed op onze receptoren.
Een alien met receptoren voor een breed scala aan golflengtes zal waarschijnlijk denken: "WTF waarom zenden jullie TV's alleen mengels van R, G en B uit?!"
Een alien met receptoren voor een breed scala aan golflengtes zal waarschijnlijk denken: "WTF waarom zenden jullie TV's alleen mengels van R, G en B uit?!"
[ Voor 22% gewijzigd door .oisyn op 18-09-2013 17:12 ]
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.
Codestijl zou je ook het effect kunnen meten met verschillende tests.
Dat over die golflengtes is wel grappig: dus dan kunnen we het eens zijn over het feit dat deze twee honden dezelfde kleur hebben, zelfde golflengtes immers
Even tussendoor
Anyway, ik ga geen hele discussie aan over wat waarheid is etc. Ook een mening moet je verdedigen als je hem uit.
[ Voor 6% gewijzigd door Zoijar op 18-09-2013 18:41 ]
/u/100634/GoT.png?f=community)
Ik moest een intersectie maken van twee lijsten, heb ik het volgende 'pareltje' bedacht:
Was niet helemaal de bedoeling
.
Python: intersect.py
35
36
37
38
39
40
41
42
| inters = [] for result in results: p1 = result['pos'][q1] p2 = result['pos'][q2] for item in p1: if item + 1 in p2 and item not in result: inters.append(item) |
Was niet helemaal de bedoeling
.
My favorite programming language is solder.
Uiteraard heeft Python een intersect methode . Gaat ongeveer zo geloof ik:
In dit geval wilde ik een intersectie maken van twee lijsten, zonder gebruik te maken van build-ins.
Mijn oplossing werkte wel (soort van), maar is niet echt efficiënt.
. Gaat ongeveer zo geloof ik:Python:
1
2
3
4
| b1 = [1,2,3,4] b2 = [0,3,4,5] inters = set(b1).intersection(set(b2)) |
In dit geval wilde ik een intersectie maken van twee lijsten, zonder gebruik te maken van build-ins.
Mijn oplossing werkte wel (soort van), maar is niet echt efficiënt.
My favorite programming language is solder.
Afhangend van verschillende factoren is hashen waarschijnlijk nog sneller. Theoretisch ammortized O(n). Wel meer space O(n) itt O(log n) voor inplace quicksort. Het hangt van je specifieke use-case af. Maar in ieder geval allebei een stuk beter dan O(n^2)
En parallel wordt het weer een heel ander verhaal.
[ Voor 5% gewijzigd door Zoijar op 18-09-2013 20:02 ]
Anoniem: 529096
Een paar dagen geleden had ik iets gepost over Java, dat ik niet haat overigens zoals werd gesuggereerd. Ik steek er alleen de draak mee 
Op dezelfde site staat ook een interview met Bjarne Stroustrup waarin hij uitlegt wat zijn motieven waren om de taal C++ te bedenken. http://harmful.cat-v.org/software/c++/I_did_it_for_you_all
:strip_exif()/u/72391/Pinky_and_the_Brain_-_Brain.gif?f=community)
Coding style zijn niet "zomaar" afspraken. Goeie coding style regels komen voort uit:
- observaties over hoe het menselijk brein (efficientst) informatie verwerkt
- observaties over hoe het menselijk brein (efficientst) informatie verzamelt
- observaties uit het verleden over wat wel en niet werkt
- observaties over de psyche van de programmeur (lazy bastards
)- etc.
Dat jij ze afdoet als "zomaar afspraken" (met een naar ik lees denigrerende ondertoon) toont aan dat je dit nog niet helemaal begrepen hebt.
Er zijn maar 2 stakeholders wanneer je codeert: de eindgebruiker en de persoon die na jou die code moet wijzigen (wat jijzelf kan zijn). Coding rules zijn een onderdeel van het kwaliteitsprocess die voor die 2de stakeholder zorgt.
ASSUME makes an ASS out of U and ME
Voor iemand anders moest ik op een website wijzigen dat eerder door een developer ontwikkeld. Dus ik kreeg toeganggegevens. Even snel rondneuzen, en het enige wat mij meteen opviel, was dat alle 20 pagina's dezelfde inhoud hadden. Er was GEEN enkel verschil tussen alle pagina's, alleen het bestandsnaam was anders. 
Inhoud is:
Hmmz. Wat doet het nu?
Zo te zien maakt-ie een sql query, op basis van URL. Wat zou er dan in de tabel 'website' kunnen zitten waar de page_nr zit? Ik keek in de database, onder tabel websites.
En wat tref ik aan in content of page_nr 1?
Iedere content van ieder page_nr (cijfer verschillend) bevat HTML-pagina die allemaal van elkaar verschillen. Okay.. het werkt. Maar ben ik dom, of ontgaat mij het nut hiervan?
Die worden als code uitgespuugd met echo $row, zodat iedere pagina op basis van URL HTML-inhoud weergeeft. Wat is dit.
Inhoud is:
PHP:
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
32
33
34
35
36
37
38
39
40
41
42
43
44
| <?php //makes $page_nr in statement available; global $page_nr; $host="XXX-weggehaald"; $user="XXX-weggehaald"; $password="XXX-weggehaald"; $dbname = "website"; $cxn = mysqli_connect($host, $user, $password, $dbname) or exit ("Couldn't connect to server"); $paginaurl=$_SERVER['REQUEST_URI']; $catchpag1 = 'disclaimer.php'; $pag1 = strpos($paginaurl, $catchpag1); if ($pag1 == 1){ $page_nr = 1; } $catchpag2 = 'contact.php'; $pag2 = strpos($paginaurl, $catchpag2); if ($pag2 == 1){ $page_nr = 2; } $catchpag3 = 'assistance.php'; $pag3 = strpos($paginaurl, $catchpag3); if ($pag3 == 1){ $page_nr = 3; } [..KNIP..] Dat ging dus nog stuk of 30 regels door 8)7 [.. KNIP ..] $query = "SELECT * FROM website WHERE page_nr = ".$page_nr; $result = mysqli_query($cxn,$query) or exit ("Couldn't execute query."); $row = mysqli_fetch_array($result); echo $row['page_content']; mysqli_close($cxn); ?> |
Hmmz. Wat doet het nu?
Zo te zien maakt-ie een sql query, op basis van URL. Wat zou er dan in de tabel 'website' kunnen zitten waar de page_nr zit? Ik keek in de database, onder tabel websites.
En wat tref ik aan in content of page_nr 1?
HTML:
1
2
3
4
5
6
| <html> <title>blabla</title> <body> Inhoud van die pagina's, overeenkomend met wat er opgevraagd is. </body> </html> |
Iedere content van ieder page_nr (cijfer verschillend) bevat HTML-pagina die allemaal van elkaar verschillen. Okay.. het werkt. Maar ben ik dom, of ontgaat mij het nut hiervan?
Die worden als code uitgespuugd met echo $row, zodat iedere pagina op basis van URL HTML-inhoud weergeeft. Wat is dit.
[ Voor 11% gewijzigd door Hatsieflatsie op 24-10-2013 19:17 ]
:strip_icc():strip_exif()/u/36083/crop5dca9e34ce091_cropped.jpeg?f=community)
Slecht of niet, creatief is het wel.
Zelden zo'n creatieve 'CMS'-like oplossing gezien. Ben wel benieuwd waarom-ie het zo doet.
Zelden zo'n creatieve 'CMS'-like oplossing gezien. Ben wel benieuwd waarom-ie het zo doet.
[ Voor 27% gewijzigd door Hatsieflatsie op 25-10-2013 13:09 ]
:strip_icc():strip_exif()/u/373712/bb.jpg?f=community)
/u/38502/icon.JPG?f=community){kind=icon}
Ik denk dat hij niet snapt hoe .htaccess werkt en dus maar een losse php pagina voor elke URL heeft gemaakt. En dan natuurlijk de html uit de database halen, want dat is hardstikke dynamisch
After all, we are nothing more or less than what we choose to reveal.
:strip_icc():strip_exif()/u/261244/avatar4-64x64.jpg?f=community){kind=avatar}
Ik zat net een (saaie en lange) vragenlijst in te vullen van HBO Monitor, maar wat daar onderaan in de broncode staat ...
if true or false.
JavaScript:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
| <!-- dirty patch --> <script type="text/javascript"> function checkDoRedirect() { if (window.location.protocol == 'http:'){ var secureLink = 'https://' + window.location.host + window.location.pathname + window.location.search; window.location.href = secureLink; } } //change || to && to disable quickly! if ( true || false ){ addEvent(window,"load",function(e){checkDoRedirect();}); } </script> |
if true or false.
:strip_exif()/u/306933/ezgif.com-optimize.gif?f=community)
Want // kende je nog niet
Oh wacht....
Oh wacht....
2x Dell UP2716D | R9 7950X | 128GB RAM | 980 Pro 2TB x2 | RTX2070 Super
.oisyn: Windows is net zo slecht in commandline als Linux in GUI
/u/313216/crop679f937b7f4c1_cropped.png?f=community)
Gebruik ik ook regelmatig om stukken code aan en af te zetten hoor :-). Het is slordig dat het zichtbaar is op niet-development.:
Ik zat net een (saaie en lange) vragenlijst in te vullen van HBO Monitor, maar wat daar onderaan in de broncode staat ...
[...]
if true or false.
Comments hebben het nadeel niet uitvoerbaar te zijn. Ik gebruik het bijvoorbeeld om stukken code snel uit te schakelen en eventueel tijdens debugging een breakpoint te zetten op de IF en geforceerd de IF laten uitvoeren; of rechtstreeks code wijzigen tijdens het debuggen :-) handig hoor.:
Want // kende je nog niet
Oh wacht....
[ Voor 36% gewijzigd door Styxxy op 30-10-2013 00:18 ]
:strip_icc():strip_exif()/u/393369/crop55d593566408c.jpeg?f=community)
Dit is natuurlijk gewoon wachten op een developer die er "true && false" in zet en het per ongeluk commit:
Ik zat net een (saaie en lange) vragenlijst in te vullen van HBO Monitor, maar wat daar onderaan in de broncode staat ...
JavaScript:
if true or false.
/u/142011/crop65b383c6c6c2f_cropped.png?f=community)
/u/57387/claimedavatar-70.png?f=community){kind=avatar}
:strip_icc():strip_exif()/u/86654/krat_toren.jpg?f=community)
https://oneerlijkewoz.nl
Op papier is hij aan het tekenen, maar in de praktijk...
:strip_icc():strip_exif()/u/78891/W-Seawolf.jpg?f=community)
:strip_exif()/u/294184/20150204_SITE_GIF_VALKUIL-resized.gif?f=community)
1x Mosselen met friet AUB!
Het enige wat ik hiet kan aan opmerken is: waarom een tabel als je bootstrap (of een ander framework) gebruikt? Dat is het enige "slechte" wat ik eraan kan zien
Het enige wat ik hiet kan aan opmerken is: waarom een tabel als je bootstrap (of een ander framework) gebruikt? Dat is het enige "slechte" wat ik eraan kan zien
Anoniem: 541917
Het is eigenlijk best useless! en gewoon cellspacing enz is toch niet meer van deze tijd? Wel? ben maar een trainee:
1x Mosselen met friet AUB!
Het enige wat ik hiet kan aan opmerken is: waarom een tabel als je bootstrap (of een ander framework) gebruikt? Dat is het enige "slechte" wat ik eraan kan zien
:strip_icc():strip_exif()/u/43400/avatar.jpeg?f=community){kind=avatar}
Een table is nog wel te verdedigen, border/cellspacing/alignment/widths e.d. doe je in je CSS, maar ik zie nog wel wat andere WTF's:
- "Hoeveelheid" waar ik persoonlijk "Aantal" gebruikt zou hebben, "Tel" i.p.v. voluit "Telefoon(nummer)" (want wat kosten die paar letters extra je? En waarom in de placeholder dan "Tel. Nr."?) en dat soort (kleine) tekstuele dingetjes...
- Voor alles een textbox
- Het zelf laten invullen van 't totaal
- Het inconsistente gebruik van placeholders (dan weer wel, dan weer niet)
- Hoewel "toegestaan", de tabindex vind ik erg onlogisch (waarom niet gewoon sequentieel?)
- Bovenop voorgenoemde border/cellspacing/alignment/widths e.d. ook nog eens inline styles
- Table cells met ... (Her-en-der had mogelijk zelfs een colspan gekund i.p.v. een <td> </td>)
- Markup voor "kolomkoppen" (<td><b>Blaat</b></td> i.p.v. gewoon th's gebruiken: <th>Blaat</th>)
- onclick="this.form.submit()" op de submitbutton. Why?
[ Voor 24% gewijzigd door RobIII op 31-10-2013 01:18 ]
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
Anoniem: 541917
Mijn excuses was de bedoeling niet. Ik dacht dat het een slechtste programmeervoorbeeld waardig was!:
[...]
Een table is nog wel te verdedigen, border/cellspacing/alignment/widths e.d. doe je in je CSS, maar ik zie nog wel wat andere WTF's:
[afbeelding]Maar having said all that: ik vind 't niet echt "Slechtste programmeervoorbeelden"-waardig en het is niet de bedoeling dat je hier stiekem/verkapt code-reviews gaat (laten) doen.
- "Hoeveelheid" waar ik persoonlijk "Aantal" gebruikt zou hebben, "Tel" i.p.v. voluit "Telefoon(nummer)" (want wat kosten die paar letters extra je? En waarom in de placeholder dan "Tel. Nr."?) en dat soort (kleine) tekstuele dingetjes...
- Voor alles een textbox
- Het zelf laten invullen van 't totaal
- Het inconsistente gebruik van placeholders (dan weer wel, dan weer niet)
- Hoewel "toegestaan", de tabindex vind ik erg onlogisch (waarom niet gewoon sequentieel?)
- Bovenop voorgenoemde border/cellspacing/alignment/widths e.d. ook nog eens inline styles
- Table cells met ... (Her-en-der had mogelijk zelfs een colspan gekund i.p.v. een <td> </td>)
- Markup voor "kolomkoppen" (<td><b>Blaat</b></td> i.p.v. gewoon th's gebruiken: <th>Blaat</th>)
- onclick="this.form.submit()" op de submitbutton. Why?
:strip_icc():strip_exif()/u/73625/Tetris_failure_ava_forum.jpg?f=community)
Kwam dit gisteren nog tegen in de krochten van wat oude probeersels... Refactoring zou wel eens nodig kunnen zijn
code:
1
| ClientSubsystems::GetInstance()->GetGraphicsSubsystem()->GetRenderer()->GetDevice()->GetDeviceFactory()->GetAssetResourceFactory() |
Waarschijnlijk omdat bootstrap juist styling heeft voor tables? .. bedoel, een nette grid bouwen met divs is leuk en aardig, maar zebra-stripes maken gaat een stuk makkelijker met tables in bootstrap .:
1x Mosselen met friet AUB!
Het enige wat ik hiet kan aan opmerken is: waarom een tabel als je bootstrap (of een ander framework) gebruikt? Dat is het enige "slechte" wat ik eraan kan zien
Mind, voor formulieren hebben ze weer andere (discuseerbaar betere) oplossingen, maar ik zou dat niet de wtf noemen.
:strip_icc():strip_exif()/u/39264/notpingu.jpg?f=community)
:strip_exif()/u/69716/disgaea2_thief.gif?f=community)
Je kan het ook omdraaien: Waarom bootstrap gebruiken om het gedrag van tabellen te emuleren?:
1x Mosselen met friet AUB!
Het enige wat ik hiet kan aan opmerken is: waarom een tabel als je bootstrap (of een ander framework) gebruikt? Dat is het enige "slechte" wat ik eraan kan zien
'Multiple exclamation marks,' he went on, shaking his head, 'are a sure sign of a diseased mind.' - Pratchett.
Hopelijk niet 100x hetzelfde nummer, anders is die willekeurige persoon (indien bestaand) redelijk gepest
/u/200133/artemis_square_e_60-60.png?f=community)
Ik heb hem ook heel vaak in databases van testomgevingen gezet, en ik had al zo'n vaag vermoeden dat ik er niet echt origineel mee was Ook mijn eigen nummer stond er vaak in, maar ja, je hebt vaak toch 2 verschillende accounts nodig
Ook mijn eigen nummer stond er vaak in, maar ja, je hebt vaak toch 2 verschillende accounts nodig
:strip_icc():strip_exif()/u/87691/avatar_alex.jpg?f=community){kind=avatar}
06-12345678 is toch gewoon een geldig nummer? Een vriend van me heeft dat nummer wel eens gebeld en kreeg iemand z'n voicemail. 
We are shaping the future
Daarom is het ook zo'n handig nummer in testsystemen, ongeldige nummers worden vaak geweigerd Ik vind het wel mooi dat ze voor dat soort doeleinden example.com hebben gereserveerd, heel jammer dat 3/4 van de boeken en websites gewoon een willekeurig adres gebruikt.
Ik vind het wel mooi dat ze voor dat soort doeleinden example.com hebben gereserveerd, heel jammer dat 3/4 van de boeken en websites gewoon een willekeurig adres gebruikt.
/u/12668/hydra.png?f=community)
:strip_icc():strip_exif()/u/23872/Y2.jpg?f=community)
06-12345678 was ooit van iemand bij KPN.:
06-12345678 is toch gewoon een geldig nummer? Een vriend van me heeft dat nummer wel eens gebeld en kreeg iemand z'n voicemail.
Door alle kansloze telefoontjes wordt dit nummer nu niet meer gebruikt
Death smiles at us all, all a man can do is smile back.
PSN
:strip_icc():strip_exif()/u/226826/aardblij-60.jpg?f=community)
/u/262310/ava.png?f=community)
PHP:
1
| if($blaat == '01110' || $blaat == '01150' || $blaat == '01165' || $blaat == '02040' || $blaat == '02106' || $blaat == '02910' || $blaat == '02946' || $blaat == '02965' || $blaat == '02966' || $blaat == '02968' || $blaat == '02969' || $blaat == '02971' || $blaat == '02974' || $blaat == '02975' || $blaat == '03683' || $blaat == '03831' || $blaat == '03922' || $blaat == '03993' || $blaat == '03994' || $blaat == '03995' || $blaat == '03996' || $blaat == '05017' || $blaat == '05018' || $blaat == '05019' || $blaat == '05038' || $blaat == '05040' || $blaat == '05045' || $blaat == '05060' || $blaat == '05193' || $blaat == '05194' || $blaat == '05195' || $blaat == '05196' || $blaat == '05197' || $blaat == '05200' || $blaat == '05210'|| $blaat == '05202' || $blaat == '05201' || $blaat == '05250' || $blaat == '05385' || $blaat == '05386'|| $blaat == '05387' || $blaat == '05395' || $blaat == '05396' || $blaat == '05409' || $blaat == '05411' || $blaat == '05412' || $blaat == '05413' || $blaat == '05414' || $blaat == '05417' || $blaat == '05421' || $blaat == '05423' || $blaat == '05431' || $blaat == '05425' || $blaat == '05426' || $blaat == '05430' || $blaat == '05431' || $blaat == '05432' || $blaat == '05433' || $blaat == '05434' || $blaat == '05436' || $blaat == '05561' || $blaat == '05591' || $blaat == '05610' || $blaat == '05636' || $blaat == '05638' || $blaat == '05645' || $blaat == '05681' || $blaat == '06010' || $blaat == '06110' || $blaat == '06115' || $blaat == '06117' || $blaat == '06118' || $blaat == '06119' || $blaat == '10905' || $blaat == '10910' || $blaat == '10911' || $blaat == '10912' || $blaat == '10913' || $blaat == '10950' || $blaat == '10955' || $blaat == '10990' || $blaat == '10991' || $blaat == '10992' || $blaat == '10993' || $blaat == '55000' || $blaat == '55002' || $blaat == '55003' || $blaat == '55004' || $blaat == '55006') |
Is toch makkelijker te lezen dan het gebruik van in_array():
PHP:
RTFM!
eh ja/nee...
Het gaat hier niet perse om DIT voorbeeld, maar om de gedachte erachter. Als je dit soort code -moet- gaan gebruiken heb je iets structureel niet goed gedaan als je uiteindelijk zover komt dat je een IF vergelijking moet maken van zoveel karakters..
:strip_exif()/u/138622/kikker.gif?f=community)
Ik zou een polynoom bepalen die door al die waarden heen gaat, en vervolgens kun je gewoon in een loopje verschillende punten van de polynoom evalueren
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.
:strip_exif()/u/47168/loop.gif?f=community)
Of iets met bloom filters.
Overigens (als je dit wil doen) is een in_array() imo waaaaay beter leesbaar. De array var kan je een goede naam geven en het scheelt tig keer '|| $blaat = ' pasten.
Overigens (als je dit wil doen) is een in_array() imo waaaaay beter leesbaar. De array var kan je een goede naam geven en het scheelt tig keer '|| $blaat = ' pasten.
{signature}
/u/24411/img.png?f=community)
Is er dan niemand die er een regex voor kan verzinnen?:
Of iets met bloom filters.
Ik weet het gebruik niet, maar verschil leesbaarheid tussen een goed geindente array en een goed geindente if-constructie vind ik niet echt heel erg groot hoor.
Als het echt 100% zeker eenmalig is (zijn dit soort constructies nooit maargoed) dan vind ik dit mits goed geindent en van commentaar voorzien wel te verdedigen (of beter gezegd ik zie de toegevoegde waarde van een array / in_array niet, de leesbaarheid is toch minimaal met een array-constructor of een if-statement van 80 regels).
Alhoewel ik het persoonlijk wel 100% zeker in een array zou dumpen omdat dit lijstje over 6 maanden ergens anders ook moet gaan werken en over 7 maanden moet het uitgebreid gaan worden met 2 waardes en over 8 maanden moeten er 15 waardes uit en over 8 maanden en 1 week krijg je een rare bug-melding binnen omdat de lijstjes toch niet 100% gelijk zijn ( of nog leuker, je moet het lijstje ergens anders hergebruiken maar dan minus 1 waarde, dan denk je een bug te fixen door het 1e lijstje te copy-pasten naar het 2e en dan introduceer je een nieuwe bug)
Natuurlijk niet, dan zou het een slecht programmeervoorbeeld zijn.:
[...]
Is er dan niemand die er een regex voor kan verzinnen?
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.
Een goed programmeervoorbeeld zou zoiets zijn:
PHP:
1
2
3
| $blaatValues = array('01110', '01150', ... ); $blaatTest = array_map($blaatValues, function ($value) { return '$blaat == ' . "'$value'"; }); $blaatInBlaatValues = eval(implode($blaatTest, ' || ')); |
:
Een goed programmeervoorbeeld zou zoiets zijn:
PHP:
RTFM!
eh ja, behalve dat ik alles kan zien. Maar verder prima hoor als jij dit netjes vindt!
Op het moment dat een IF statement moet gaan maken van 500 karakters+ dan doe je toch echt iets fout..
[ Voor 15% gewijzigd door Douweegbertje op 19-11-2013 11:12 ]
/u/280004/HappyFamily.png?f=community)
Ik dacht in eerste instantie ook: "is toch niets mis mee"....totdat ik de scollbar onder de code-snippet zag...:
[...]
eh ja, behalve dat ik alles kan zien. Maar verder prima hoor als jij dit netjes vindt!
Op het moment dat een IF statement moet gaan maken van 500 karakters+ dan doe je toch echt iets fout..
Diablo III: <GOT> Pteer#2475 --- POE: Dwergux
Ja, jij... maar wij niet. Eén zo'n regel duidt niet per se op "structurele" fouten, het kan ook een maandagochtendregel zijn.:
[...]
eh ja, behalve dat ik alles kan zien. Maar verder prima hoor als jij dit netjes vindt!
Op het moment dat een IF statement moet gaan maken van 500 karakters+ dan doe je toch echt iets fout..
Op basis van één regel iets structureel (kan ik de nadruk nog harder op "structureel" leggen?) fout noemen is nogal voorbarig.
Overigens zeg ik nergens dat ik het netjes vind. Ik ken de context verder ook niet. Maar om op basis van één regel, één regel (!), iets "structureel" (dat is: alle regels) fout te noemen... nah, daar krijg je mij niet in mee.
Met één regel als 'voorbeeld' kun je toch concreet zien of het nou goed of slecht is? Vervolgens kun je toch heel makkelijk zeggen of dát nu fout is, of dat het structureel fout gaat?
Ik bedoel, stel dat je een function rand hebt, die hard 5 returned.. dan kun je zeggen dat die functie gewoon ruk is. Verder kun je dan weer geen conclusie trekken of de rest structureel fout is. Echter met mijn voorbeeld weet je dat het structureel niet goed zit, anders hoefde je namelijk geen bizare IF constructie te maken met die gare data.
Indien het structureel goed zat, heb je geen vage statements nodig. Dan heb je bijvoorbeeld de data beter geordend al dan niet mooi in een DB gezet. Je wilt gewoon -niet- zulke statements hebben, en heb je dat wel.. dan gaat er -ergens- iets mis. Structureel dus.
Misschien definieer ik structureel in deze context wat verkeerd. Voor mij betekend in dit geval dat het 'vanaf het begin' al fout gaat, dat je uiteindelijk bij zo'n constructie komt.
In ieder geval heb ik al 3000 regels aan code aangepast naar een kleine 200-300.
:strip_icc():strip_exif()/u/438947/crop575c94eea6d14.jpeg?f=community)
Need for Speed: Rivals is ook een voorbeeld van slechtste programmeervoorbeelden
Als je de framerate unlocked via een console command naar 60 fps dan loopt het spel 2x zo snel, want normaal is het locked op 30 fps
YouTube: Let's not play Need for Speed: Rivals
Als je de framerate unlocked via een console command naar 60 fps dan loopt het spel 2x zo snel, want normaal is het locked op 30 fps
YouTube: Let's not play Need for Speed: Rivals
Och tegenwoordig is het niet meer echt gebruikelijk om de simulatiesnelheid aan de framerate te koppelen, maar zo enorm slecht is het nou ook weer niet.:
Need for Speed: Rivals is ook een voorbeeld van slechtste programmeervoorbeelden
Als je de framerate unlocked via een console command naar 60 fps dan loopt het spel 2x zo snel, want normaal is het locked op 30 fps
YouTube: Let's not play Need for Speed: Rivals
Ik kom zojuist dit pareltje tegen. Moet ik mezelf hier voor schamen? 
It gets the job done... (automatisch punten zetten als je voorletters typt, abc wordt A.B.C.).
code:
1
| var newval = element.value.split('.').join('').toUpperCase().split('').join('.'); |
It gets the job done... (automatisch punten zetten als je voorletters typt, abc wordt A.B.C.).
Roses are red, violets are blue, unexpected '{' on line 32.
'Multiple exclamation marks,' he went on, shaking his head, 'are a sure sign of a diseased mind.' - Pratchett.
/u/26742/000000751.png?f=community)
Mmh, ik vind hem wel ok eigenlijk. Het enige wat ik liever zou zien is een filter op [a-zA-Z] ipv .split('.').join(''). Het is wel typisch zo'n stukje code dat je van commentaar moet voorzien om uit te leggen dat hij initialen format:
Ik kom zojuist dit pareltje tegen. Moet ik mezelf hier voor schamen?
code:
var newval = element.value.split('.').join('').toUpperCase().split('').join('.');
It gets the job done... (automatisch punten zetten als je voorletters typt, abc wordt A.B.C.).
Er zit alleen wel een bugje in, hij genereert geen punt na de laatste letter.
Ik ben eigenlijk wel benieuwd wie een mooiere constructie weet te verzinnen. Voor Javascript weet ik weinig beters (maar ik ben ook geen JS-programmeur meestal), maar in Python zou ik het zo doen:
Python:
1
2
| initials = 'abc' ''.join([c.upper() + '.' for c in initials if c.isalpha()]) |
:strip_icc():strip_exif()/u/45523/jack_nicholson.jpg?f=community)
Ja, kan er wel uit inderdaad die nutteloze commentaar.
maar is dit wel oké?
if ($mobile=="yes") {
$mobile = ('no');
ik zie daar eerst twee == en daaronder eentje. dan zie ik "yes" en daaronder 'no' en die no staat dan tussen haakjes, het zal allemaal wel, ik heb het zo verzonnen, meer trial and error geweest dan kunde. Dus vraag me wel af of dit niet simpeler kan, of beter. Ik zie mijn code altijd als poep als het werkt, spring ik al een gat in de lucht.
maar is dit wel oké?
if ($mobile=="yes") {
$mobile = ('no');
ik zie daar eerst twee == en daaronder eentje. dan zie ik "yes" en daaronder 'no' en die no staat dan tussen haakjes, het zal allemaal wel, ik heb het zo verzonnen, meer trial and error geweest dan kunde. Dus vraag me wel af of dit niet simpeler kan, of beter. Ik zie mijn code altijd als poep
als het werkt, spring ik al een gat in de lucht.
/u/14091/icon.png?f=community){kind=icon}
:strip_icc():strip_exif()/u/18606/crop63ebc46666763_cropped.jpg?f=community)
Do you seek to engage in or have you ever engaged in terrorist activities, espionage, sabotage, or genocide?
:strip_icc():strip_exif()/u/300196/jj2.jpg?f=community)
And the winner is:
Een linq statement van 66 regels...
EN natuurlijk gaat het nog fout ook....
C#:
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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
| var toSend = from q1 in Connections where (from q2 in contractRules where (from q3 in contracts where q3.LabelID == Label && (from q4 in switches where ( (from q5 in switchRules where q5.Status == "Done" && new[] { "Requirement1", "Requirement2" } .Contains(q5.Message) && q5.SwitchDate != null && q5.SwitchDate <= DateTime.Today select q5.LineID) .Contains( q4.BeginLineID ?? 0) || (from q5 in switchRules where q5.Status == "Done" && new[] { "Requirement3", "Requirement4" } .Contains(q5.Message) && q5.SwitchDate != null && q5.SwitchDate <= DateTime.Today select q5.LineID) .Contains( q4.EndLineID ?? 0) ) && (from q8 in switches where q8.ContractLineID == q4.ContractLineID group q8 by q8.ContractLineID into g1 select (from q9 in g1 select q9.SwitchID) .Max()).Contains(q4.SwitchID) select q4.ContractLineID).Contains( q2.ContractLineID) select q3.ContractID). Contains(q2.ContractID) select q2.ID).Contains( q1.ID) && q1.SegmentID == segmentId && (from q6 in meters where (from q7 in Reading where q7.Uploaded == false && new[] { "Customer", "Internal" }.Contains(q7.BronID) && (q7.IsJunk == null || !(bool)q7.IsJunk) select q7.MeterID).Contains(q6.MeterID) select q6.ID).Contains(q1.ID) select q1; var result = toSend.ToList(); |
Een linq statement van 66 regels...
EN natuurlijk gaat het nog fout ook....
Lambda expressies 
maar dat 
.
Behalve dat het slecht leesbaar is, zijn de performance ongelofelijk slecht.
maar dat . Behalve dat het slecht leesbaar is, zijn de performance ongelofelijk slecht.
[ Voor 53% gewijzigd door Kwastie op 23-11-2013 21:30 ]
When I get sad i stop being sad and be awesome instead
als je ze goed gebruikt zijn ze snel hoor, maar dit is inderdaad een hoogtedieptepuntje:
Lambda expressies
Behalve dat het slecht leesbaar is, zijn de performance ongelofelijk slecht.
This message was sent on 100% recyclable electrons.
:strip_icc():strip_exif()/u/116600/crop5be04a1ae3ded_cropped.jpeg?f=community)
Kwam net dit pareltje tegen. Heb even de namen van de variabelen etc. gewijzigd, maar het punt is duidelijk.
C#:
1
2
3
4
5
6
| switch (booleanVar) { case false: booleanVar2 = false; break; } |
[ Voor 4% gewijzigd door PdeBie op 02-12-2013 14:35 ]
:strip_exif()/u/13818/episodebutler_60x60.gif?f=community)
Voor alle mensen die vinden dat het commentaar van _js_ ("[alg] Slechtste programmeervoorbeelden deel 5") wat vergezocht is hoop ik niet dat ze ooit bv een IJsbrandt als gebruiker krijgen, of iemand die de schrijver A. F. Th. van der Heijden ergens in kwijt moet:
Ik kom zojuist dit pareltje tegen. Moet ik mezelf hier voor schamen?
code:
var newval = element.value.split('.').join('').toUpperCase().split('').join('.');
It gets the job done... (automatisch punten zetten als je voorletters typt, abc wordt A.B.C.).
Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'
Verwijderd
Collega vanmiddag:
"MIJN CLEANUP ROUTINE WERKT NIET MEER"
PHP:
1
2
3
4
5
6
7
8
9
| <?php if ($not_found) { ob_clean(); header('HTTP/1.1 301 Moved Permanently'); header("Location: http://www.nieuwe-url-yolo.com/"); exit; clean_up_routine_that_does_something_important(); } |
"MIJN CLEANUP ROUTINE WERKT NIET MEER"
[ Voor 5% gewijzigd door Verwijderd op 04-12-2013 19:54 ]
:strip_icc():strip_exif()/u/36003/crop584e9e5c223bd.jpeg?f=community)
Of ik even wat belangrijke productiecode wil aanpassen, een job die 's nachts draait.
/me opent solution
Een berg moeilijk te ontcijferen code (want fuck SRP), binnen iedere iteratie van een foreach excepties afvangen die gecheckt hadden kunnen worden of op z'n minst buiten de loop eenmalig opgevangen kunnen worden, yay!
Hard-wired debugcode, en zelfs dát verkeerd geimplementeerd
/me opent solution
C#:
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
| static void Main(string[] args) { try { // Honderdvierenzeventig regels code met inline bijna alle domeinlogica, // transactieafhandeling, data access en logging foreach(...) { try { // ... } catch (IndexOutOfRangeException ex) { // meld dat het aantal argumenten (args[]) niet klopt } } } catch (Exception ex) { // Log exceptie } } |
Een berg moeilijk te ontcijferen code (want fuck SRP), binnen iedere iteratie van een foreach excepties afvangen die gecheckt hadden kunnen worden of op z'n minst buiten de loop eenmalig opgevangen kunnen worden, yay!
edit:
Hoe kon ik dit over het hoofd zien, prachtig:C#:
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
32
33
34
35
| if (ConfigurationManager.AppSettings["Debug"] == "1") { // doe wat debug-dingen } else { // doe wat echte dingen } if (ConfigurationManager.AppSettings["Debug"] == "1") { // doe andere debug-dingen } else { // doe andere echte dingen } if (ConfigurationManager.AppSettings["Debug"] == "1") { // doe nog meer debug-dingen } else { // doe nog meer echte dingen } if (ConfigurationManager.AppSettings["Debug"] == "1") { // doe verschillende debug-dingen } else { // doe verschillende echte dingen } |
Hard-wired debugcode, en zelfs dát verkeerd geimplementeerd
[ Voor 31% gewijzigd door kenneth op 05-12-2013 11:54 ]
Look, runners deal in discomfort. After you get past a certain point, that’s all there really is. There is no finesse here.
Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'
/u/172597/crop5e1225c8b1011.png?f=community)
Ik denk dat die gast het wel geprobeerd heeft te debuggen:
Hard-wired debugcode, en zelfs dát verkeerd geimplementeerd
No trees were harmed in creating this message. However, a large number of electrons were terribly inconvenienced.
/u/7375/1fb4348bd035882afac185e65c902b85.png?f=community)
Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'
Let op:
Uiteraard is het in dit topic niet de bedoeling dat andere users en/of topics aangehaald worden om ze voor gek te zetten. Lachen om je eigen code, of over dingen die je "wel eens tegengekomen bent" is prima, maar hou het onderling netjes.
Uiteraard is het in dit topic niet de bedoeling dat andere users en/of topics aangehaald worden om ze voor gek te zetten. Lachen om je eigen code, of over dingen die je "wel eens tegengekomen bent" is prima, maar hou het onderling netjes.