If, else Problem

[Taurus]

Hallo ,
ich will ein Programm, mit dem man Porto Gebuehren berechnen kann machen.
Der erste Abschnitt also der preis fuer LOW_WEIGHT_SAME_STATE_FEE; und LOW_WEIGHT_INTERSTATE_FEE; funktioniert.
Mein Problem ist, das ich per else einfuegen will, wie hoch die Portogebuehren fuer ueber LOW_WEIGHT ist. (Das Problem beginnt ab dem dritten else.)

Danke im vorraus .

 Double.parseDouble(txtWeight.getText());
    if (weight < 0 )
    {
        txtStatus.setText("weight has to be positive");     
   }  
  else
  {  
        if    (  ( destinationNo == fares[0] ) || (destinationNo == fares[1] )) 
      {
           if (weight<= LOW_WEIGHT) 
         
                  if (rbtSameState.isSelected())           
                  { 
                        postage = LOW_WEIGHT_SAME_STATE_FEE;
                  }
             else
                  {
                        postage=LOW_WEIGHT_INTERSTATE_FEE;
                  }           
                  
       }
          else
       {           
                  if(rbtSameState.isSelected())
                  {
                        postage= HI_WEIGHT_SAME_STATE_FEE;
                  }
                else
                  {
                        postage=HI_WEIGHT_INTERSTATE_FEE;
                  }
             
         
       }
  } 
    txtPostage.setText(postage+"$");

 

[Traxter]

Der Schlüssel zu deinem Problem lautet "else if".


If-Anweisung in Java



Versuch es mal so:

    Double.parseDouble(txtWeight.getText());
        if (weight < 0 )
        {
            txtStatus.setText("weight has to be positive");    
       }  
      else
      {  
            if    (  ( destinationNo == fares[0] ) || (destinationNo == fares[1] ))
          {
               if (weight<= LOW_WEIGHT)
             
                      if (rbtSameState.isSelected())          
                      {
                            postage = LOW_WEIGHT_SAME_STATE_FEE;
                      }
                 else
                      {
                            postage=LOW_WEIGHT_INTERSTATE_FEE;
                      }          
                     
           }
              else if
           {          
                      if(rbtSameState.isSelected())
                      {
                            postage= HI_WEIGHT_SAME_STATE_FEE;
                      }
                    else
                      {
                            postage=HI_WEIGHT_INTERSTATE_FEE;
                      }
                 
             
           }
      }
        txtPostage.setText(postage+"$");

 

[Taurus]

Danke fuer deine Antwort, klappt aber trotzdem nicht. Also mein Code sieht momentan so aus

     Double.parseDouble(txtWeight.getText());
            if (weight < 0 )
            {
                txtStatus.setText("weight has to be positive");    
            }
    else {
           
             if  (  ( destinationNo == fares[0] ) || (destinationNo == fares[1] ))
               {
                   if (weight<= LOW_WEIGHT)
                    {    
                          if (rbtSameState.isSelected())          
                          {
                                postage = LOW_WEIGHT_SAME_STATE_FEE;
                          }
                     else
                          {
                                postage=LOW_WEIGHT_INTERSTATE_FEE;
                          }          
                       
                    }else if (rbtSameState.isSelected())
                          {
                                postage= HI_WEIGHT_SAME_STATE_FEE;
                          }
                        else
                          {
                                postage=HI_WEIGHT_INTERSTATE_FEE;
                          }
                         
                }  
            }
         
            txtPostage.setText(postage+"$");

 

[Traxter]

Könntest du das Problem eventuell genauer erläutern ? Was genau funktioniert denn nicht ? Gibts Fehlermeldungen ?

 

[Taurus]

Es wird immer nur LOW_WEIGHT_SAME_STATE_FEE; oder LOW_WEIGHT_INTERSTATE_FEE; angezeigt, (je nachdem ob rbtSameState angeklickt wurde). ab else if { das ja eine hoehere Summe in postage zeigen soll, wenn das Gewicht nicht weniger oder gleich LOW_WEIGHT ist wird wieder LOW_WEIGHT_SAME_STATE_FEE; oder LOW_WEIGHT_INTERSTATE_FEE; angezeigt.

wenn du moechtest, hier ist der komplette code.
Danke fuer deine hilfe

    final double LOW_WEIGHT_SAME_STATE_FEE= 5;
    final double LOW_WEIGHT_INTERSTATE_FEE= 10;
    final double HI_WEIGHT_SAME_STATE_FEE= 8;
    final double HI_WEIGHT_INTERSTATE_FEE= 12;
    final double LOW_WEIGHT=2;
    String [] destinations ={"Australia - Capital City", "Australia - Country", "Overseas"};
    int []fares={0,1,2};
    int destinationNo= cbxDestination.getSelectedIndex();
    String destination= destinations[destinationNo]; 
    int fare =fares [destinationNo];
    double postage=0;
    double weight=0;
       Double.parseDouble(txtWeight.getText());
                if (weight < 0 )
                {
                    txtStatus.setText("weight has to be positive");    
                }
        else {
               
                 if  (  ( destinationNo == fares[0] ) || (destinationNo == fares[1] ))
                   {
                       if (weight<= LOW_WEIGHT)
                        {    
                              if (rbtSameState.isSelected())          
                              {
                                    postage = LOW_WEIGHT_SAME_STATE_FEE;
                              }
                         else
                              {
                                    postage=LOW_WEIGHT_INTERSTATE_FEE;
                              }          
                           
                        }else if (rbtSameState.isSelected())
                              {
                                    postage= HI_WEIGHT_SAME_STATE_FEE;
                              }
                            else
                              {
                                    postage=HI_WEIGHT_INTERSTATE_FEE;
                              }
                             
                    }  
                }
             
                txtPostage.setText(postage+"$");

 

[qdoi123]

1. Du solltest den Code besser formatieren mit Einrückungen etc.

2. Muss das elseif VOR dem else zweig stehen

 

[Traxter]

Habe mich gerade etwas schwer getan die zusammenhänge zu finden aber es liegt auf jeden Fall an den Bedingungen. Eigentlich sollte das kein problem sein wenn du dir den link anschaust den ich dir gepostet habe.

 

[Taurus]

Du meinst so stimmts? hab ich auch schon probiert, aber funkt nicht. Wieder gleiches Problem.

  Double.parseDouble(txtWeight.getText());
    if (weight < 0 )
            {
                txtStatus.setText("weight has to be positive");    
            }
    else if  (  ( destinationNo == fares[0] ) || (destinationNo == fares[1] ))
         {
                 if (weight<= LOW_WEIGHT)
                   {    
                         if (rbtSameState.isSelected())          
                          {
                                postage = LOW_WEIGHT_SAME_STATE_FEE;
                          }
                         else
                          {
                                postage=LOW_WEIGHT_INTERSTATE_FEE;
                          }          
                    }
                   else
                    {   
                    
                          if (rbtSameState.isSelected())
                          {
                                postage= HI_WEIGHT_SAME_STATE_FEE;
                          }
                        else
                          {
                                postage=HI_WEIGHT_INTERSTATE_FEE;
                          }
                         
                    }  
           }
    
            txtPostage.setText(postage+"$");

 

[Traxter]

if -> else if -> else


In deinem Code sind jede Menge "else" an stellen wo sie nicht hingehören und durch "else if" ersetzt werden müssen.

 

[VfL_Freak]

Moin,

so, hier mal sinnvoller formatiert:

Double.parseDouble( txtWeight.getText() );

if( weight < 0 )
{
    txtStatus.setText("weight has to be positive");    
}
// hier würde dann auch ein 'else' reichen, da keine weiteren Abfragen kommen !!
else if( (destinationNo == fares[0]) || (destinationNo == fares[1]) )
{
    if( weight<= LOW_WEIGHT )
    {    
        if( rbtSameState.isSelected() )          
        {
            postage = LOW_WEIGHT_SAME_STATE_FEE;
        }
        else
        {
            postage=LOW_WEIGHT_INTERSTATE_FEE;
        }          
    }
    else
    {  
        if( rbtSameState.isSelected() )
        {
            postage= HI_WEIGHT_SAME_STATE_FEE;
        }
        else
        {
            postage=HI_WEIGHT_INTERSTATE_FEE;
        }
    }  
}

txtPostage.setText(postage+"$");

Sieht sonst auf den ersten Blick erstmal ganz gut aus! Mehr kann mal ohne weitere Infos nicht sagen!
Womit hast Du denn nun ein konkretes Problem ???:L

Gruß
Klaus

 

[Thallius]

Ich würde mal wetten auf Grund der ersten Zeile ist weight immer 0 und kann deshalb gar nicht >LOW_WEIGHT sein.

Gruß

Claus

 

[VfL_Freak]

Moin,

Ich würde mal wetten auf Grund der ersten Zeile ist weight immer 0 und kann deshalb gar nicht >LOW_WEIGHT sein.

tja, ist ohnehin die Frage, wie die ganzen variablen deklariert sind :autsch:

Gruß
Klaus

 

[Taurus]

Danke Leute, es klappt!
Thallius hat den Fehler entdeckt, es lag daran das ich bei weight anstatt

double weight=
        Double.parseDouble( txtWeight.getText() );

double weight=0;
        Double.parseDouble( txtWeight.getText() );

eingegeben hatte.

Echt super forum!