Screenshot

[himitsu]

Passt alles nicht.
Hier muß ja nichts zusätzlich implementiert werden, sondern es braucht nur die eine Ausgangsposition genutzt werden, welche immer passt.
Es ist sogar einfacher übersichtlicher das fertige Rect zu verwenden, anstatt selbst Eines zu bauen.

Aber klar, wenn man es jetzt nicht braucht, dann könnte man auch "einfach" nur 0 verwenden (KISS) und dann später erstmal den Fehler zu suchen, wenn man das Programm doch weitergegeben oder sich einen zweiten Monitor gekauft hat.

[freimatz]

Ich bin auch ein Verfechter von YAGNI und er muss sicher nicht zwei Monitore unterstützen.

Aus Sicht Clean-Code ist es trotzdem falsch. Es geht um "Screenshots". Wenn mal also den Screen möchte dann folgt daraus, dass man auch die Werte vom Screen nimmt und nicht ein "magic value" 0.

[Redeemer]

Zitat von himitsu:

Aber klar, wenn man es jetzt nicht braucht, dann könnte man auch "einfach" nur 0 verwenden (KISS) und dann später erstmal den Fehler zu suchen, wenn man das Programm doch weitergegeben oder sich einen zweiten Monitor gekauft hat.

Richtig, da sucht man sich zu Tode. Korrekte Begründung für das Weglassen wäre somit eher:

Zitat:

Bei nur einem Monitor (den ich auch nur habe und auch immer nur haben werde) geht es.