Condition if

Question

Bonjour les amis....Je ne sais pas ce qui ne fonctionne pas sur mon code...à propos du condition if
Il me donne toujour la valeur 2000 or normalement ca ne doit pas être cela

public double[] getIRSA(Connection n)
 {
  
  double[] salaire=getValueOfIrsa(n);
  for(int i=0;i<salaire.length;i++)
  {
   System.out.println(salaire[i]);
  }
  
  double[] irsa=new double[salaire.length];
  for(int i=0;i<salaire.length;i++)
  {
    if(salaire[i]<=350000)
    {
     irsa[i]=2000;
    }

if(350001<=salaire[i] && salaire[i]<=400000)
    {
     irsa[i]=(salaire[i]*5)/100;
    }

if(400001<=salaire[i] && salaire[i]<=500000)
    {
     irsa[i]=(salaire[i]*10)/100;
    }

if(500001<=salaire[i] && salaire[i]<=600000)
    {
     irsa[i]=(salaire[i]*15)/100;
    }

if(salaire[i]>=600000)
    {
     irsa[i]=(salaire[i]*20)/100;
    }
   System.out.println(irsa[i]);
  }

return irsa;
 }

Merci d'avance de votre aide. C'est le irsa qui me donne toujour 2000....alors que ce n'est pas normal

Whismeril · Answer

Bonjour 

Tout d’abord, tu as essayé d’utiliser les balises de code mais pas tout à fait comme il faut, voir ici pour te prochain messages https://codes-sources.commentcamarche.net/faq/11288-les-balises-de-code

Pour ton problème, as tu exécuté le code en pas à pas et espionné le contenu des variables?

KX · Answer

Bonjour,

Une remarque sur ton code : un double n'est pas un nombre entier.
Donc si tu fais if(salaire[i]<=350000) et if(350001<=salaire[i]), il te manque le cas où le salaire est entre 350000.01 et 350000.99

Voici une alternative à ton code qui permet de résoudre ce problème :
public double computeIrsa(double salaire) {
    if (salaire <= 350_000)
        return 2000;
    if (salaire <= 400_000)
        return salaire*5/100;
    if (salaire <= 500_000)
        return salaire*10/100;
    if (salaire <= 600_000)
        return salaire*15/100;
    return salaire*20/100;
}
for (int i = 0; i < salaire.length; i++)
    irsa[i] = computeIrsa(salaire[i]);

Condition if

2 réponses

Votre réponse