Errore calcolo ?

[AleJAVA]

Ciao

volevo porvi un problema:

metto il codice e spiego :

      do {
                 System.out.println ("Livello Negozio del Merchandising");
                 negozio=tast.nextInt ();
               } while  ((negozio<1) || (negozio>10));
               
                 if (negozio==1) {
                     manutenzioneNeg=50000*2;   //-100.000 ogni livello
                     ricavoNeg=20*negozio*tifosi*2; //20 per (livello negozio)
                 }
                 if (negozio==2) {
                     manutenzioneNeg=150000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==3) {
                     manutenzioneNeg=250000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==4) {
                     manutenzioneNeg=350000*2; 
                     ricavoNeg=20*negozio*tifosi*2;
                 }
                 if (negozio==5) {
                     manutenzioneNeg=450000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==6) {
                     manutenzioneNeg=550000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==7) {
                     manutenzioneNeg=650000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==8) {
                     manutenzioneNeg=750000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==9) {
                     manutenzioneNeg=850000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==10) {
                     manutenzioneNeg=950000*2; 
                     ricavoNeg=20*negozio*tifosi*2;
                         
                 }
                 System.out.println ("Negozio tot setttimana : "  +ricavoNeg); //PROVVISORIO
                 System.out.println ("Manutenzione negozio : "  +manutenzioneNeg); //PROVVISORIO

ricavoNeg l'ho inizializzato (ricavoNeg=0)
ora nella riga

System.out.println ("Negozio tot setttimana : " +ricavoNeg);

mi dovrebbe visualizzare ricavoNeg,il problema è che mi visualizza in ogni caso "0" .
ho provato anche a sostituire quello che c'è dentro ricavoNeg con "1+2" e il risultato mi viene corretto .

Dentro di variabili c'è negozio che ho messo in int (ho provato anche ad inizializzarla ma non mi cambia nulla,sempre 0)

tifosi che ho invece inizializzato,sempre in int tifosi=0

e infine anche l'ho inizializzato come ricavoNeg=0 sempre in int

qualcuno sa darmi una mano ? grazie

[francesco.muia]

Bè matematica elementare.....
se come hai scritto tu tifosi è uguale a 0 alle elementari mi hanno insegnato che qualsiasi numero moltiplicato per 0 fa sempre 0. È dato che in ogni condizione in cui aggiorni la tua propety vi è sempre una moltiplicazione per 0 il risultato sarà sempre 0 a prescindere dei valori delle altre variabili

[AleJAVA]

non si vede tutto il codice perchè è abbastanza lungo,ma tifosi lo digiti tu da tastiera,l'ho dovuto inizializzare a 0 perchè senò mi dava errore

[francesco.muia]

Dal codice postato, se effettivamente sia negozio che tifosi non valgono 0 allora non si spiega il motivo per il quale non funzioni.
Quindi prova a postare il resto

[AleJAVA]

ecco a te

import java.util.Scanner;
class Bilancio {




public static void main(String[] args) {

Scanner tast=new Scanner (System.in);

int ricavoS=0,e,r,r2,rs1,tifosi=0,ms,st,et,cs,a,b,c,d, f,ba,v,m,cibo,m1=0,manutenzioneFF=0,ricavoFF=0,pa, manutenzioneBan=0,mb,ff,ricavoBan=0,bancarelle,ris torante,manutenzioneRist=0,ricavoRist=0,negozio=0, salamelle,Sett,rPOSITIVO,srim,srimAnnuo,srim2,srim 3,manutenzioneNeg=0,ricavoNeg=0,manutenzioneS=0,ri cavoNeg2=0;
// double ricavoS,e,r,r2,rs1;
System.out.println("Bilancio attuale");
ba=tast.nextInt();

System.out.println ("Manutenzione settimanale ");
ms=tast.nextInt();

a=ms*2;
System.out.println ("Stipendi settimanali");
st=tast.nextInt();

b=st*2;
System.out.println("Entrate TV");
et=tast.nextInt ();

c=et*2;
System.out.println("Capienza stadio");
cs=tast.nextInt();

System.out.println ("Tifosi ");
tifosi:tast.nextInt();

do {
System.out.println("Livello fast food");
ff=tast.nextInt();
} while ((ff<1) || (ff>10));

if (ff==1) {
manutenzioneFF=45000*2;
ricavoFF=(45*cs)*3;

}

if (ff==2) {
manutenzioneFF=90000*2;
ricavoFF=(8*cs)*3;

}
if (ff==3) {
manutenzioneFF=135000*2;
ricavoFF=(12*cs)*3;
}
if (ff==4) {
manutenzioneFF=180000*2;
ricavoFF=(16*cs)*3;
}
if (ff==5) {
manutenzioneFF=225000*2;
ricavoFF=(20*cs)*3;
}
if (ff==6) {
manutenzioneFF=270000*2;
ricavoFF=(24*cs)*3;
}
if (ff==7) {
manutenzioneFF=315000*2;
ricavoFF=(28*cs)*3;
}
if (ff==8) {
manutenzioneFF=360000*2;
ricavoFF=(32*cs)*3;
}
if (ff==9) {
manutenzioneFF=405000*2;
ricavoFF=(36*cs)*3;
}
if (ff==10) {
manutenzioneFF=450000*2;
ricavoFF=(40*cs)*3;
}



do {
System.out.println ("Livello bancarelle");
bancarelle=tast.nextInt ();
} while ((bancarelle<1) || (bancarelle>10));

if (bancarelle==1) {
manutenzioneBan=1125000*2; //manutenzione bancarelle
ricavoBan=(10*cs)*3;
}
if (bancarelle==2) {
manutenzioneBan=225000*2;
ricavoBan=(20*cs)*3;
}
if (bancarelle==3) {
manutenzioneBan=337500*2;
ricavoBan=(30*cs)*3;
}
if (bancarelle==4) {
manutenzioneBan=450000*2;
ricavoBan=(40*cs)*3;
}
if (bancarelle==5) {
manutenzioneBan=562000*2;
ricavoBan=(50*cs)*3;
}
if (bancarelle==6) {
manutenzioneBan=675000*2;
ricavoBan=(60*cs)*3;
}
if (bancarelle==7) {
manutenzioneBan=787500*2;
ricavoBan=(70*cs)*3;
}
if (bancarelle==8) {
manutenzioneBan=900000*2;
ricavoBan=(80*cs)*3;
}
if (bancarelle==9) {
manutenzioneBan=1012500*2;
ricavoBan=(90*cs)*3;
}
if (bancarelle==10) {
manutenzioneBan=1125000*2;
ricavoBan=(100*cs)*3;
}

do{
System.out.println ("Livello ristorante");
ristorante=tast.nextInt ();
} while ((ristorante<1) || (ristorante>10));

if (ristorante==1) {
manutenzioneRist=56250*2;
ricavoRist=(5*cs)*3;
}
if (ristorante==2) {
manutenzioneRist=112500*2;
ricavoRist=(10*cs)*3;
}
if (ristorante==3) {
manutenzioneRist=168750*2;
ricavoRist=(15*cs)*3;
}
if (ristorante==4) {
manutenzioneRist=225000*2;
ricavoRist=(20*cs)*3;
}
if (ristorante==5) {
manutenzioneRist=281250*2;
ricavoRist=(25*cs)*3;
}
if (ristorante==6) {
manutenzioneRist=337500*2;
ricavoRist=(30*cs)*3;
}
if (ristorante==7) {
manutenzioneRist=393750*2;
ricavoRist=(35*cs)*3;
}
if (ristorante==8) {
manutenzioneRist=450000*2;
ricavoRist=(40*cs)*3;
}
if (ristorante==9) {
manutenzioneRist=506250*2;
ricavoRist=(45*cs)*3;
}
if (ristorante==10) {
manutenzioneRist=562500*2;
ricavoRist=(50*cs)*3;
}

do {
System.out.println ("Livello Negozio del Merchandising");
negozio=tast.nextInt ();
} while ((negozio<1) || (negozio>10));

if (negozio==1) {
manutenzioneNeg=50000*2; //-100.000 ogni livello
ricavoNeg=20*negozio*tifosi*2; //20 per (livello negozio)
}
if (negozio==2) {
manutenzioneNeg=150000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==3) {
manutenzioneNeg=250000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==4) {
manutenzioneNeg=350000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==5) {
manutenzioneNeg=450000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==6) {
manutenzioneNeg=550000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==7) {
manutenzioneNeg=650000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==8) {
manutenzioneNeg=750000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==9) {
manutenzioneNeg=850000*2;
ricavoNeg=20*negozio*tifosi*2;
}
if (negozio==10) {
manutenzioneNeg=950000*2;
ricavoNeg=20*negozio*tifosi*2;

}
System.out.println ("Negozio tot setttimana : " +ricavoNeg); //PROVVISORIO
System.out.println ("Manutenzione negozio : " +manutenzioneNeg); //PROVVISORIO


/*

do {
System.out.println ("Livello Camion delle Salamelle");
salamelle=tast.nextInt ();
} while ((salamelle<1) || (salamelle>10));

if (salamelle==1) {
manutenzioneS=28125*2;
ricavoS=2.50*salamelle*cs*3; //2,50 * (livello salamelle)
}
if (salamelle==2) {
manutenzioneS=56250*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==3) {
manutenzioneS=84375*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==4) {
manutenzioneS=112500*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==5) {
manutenzioneS=140625*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==6) {
manutenzioneS=168750*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==7) {
manutenzioneS=196875*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==8) {
manutenzioneS=225000*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==9) {
manutenzioneS=253125*2;
ricavoS=2.50*salamelle*cs*3;
}
if (salamelle==10) {
manutenzioneS=281250*2;
ricavoS=2.50*salamelle*cs*3;
}

System.out.println ("Salamelle ricavo:" +ricavoS); //PROVVISORIO

*/






d=(cs*200)*3;
e=d+c+m1+ricavoFF+ricavoBan+ricavoRist+ricavoNeg; //attivi double
f=a+b+manutenzioneFF+manutenzioneBan+manutenzioneR ist+manutenzioneNeg; // passivi
r=e-f; //double
rs1=r/2;
System.out.println("Bilancio bisettimanale: "+r);
System.out.println("Bilancio settimanale : "+rs1);




r2=r/2;
Sett=(ba/r2)-1;
Sett=-Sett;
srimAnnuo=rs1*12;
System.out.println ("Settimane al passivo: " +Sett);
System.out.println ("Passivo annuo: " +srimAnnuo);

do {
System.out.println ("Quante settimane mancano a fine stagione ?");
srim=tast.nextInt ();
} while ((srim<1) || (srim>12));


srim2=rs1*srim;
srim3=ba+srim2;
System.out.println ("Senza operazioni di mercato,finirai la stagione con " +srim3);



} // public

} //class

[Alex'87]

Illeggibile, usa il tag CODE per favore

[francesco.muia]

Quoto quello detto da Alex e aggiungo che un metodo main in java così lungo non lo mai visto...
Mi riporta ai i miei primi tentativi di programmazione C...
È per caso un esercizio di scuola/università ?

[AleJAVA]

import java.util.Scanner;
class Bilancio {




    public static void main(String[] args) {
     
         Scanner tast=new Scanner (System.in);
           
            int ricavoS=0,e,r,r2,rs1,tifosi=0,ms,st,et,cs,a,b,c,d,f,ba,v,m,cibo,m1=0,manutenzioneFF=0,ricavoFF=0,pa,manutenzioneBan=0,mb,ff,ricavoBan=0,bancarelle,ristorante,manutenzioneRist=0,ricavoRist=0,negozio=0,salamelle,Sett,rPOSITIVO,srim,srimAnnuo,srim2,srim3,manutenzioneNeg=0,ricavoNeg=0,manutenzioneS=0,ricavoNeg2=0;
            // double ricavoS,e,r,r2,rs1;
            System.out.println("Bilancio attuale");
            ba=tast.nextInt();
            
            System.out.println ("Manutenzione settimanale ");
            ms=tast.nextInt();
            
            a=ms*2;
            System.out.println ("Stipendi settimanali");
            st=tast.nextInt();
           
            b=st*2;
            System.out.println("Entrate TV");
            et=tast.nextInt ();
            
            c=et*2;
            System.out.println("Capienza stadio");
            cs=tast.nextInt();
            
            System.out.println ("Tifosi ");
            tifosi:tast.nextInt();
            
            do {
            System.out.println("Livello fast food");
            ff=tast.nextInt();
            } while  ((ff<1) || (ff>10));
           
              if (ff==1) {
                  manutenzioneFF=45000*2;
                          ricavoFF=(45*cs)*3;
                        
              }
              
              if (ff==2) {
                  manutenzioneFF=90000*2;
                  ricavoFF=(8*cs)*3;
                 
              }
              if (ff==3) {
                  manutenzioneFF=135000*2;
                  ricavoFF=(12*cs)*3;
              }
              if (ff==4) {
                  manutenzioneFF=180000*2;
                  ricavoFF=(16*cs)*3;
              }
              if (ff==5) {
                  manutenzioneFF=225000*2;
                  ricavoFF=(20*cs)*3;
              }
              if (ff==6) {
                  manutenzioneFF=270000*2;
                  ricavoFF=(24*cs)*3;
              }
              if (ff==7) {
                  manutenzioneFF=315000*2;
                  ricavoFF=(28*cs)*3;
              }
              if (ff==8) {
                  manutenzioneFF=360000*2;
                  ricavoFF=(32*cs)*3;
              }
              if (ff==9) {
                  manutenzioneFF=405000*2;
                  ricavoFF=(36*cs)*3;
              }
              if (ff==10) {
                  manutenzioneFF=450000*2;
                  ricavoFF=(40*cs)*3;
              }
           
            
                  
                   do {
                  System.out.println ("Livello bancarelle");
                  bancarelle=tast.nextInt ();
                   } while  ((bancarelle<1) || (bancarelle>10));
                 
                    if (bancarelle==1) {
                        manutenzioneBan=1125000*2;          //manutenzione bancarelle
                        ricavoBan=(10*cs)*3;
                    }
                    if (bancarelle==2) {
                        manutenzioneBan=225000*2;
                        ricavoBan=(20*cs)*3;
                    }
                    if (bancarelle==3) {
                        manutenzioneBan=337500*2;
                                ricavoBan=(30*cs)*3;
                    }
                    if (bancarelle==4) {
                        manutenzioneBan=450000*2;
                                ricavoBan=(40*cs)*3;
                    }
                    if (bancarelle==5) {
                        manutenzioneBan=562000*2;
                                ricavoBan=(50*cs)*3;
                    }
                    if (bancarelle==6) {
                        manutenzioneBan=675000*2;
                                ricavoBan=(60*cs)*3;
                    }
                    if (bancarelle==7) {
                        manutenzioneBan=787500*2;
                                ricavoBan=(70*cs)*3;
                    }
                    if (bancarelle==8) {
                        manutenzioneBan=900000*2;
                                ricavoBan=(80*cs)*3;
                    }
                    if (bancarelle==9) {
                        manutenzioneBan=1012500*2;
                                ricavoBan=(90*cs)*3;
                    }
                    if (bancarelle==10) {
                        manutenzioneBan=1125000*2;
                                ricavoBan=(100*cs)*3;
                    }
                    
                    do{
                    System.out.println ("Livello ristorante");
                      ristorante=tast.nextInt ();
                    } while  ((ristorante<1) || (ristorante>10));
                      
                    if (ristorante==1) {
                          manutenzioneRist=56250*2;
                          ricavoRist=(5*cs)*3;
                      }
                    if (ristorante==2) {
                        manutenzioneRist=112500*2;
                        ricavoRist=(10*cs)*3;
                    }
                   if (ristorante==3) {
                       manutenzioneRist=168750*2;
                       ricavoRist=(15*cs)*3;
                   }
                   if (ristorante==4) {
                       manutenzioneRist=225000*2;
                       ricavoRist=(20*cs)*3;
                   }
                   if (ristorante==5) {
                       manutenzioneRist=281250*2;
                       ricavoRist=(25*cs)*3;
                   }
                   if (ristorante==6) {
                       manutenzioneRist=337500*2;
                       ricavoRist=(30*cs)*3;
                   }
                   if (ristorante==7) {
                       manutenzioneRist=393750*2;
                       ricavoRist=(35*cs)*3;
                   }
                   if (ristorante==8) {
                       manutenzioneRist=450000*2;
                       ricavoRist=(40*cs)*3;
                   }
                   if (ristorante==9) {
                       manutenzioneRist=506250*2;
                       ricavoRist=(45*cs)*3;
                   }
                   if (ristorante==10) {
                       manutenzioneRist=562500*2;
                       ricavoRist=(50*cs)*3;
                   }
                      
               do {
                 System.out.println ("Livello Negozio del Merchandising");
                 negozio=tast.nextInt ();
               } while  ((negozio<1) || (negozio>10));
               
                 if (negozio==1) {
                     manutenzioneNeg=50000*2;   //-100.000 ogni livello
                     ricavoNeg=20*negozio*tifosi*2; //20 per (livello negozio)
                 }
                 if (negozio==2) {
                     manutenzioneNeg=150000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==3) {
                     manutenzioneNeg=250000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==4) {
                     manutenzioneNeg=350000*2; 
                     ricavoNeg=20*negozio*tifosi*2;  
                 }
                 if (negozio==5) {
                     manutenzioneNeg=450000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==6) {
                     manutenzioneNeg=550000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==7) {
                     manutenzioneNeg=650000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==8) {
                     manutenzioneNeg=750000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==9) {
                     manutenzioneNeg=850000*2; 
                     ricavoNeg=20*negozio*tifosi*2; 
                 }
                 if (negozio==10) {
                     manutenzioneNeg=950000*2; 
                     ricavoNeg=20*negozio*tifosi*2;
                         
                 }
                 System.out.println ("Negozio tot setttimana : "  +ricavoNeg); //PROVVISORIO
                 System.out.println ("Manutenzione negozio : "  +manutenzioneNeg); //PROVVISORIO
               
                 
                /*
                 
               do {
              System.out.println ("Livello Camion delle Salamelle");
              salamelle=tast.nextInt ();
               } while  ((salamelle<1) || (salamelle>10));
               
               if (salamelle==1) {
                   manutenzioneS=28125*2;
                           ricavoS=2.50*salamelle*cs*3; //2,50 * (livello salamelle)
               }
               if (salamelle==2) {
                   manutenzioneS=56250*2;
                           ricavoS=2.50*salamelle*cs*3;
               }
               if (salamelle==3) {
                   manutenzioneS=84375*2;
                           ricavoS=2.50*salamelle*cs*3;
               }
               if (salamelle==4) {
                   manutenzioneS=112500*2;
                           ricavoS=2.50*salamelle*cs*3;
               }
               if (salamelle==5) {
                   manutenzioneS=140625*2;
                           ricavoS=2.50*salamelle*cs*3;
               }
               if (salamelle==6) {
                   manutenzioneS=168750*2;
                           ricavoS=2.50*salamelle*cs*3;
               }
               if (salamelle==7) {
                   manutenzioneS=196875*2;
                           ricavoS=2.50*salamelle*cs*3;
               }
               if (salamelle==8) {
                   manutenzioneS=225000*2;
                           ricavoS=2.50*salamelle*cs*3;
                           }
               if (salamelle==9) {
                   manutenzioneS=253125*2;
                           ricavoS=2.50*salamelle*cs*3;
               }
               if (salamelle==10) {
                   manutenzioneS=281250*2;
                           ricavoS=2.50*salamelle*cs*3;
                           }
               
               System.out.println ("Salamelle ricavo:"   +ricavoS);  //PROVVISORIO
                   
                */
              
        
              
             
        
           
            d=(cs*200)*3;
            e=d+c+m1+ricavoFF+ricavoBan+ricavoRist+ricavoNeg;   //attivi double
            f=a+b+manutenzioneFF+manutenzioneBan+manutenzioneRist+manutenzioneNeg;   // passivi
            r=e-f; //double
            rs1=r/2;
            System.out.println("Bilancio bisettimanale:      "+r);
            System.out.println("Bilancio settimanale  :      "+rs1);
            
            
           
            
            r2=r/2;
            Sett=(ba/r2)-1;
            Sett=-Sett;
            srimAnnuo=rs1*12;
            System.out.println ("Settimane al passivo: " +Sett);
            System.out.println ("Passivo annuo:    " +srimAnnuo);
            
            do {
            System.out.println ("Quante settimane mancano a fine stagione ?");
            srim=tast.nextInt ();
               }  while ((srim<1) || (srim>12));


                 srim2=rs1*srim;
                 srim3=ba+srim2;
                 System.out.println ("Senza operazioni di mercato,finirai la stagione con "  +srim3);         
                 


          } // public
        
    } //class

Ecco scusatemi
Comunque no,è un programmino di un gioco manageriale online,è lungo perchp dovevo scrivere i valori dei vari livelli delle varie strutture =)

[francesco.muia]

Ciao Ale, allora l'errore è nell'assegnamento del valore della variabile tifosi, l'operatore di assegnamento non è il ":" ma "=" quindi

tifosi: tast.nextInt(); //SBAGLIATO
tifosi = tast.nextInt(); //MEGLIO

Poi il mio commento non riguardava il numero di righe di codice che hai scritto ma il fatto che stanno tutte dentro un metodo (tra l'altro il main). E' sempre buona norma dividere le diverse istruzioni in + metodi o classi. Il commento che mi ricordasse tanto il codice C nasce dal fatto che appunto quest'ultimo è un linguaggio procedurale e non a oggetti cosa che java lo è, e sarebbe opportuno che lo utilizzassi come tale.

P.S
Tra l'altro non vedo come lo potresti mettere online solamente con il metodo main.....

[AleJAVA]

Cavolo grazie,so che si faceva con l'uguale,non ne ho idea del perchè ho messo i ":"

Comunque,sto studiando java a scuola,sono abbastanza all'inizio ! quindi dovrei mettere più main e class ? ma tutte nello stesso foglio ? e quando devo cambiare main/class ? comunque dovevo visualizzare alla fine dei risultati che collegava tutto il procedimento dalla prima all'ultima riga insomma .

Grazie e ciao Francesco