Hilfe bei Zufallzahlen

[Guest]

Hallo habe ein Problem wir sollen zufallszahlen zwischen 5 und 200 in einem Feld speichern und dann die kleinste und größte Zahl im Feld aus geben.Hier ist mein Code.

public class numbers {
	public static void main(String args[]) {
		int n = 1000;
		int R[] = new int[n];
		int i;
		for (i = 0; i < n; i++)
			R[i] = (int) (200 * Math.random());
		int max = R[0];
		for (i = 1; i < n; i++)
			if (R[i] > max)
				max = R[i];
		System.out.println(max);
		int min = R[0];
		for (i = 1; i > n; i++)
			if (R[i] < min)
				min = R[i];
		System.out.println(min);
	}
}

Nun mein Problem bei der Anzeige für Maximum zeigt er mir immer 199 an.Die min Zahlen wechselt er.Warum bekomme ich immer die gleiche Zahl für max? Und wie kann ich es einbauen das er erst ab 5 anfängt.
Danke für eure Hilfe

 

[anfänger15]

Es gilt: random()*(to-from)+from;

Für zufahlszahlen zwischen 5 und 200 müsste das dann so heißen: random()*(195)+5;

woher weißt du eigentlich das R[0] die max zahl ist???

meiner Meinung nach musst du das gesamte array durchgehen und dir jedesmal die größte zahl merken und wenn beim gleichen durchgang eine größere Zahl gefunden wird ist das die max zahl und so weiter bis du das ganze array durchlaufen hast und mit min das gleiche.


Klassen werden normalerweiße groß geschrieben (Numbers)

 

[anfänger15]

so sollte es funktionieren

public class Numbers {
   public static void main(String args[]) {
      int n = 1000;
      int R[] = new int[n];
      int max =0;
      int min =0;
      for (int i = 0; i < n; i++){
         R[i] = (int) (Math.random()*(195)+5);
        
      }

      for(int i=0;i<R.length;i++){
      	if(max<R[i]){
      		max = R[i];
        }
    }
    min = max;
          for(int i=0;i<R.length;i++){
      	
        if(min>R[i]){
        min=R[i];	
        }
    }
    System.out.println(max);
    System.out.println(min);
    
   }
   
   
}

 

[Quaxli]

Das Ganze mit weniger Schleifen und Variablen:

public class Numbers {
   public static void main(String args[]) {

      int r[] = new int[1000];
      int max =0;
      int min =200;
      
      for (int i = 0; i < r.length; i++){
         r[i] = (int) (Math.random()*(195))+5;
         
         System.out.println(r[i]);
         
         if(r[i]<min){
        	 min = r[i];
         }
         
         if(r[i]>max){
        	 max = r[i];
         }
      }


    System.out.println(max);
    System.out.println(min);
   
   }
   
   
}

 

[Leroy42]

Warum bekomme ich immer die gleiche Zahl für max?

Weil die Wahrscheinlichkeit bei 1000 Zufallszahlen mindestens einmal die 199 zu bekommen
bei 100*(1-(199/200)^1000) = 99,33...% liegt.

Es ist also eher unwahrscheinlich, die 0 nicht als Minimum zu erhalten.

 

[Guest]

Ich danke euch für die schnelle Hilfe. Werde es mit euren Hilfen probieren!

 

[Guest]

Ich bins nochmal habe jetzt alles fertig und haut auch hin danke nochmal.Mein nächstes Problem ist wie kann ich diesen Quelltext jetzt auseinandernehmen das die berchnungen immer in einzelnen Methoden stehen.Hier mein Text:

public class numbers {
	   public static void main(String args[]) {
		      int n = 11;//Feldlänge
		      int a[] = new int[n];
		      int i;
		      for (i = 0; i < n; i++)
		         a[i] = (int) (5+(195) * Math.random());
		      
		      int max = a[0];
		      for (i = 1; i < n; i++)
		         if (a[i] > max)
		            max = a[i];
		      System.out.println("Größte Zahl der Felder ist:"+max);
		      
		      int min = a[0];
		      for (i = 1; i < n; i++)
		         if (a[i] < min)
		            min = a[i];
		      System.out.println("Kleinste Zahl der Felder ist:"+min);
		      
		      int s=0;
		  	for(i=0; i<n; ++i)
		  		s+=a[i];
		       System.out.println("Die Summe aller Felde beträgt:"+s);
		       
		       int ii=0;
		       while(ii<a.length){
		          System.out.println("Feld:"+ii+" hat den Wert "+a[ii]);
		          ii++;  
		   }
		
		       int w=0;
		       for(i=0;i<n;i++)
		    	  
		       if (a[i]%13==0 && a[i]%4==0) ;
		       else w+=a[i]; 
		       System.out.println("Die Summe aller Zahlen die nicht durch 4 und 13 teilbar sind:"+w);
		     
		       
		      

  }}

 

[anfänger15]

z.B.

public int getMaximum(int a[]){
int max = a[0];
            for (i = 1; i < n; i++){
               if (a[i] > max){
                  max = a[i]; 
}
}
return max;
}

und das jetzt eben mit den anderen sachen die du sonst noch hast auch noch

 

[Guest]

Danke hab es fast hinbekommen. Nur bei der Methode "feldausgabe" gibt er mir nur ein feld aus.Was mache ich da falsch bzw. ich denke in der syso ausgabe stimmt was nicht was ich da gemacht habe.

public class shit {
	public static void main(String args[]) {
		final int n = 11;// Feldlänge
		int a[] = new int[n];
		int ii = 0;
		int i;
		for (i = 0; i < n; i++)
			a[i] = (int) (5 + (195) * Math.random());
		System.out.println("Größte Zahl der Felder ist:" + getMaximum(a));
		System.out.println("Kleinste Zahl der Felder ist:" + getMinimum(a));
		System.out.println("Die Summe aller Felde beträgt:"+sum(a));
		System.out.println("Feld:"+feldwerte(a)+" hat den Wert "+a[ii]);
		System.out.println("Die Summe aller Zahlen die nicht durch 4 und 13 teilbar sind:"+vierdreizehn(a));
	}

	public static int getMaximum(int a[]) {
		int i;
		int n = 11;
		int max = a[0];
		for (i = 1; i < n; i++) {
			if (a[i] > max) {
				max = a[i];
			}
		}
		return max;
	}

	public static int getMinimum(int a[]) {
		int i;
		int n = 11;
		int min = a[0];
		for (i = 1; i < n; i++) {
			if (a[i] < min) {
				min = a[i];
			}
		}
		return min;
	}
	public static int sum(int a[]){
	int s=0;int i;int n=11;
  	for(i=0; i<n; ++i){
  		s+=a[i];}
  	return s;}
       
    public static int feldwerte(int a[]){
    	int ii=0;
    	while(ii<a.length){
    		
    	ii++;
    		}
	    	  
	      return ii;}
	          
	
    
    public static int vierdreizehn(int a[]){
    	int w=0;int i;int n=11;
	       for(i=0;i<n;i++){
	    	  
	       if (a[i]%13==0 && a[i]%4==0) ;
	       else w+=a[i]; }
	       return w;
}}

 

[Leroy42]

Das n=11 würde ich in den Methoden nicht fest verdrahten.

Eher:

public static int getMaximum(int a[]) { 
      int max = a[0]; 
      for (int i = 1; i < a.length; i++) { 
         if (a[i] > max) { 
            max = a[i]; 
         } 
      } 
      return max; 
   }

Was meinst du mit Methode "feldausgabe"? ???:L

 

[Guest]

Meinte Feldwerte habe es aber jetzt hinbekommen.Danke euch alle für eure Tips!

 

[Leroy42]

Das hier:

public static int feldwerte(int a[]){ 
       int ii=0; 
       while (ii<a.length){ 
         ii++; 
        } 
       return ii;
}

geht auch einfacher:

public static int feldwerte(int a[]) {return a.length;}