bräuchte hilfe bei einem kleinen prog

[Stoffl]

es funktioniert so weit , aber es schaut irgendwie sehr um ständlich aus- kann ich die einage bei den monaten nicht irgendwie optimieren?
und kann zum schluss spuckt er mir ja den minum betrag und maxium betrag aus- tolle wäre noch wenn er mir sagen würde welcher monat das ist. danke im voraus wenn jemand sich zeit nimmt um mir zu helfen. greetz stoffl

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class x
public static void main (String[] args ) throws IOException
        {  BufferedReader userIn = new BufferedReader
        (new InputStreamReader(System.in));
            double zinsatz,x,y;
          double[] monat ={1,2,3,4,5,6,7,8,9,10,11,12,13};
         
            
   
    
 String inputData;
   
    System.out.println( "zinsatz in %: " );
    inputData  = userIn.readLine();
   zinsatz = Double.parseDouble( inputData );
   
   System.out.println( "Wie viele Euros wurden eingezahlt.. " );
   System.out.println("..im Jänner");
    inputData  = userIn.readLine();
   monat[1] = Double.parseDouble( inputData );      
   System.out.println("..im Feburar");
    inputData  = userIn.readLine();
   monat[2] = Double.parseDouble( inputData ); 
   System.out.println("..im März");
    inputData  = userIn.readLine();
   monat[3] = Double.parseDouble( inputData ); 
   System.out.println("..im April");
    inputData  = userIn.readLine();
   monat[4] = Double.parseDouble( inputData ); 
   System.out.println("..im Mai");
    inputData  = userIn.readLine();
   monat[5] = Double.parseDouble( inputData ); 
   System.out.println("..im Juni");
    inputData  = userIn.readLine();
   monat[6] = Double.parseDouble( inputData ); 
   System.out.println("..im Juli");
    inputData  = userIn.readLine();
   monat[7] = Double.parseDouble( inputData ); 
   System.out.println("..im August");
    inputData  = userIn.readLine();
   monat[8] = Double.parseDouble( inputData ); 
   System.out.println("..im September");
    inputData  = userIn.readLine();
   monat[9] = Double.parseDouble( inputData ); 
   System.out.println("..im Oktober");
    inputData  = userIn.readLine();
   monat[10] = Double.parseDouble( inputData ); 
   System.out.println("..im November");
    inputData  = userIn.readLine();
   monat[11] = Double.parseDouble( inputData ); 
   System.out.println("..im Dezember");
    inputData  = userIn.readLine();
   monat[12] = Double.parseDouble( inputData ); 
   
   y=monat[12]*Math.pow((1+zinsatz/100),12)+monat[12]*Math.pow((1+zinsatz/100),11)+monat[12]*Math.pow((1+zinsatz/100),10)+monat[12]*Math.pow((1+zinsatz/100),9)+monat[12]*Math.pow((1+zinsatz/100),8)+monat[12]*Math.pow((1+zinsatz/100),7)+monat[12]*Math.pow((1+zinsatz/100),6)+monat[12]*Math.pow((1+zinsatz/100),5)+monat[12]*Math.pow((1+zinsatz/100),4)+monat[12]*Math.pow((1+zinsatz/100),3)+monat[12]*Math.pow((1+zinsatz/100),2)+monat[12]*Math.pow((1+zinsatz/100),1);
   
        DecimalFormat df = new DecimalFormat("0.00");
        String gesamtsumme = df.format(y);
        System.out.println( "Am Ende sind "+gesamtsumme+"€ auf dem Konto." );

   x=(monat[1]+monat[2]+monat[3]+monat[4]+monat[5]+monat[6]+monat[7]+monat[8]+monat[9]+monat[10]+monat[11]+monat[12])/12;
        DecimalFormat df2 = new DecimalFormat("0.00");
        String durchschnitt = df2.format(x);
        System.out.println( "Im Durchschnitt wurden "+durchschnitt+"€ auf das Konto eingezahlt" );
int minIndex = 0;
int maxIndex = 0;
    for (int i=1; i < monat.length; i++)
      if (monat[i] < monat[minIndex])
          minIndex = i;
    for (int j=1; j < monat.length; j++)
      if (monat [j] > monat[maxIndex])    
            maxIndex = j;
    System.out.println("Minimum bei "+monat[minIndex]);
    System.out.println("Minimum bei "+monat[maxIndex]);
  }
}

 

[Frank_H]

kleiner tipp:

nimm mal ne for-schleife und iteriere über das array drüber... dann wirds auch nicht so "lang" ;-)

 

[Stoffl]

ich eine stunde jetzt rumporbiert und komm auf ekien grünen zweig kann mir einer eine größer tipp geben - danke

 

[SlaterB]

for (int i=0; i<12; i++) {
System.out.println("..im "+monatsName);
inputData = userIn.readLine();
monat = Double.parseDouble( inputData );
}

Achtung: i läuft dann von 0 bis 11, nicht von 1 bis 12

brauchst dafür ein Array mit den Monatsnamen

das kannst du dann gleich am Ende nutzen und noch den Monatsindex oder per Array wieder den Namen des Minnium/ Maximum-Monats ausgeben

--------

die Rechnung

y=monat[12]*Math.pow((1+zinsatz/100),12) ......

kann man sicher auch in einer for-Schleife erledigen,
i läuft von 0 bis 11, schau nach was du pro Monat ausrechnen musst


genauso für x,


wieso wird bei der y-Rechnung eigentlich so oft monat[12] verwendet, braucht man die anderen Monate nicht?


----

edit:

zwei gleiche DecimalFormat-Objekte zu erzeugen ist wenig elegant, eins reicht


------

die Krönung wäre am Ende, alles in einer einzigen for-Schleife zu schreiben,
sofort nachdem eine Zahl eingelesen wurde kannst du gleich y und x zum Teil berechnen und Minnium + Maximum-Index aktualisieren usw.
das monat-Array könnte dann wegfallen, obwohl es am Ende zur Ausgabe von Min/ Max schon nützt, sonst müsstest du noch zwei weitere Variablen einführen