Arrays mit Zufallzahlen füllen und Statistiken berechnen

[Nash404]

Hallo liebe Leute ,
Ich studiere seit kurzem Bioinformatik und sitze gerade an einer Programmieraufgabe.
Es geht darum ,dass ich ein Array mit 100 Elementen, 100 zufällige Zahlen zwischen 1-50 zuweisen soll. Die Generierung der Zufälligen Zahlen habe ich schon hinbekommen. auch der maxscore klappt schon. Ob es elegant ist bezweifle ich.
Mein Problem ist es den MinimalWert der Zufallszahlen zu bestimmmen. Außerdem kriege ich eine extrem unschöne Ausgabe (in der die einzelnen Printbefehle immer nacheinander ausgeführt werden. Sobald ich den log befehl jedoch aus der for schleife nehme. Kriege ich keine Zahlen mehr ausgegeben.
Hoffe auf ein paar Tipps wie ich das hinbekommen könnte.

import java.util.Random;
public class ZufallsZahlenArray {

 public static void main(String[] args) {
  int minscore = 0;
  int maxscore = 0;
  int[] zahlen = new int[50];
  int[] doppelteZahlen = new int[100];
  Random rnd = new Random();

  for (int i = 0; i < doppelteZahlen.length; i++) {
   zahlen[i] = rnd.nextInt(49) + 1;
   log("generiert " + zahlen[i]);
   if (zahlen[i] > maxscore) {
    maxscore = zahlen[i];
   }
   log("maxscore liegt bei : " + maxscore);

   if (minscore < zahlen[i]) {
    zahlen[i] = minscore;
   }
   log("MINSCORE liegt bei : " + minscore);
  }
 }
 
 private static void log(String aMessage) {
  System.out.println(aMessage);
 }
}

 

[Flown]

Code-Tags sehen hier im Forum so aus: [code=java]//JAVA CODE HERE [/code]

 

[Flown]

Also wenn du Tipps möchtest:
- Lagere bestimmte Funktionen in Methoden aus
- Bestimme default-Werte für min und max (Sinnvoll dafür wäre für min=Integer.MAX_VALUE und max = Integer.MIN_VALUE)
- Arrays kann man bequem in einen String mit Arrays.toString(zahlen) umwandeln
- Benutz doch eine eigene Klasse um deine Int-Statistiken zu berechnen (Gibt es auch schon in Java 8 vorgefertigt)

Dann kann es eventuell so aussehen:

import java.util.Arrays;
import java.util.Random;

public class Test {
  public static void main(String... args) {
    int[] ints = nRandomInts(100, 1, 50);
    System.out.println(Arrays.toString(ints));
    IntStatistics statistics = computeStatistics(ints);
    System.out.println(statistics);
  }
  
  private static int[] nRandomInts(int n, int min, int max) {
    int[] rands = new int[n];
    Random rand = new Random();
    for (int i = 0; i < n; i++) {
      rands[i] = rand.nextInt(max - min) + min;
    }
    return rands;
  }
  
  private static IntStatistics computeStatistics(int[] ints) {
    IntStatistics stats = new IntStatistics();
    for (int i : ints) {
      stats.add(i);
    }
    return stats;
  }
  
  static class IntStatistics {
    private int min = Integer.MAX_VALUE;
    private int max = Integer.MIN_VALUE;
    private int sum = 0;
    private int n = 0;
    
    public void add(int i) {
      if (i < min) {
        min = i;
      }
      if (max < i) {
        max = i;
      }
      sum += i;
      n++;
    }
    
    @Override
    public String toString() {
      return "[minimum=" + min + ", maximum=" + max + ", average=" + (n == 0 ? 0 : sum / n) + "]";
    }
  }
}

 

[JStein52]

Also den Min-Wert krioegst du zunächst mal so:

public static void main(String[] args) {
  int minscore = Integer.MAX_VALUE;
  int maxscore = 0;
  int[] zahlen = new int[50];
  int[] doppelteZahlen = new int[100];
  Random rnd = new Random();

  for (int i = 0; i < doppelteZahlen.length; i++) {
   zahlen[i] = rnd.nextInt(49) + 1;
   log("generiert " + zahlen[i]);
   if (zahlen[i] > maxscore) {
    maxscore = zahlen[i];
   }
 

   if (zahlen[i] < minscore) {
    minscore = zahlen[i];
   }
  }
log("MINSCORE liegt bei : " + minscore);
log("maxscore liegt bei : " + maxscore);
}

Und warum solltest du keine Ausgabe kriegen wenn du die log-Aufrufe hinter die Schleife schreibst ?

Edit: zu spät

 

[Nash404]

Vielen Dank schonmal für die beiden Antworten. Ich habe mich letztendlich für Die Variante von Flow entschieden. Da ich noch ziemlich neu auf dem Gebiet bin fehlt mir eine gewisse Übersichtlichkeit in meinem Code. Und das Schachteln in Methoden macht die Sache ungemein verständlicher. Bin mir allerdings nciht sicher ob ich die ersten beiden Methoden wirklich verstanden habe.

//  [COLOR=#009900]
  private static IntStatistics computeStatistics(int[] ints) {
    IntStatistics stats = new IntStatistics();
    for (int i : ints) {
      stats.add(i);[/COLOR]

Könntest du mir den Abschnitt nochmal erklären ? verstehe nicht was dort passiert.

Außerdem versuche ich jetzt diese 100 Zufallszahlen auf Doppelte Zahlen zu prüfen und wenn welche vorkommen, wann sie das erste mal aufgetaucht bin.
Ein weiteres Problem , an dem ich seit ein paar Stunden grübele.
Ich muss ja irgendwie den Inhalt der Arrays mit einander Vergleichen... Ich vergleiche aber immer das Array mit sich selbst und da ist klar das jede Zahl doppelt ist. Finde keinen Ansatz das Problem zu lösen.

Danke auch andich JStein52 deine Antwort habe ich auch ausprobiert aber gemerkt , dass ich für die Folgeaufgaben meine Sachen besser in Methoden packe.

Grüße
Nash404

 

[Jardcore]

Könntest du mir den Abschnitt nochmal erklären ?

Dort wird eine Klasse IntStatistics erstellt und dieser werden die als Parameter übergebenen Werte des int[]-Arrays mithilfe einer forEach Schleife übergeben.