Kwam dit laatst tegen in een project waar ik vroeger met wat mede studenten aan heb gewerkt.
:strip_icc():strip_exif()/u/92491/crop64a1593f33a7b_cropped.jpg?f=community)
Dus je krijg nooit Vrouw terug 
Al wat ik aanraak werk niet meer zoals het hoort. Damic houd niet van zijn verjaardag
:strip_exif()/u/396800/D-_3427da0a9c293b63f2a66dea2e642102.gif?f=community)
Dacht het niet
als de tekst Male bevat ga je 1 if verder dan doe je weer een check en daar bepaal je man/vrouw. of die else moest op de eerst if slagen maar denk het niet.
[ Voor 9% gewijzigd door Damic op 09-06-2016 21:30 ]
Al wat ik aanraak werk niet meer zoals het hoort. Damic houd niet van zijn verjaardag
:strip_exif()/u/69039/MetallicA.gif?f=community)
Er wordt Equals gebruikt, niet Contains.:
[...]
Dacht het niet
En die rode regel is vervangen door de groene. (of begrijp ik ne post nou verkeerd)
🇪🇺 pro Europa! | Puinhoop Veroorzaken en Vertrekken (PVV)
In de oude situatie was een SheMale een Man geweest, in de nieuwe situatie is het een Vrouw.
Nu ga ik zelf de fout in
Het is equals inderdaad.
Nu ga ik zelf de fout in
Het is equals inderdaad.
[ Voor 24% gewijzigd door CurlyMo op 09-06-2016 22:00 ]
Sinds de 2 dagen regel reageer ik hier niet meer
/u/142011/crop65b383c6c6c2f_cropped.png?f=community)
Nog nooit een diff tool gezien? ^^:
[...]
Dacht het niet
:strip_exif()/u/259931/stuiterbal2.gif?f=community)
Ach, als ik naar de commit message kijk dan heb ik ook niet zo'n hoge pet op van de programmeur in kwestie...:
[...]
De laatste edit is van 9 juni 2016
Dit is echt van het nivo : git commit -m "Wrote some code"
Als die mensen nu eens de { ook zouden sluiten is het makkelijker om te zien wat er zou gebeuren }, nu zien ik het als
if(.contains()){
if(sex==male){add('man');}else{add(vrouw');}
}
:
In de oude situatie was een SheMale een Man geweest, in de nieuwe situatie is het een Vrouw.
Nu ga ik zelf de fout in
Nope
Al wat ik aanraak werk niet meer zoals het hoort. Damic houd niet van zijn verjaardag
Ik maak mij niet druk hoor, ik heb wel betere dingen te doen:
[...]
Maak je toch niet zo druk yo
Verder ook geen zin om hier nog woorden aan vuil te maken, ik heb gezegd wat ik wilde zeggen dus voor mij is het verder klaar (ik heb ook geen zin om dit topic verder te vervuilen, ik lees liever voorbeelden van slechte code
)
:strip_exif()/u/47168/loop.gif?f=community)
De code style is helder en consistent in beide revisies.
Dat het een commit is, met een view met dubbele regelnummers, -/+ aan begin van regel en ook nog een achtergrondkleur zou toch voldoende hint moeten zijn, zelfs al diff je niet vaak.
{signature}
Over de achtergrondkleur valt te twisten... zo'n 5% van de mannelijke bevolking kan het verschil tussen rood en groen niet zien:
[...]
De code style is helder en consistent in beide revisies.
Dat het een commit is, met een view met dubbele regelnummers, -/+ aan begin van regel en ook nog een achtergrondkleur zou toch voldoende hint moeten zijn, zelfs al diff je niet vaak.
Eigenlijk is zo'n diff-tool dus ook een voorbeeld van een slecht programmeervoorbeeld, vanuit usability bekeken.
[ Voor 10% gewijzigd door Jerrythafast op 10-06-2016 13:28 ]
/u/470671/elephant.png?f=community)
Nope, helemaal niet, usability zegt niet dat je geen gebruik van kleur mag maken, alleen dat je niet uitsluitend op kleur mag vertrouwen om betekenis over te dragen.:
[...]
Over de achtergrondkleur valt te twisten... zo'n 5% van de mannelijke bevolking kan het verschil tussen rood en groen niet zien
Maar dat is nadrukkelijk niet het geval, je hebt namelijk ook de regelnummering en de +/- indicator.
Never explain with stupidity where malice is a better explanation
/u/301967/pingu2.png?f=community)
5%? Dus 1 op de 20 mannen kunnen het verschil tussen rood en groen niet zien? Lekker gevaarlijk in het verkeer.:
[...]
Over de achtergrondkleur valt te twisten... zo'n 5% van de mannelijke bevolking kan het verschil tussen rood en groen niet zien
Ik wil even opkomen voor de kleurenblinden onder ons, gezien ik er zelf bij hoor.
Ik kan prima het verschil tussen het rood en groen in deze screenshot zien.Echter zijn er wel enkele difftools die met een scherm met slecht contrast (e.g., laptop met scherm te ver naar voren), waarbij het voor mij soms moeilijk te herkennen is.
/u/207803/obly.PNG?f=community)
"There are two hard things in computer science: cache invalidation, naming things, and off-by-one errors."
:strip_icc():strip_exif()/u/109161/crop5a7da4f89d10d_cropped.jpeg?f=community)
/u/318471/crop5db76117a451a_cropped.png?f=community)
/u/468514/crop67a52a3ca0480_cropped.png?f=community)
/u/313216/crop679f937b7f4c1_cropped.png?f=community)
C#:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
| public static class DataTableExtensions { public static DataTable Table(this DataSet ds, string tableName) { if (ds.Tables.Contains(tableName)) return ds.Tables[tableName]; return null; } public static DataTable Table(this DataSet ds, int tableIndex) { return ds.Tables[tableIndex]; } } |
Why???
Extension Methods maken om Extension Methods te maken denk ik.
[ Voor 7% gewijzigd door Styxxy op 29-06-2016 18:37 ]
:strip_icc():strip_exif()/u/88819/larry_new.jpg?f=community)
Ben nog vers in het programmeercircuit, ik vroeg me dus af wat het probleem was.:
C#:
Why???
Extension Methods maken om Extension Methods te maken denk ik.
Is het omdat de klasse zelf geen nut heeft en het net zo goed methodes van DataSet hadden kunnen zijn?
:strip_exif()/u/306933/ezgif.com-optimize.gif?f=community)
Als Tables een package-local item is en deze extensions class in dezelfde package zit is het niet 100% overbodig. Alleen had het dan net zo goed direct geïmplementeerd kunnen worden.:
C#:
Why???
Extension Methods maken om Extension Methods te maken denk ik.
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/524336/crop57ffe766deaff_cropped.png?f=community)
| Mijn joystick / simpit project | Blog | Elite Dangerous CMDR Cataclysm72
:strip_icc():strip_exif()/u/427720/crop5656ff33ad7d0_cropped.jpeg?f=community)
JSON:
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
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
| { "BookDate": "2016-12-18T10:39:05.276Z", "BusinessObject": "SO1", "Table": { "Definition": { "Name": "SO_HDR", "Fields": [ { "Name": "RowId", "FieldType": "C" }, { "Name": "INV_DATE", "FieldType": "T" }, { "Name": "RPL_INV", "FieldType": "C" }, { "Name": "DEL_DATE", "FieldType": "T" }, { "Name": "COMMENT1", "FieldType": "C" }, { "Name": "ORD_DATE", "FieldType": "T" }, { "Name": "ORD_NR", "FieldType": "C" }, { "Name": "RPL_DEL", "FieldType": "C" }, { "Name": "VAT_TOT", "FieldType": "N" } ] }, "DetailDefinitions": [ { "Name": "SO_LINE", "Fields": [ { "Name": "RowId", "FieldType": "C" }, { "Name": "HeaderId", "FieldType": "C" }, { "Name": "DEL_DATE", "FieldType": "T" }, { "Name": "REC_ID", "FieldType": "C" }, { "Name": "ORD_NR", "FieldType": "C" }, { "Name": "ART_CODE", "FieldType": "C" }, { "Name": "VART_CODE", "FieldType": "C" }, { "Name": "ART_DESC1", "FieldType": "C" }, { "Name": "ORD_QTY", "FieldType": "N" } ] } ] }, "TableData": { "Data": { "Rows": [ { "Values": [ "1", "2016-11-30T13:37:37.450Z", "TEST", "2016-11-30T13:37:37.450Z", "Test", "2016-11-30T13:37:37.450Z", "TEST", "TEST", 1337 ] } ] }, "DetailData": [ { "Rows": [ { "Values": [ "1", "1", "2016-11-30T13:37:37.450Z", "TEST", "TEST", "TEST", "TEST", "Test", 1337 ] } ] } ] } } |
Op zich niet heel verkeerd, tenzij je dit natuurlijk de betreffende partij moet sturen waaronder dus zijn eigen veldnamen en definities en in welke volgorde ze komen, en natuurlijk niet te vergeten wat voor sort data er in het veld zit.
Dit kan makkelijke gecomprimeerd worden tot een klein stukje JSON die ik elke keer als ik er een Order in wil schieten kan gebruiken maar nee dit is stukken handiger
“Never underestimate the bandwidth of a station wagon full of tapes hurtling down the highway.”
:strip_icc():strip_exif()/u/104670/66407.jpg?f=community)
:strip_exif()/u/267916/gif_70x53_b432ec.gif?f=community)
Voor abonnementen in ons interne systeem gebruiken we periodes. Deze staan als volgt in een tabel:
name, seconds
Dus bijvoorbeeld:
Jaarlijks, 31557600
Opzich snap ik de logica erachter. Maar zou ik het zelf aan pakken. Nu ging ik de code voor het berekenen van de verlengingsdatum bekijken:
name, seconds
Dus bijvoorbeeld:
Jaarlijks, 31557600
Opzich snap ik de logica erachter. Maar zou ik het zelf aan pakken. Nu ging ik de code voor het berekenen van de verlengingsdatum bekijken:
PHP:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
| public function calculate_date_extension($subscription_id) { $subscription = new Subscription($subscription_id); $subscription_period = new Subscription_Period($subscription->period_id); $subscription_invoice = new Subscription_invoice(); $subscription_invoice->where("subscription_id", $subscription_id)->get(); $count = 0; if ($subscription_invoice->exists()) { $count = $subscription_invoice->where("subscription_id", $subscription_id)->count(); } $count++; if ($subscription_period->name == "Jaarlijks") { return date("Y-m-d", strtotime($subscription->date_start . "+ {$count} year")); } } |
/u/262310/ava.png?f=community)
:
Voor abonnementen in ons interne systeem gebruiken we periodes. Deze staan als volgt in een tabel:
name, seconds
Dus bijvoorbeeld:
Jaarlijks, 31557600
Opzich snap ik de logica erachter. Maar zou ik het zelf aan pakken. Nu ging ik de code voor het berekenen van de verlengingsdatum bekijken:
PHP:
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
:strip_exif()/u/33451/kirk_bones.gif?f=community)
| Mijn joystick / simpit project | Blog | Elite Dangerous CMDR Cataclysm72
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
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
| Mijn joystick / simpit project | Blog | Elite Dangerous CMDR Cataclysm72
Laatst een keer tegengekomen in iemands C# code...
ToUpper() op een string doen om vervolgens te vergelijken met een numerieke waarde
if(value.ToUpper() == "3") { ... }
ToUpper() op een string doen om vervolgens te vergelijken met een numerieke waarde
if(value.ToUpper() == "3") { ... }
1 + 1 = 3, for large values of 1?:
Laatst een keer tegengekomen in iemands C# code...
ToUpper() op een string doen om vervolgens te vergelijken met een numerieke waarde
if(value.ToUpper() == "3") { ... }
| Mijn joystick / simpit project | Blog | Elite Dangerous CMDR Cataclysm72
/u/310759/crop5a8fa260a4c33_cropped.png?f=community)
De vergelijking is daar gewoon met een string.:
Laatst een keer tegengekomen in iemands C# code...
ToUpper() op een string doen om vervolgens te vergelijken met een numerieke waarde
if(value.ToUpper() == "3") { ... }
Komt d'r in, dan kö-j d’r oet kieken
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/200133/artemis_square_e_60-60.png?f=community)
De vraag is dan wel wat ToLower() dan doet als je het toepast op "3". Deze code lijkt te impliceren dat de lowercase van "3" niet gelijk is aan "3":
[...]
De vergelijking is daar gewoon met een string.
Misschien denk de auteur dat "3.14".toLower() gelijk is aan "3". Wellicht door PHP achtergrond. /satire:
[...]
De vraag is dan wel wat ToLower() dan doet als je het toepast op "3". Deze code lijkt te impliceren dat de lowercase van "3" niet gelijk is aan "3"
| Mijn joystick / simpit project | Blog | Elite Dangerous CMDR Cataclysm72
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/57387/claimedavatar-70.png?f=community){kind=avatar}
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
Precies dit inderdaad. En het stond niet eens op 1 plek haha
[ Voor 6% gewijzigd door PWSteal op 24-07-2016 11:21 ]
:strip_icc():strip_exif()/u/36143/stowa.jpg?f=community)
Wat is er mis met de oude?
Ik kwam een tijdje terug nog een constructie tegen in de code hier waarvan ik even op moest zoeken wat het was, en vervolgens niet kon bedenken waarom het a) mogelijk gemaakt is, en b) je dat zou willen gebruiken:
code:
1
2
3
4
| public void SomeMethod()
{
var @break = new Klant();
} |
Het gaat om het gebruik van het @-teken voor de naam van de variabele. Voor degenen die het ook zouden moeten opzoeken: hiermee kun je in C# keywords (zoals class, in, out, goto, break, continue, etc etc) gebruiken als variabel-namen. Leuk dat het kan, maar waarom zou je in hemelsnaam keywords willen gebruiken hiervoor?
:strip_icc():strip_exif()/u/36003/crop584e9e5c223bd.jpeg?f=community)
Look, runners deal in discomfort. After you get past a certain point, that’s all there really is. There is no finesse here.
Goeie, Razor views was ik even vergeten. Nu ik dat zo teruglees bedenk ik me dat ik het waarschijnlijk al veel eerder ben tegengekomen.
Je bedoelt een event zoals een feestje, optreden of braderie ergens ofzo? Dat kan ik me nog wel voorstellen ja, al zou ik dan toch m'n best doen om dat keyword te omzeilen.
Look, runners deal in discomfort. After you get past a certain point, that’s all there really is. There is no finesse here.
ben benieuwd waar die 3 vandaan komt dan, leg uit:
[...]
Of je mist even de '/satire' aan het einde van zijn bericht
Je gaat nog al hard in op de zin 'Wellicht door PHP achtergrond', terwijl dat helemaal niet serieus bedoeld was. Je kan er een bloedserieus codevoorbeeld tegenaangooien, maar dat maakt niet uit.:
[...]
ben benieuwd waar die 3 vandaan komt dan, leg uit
/u/224026/crop5a1c1a6fcf1b1.png?f=community)
Met ToUpper heb ik nog een keer leuke meegemaakt, als je een case sensitive database hebt, en je doet een to upper, dan word de index genegeerd:
Laatst een keer tegengekomen in iemands C# code...
ToUpper() op een string doen om vervolgens te vergelijken met een numerieke waarde
if(value.ToUpper() == "3") { ... }
:strip_icc():strip_exif()/u/51489/heforshe_logo_detail.jpg?f=community){kind=logo}
:strip_icc():strip_exif()/u/406001/crop577a0fb20a5ab_cropped.jpeg?f=community)
Of...
Erg veel quirks blijven ook nog wel even... Maar het blijft voorlopig de taal waarin ik het snelst resultaat boel
[edit]Hier klaagde een collega recent over. Ik kan het alleen niet reproduceren
code:
1
2
| $res = true && false; // false $res = true AND false; // true |
Erg veel quirks blijven ook nog wel even... Maar het blijft voorlopig de taal waarin ik het snelst resultaat boel
[edit]Hier klaagde een collega recent over. Ik kan het alleen niet reproduceren
[ Voor 17% gewijzigd door mbarie op 29-07-2016 09:02 ]
Tsja, voor velen is het redelijk hetzelfde, maar het eerste is een logische AND (beide waarden moeten TRUE opleveren) terwijl het tweede een bitwise AND (beter: &) is (vergelijkt per bit of het gelijkwaardig is en geeft de resulterende bitwaarde terug).:
Of...
code:
Erg veel quirks blijven ook nog wel even... Maar het blijft voorlopig de taal waarin ik het snelst resultaat boel
[edit]Hier klaagde een collega recent over. Ik kan het alleen niet reproduceren
Dat vrijwel niemand de bitwise operators gebruikt en daarom niet snapt wat er gebeurt maakt de taal natuurlijk niet meteen fout.
Feitelijk moet je AND gewoon niet gebruiken in PHP om het helder te houden. Gewoon & vs &&, dat is duidelijk voor iedereen.
[ Voor 8% gewijzigd door Tjolk op 29-07-2016 09:21 ]
Tjolk is lekker. overal en altijd.
:strip_icc():strip_exif()/u/289675/crop6401bf2c85501_cropped.jpg?f=community)
Krijg toch echt::
Of...
code:
Erg veel quirks blijven ook nog wel even... Maar het blijft voorlopig de taal waarin ik het snelst resultaat boel
[edit]Hier klaagde een collega recent over. Ik kan het alleen niet reproduceren
code:
1
| bool(false) bool(false) |
Je kan volgensmij ook OR gebruiken i.p.v. ||, maar mijzelf aangeleerd dit nooit te doen.
/u/341393/zandloper2.png?f=community)
Zou wel het gedocumenteerde gedrag zijn. && heeft een hogere presedence dan =, terwijl 'and' een lagere heeft. Daardoor zou de 2e expressie in je voorbeeld equivalent zijn aan:
Of...
code:
Erg veel quirks blijven ook nog wel even... Maar het blijft voorlopig de taal waarin ik het snelst resultaat boel
[edit]Hier klaagde een collega recent over. Ik kan het alleen niet reproduceren
code:
1
| ($res=TRUE) and FALSE; |
Al zou ik ook geloven dat dit soort quirks heel stil gefixed worden, om de fuckup maar te verbergen.
:strip_icc():strip_exif()/u/249917/jaapschaap.jpg?f=community)
Look, runners deal in discomfort. After you get past a certain point, that’s all there really is. There is no finesse here.
/u/286895/icon1.png?f=community)
:strip_exif()/u/13818/episodebutler_60x60.gif?f=community)
Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'
:strip_icc():strip_exif()/u/78891/W-Seawolf.jpg?f=community)
hmm ja, daar heb je een punt
Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'
/u/126369/Daos2.png?f=community)
Collega van het vele nesten zoals:
heeft gelukkig LINQ ontdekt:
maar hij kent nog niet alle functies
De if kan ook zo:
maar alles kan ook met 1 functie-aanroep ipv 3:
C#:
1
2
3
4
5
6
7
8
9
| foreach (var item in items) { if (item.ID == id) { // do something with item break; } } |
heeft gelukkig LINQ ontdekt:
C#:
1
2
3
4
5
6
7
8
9
10
| var res = items.Where(i => i.ID == id); if (res.Count() < 1) { return; } var item = res.ElementAt(0); // do something with item |
maar hij kent nog niet alle functies
De if kan ook zo:
C#:
3
| if (!res.Any()) |
maar alles kan ook met 1 functie-aanroep ipv 3:
C#:
1
2
3
4
5
6
7
8
| var item = items.FirstOrDefault(i => i.ID == id); if (item == null) { return; } // do something with item |
| Mijn joystick / simpit project | Blog | Elite Dangerous CMDR Cataclysm72
HEt is nu niet het allerslechte maar echt goed is het nu ook weer niet
*gelukkig draai ik T/F niet om
Visual Basic .NET:
1
| Using objWriter As New StreamWriter(FileToSaveTo, IIf(File.Exists(FileToSaveTo), True, False)) |
*gelukkig draai ik T/F niet om
[ Voor 12% gewijzigd door Damic op 11-08-2016 22:07 ]
Al wat ik aanraak werk niet meer zoals het hoort. Damic houd niet van zijn verjaardag
: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/87691/avatar_alex.jpg?f=community){kind=avatar}
:strip_icc():strip_exif()/u/419131/crop647054edc1b07_cropped.jpg?f=community)
Verwijderd
Inderdaad, ik heb werkelijk nog nooit een regel LINQ to Entities geschreven. LINQ to Objects daarentegen
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.