If Bedingungen kombinieren?

[berndoa]

Hallo,
ich habe mal ein für meine Verhältnisse simples Problem/Frage:
Ich lese vom bildschirm die Farbe (Datentyp color) eines bestimmten Pixels ein.
Dann zerlege ich es in den rot, grün und blau Wert.

Nun will ich immer dann, wenn das Pixel sich innerhalb eines gewissen Farbbereich befindet, was tun.
Falls es aussehralb liegt, will ich was Anderes tun.

Aktuell sieht die Bedingung bei mir so aus:

if((r>=180&&r<=200)&&(g>=180&&g<=200)&&(b>=180&&b<=200)){...}
else{...}

Was sich nicht schön liest .

Da habe ich mich gefragt ob man das nicht einfacher schreiben kann.

Mathematisch würde man es einfach schreiben als
180<=r,g,b<=200

Aber ich bin mir unklar ob java solche Doppelungleichungen erlaubt,
geschweigedenn das kombinieren der Bedingungen dreier Variabeln.

Im Prinzip selbe Frage wäre es wie ich ein Ungleichungssystem wie
20<=r<=40
b>80
c<55

oder Vergleichbares irgendwie sinnvoll in eine if Bedingung packen kann ohne 20 Ungleichungen mit && verbinden zu müssen?

 

[httpdigest]

Naja, das hier:

20<=r<=40
b>80
c<55

Ist ja quasi in Java:

if (20 <= r && r <= 40
  && b > 80
  && c < 50) {
}

Ich behelfe mir da auch immer gerne mit Aufteilen in einzelne Zeilen und manuellem Eindrücken des Codes, damit's übersichtlicher aussieht.

Und für 180<=r,g,b<=200 kannst du dir ja eine Hilfsmethode schreiben:

private static boolean between(int min, int value, int max) {
  return min <= value && value <= max;
}

Dann ist's:

if (between(180, r, 200) &&
    between(180, g, 200) &&
    between(180, b, 200)) {
  ..
}

 

[yfons123]

warum machst du dir nicht eine tool klasse ?

public class RangeTester{
    private int upperBorder;
    private int lowerBorder;
    public void RangeTester(int upperBorder ,int lowerBorder){
        this.upperBorder = upperBorder;
        this.lowerBorder = lowerBorder;
    }
    public bool isInRange(int val){
        return (val >=lowerBorder&& upperBorder<=200;
    }
}
if(tester.isInRange(r) && tester.isInRange(g) && tester.isInRange(b))

geht bestimmt noch kürzer aber java hat noch nie viel auf kürze gegeben

 

[Robert Zenz]

Um den Gedanken von @httpdigest weirerzufuehren, ich finde es immer gut wenn man die Hauptlogik selbst so aussagekraeftig und lesbar hat wie es moeglich ist. Damit bietet sich das kapseln von komplexeren Bedingungen in Methoden mit guten Namen an:

if((r>=180&&r<=200)&&(g>=180&&g<=200)&&(b>=180&&b<=200)){...}
else{...}

private boolean isMediumGray(int red, int green, int blue) {
    return red >= 180 && red <= 200
        && green >= 180 && green <= 200
        && blue >= 180 && blue <= 200
}

if (isMediumGray(r, g, b)) {
    // ...
} else {
    // ...
}

 

[berndoa]

Dass du das dem Code angesehen hast dass das ein mittleres Grau ist, schockiert mich gerade :O

 

[Robert Zenz]

Dass du das dem Code angesehen hast dass das ein mittleres Grau ist, schockiert mich gerade :O

Ja, ja, ist eher ein leichtes Grau, jetzt wo ich so darueber nachdenke...