0 registered members (),
1,173
guests, and 0
spiders. |
Key:
Admin,
Global Mod,
Mod
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: Uhrwerk]
#437740
02/24/14 20:10
02/24/14 20:10
|
Joined: Apr 2007
Posts: 3,751 Canada
WretchedSid
Expert
|
Expert
Joined: Apr 2007
Posts: 3,751
Canada
|
Mir wird bestimmt wieder vorgeworfen ich sei ein Fanboy, aber hat keiner von euch jemals n merge versaut und dabei solche überreste hinterlassen? Mir persönlich bereits passiert.
Dass das nicht durch unit tests aufgefangen wurde, DAS ist viel mehr eine schweinerei. Aber dafür wäre dann wohl jemand anders zu erschlagen.
Shitlord by trade and passion. Graphics programmer at Laminar Research. I write blog posts at feresignum.com
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: WretchedSid]
#437741
02/24/14 20:18
02/24/14 20:18
|
Joined: Sep 2003
Posts: 9,859
FBL
Senior Expert
|
Senior Expert
Joined: Sep 2003
Posts: 9,859
|
Die doppelte Zeile war sicher ein versehentliches CTRL+D statt CTRL+C. Aber unabhängig davon... die Art und Weise wie dieser Code geschrieben ist - und mir ist egal ob da Apple oder sonstwer im Header oben steht - das geht einfach gar nicht . Und allein darauf wollte ich mich beziehen, nicht darauf, dass sich da jemand vertippt hat.
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: FBL]
#437742
02/24/14 20:24
02/24/14 20:24
|
Joined: Apr 2007
Posts: 3,751 Canada
WretchedSid
Expert
|
Expert
Joined: Apr 2007
Posts: 3,751
Canada
|
Was genau stört dich denn daran? Die gotos?
Shitlord by trade and passion. Graphics programmer at Laminar Research. I write blog posts at feresignum.com
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: FBL]
#437745
02/24/14 21:07
02/24/14 21:07
|
Joined: Apr 2007
Posts: 3,751 Canada
WretchedSid
Expert
|
Expert
Joined: Apr 2007
Posts: 3,751
Canada
|
Mhhh, ich muss sagen das mich das goto nicht stört. Die alternative ist entweder ein deeply nested scope oder jede Menge code duplizieren durch lokale variablen aufräumen. Goto ist schlecht, aber es hat durchaus nutzen. Das mit den Klammern... Sicher? Ich kann mir vorstellen das ein kaputter merge auch das hier durchgelassen hätte:
{ goto fail; }
{ goto fail; }
Würde auch kompilieren, selbes Problem. YMMV. Ich steh übrigens total auf zuweisungen in ifs, aber ich steh auch auf ternary operatoren. Und das extra klammer paar, ohne das es beim kompilieren eh ne Warnung unter Clang gibt, macht es deutlich das es kein Vergleich ist. Generell finde ich expressions extrem sexy, PLUS, selbst simple compiler können aus einem ternary ein conditional move machen. Aber das ist durchaus Geschmackssache. Solange man konsequent bei einem stil in einem Projekt bleibt kann ich damit leben.
Shitlord by trade and passion. Graphics programmer at Laminar Research. I write blog posts at feresignum.com
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: WretchedSid]
#437746
02/24/14 21:12
02/24/14 21:12
|
Joined: Sep 2003
Posts: 9,859
FBL
Senior Expert
|
Senior Expert
Joined: Sep 2003
Posts: 9,859
|
{ bla; } macht man auch nicht.
{ bla; }
so und nicht anders (+ Indent, danke Forum).
Last edited by Firoball; 02/24/14 21:12.
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: FBL]
#437747
02/24/14 21:17
02/24/14 21:17
|
Joined: Apr 2007
Posts: 3,751 Canada
WretchedSid
Expert
|
Expert
Joined: Apr 2007
Posts: 3,751
Canada
|
Das ist jetzt aber wieder zurück zur persönlichen Geschmackssache. Ich persönlich würde keine Klammern drum machen weil ich es ästhetisch als grauen empfinde.
Edit: Solche bugs sollten sowieso durch unit tests gefunden werden, nicht über die persönliche Präferenz code zu schreiben. Und das ist, wie bereits erwähnt, was ich als größten fail an der Sache finde: Apple hatte keine unit tests für die Zertifikat validierung. What. The. Fuck?!
Last edited by JustSid; 02/24/14 21:18.
Shitlord by trade and passion. Graphics programmer at Laminar Research. I write blog posts at feresignum.com
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: WretchedSid]
#437748
02/24/14 21:22
02/24/14 21:22
|
Joined: Sep 2003
Posts: 9,859
FBL
Senior Expert
|
Senior Expert
Joined: Sep 2003
Posts: 9,859
|
Unit Test Fail ist natürlich ein echtes Übel. Dennoch gibt es viele Codierrichtlinien nicht ohne Grund. Man darf nicht vergessen, dass man bei nem großen Projekt nie alle Coder auf gleich hohem Niveau haben wird. Ich erleb ja regelmäßig in der Arbeit, was für ein Wahnsinn da in die Codebasis geraged wird. Deswegen sind bei uns solche Abkürzungen um Codezeilen zu sparen eigentlich verboten... eigentlich. Außerdem... noch vor nem Unit Test sollte es Code Reviews geben. Zumindest bei solchen sicherheitsrelevanten Sachen. Sowas darf der Reviewer nicht verpennen!
|
|
|
Re: Der "mich kotzt etwas verdammt an"- Thread
[Re: FBL]
#437750
02/24/14 22:10
02/24/14 22:10
|
Joined: Nov 2007
Posts: 2,568 Germany, BW, Stuttgart
MasterQ32
Expert
|
Expert
Joined: Nov 2007
Posts: 2,568
Germany, BW, Stuttgart
|
@Firo: Ternary zu verbieten ist aber teilweise auch grauenhaft, führt zu schlechtem Code. Oder zu tollen, unlösbaren Problemem. Beispiel:
using (GameWindow window = new GameWindow(
presentation.Resolution.Width,
presentation.Resolution.Height,
presentation.GraphicsMode,
presentation.Title,
presentation.IsFullscreen ? GameWindowFlags.Fullscreen : GameWindowFlags.Default,
presentation.DisplayDevice ?? DisplayDevice.Default,
3, 3,
#if DEBUG
GraphicsContextFlags.Default | GraphicsContextFlags.ForwardCompatible | GraphicsContextFlags.Debug))
#else
GraphicsContextFlags.Default | GraphicsContextFlags.ForwardCompatible))
#endif
{
}
Klar, man könnte ne Variable verwenden, aber ist das SO viel besser? Ich hab ne Variable mehr, senkt auch wieder die Wartbarkeit und die Möglich für Fehler und Fehlnutzung.
|
|
|
|