Mehrere Bedinungen in einer if Abfrage

Ratbag · 2. März 2016

Hallo an all!

Ich versuche gerade meine ersten kleinen Programme zu schreiben und brauche Hilfe bei einer Bedingungsabfrage.

Es soll ein einfacher Lotto Generator werden.

Einfache onClickListener Struktur und ein Random:

Code

// Zufallszahl erzeugen
Random Zufallszahl = new Random ();
Zahl1 = Zufallszahl.nextInt(49)+7;
Zahl2 = Zufallszahl.nextInt(49)+7;
Zahl3 = Zufallszahl.nextInt(49)+7;
Zahl4 = Zufallszahl.nextInt(49)+7;
Zahl5 = Zufallszahl.nextInt(49)+7;
Zahl6 = Zufallszahl.nextInt(49)+7;
Zusatzzahl = Zufallszahl.nextInt(49)+7;
// Abfrage, dass keien Zahl doppelt vorkommt

Nun möchte ich vor der Ausgabe der Zahlen abfragen ob Zahlen doppet vor kommen, wollte es mit einer if Abfrage machen und das mit != lösen. Jedoch komm ich nicht weiter.

Kann mir da wer weiter helfen?

Danke schon mal!

Pommes9485 · 2. März 2016

Ich würde es in einer Array machen, dann kannst du durch die einzelnen Zahlen in einer Schleife durchgehen und dabei jede Zahl mit jeder anderen vergleichen. Am Einfachsten wäre wohl etwas ähnliches wie dieses hier:

Java

int[] zahlen = new int[7];
//...zahlen reinschreiben, am besten auch direkt in einer Schleife
//Abfrage ob gleiche vorkommen
for(int i=0; i&lt;=zahlen.length; i++)
{
    for(int a=i+1; a&lt;=zahlen.length;a++)
       if(zahlen[i]==zahlen[a])
           //zahlen[i] bzw. zahlen[a] sind gleich 
}

Zur Erklärung:
Die erste Schleife geht einfach nur die Zahlen nacheinander durch. Für jede Zahl wird aber eine weitere Schleife gestartet. Diese startet nicht bei 0, sondern bei i+1, daher werden bereits verglichene Zahlen nicht noch einmal verglichen und die Zahl wird nicht mit sich selber verglichen. Sollte so stimmen

Ben · 2. März 2016

Java

private final static int COUNT = 6;


public void main(String[] arg0) {
  List&lt;Integer&gt; numbers = new ArrayList&lt;Integer&gt;();
  for (int i = 0; i &lt; COUNT; i++) {
    int randomNum = genRandomInt();


    if (!numbers.contains(randomNum)) {
      numbers.add(randomNum);
    }
  }


  for (Integer number : numbers) {
    System.out.println("Zufallszahl: " + number);
  }
}


private int genRandomInt() {
  Random Zufallszahl = new Random ();
  return Zufallszahl.nextInt(49)+7;
}

Alles anzeigen

Wäre da mein vorschlag.

Ratbag · 2. März 2016

Erst mal Danke für die Antworten, hab erst mal Pommes seine Idee genutzt:

Code

Zufallsschalter.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                // Zufallszahl erzeugen
                Random Zufallszahl = new Random ();
                Zahl1 = Zufallszahl.nextInt(49)+7;
                Zahl2 = Zufallszahl.nextInt(49)+7;
                Zahl3 = Zufallszahl.nextInt(49)+7;
                Zahl4 = Zufallszahl.nextInt(49)+7;
                Zahl5 = Zufallszahl.nextInt(49)+7;
                Zahl6 = Zufallszahl.nextInt(49)+7;
                Zusatzzahl = Zufallszahl.nextInt(49)+7;
                // Array erzeugen
                int[] Zahlen = new int[7];
                Zahlen[0] = (Zahl1);
                Zahlen[1] = (Zahl2);
                Zahlen[2] = (Zahl3);
                Zahlen[3] = (Zahl4);
                Zahlen[4] = (Zahl5);
                Zahlen[5] = (Zahl6);
                Zahlen[6] = (Zusatzzahl);
                //Abfrage ob gleiche vorkommen
                for(int i=0; i&lt;=Zahlen.length; i++)
                {
                    for(int a=i+1; a&lt;=Zahlen.length;a++)
                        if(Zahlen[i]!=Zahlen[a]);
                        //zahlen[i] bzw. zahlen[a] sind ungleich
                        Lotto_Zahl_1.setText(Integer.valueOf(Zahl1));
                        Lotto_Zahl_2.setText(Integer.valueOf(Zahl2));
                        Lotto_Zahl_3.setText(Integer.valueOf(Zahl3));
                        Lotto_Zahl_4.setText(Integer.valueOf(Zahl4));
                        Lotto_Zahl_5.setText(Integer.valueOf(Zahl5));
                        Lotto_Zahl_6.setText(Integer.valueOf(Zahl6));
                        Lotto_Zusatzzahl.setText(Integer.valueOf(Zusatzzahl));


                }
            }
        });

Alles anzeigen

Leider mit nem dicken Error! Unfortunaly, Projekt wurde gestoppt.

Fehler in der Syntax werden mir jedoch nicht angezeigt.

Pommes9485 · 2. März 2016

Dann würde ich mir den Logcat Bericht einmal anschauen, das ist sehr wichtig, auch wenn dieser Fehler relativ einfach ist.

TextView.setText() nimmt verschiedene Argumente. Das einfachste ist ein String, dieser wird dann direkt angezeigt. Ansonsten geht auch ein int-Wert, welcher meistens eine Referenz auf einen String ist. Zum Beispiel "R.string.teststring". Das wäre dann aber extra in der xml Datei festgelegt.

Integer.valueOf() gibt dir den Integer Wert des Parameters wieder. Das möchtest du aber nicht. Also am Besten einfach

Code

Lotto_Zahl_1.setText(""+(Zahl1));

benutzen.

Ratbag · 2. März 2016

Ich dachte mit Integer.valueOf() wird eine ganze Zahl als String wieder gegeben. Machte ich bisher immer so. Alternativ nutze ich Float.valueOf() um eine Fließkommazahl anzuzeigen.

Ich möchte ja eigentlich eine Zahl in einen String umwandeln, der dann angezeigt wird. Alternativ, könnte man String.valueOf nutzen?

Ach ja, der Fehler kommt immer noch:

Code

03-02 20:07:14.268 1787-1787/com.example.tazgosh.lottoprojekt1 E/AndroidRuntime: FATAL EXCEPTION: main
                                                                                 java.lang.ArrayIndexOutOfBoundsException: length=7; index=7
                                                                                     at com.example.tazgosh.lottoprojekt1.MainActivity$1.onClick(MainActivity.java:63)
                                                                                     at android.view.View.performClick(View.java:3511)
                                                                                     at android.view.View$PerformClick.run(View.java:14105)
                                                                                     at android.os.Handler.handleCallback(Handler.java:605)
                                                                                     at android.os.Handler.dispatchMessage(Handler.java:92)
                                                                                     at android.os.Looper.loop(Looper.java:137)
                                                                                     at android.app.ActivityThread.main(ActivityThread.java:4424)
                                                                                     at java.lang.reflect.Method.invokeNative(Native Method)
                                                                                     at java.lang.reflect.Method.invoke(Method.java:511)
                                                                                     at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:784)
                                                                                     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:551)
                                                                                     at dalvik.system.NativeStart.main(Native Method)

Alles anzeigen

Wichtig wäre das hier:

java.lang.ArrayIndexOutOfBoundsException: length=7; index=7

Arrays hab ich nie vorher genutzt, ist Neuland für mich.^^

Pommes9485 · 2. März 2016

Oh, das ist mein Fehler. Mach aus dem <= in den Schleifen ein <, dann sollte es gehen.

Ratbag · 2. März 2016

OK, jetzt klappts anscheinend, gibt nur noch nen Anzeigefehler.

Noch mal kurz zur Nachfrage:

Zitat

Ich dachte mit Integer.valueOf() wird eine ganze Zahl als String wieder gegeben. Machte ich bisher immer so. Alternativ nutze ich
Float.valueOf() um eine Fließkommazahl anzuzeigen.

Ich möchte ja eigentlich eine Zahl in einen String umwandeln, der dann
angezeigt wird. Alternativ, könnte man String.valueOf nutzen?

Wäre dies richtig?

Edit:

Könntest du mir noch was erklären.

Was ist der Unterschied von Integer.valueOf() und String.valueOf().

Durch Integer in der Anzeige bekam ich gerade nen Error, nutze ich String funktioniert alles. Ist das nicht beides das selbe, wo ist da der Unterschied und wann nutze ich welches von beiden? Wäre nett, wenn du mir da noch mal weiterhelfen könntest.

Edit 2:

Aus irgend nem Grund kommen ab und an doch noch doppelte Zahlen vor. *grübel*
Bisschen viel vorgenommen, dachte wäre einfacher, mit Arrays hab ich mich noch nie beschäftigt. Alles schwere Kost.^^

Pommes9485 · 2. März 2016

Integer.valueOf gibt dir einen Integer wieder und String.valueOf gibt dir einen String wieder. Integer sind nur Zahlen, String sind Zahlen und Buchstaben und co. Da du hier Text anzeigen möchtest, ist String.valueOf nötig, oder eben " "" + zahlen1".

Das liegt daran, dass du ja noch nichts machst, wenn die Zahl doppelt ist. Es wird überprüft, dafür ist die if-Abfrage drinne. Aber nichts darauf hin dagegen getan. Ich denke eine neue Zahl für die doppelte zu wählen ist am sinnvollsten. Ich würde das ganze aber, ähnlich wie Champs noch in extra Methoden auslagern. Dann kannst du das überprüfen danach direkt noch mal machen, ohne dafür doppelten Code zu haben.

Ratbag · 3. März 2016

Verstehe, vielen Dank für die ganzen Antworten. War wohl doch etwas aufwendiger als gedacht, dass kleine Lotto Spiel.^^

Edit:

Also um es nicht falsch zu verstehen, zum Anzeigen lassen benutz ich immer String.valueOf, egal ob es ein Double, Integer oder Float ist? Wann nutze ich dann eigentlich noch Integer.valueOf?

Pommes9485 · 3. März 2016

Zitat

Also um es nicht falsch zu verstehen, zum Anzeigen lassen benutz ich immer String.valueOf, egal ob es ein Double, Integer oder Float ist?

Im Prinzip schon, ja.

Wenn du einen String hat, der eine Zahl repräsentierst und du damit wieder rechnen möchtest, dann brauchst du einen Integer. Und dafür könntest du dann schreiben:

Java

String meine_zahl_als_string = "33";
int meine_zahl_als_int = Integer.valueOf(meine_zahl_als_string);

Allerdings gibt es noch einen Unterschied zwischen "Integer" und "int". Das würde jetzt aber den Rahmen sprengen. In den meisten Fällen würdest du vermutlich Integer.parseInt(String) benutzen

Mehrere Bedinungen in einer if Abfrage

Jetzt mitmachen!

Teilen