if-else Logikproblem

[quidnovi]

Hi @ all

sitze offenbar ziemlich auf der Leitung und bitte Euch um Eure Hilfe.
Eigentlich sollte das folgende if-else Problem gar keines sein.

Also:
Ich überprüfe 4 Eingabefelder.

Wenn alle 4 Eingaben stimmen, so sollte eine Weiterleitung stattfinden.
Leider habe ich meinen Code momentan so, dass der "ok" - Button für jedes Eingabefeld extra geklickt werden muss, was natürlich nicht Userfreundlich ist.

Hier mal der Code:

public void checkCalc4(String mlb, String mlh, String lw, String dlh) {

		// Eingabe ueberpruefen
		String lineMlb = mlb;
		String lineMlh = mlh;
		String lineLw = lw;
		String lineDlh = dlh;
		
		String pattern = "^[0-9]{3,4}$";
		Pattern r = Pattern.compile(pattern);
		
		Matcher mMlb = r.matcher(lineMlb);
		Matcher mMlh = r.matcher(lineMlh);
		Matcher mLw = r.matcher(lineLw);
		Matcher mDlh = r.matcher(lineDlh);
	
		if(mMlb.find()){
			validMlb = lineMlb;
			Masse.ok();
		}
		else{
			errPattern("Mauerlichte Breite");
			return;
		}
		if(mMlh.find()){
			validMlh = lineMlh;
			Masse.ok();
		}
		else{
			errPattern("Mauerlichte H\u00F6he");
			return;
		}
		if(mLw.find()){
			validLw = lineLw;
			Masse.ok();
		}
		else{
			errPattern("calc4 - Lichte Weite");
			return;
		}
		if(mDlh.find()){
			validDlh = lineDlh;
			Masse.ok();
		}
		else{
			errPattern("Duchgangslichte H\u00F6he");
			return;
		}
		
		// casten
		int mlbInt = Integer.parseInt(validMlb);
		int mlhInt = Integer.parseInt(validMlh);
		int lwInt = Integer.parseInt(validLw);
		int dlhInt = Integer.parseInt(validDlh);

		// ---------------------------------------------------------

		// Berechnen
		int ergLw = mlbInt - lwInt;
		int ergDlh = mlhInt - dlhInt;

		// Ergebnis ueberpruefen
		if (ergLw <= 250) {
			errCalcLw("Lichte Weite");
			return;
		}
		else{
			Masse.inDb();
		}
		if (ergDlh <= 250) {
			errCalcDlh("Durchgangslichte H\u00F6he");
			return;
		}
		else{
			Masse.inDb();
		}

	}

Ich habe momentan eine komplette Blockade im Hirn, die erforderliche Logik bringt meine graue Masse nicht zu Wege

Bitte lotst mich auf den richtigen Weg, damit der Anwender nur einmal klicken muss (natürlich nur dann, wenn alle 4 Eingabefelder verifiziert sind), danke

 

[Marco13]

Was soll denn Masse.ok() sein?

boolean a = ...
boolean b = ...
if (a && b)
{
     System.out.println("Beides");
}

 

[quidnovi]

Hi Marco13

auja...boolean....danke für Deinen Tip...damit sollte ich's erledigen können.

Masse.ok():

public static void ok(){
		btnWeiter.setActionCommand("LUFTTEILUNGST");
	}

Diese befindet sich eben in der beschriebenen class Masse.
Von der class Masse rufe ich ja die calc - Methode auf.

btnWeiter.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent e) {
					Calc calc = new Calc();
					calc.checkCalc2(txtMlb.getText(),txtMlh.getText());
				}
			});

Vielen Dank nochmal für den Tip

 

[Marco13]

Static in dieser Form ist Murks, aber das ist vermutlich ein anderes Thema...

 

[quidnovi]

yo...hab beim copy des Codes hier ins Forum, eine überholte Version erwischt.