:strip_icc():strip_exif()/u/126096/crop580e4f8566fec_cropped.jpeg?f=community)
:strip_exif()/u/240696/Darth%2520Stewie.gif?f=community)
Ik ook, iemand die een boolean als string in de DB opsloeg leidde tot dit soort wildgroei aan mogelijke waarden.:
[...]
Het gaat me er niet om dat hij yes / no tikt in plaats van true / false. Mijn probleem met die code is dat hij strings en stringoperaties gebruikt in plaats van booleans voor booleaanse waardes. Dat is toch wel zo ongeveer een van de meest basic fouten die je kan maken.
Temeer omdat je op een gegeven moment tests krijgt als if (value == "yes" || value == "true" || value == "waar" || value == "ok"). Lach maar, ik heb het gezien
http://www.simple-hosting.nl/diensten/webhosting
daar hostte ik eerst wel eens wat, zijn niet duur. Eerste jaar helemaal niet, want ze hebben een actie (hebben ze meestal
)
:strip_exif()/u/22119/ch.gif?f=community)
Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz
). Ook genoeg code geschreven waarvan ik me achteraf afvroeg wat me bezielde, maargoed 
/u/35984/crop67a0be11e3453_cropped.png?f=community)
Je kan net zo goed ranzige code schrijven in willekeurig welke taal (nouja, op een paar academische talen na wellicht). Dat PHP een lage drempel heeft en juist beginnende (lees: slechte) programmeurs het gebruiken zorgt ervoor dat er relatief veel rotzooi in geschreven wordt, maar iemand die netjes in bijvoorbeeld JAVA kan coden kan ook netjes in PHP coden.
Sterker nog, doordat er wat meer abstractie in zit kun je juist hele nette classes krijgen. Geen lijst met includes, geen headerfiles, etc. Ik vind mijn PHP code er meestal netter uitzien dan mijn JAVA en C++ code
Functionele programmeertalen bedoel je? Side-effects zal je dan geen last van hebben, maar je kan je code zo compact schrijven dat je een maand bezig bent voor je 10 regels code snapt
Geen includes en headerfiles? Dat vind ik juist onhandig: die geven een mooie indicatie waar je dependencies liggen.Sterker nog, doordat er wat meer abstractie in zit kun je juist hele nette classes krijgen. Geen lijst met includes, geen headerfiles, etc. Ik vind mijn PHP code er meestal netter uitzien dan mijn JAVA en C++ code
Dat zag ik op Project Euler veel.:
[...]
Functionele programmeertalen bedoel je? Side-effects zal je dan geen last van hebben, maar je kan je code zo compact schrijven dat je een maand bezig bent voor je 10 regels code snapt
[...]
Geen includes en headerfiles? Dat vind ik juist onhandig: die geven een mooie indicatie waar je dependencies liggen.
Had ik met moeite met mijn imperatieve taal een probleem in 50 regels opgelost en ge-brute-forcet, zaten er van die 1-regelige functionele parels tussen.
/u/172597/crop5e1225c8b1011.png?f=community)
No trees were harmed in creating this message. However, a large number of electrons were terribly inconvenienced.
:strip_icc():strip_exif()/u/151534/crop5a48e625630a0_cropped.jpeg?f=community)
:strip_icc():strip_exif()/u/12461/crop65b195553d948.jpg?f=community)
Cool, 3D bugs
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.
@orf: er zijn minder tools beschikbaar, omdat er minder vaak serieus in wordt geprogrammeerd.
Er zijn genoeg tools die coding standards checken, en uiteindelijk heb je er maar 1 nodig, dus zelfs als het er minder zijn (wat ik betwijfel) is dat meer dan genoeg.
No trees were harmed in creating this message. However, a large number of electrons were terribly inconvenienced.
:strip_icc():strip_exif()/u/226826/aardblij-60.jpg?f=community)
Heb ik liever Y en N:
[...]
1 en 0 zijn nog korter
Let op: Mijn post bevat meningen, aannames of onwaarheden
:strip_exif()/u/13818/episodebutler_60x60.gif?f=community)
Leuk dat ze coding standards checken, maar daar gaat het hier niet om. Hoeveel tooling is er die een static code analysis kan doen? Dat je kunt afdwingen dat bij een if altijd {}'s worden gebruikt en of ze nu wel of niet op dezelfde regel staan is leuk, maar metrics en coverage zijn natuurlijk een stuk belangrijker voor je software kwaliteit.:
Mbv, ik weet niet in welke kringen jij rondhangt, maar jouw beeld van PHP kan niet gebasseerd zijn op feiten uit deze eeuw
Er zijn genoeg tools die coding standards checken, en uiteindelijk heb je er maar 1 nodig, dus zelfs als het er minder zijn (wat ik betwijfel) is dat meer dan genoeg.
Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'
/u/4024/burne.png?f=community)
Tja, metrics is vrij makkelijk: als je accolades bij ifjes kan afdwingen kan je ook McCabe, Halstead en LOC uitrekenen. Omdat een parser schrijven voor PHP zal hij niet in alle gevallen 100% kloppen, maar veel tools voor C++ zijn net zo goed in een paar regexjes geschreven
Code coverage bij PHP lijkt me onmogelijk, bij gebrek aan een compiler.
/u/147737/foto.png?f=community)
:strip_exif()/u/151465/Untitled-1.gif?f=community)
De functie heet natuurlijk "string compare" en niet "strings are equal"
(Je krijgt een verschil. Géén verschil: 0)
[ Voor 7% gewijzigd door Zeebonk op 26-03-2011 12:08 ]
Je bedoelt als naamA en naamB gelijk zijn aan elkaar neem ik aan? Of mis ik een quirk in C?
Probleem waar ik vandaag tegenaan liep. Een vrij grote applicatie geschreven in C++ waarbij je onder andere nieuwe characters aan kan maken. Index van deze characters (en heel veel meer zaken for that matter) moet uniek zijn maar is niet auto-incremental in de (MySQL) database (al een WTF op zich, maar goed).
Enfin, ik ging er vanuit dat ze iets deden als MAX(guid) + 1. Prima, kan beter, maar valt mee te leven. En in eerste instantie leek dat ook zo:
1
2
3
4
5
6
| QueryResult *result = CharacterDatabase.Query( "SELECT MAX(guid) FROM characters" ); if( result ) { m_CharGuids.Set((*result)[0].GetUInt32()+1); delete result; } |
Dat verklaart echter niet waarom, als ik zelf via een andere applicatie nieuwe guid's aanmaak, deze overschreven leken te worden. Tot ik las hoe ze het volgende guid genereren:
1
2
3
4
| T IdGenerator<T>::Generate() { return m_nextGuid++; } |
Tijdens startup wordt het hoogste guid in de database bepaald en vervolgens wordt elk guid gegenereert door (vorige guid) + 1 te pakken
En dit is een serverapplicatie die makkelijk weken achter elkaar draait. Kun je dus nooit via een andere applicatie een record inserten zonder dat die weer overschreven wordt 
[ Voor 69% gewijzigd door FragFrog op 26-03-2011 12:20 ]
Natuurlijk, maar als je een poosje bezig bent ben je niet meer zo helder. Dan vergeet je dat soort dingen en dan kan je je hoofd er wel over breken, het valt niet echt op als foutieve code.:
[...]
De functie heet natuurlijk "string compare" en niet "strings are equal"
(Je krijgt een verschil. Géén verschil: 0)
Overigens heb ik een rothekel aan C met een berg aan pointers, dus dan ben je al helemaal niet helder meer
. En voor OSX is er ook geen fatsoenlijke debug tool waarmee ik kan monitoren hoe references zitten (en in elkaar verweven zijn/raken).
:strip_icc():strip_exif()/u/77332/crop5e54ec42433bc_cropped.jpeg?f=community)
De lexer & parser zijn open-source. Als iemand degelijke tools had willen maken had dat al lang het geval kunnen zijn.:
[...]
Tja, metrics is vrij makkelijk: als je accolades bij ifjes kan afdwingen kan je ook McCabe, Halstead en LOC uitrekenen. Omdat een parser schrijven voor PHP zal hij niet in alle gevallen 100% kloppen, maar veel tools voor C++ zijn net zo goed in een paar regexjes geschreven
Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz
No trees were harmed in creating this message. However, a large number of electrons were terribly inconvenienced.
Ja, zo heb je er nog veel meer, maar die zouden allemaal niet bestaan als PHP niet zelf een compiler had, want dan viel er niet zoveel te cachen
Vrijwel alle analysetools die ik noemde maken daar idd gebruik van.
Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz
. Er werd gezegd dat PHP geen compiler heeft, en ik zeg dus dat 'ie die wel heeft, alleen zie je daar niks van.
Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz
Ga's met facebook praten? Die hebben HipHop geschreven omdat ze hun bestaande ontwikkelaars aan hun bestaande code willen laten prutsen maar ook geld wilden besparen op het aantal dozen in het datacenter. Tweemaal meer data uit een server met een iets lagere CPU-load dan wanneer je het op de php-intepreter draait.:
Ja, er zijn nog veel meer coole toolies, maar daar gaat het niet om
Ik denk dat er genoeg sites dezelfde kant op zijn gegaan zonder het aan de grote klok te hangen.
De laatste keer dat ik keek was Hyves bezig met z'n 4500-ste server. Gebruik van HipHop zou dat kunnen halveren. Reken op een euro of 50 per doos per maand, en reken eens uit wat dat bespaart? Wie zou dat laten liggen?
I don't like facts. They have a liberal bias.
/u/86654/crop68d4ff30987a8.png?f=community)
https://oneerlijkewoz.nl
Op papier is hij aan het tekenen, maar in de praktijk...
Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz
np
Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz
/u/233413/crop574768220d598_cropped.png?f=community)
Form:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
| private void BeginButton1Click(object sender, EventArgs e) { int teller = 0; foreach (RadioButton rb in algorGroupBox1.Controls) { if (rb.Checked) { _controller.CalculatePath(rb, teller, delayCheckBox1.Checked); } teller++; } DrawBoard(); DisplayGraphics(); } |
"Controller":
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
| internal void CalculatePath(RadioButton rb, int alg, bool p2) { _algorithms = new Algorithms(_board, _drawing); if (alg == 1 && rb.Name == "radioButton1" && rb.Checked == true) { _algorithms.Bfs(); } else if (alg == 2 && rb.Name == "radioButton2" && rb.Checked == true) { _algorithms.Dfs(); } else if (alg == 3 && rb.Name == "radioButton3" && rb.Checked == true) { _algorithms.Astar(); } else { MessageBox.Show("Er is een fout opgetreden."); } } |
Maar even alles verwijderd en LINQ gebruikt. Ik begon al te zweten toen er "using System.Windows.Forms;" in een non-form klasse stond.
Het A-Star gedeelte had ik voor ze gedaan... Meteen even een interface voor ieder algoritme gemaakt en wat enums gebruikt voor A-Star.
Uiteraard ook alles in dezelfde thread als het forum. Altijd leuk die witte formpjes met het Not-Responding bericht.
[ Voor 14% gewijzigd door Paprika op 27-03-2011 22:49 ]
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
| DECLARE @Foo VARCHAR(10); SELECT @Foo = 'Foo'; IF (@Foo <> 'Foo') PRINT '@Foo <> ''Foo''' ELSE PRINT '@Foo == ''Foo'''; SELECT @Foo = 'Bar'; IF (@Foo <> 'Foo') PRINT '@Foo <> ''Foo''' ELSE PRINT '@Foo == ''Foo'''; SELECT @Foo = NULL; IF (@Foo <> 'Foo') PRINT '@Foo <> ''Foo''' ELSE PRINT '@Foo == ''Foo'''; IF (ISNULL(@Foo, '') <> 'Foo') PRINT '@Foo <> ''Foo''' ELSE PRINT '@Foo == ''Foo'''; |
Uitvoer:
@Foo == 'Foo' @Foo <> 'Foo' @Foo == 'Foo' @Foo <> 'Foo'
Ik ook met m'n stomme kop.
[ Voor 10% gewijzigd door CodeCaster op 28-03-2011 17:39 ]
https://oneerlijkewoz.nl
Op papier is hij aan het tekenen, maar in de praktijk...
Verwijderd
1
2
3
| counter = 0; if (counter > 200) // if 2 s nothing start = 1; // Set start |
En dan verbaast zijn dat het niet werkt
:strip_icc():strip_exif()/u/59957/got-icon.jpg?f=community){kind=icon}
Die comment is nog ernstiger dan dat gebeuren met die counter imo.:
C:
En dan verbaast zijn dat het niet werkt
:strip_icc():strip_exif()/u/206968/325134.jpg?f=community)
/u/81985/crop566b00b43645a.png?f=community)
De define doet iig wat hij zegt te doen, return Positive Negative votes. Dat degene die de functie/define aanroept de waardes verkeerd wegzet, dat is idd slordig
If money talks then I'm a mime
If time is money then I'm out of time
Verwijderd
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
| public void downloadRooster(Context context) { final SharedPreferences settings = getPreferences(MODE_PRIVATE); try { HttpClient http = new DefaultHttpClient(); HttpGet requestToday = new HttpGet(); URL today = new URL("www.stijnmeijer.nl/rw/app.json/1/"+settings.getString("classIsSet", "error")); requestToday.setURI(new URI("http://www.stijnmeijer.nl/rw/app.json/1")); HttpResponse responseToday = http.execute(requestToday); Log.e(responseToday.getEntity().toString(), "entity"); url = requestToday.getURI().toString(); } catch(URISyntaxException lol) { // de exceptions die mogelijk voor kunnen komen catchen. Log.e("NOOOEE","zomg URISYNTAXEXCEPTION!!!!"); } catch(MalformedURLException MUE) { Log.e("MUE!", "MUE Exception in downloadRooster()"); Log.e(url, "The URL"); } catch(ClientProtocolException CPE) { Log.e("CPE!", "CPE Exception in downloadRooster()"); } catch(IOException IOE) { Log.e("IOE!", "IOE Exception in downloadRooster()"); } } |
Vooral lolz @ de
1
| Log.e("NOOOEE","zomg URISYNTAXEXCEPTION!!!!"); |
:strip_icc():strip_exif()/u/46721/wolf-av.jpg?f=community)
/u/89535/crop56f4e5c063de4.png?f=community)
:strip_icc():strip_exif()/u/87691/avatar_alex.jpg?f=community){kind=avatar}
/u/313216/crop679f937b7f4c1_cropped.png?f=community)
:strip_icc():strip_exif()/u/14965/crop6362b4571b01a_cropped.jpg?f=community)
Magic constants, awesome. Gebruik constanten die weergeven waarom het nummer 4 is. En als dat niet zo is, typ dan gewoon een 4. Het maakt qua code netheid niet uit of je INTEGER4 of '4' typt, iedereen weet dat 4 een integer is.
Het idee dat magic numbers pure evil zijn moet je bij algoritmen soms even aan je voorbij laten gaan. De methodenaam of eventueel inline commentaar kan ook prima aangeven waarom dat magic number er staat.
Dat C voorbeeld, als je je daar fijn bij voelt kan het geen enkele kwaad toch? Het is wel een beetje anders, omdat INTEGER4 => 4 iets meer dubbel op is dan BIT4 => 0x00000010
[ Voor 13% gewijzigd door Makkelijk op 10-04-2011 19:35 ]
Badieboediemxvahajwjjdkkskskskaa
:strip_icc():strip_exif()/u/78891/W-Seawolf.jpg?f=community)
:strip_icc():strip_exif()/u/104670/66407.jpg?f=community)
Verwijderd
. Dat had ik misschien niet helemaal duidelijk gemaakt.
1
2
| float n = 1.234; int k = n + .4; |
En dan afvragen waarom er geen 2 uitkomt (magisch afronden in me hoofd
)Volgende keer iets minder uitgaan van de wiskundige regels
.
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/6143/rincewind.jpg?f=community)
“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.”
/u/142011/crop65b383c6c6c2f_cropped.png?f=community)
Ken Thompson's famous line from V6 UNIX is equaly applicable to this post:
'You are not expected to understand this'
“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.”
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/315253/boy-tweakers.jpg?f=community)
. Als je een 0 ofzo bent vergeten hoef je niet overal dat 0tje toe te voegen
[ Voor 17% gewijzigd door Rutix op 11-04-2011 14:12 ]
Nothing to see here!
:strip_icc():strip_exif()/u/49080/Tealon.jpg?f=community)
1
2
3
4
| Validation of the transaction failed.: The specified value does not satisfy the expression ^((?![P|p][O|o][S|s][T|t][B|b][U|u][S|s]).)*$. |
Dit had natuurlijk ook eenvoudiger gekund
/dev/null
Verwijderd
Oh wacht, negative assertions... 
[ Voor 36% gewijzigd door Verwijderd op 12-04-2011 11:06 ]
ieuw regex:
code:
Dit had natuurlijk ook eenvoudiger gekund
1
2
3
4
| // /me negeert verhaal over magic constants even. if (!"postbus".equalsIgnoreCase(input)) { throw new TransactionValidationException("Validation of the transaction failed: The specified value is not 'postbus', but " + input); } |
Is waarschijnlijk ook een stuk sneller dan regex. En leesbaarder. En exception is optioneel.
: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.
:strip_icc():strip_exif()/u/237649/FFA.jpg?f=community)
Ja, dat wel. Maar als ik mij niet vergis zijn het geen identieke oplossingen. De regex gaat op meer situaties een TRUE geven dan jou IF:
[...]
ieuw regex
Java:
Is waarschijnlijk ook een stuk sneller dan regex. En leesbaarder. En exception is optioneel.
When you talk to God it's called prayer, but when God talks to you it's called schizophrenia
Dat is volgens mij niet equivalent aan de regex. De regex controleert als ik mij niet vergis of iets begint met postbus (case insensitive), niet of de hele input gelijk is aan postbus.:
[...]
ieuw regex
Java:
Is waarschijnlijk ook een stuk sneller dan regex. En leesbaarder. En exception is optioneel.
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.
“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.”
[ Voor 46% gewijzigd door .oisyn op 12-04-2011 12:42 ]
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.
O ja nu zie ik het. Lekker vreemde validatie dan, ik vraag me af wat er exact de bedoeling van is ( En of die Regexp dat ook doet ):
Nee, want jij accepteert ook postbusABCDEF. En je accepteert een lege string niet, terwijl de regex dat wel doet
@PrisonerOfPain: Sowieso is het natuurlijk een slecht idee om werkende dingen te gaan wijzigen zonder dat daar een goede reden voor is, maar ik denk dat veel van de Regex oplossingen ook niet exact doen wat de maker bedacht heeft dat ze moeten doen.
[ Voor 26% gewijzigd door Woy op 12-04-2011 13:21 ]
“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.”
Maar voor andere mensen haalt het toch vaak een beetje de readability weg.
Nothing to see here!
:strip_exif()/u/67459/m002.gif?f=community)
Dat lijkt wel verdomd veel op een regex van SIDN. Die houden er niet van als je een postbus als bedrijfsadres invult. Wij hadden dit in eerste instantie wel.:
code:
Dit had natuurlijk ook eenvoudiger gekund
Toen hebben ze bij de introductie van "DRS5" als oplossing de volgende regular expression toegepast:
1
| ^((?![P|p]ostbus).)*$. |
Ze waren dan ook not amused toen mijn collega daar leuk 'POSTBUS' invulde. Nu hebben ze dat 'opgelost', met jouw regexp tot gevolg
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.
“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.”
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/104956/epica_got.jpg?f=community)
Bedoel je het foutieve gebruik van de Logtags?:
Ohh hoe slecht van mij
Java:
Vooral lolz @ deJava:
:strip_icc():strip_exif()/u/53213/crop57ace20969734.jpeg?f=community)
/u/24411/img.png?f=community)
Nou, schrijf deze maar eens om naar iets leesbaars:
1
2
| import re m = re.compile('^((00|11)*(01|10)(11|00)*(10|01))*(00|11)*(01|10)(11|00)*$') |
Ipsa Scientia Potestas Est
NNID: ShinNoNoir
:strip_icc():strip_exif()/u/190884/dug.jpg?f=community)
Trok je zo oude versies van IE niet over de zeik? Als je doctype niet als eerste stond negeerde oude versies van IE deze en ging hij sowieso in Quicks mode. Erg fijne "feature". Weet niet of het voor lege regels ook zo was, maar als je commentaar boven je doctype plaatste had IE sowieso schijt aan je doctype...
IE + Doctype, the horror
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.
A potentially dangerous Request.Form value was detected from the client (ctl00$Main$IssueUC1$rtcDescription$txtTextBox=".... [b]Interesse[/b]..."). at System.Web.HttpRequest.ValidateString(String s, String valueName, String collectionName)
Let op het vetgedrukte Interesse, dat wordt dus wel vrolijk zonder controle naar de browser gestuurd
WTF?
Look, runners deal in discomfort. After you get past a certain point, that’s all there really is. There is no finesse here.
Dit topic is gesloten.
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.