|
News:
|
//
// Dear maintainer:
//
// Once you are done trying to 'optimize' this routine,
// and have realized what a terrible mistake that was,
// please increment the following counter as a warning
// to the next guy:
//
// total_hours_wasted_here = 39
//
//If you don't understand this code, you should be flipping burgers instead.
|
|
| 1 |
Ich kringel mich vor lachen in der Ecke :D
– Floyd 21.07.2011
|
|
| 1 |
Hab ich oben bereits in meiner Fragestellung erwähnt, dass es auf StackOverflow bereits einen solchen Thread gibt und man bitte NICHT die Comments einfach kopieren oder übersetzen soll... ;-)
– Dustin Klein 21.07.2011
|
|
|
Der Link darf ruhig bleiben, dann haben wir wenigstens nur einen Kommentar, der einen Link dazu enthält, sonst kommen hier demnächst noch mehr auf die Idee ;-)
– Dustin Klein 21.07.2011
|
||
poRequest.HitMeBabyOneMoreTime = true; //Oh baby, baby
return this.SendRequest(poRequest);
|
|
|
Herrlich, ich finde den Kommentar schon gut, aber die HitMeBabyOneMoreTime Property is der Abschuss.
– ffordermaier 21.07.2011
|
||
// Erweiterungsmethode für boolsche
// Variablen um den Wert umzukehren
public static void Reverse(this bool b)
{
if (b == false) return true;
return false;
}
// Lehrer: Meinen Glückwunsch, du wurdest in den Grundkurs zurückversetzt!
inputValid.Reverse();
|
|
| 1 | ||
| 1 |
Ich hätte dem auch zumindest einen Pluspunkt für das unsinnigste aber kreativste Workaround gegeben :D
– Dustin Klein 21.07.2011
|
|
|
interessant ist ja auch, dass nicht die Variable auf false überprüft wird sondern direkt "bool" o_O
außerdem, wenn schon als Erweiterungsmethode, dann doch bitte einfach "return !b;" xD – Karill Endusa 21.07.2011
|
||
|
Der bool Fehler war ein Tippfehler beim abschreiben ;-) Danke für den Hinweis. Ganz so blöd war er dann auch wieder nicht :D
– Dustin Klein 21.07.2011
|
// Dear fellow programmer. You can be shure I tried my very best to make this look nicer.
// However there is no better solution to this.
// SO MOTHERFUCKER KEEP YOUR HANDS OFF.
|
|
|
Das erinnert mich an einen früheren Kollegen. Er hatte als Lieblingstätigkeit das "umorganisieren" der Sourcen. Eines Nachts hat er wieder mal "alles" umgebaut, und am Morgen lief nichts mehr *g*
– nabuchodonossor 22.07.2011
|
catch(DivideByZeroException ex){ //Only Cuck Norris can divide by zero
...
}
|
|
set
{
Debug.Assert(value != null);
if(value != null)
{
bnController = null; // darf nicht null sein
}
}
|
|
// Magic! Do not touch!!!
// Nicht anfassen!! Muß so ein!
|
|
|
normal, ne? :D
Wobei ich zugeben muss, das solche Kommentare durchaus ihre Daseinsberechtigung haben... bei der Verwendung externen Codes kann es durchaus vorkommen, dass man Sachen machen muss, die auf den ersten (bis hundertsten) Blick einfach vollkommen schwachsinnig und wahllos aussehen... interessanterweise funktioniert es aber nur dann. – Karill Endusa 16.08.2011
|
i++; // Increment i by one.
|
|
// TODO: remove me if possible
|
|
// TODO: Kommentar ändern wenn alles erledigt
// WW_Erzeugung - hat zwar keine Orientations, ist aber wurscht!
foreach (var v in this.ElementContainer.SelectedElements["WW_Erzeugung"].Orientations)
{ ...
|
|
if( IrgendEtwasKomplexesMitMehrerenAndsUndOrsUndVielleichtSogarKlammern ){
//nix
}
else{
doSomething();
}if( !(IrgendEtwasKomplexesMitMehrerenAndsUndOrsUndVielleichtSogarKlammern) )den selben Effekt hat.
|
|
|
Könnte daran liegen, dass hinter //nix früher was stand, was dann irgendwann überflüssig wurde. ;)
– puls200 12.08.2011
|
||
| 1 |
Ich finde, es spricht auch was dafür, die originale Aufteilung zu belassen: noch eine Klammerebene mehr, und davor dann dieser schrecklich unauffällige Not-Operator - leserlicher wird es dadurch nicht. Außerdem ergibt es in der Historie der Versionsverwaltung ein Diff, dem man auf den ersten Blick ansieht, was da geändert wurde.
Das eigentliche Problem ist hier m.E., dass IrgendEtwasKomplexesMitMehrerenAndsUndOrsUndVielleichtSogarKlammern nicht in eine Methode ausgelagert wurde, deren prägnanter (und hoffentlich kürzerer) Name zusammenfasst, was der Sinn des logischen Ausdrucks ist. – Matthias Hlawatsch 16.08.2011
|
|
//Anmerkung: bei iEinStr handelt es sich um einen String :D
iEinStr = iEinStr.Trim(" ".ToCharArray())
|
|
|
Ich finde das "i" - welches ich als Integer der ungarischen Notation werten würde - irgendwie viel schlimmer im Namen der String-Variable.
– Jens Duczmal 15.04.2012
|
||
|
war so nicht im Quelltext vorhanden und sollte hier nur als Abkürzung für "irgend" stehen :)
Keine Sorge, so schlimme Ausrutscher passieren mir nicht mehr – Karill Endusa 16.04.2012
|
catch
{
//Macht hier nix
}
|
|
printk(KERN_WARNING "GSCD: DMA kann ich noch nicht!\n");
linux-2.6.19/drivers/cdrom/gscd.c
(Do we have i18n'ed messages now? -- Ed.)
|
|