Zufallszahl (Statische Attribute und Methoden)

[sengul]

Hallo zusammen,
ich habe folgende aufgabe versucht zu lösen. Nun möchte ich aber gerne eure meinung dazu hören, ob ich es richtig gemacht habe, da ich meine java kenntnisse verbessern möchte, bin ich dankbar für Kritik und für andere lösungsvorschlage. Ich hoffe dass ich die Aufgabe richtig verstandan, den ich weiss nicht wo ich den Math.round() benutzen soll. Unten findet ihr die Aufgabe und meine lösung.

Aufgabe: Zufallszahl (Statische Attribute und Methoden)
Schreiben Sie eine Klasse Zufallszahl mit folgenden Eigenschaften:

– Jedes neu erzeugte Objekt dieser Klasse erhält eine eindeutige fortlaufende Nummer und berechnet bei der Initialisierung automatisch eine Zufallszahl zwischen 0 (einschließlich) und 100 (ausschließlich).
Hinweis: Sie können die Methoden Math.random() zur Erzeugung der Zufallszahl und Math.round() zur Rundung verwenden. Die Beschreibung der Methoden finden Sie in der Java-API-Spezifikation.
– Sowohl die fortlaufende Nummer als auch die berechnete Zufallszahl sollen anschließend nicht mehr verändert werden können. Die Werte sollen aber über entsprechende Methoden abgefragt werden können.
– Es soll es weiterhin möglich sein, die Gesamtzahl der erzeugten Zufallszahlen sowie das Zufallszahl-Objekt, das die höchste Zufallszahl speichert, abzufragen.​

Schreiben Sie ein Hauptprogramm, das mehrere Zufallszahlen erzeugt und anschließend die Zufallszahlen, die Anzahl der erzeugten Zufallszahlen und die größte erzeugte Zufallszahl ausgibt.

Klasse Zufallszahl

package ubungsblatt;

import java.util.Random;

public class Zufallszahl {

    public final int nummer;
    public final int randomNumber;
    private static int count;

    public Zufallszahl(int nummer) {
        count++;
        this.nummer = nummer;
        Random random = new Random();
        int randomNumber = random.nextInt(100);
        this.randomNumber = randomNumber;
    }

    public int getNummer() {
        return nummer;
    }

    public int getRandomNumber() {
        return randomNumber;
    }

    public static int getCount() {
        return count;
    }

    public static int groesteZahl(Zufallszahl... params) {
        int max = 0;

        for (Zufallszahl o : params) {
            if (max < o.getRandomNumber()) {
                max = o.getRandomNumber();
            }
        }
        return max;
    }

    public static Zufallszahl groesteZufallsObjekt(int max, Zufallszahl... params) {
        Zufallszahl p = null;
        for (Zufallszahl o : params) {
            if (max == o.getRandomNumber()) {
                p = o;
            }
        }
        return p;
    }

    public static void main(String[] args) {

        Zufallszahl z = null;
        int max = 0;
        Zufallszahl z1 = new Zufallszahl(18823678);
        Zufallszahl z2 = new Zufallszahl(74387473);
        Zufallszahl z3 = new Zufallszahl(672);
        Zufallszahl z4 = new Zufallszahl(4938872);
        Zufallszahl z5 = new Zufallszahl(827266600);

        System.out.println("Zufalszahl:\t" + z1.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z2.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z3.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z4.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z5.getRandomNumber());
        System.out.println();
        System.out.println("Zufalszahl:\t" + z1.getNummer());
        System.out.println("Zufalszahl:\t" + z2.getNummer());
        System.out.println("Zufalszahl:\t" + z3.getNummer());
        System.out.println("Zufalszahl:\t" + z4.getNummer());
        System.out.println("Zufalszahl:\t" + z5.getNummer());
        System.out.println();
        System.out.println("Anzahl der erzeugten objecte:\t" + getCount());
        max = groesteZahl(z1, z2, z3, z4, z5);
        System.out.println("Grösste Zufallszahl:\t" + max);
        z = groesteZufallsObjekt(max, z1, z2, z3, z4, z5);
        System.out.println("Nummer des Objects mit dem grössten Zufallszahl:\t" + z.getNummer());
    }
}

 

[VfL_Freak]

Unten ist die Aufgabe und meine lösung

ääh
Wo jetzt genau ???

 

[Viktim]

Sowohl die fortlaufende Nummer als auch die berechnete Zufallszahl sollen anschließend nicht mehr verändert werden können.

Bei deinem 'count' fehlt ein 'final'

– Es soll es weiterhin möglich sein, die Gesamtzahl der erzeugten Zufallszahlen sowie das Zufallszahl-Objekt, das die höchste Zufallszahl speichert, abzufragen.

hast du ja hiermit:

Zufallszahl z1 = new Zufallszahl(18823678);
        Zufallszahl z2 = new Zufallszahl(74387473);
        Zufallszahl z3 = new Zufallszahl(672);
        Zufallszahl z4 = new Zufallszahl(4938872);
        Zufallszahl z5 = new Zufallszahl(827266600);

        System.out.println("Zufalszahl:\t" + z1.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z2.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z3.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z4.getRandomNumber());
        System.out.println("Zufalszahl:\t" + z5.getRandomNumber());
        System.out.println();
        System.out.println("Zufalszahl:\t" + z1.getNummer());
        System.out.println("Zufalszahl:\t" + z2.getNummer());
        System.out.println("Zufalszahl:\t" + z3.getNummer());
        System.out.println("Zufalszahl:\t" + z4.getNummer());
        System.out.println("Zufalszahl:\t" + z5.getNummer());
        System.out.println();
        System.out.println("Anzahl der erzeugten objecte:\t" + getCount());
        max = groesteZahl(z1, z2, z3, z4, z5);
        System.out.println("Grösste Zufallszahl:\t" + max);
        z = groesteZufallsObjekt(max, z1, z2, z3, z4, z5);
        System.out.println("Nummer des Objects mit dem grössten Zufallszahl:\t" + z.getNummer());

prinzipiell gelößt, allerdings würde ich die Zufallszahlen in ein Array oder eine Liste tun und dann mit schleifen durchgehen.

 

[X5-599]

count darf nicht final sein, weil damit ja alle erzeugten Instanzen gezählt werden.

 

[Viktim]

count darf nicht final sein, weil damit ja alle erzeugten Instanzen gezählt werden.

hmm stimmt Freitag Nachmittag , dann macht die Aufgabe aber keinen Sinn

 

[X5-599]

doch schon. ich würde sagen, nur der Konstruktor mit Parameter nummer könnte weg. Davon lese ch nichts in der Aufgabe. Stattdessen würde ich nach dem count++ einfach this.nummer = count; machen. Somit hat die Instanz eine Nummer bekommen die nicht mehr verändert werden kann, weil kein setter oder ähnliches vorhanden ist.

 

[X5-599]

Mit "Konstruktor könnte weg" meine ich natürlich nur den Parameter. Ein Parameterloser Konstruktor würde hierfür reichen.

 

[sengul]

Ich habe es mit this.nummer = count; gemacht, klappt wunderbar, jetzt haben die Objekte eine fortlaufende nummer, dankeschön

 

[Viktim]

@sengul wenn du das noch nicht gemacht hast, solltest du dir wirklich mal Arrays und Listen angucken und schleifen dabei auch

 

[sengul]

Hallo Viktim, das mit List und array habe ich auch gemacht. Aber ich würde es gerne nochmal machen als übung. So würde ich vorgehen, also ich würde erst alle erzeugten Objecte in ein List einfügen und diesen list als parameter an die methode übergeben, und dann einen Iterator erzeugen und mit dem Iterator dann den list durchlaufen. So sollte ich es machen oder? Ich mache es jetzt mal

 

[sengul]

mit dem Iterator habe ich es doch noch gelassen. Die grösste Zufalszahl habe ich noch auf dieser weise herausbekommen

List<Integer> list = new ArrayList<>();
        list.add(z1.getRandomNumber());
        list.add(z2.getRandomNumber());
        list.add(z3.getRandomNumber());
        list.add(z4.getRandomNumber());
        list.add(z5.getRandomNumber());
        list.add(z6.getRandomNumber());
System.out.println("Grösste Zufallszahl:\t" + grosteZahl(list));
......
.....

.....
    public static int grosteZahl(List<Integer> list) {
        int x = Collections.max(list);
        return x;
    }

Ist es so okey?
Hat jemand ein anderen vorschalg?