Peut-on simplifier ce code conditionnel

Version imprimable

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
    private void calcul(){
        if (eDia.getText().length() == 0) {
            Toast.makeText(MainActivity.this,"Vous n'avez pas rempli le diamètre", Toast.LENGTH_SHORT).show();
        }else{
            Resultat=Double.valueOf(eDia.getText().toString()).doubleValue();
            ePoids.setText(String.valueOf(Resultat));
 
        	if (eLong.getText().length() == 0) {
        		Toast.makeText(MainActivity.this,"Vous n'avez pas rempli la longueur", Toast.LENGTH_SHORT).show();
        	}else{
        		Resultat=Double.valueOf(eDia.getText().toString()).doubleValue();
        		ePoids.setText(String.valueOf(Resultat));
 
        		} 
 
        	}
 
    }

CA me parait un peu confus, et j'ai encore des valeur a tester , avant de faire la calcul proprement dit

merci d'avance

André

Bonsoir
Si je ne me trompe pas, il me semble que ce jeu d'instructions se répète

Code:

1
2
Resultat=Double.valueOf(eDia.getText().toString()).doubleValue();
ePoids.setText(String.valueOf(Resultat));

Sur cette base vous pouvez tout aussi bien faire

Code:

1
2
3
4
5
6
7
8
9
10
11
12
private void calcul(){
        if (eDia.getText().length() == 0) {
            Toast.makeText(MainActivity.this,"Vous n'avez pas rempli le diamètre", Toast.LENGTH_SHORT).show();
        }else{
            Resultat=Double.valueOf(eDia.getText().toString()).doubleValue();
            ePoids.setText(String.valueOf(Resultat));
 
        	if (eLong.getText().length() == 0)
        		Toast.makeText(MainActivity.this,"Vous n'avez pas rempli la longueur", Toast.LENGTH_SHORT).show();
          }
 
    }

merci
Je trouve plus clair en lecture, mais je ne met pas résolu, car je suis trop débutant, et qu'il y a surement plein d'autre méthodes valables.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
    private void calcul(){
    	Resultat = 1000 ;
    	cal = 1000 ;
        if (eDia.getText().length() == 0) {
 
            cal = 0;
        }
        if (eLong.getText().length() == 0){
        	cal =1;
        }
        if (cal > 1){
        	if (ePoids.getText().length() > 0 ^ eNBTiges.getText().length() > 0 ){
        		cal = 2 ;
        	}else {
        		cal =3;
        	}
        }
 
 
        switch (cal) { 
        case 0 :
        	Toast.makeText(MainActivity.this,"Vous n'avez pas rempli le diamètre", Toast.LENGTH_SHORT).show();
        	break;
 
    	case 1 :
    		Toast.makeText(MainActivity.this,"Vous n'avez pas rempli la longueur", Toast.LENGTH_SHORT).show();
    		break;
    	case 2 :
    		//calcul 
    		Toast.makeText(MainActivity.this,"poids ou nb tiges enregistré", Toast.LENGTH_SHORT).show();
        	//Resultat=Double.valueOf(eDia.getText().toString()).doubleValue();
        	//ePoids.setText(String.valueOf(Resultat));
    		break;
    	case 3 :
    		//calcul 
    		Toast.makeText(MainActivity.this,"poids et nb tiges enregistré", Toast.LENGTH_SHORT).show();
    		break;
 
    	default: 
    		Toast.makeText(MainActivity.this,"Cas pas traité", Toast.LENGTH_SHORT).show();
        }
    }

cordialement

14/04/2013, 20h49
Feanorin
Bonjour,

Surtout quelle est l'intérêt de passer ta variable en de string -> double -> string ?
Code:

1 2 Resultat=Double.valueOf(eDia.getText().toString()).doubleValue(); ePoids.setText(String.valueOf(Resultat));