Anfänger sucht hilfe bei code

[Paddy123]

hi alle zusammen,

also um es gleich mal auf den punkt zu bringen ich besuche gerade eine schulart bei der wir einwenig in java programmierung einsteigen und die grundbefehle etc. lernen am freitag steht jetzt eine arbeit an und ich versuche ein wenig darauf zu lernen und mich vorzubereiten.

vor kurzem hatten wir ein projekt bei dem wir ein programm machen sollten (mit arrays und schleifen)
das 7 "lotto"zahlen zieht ( nach dem prinzip 1-49)
naja ich versuche mich jetzt grad an der übungsaufgabe aber scheitere, die arbeitsblätter die wir bekommen haben helfen mir auch nicht recht weiter :-/

hier mal mein bisheriger code

import java.util.Random;
/**
 * dieses Programm zieht zufällige Lottozahlen
 * 
 * @Autor (Patrick)
 * @version (1.0)
 */
public class lotto2
{
    public static void main (String[] args)
    {
            int[] GezogeneZahlen = new int[6];
            Random generator = new Random();
            int x = generator.nextInt(48) + 1;
            
            
               for (int a=0; a>=6; a++ )
               {
                
                GezogeneZahlen[a] = x;
                System.out.print(GezogeneZahlen[a]);
               }
 
            }
}

wir benutzen BlueJ (keine ahnung ob das relevant ist oder nicht)

mfg Patrick

 

[XHelp]

Warum "Codeschnipsel und Projekte"? Das Thema ist bei den Anfängerfragen besser aufgehoben. Jemand sollte es mal verschieben.

Du solltest wohl eher IN der Schleife dir x generieren lassen. Dann musst du auch prüfen, ob so ein x nicht bereits generiert wurde.

Und trenne lieber Generierung und Ausgabe von Anfang an.

 

[eRaaaa]

Habs mal verschoben!

Anmerkung:

int x = generator.nextInt(48) + 1;

zieht keine Zufallszahl von 1-inkl. 49 ! Random (Java Platform SE 6)

 

[Paddy123]

ja ich wusste nich genau wo ich es rein machen sollte ...sorry ^^

wie meinst du das generierung und ausgabe trennen ?

E: ich hab das mit absicht von 0-48 generieren lassen, und erst danach dann +1 damit die zahlen von 1-49 reichen (schonmal 0 in ner lotto ziehung gesehn ?^^)

 

[XHelp]

generator.nextInt(48) generiert die Zahlen im Bereich 0..47 und nicht 0..48. Eben so wie es in der Api steht.
Naja, du hast ja eine Schleife, wo generiert wird und direkt ausgegeben. Mach 2 Schleife: die eine füllt nur das Array und die andere gibt nur das Array aus.

 

[Paddy123]

import java.util.Random;
/**
 * dieses Programm zieht zufällige Lottozahlen
 * 
 * @Autor (Patrick)
 * @version (1.0)
 */
public class lotto2
{
    public static void main (String[] args)
    {
            int[] GezogeneZahlen = new int[6];
            Random generator = new Random();
                        
               for (int a=0; a>=6; a++ )
               {
                int x = generator.nextInt(49) + 1;
                GezogeneZahlen[a] = x;
                System.out.print(GezogeneZahlen[a]);
               }
 
            }
}

das wäre also jetzt der neue code, hab den vorschlag mal mit eingearbeitet x in der schleife zu definieren (wenn ich das jetzt richtig verstanden habe)
aber wie das mit "generierung und ausgabe trennen" gemeint ist weis ich immernoch nicht :-/

bin halt ziemlich neu was das alles angeht 🙂
--> learn it by doing ^^

E: also nur mal rein theoretisch, würde das programm so laufen, was denkt ihr ?
ich persönlich würde sagen es müsste laufen, BlueJ sieht das aber anderst. Bis zum Compilieren geht alles noch einwandfrei aber es wird einfach kein Textfeld geöffnet das die Zahlen die im array sind ausgibt

 

[XHelp]

Ne, du solltest es durch Lesen lernen!
Der Vorschlag mit der Trennung:

for (int a=0; a<gezogeneZahlen.length; a++ ) {
  int x = generator.nextInt(49) + 1;
  gezogeneZahlen[a] = x; //Variablen fangen mit Kleinbuchstaben an
}
for (int a=0; a<gezogeneZahlen.length; a++ ) {
  System.out.print(gezogeneZahlen[a]);
}

1. Ist deine Schleife falsch, zumindestmal die Bedingung.
2. Dein Algo stimmt nicht so ganz, weil es passieren kann dass du 2 gleiche Zahlen hast.

Zu dem Thema gibt es übrigens schon 1,2,999999 Theme in Forum, die lassen sich bestimmt über die Suchfunktion finden.

 

[Paddy123]

ja das mit den doppelten zahlen wollte ich erst angehn wenn ich das 1. problem gelöst habe 😉

step by step eben
ok danke, dann werd ich gleich mal suchen

E: hab mal deine schleife eingesetzt in meinen code und "GezogeneZahlen" abgeändert in "gezogeneZahlen"

das problem hierbei ist jetzt das ich anstatt 6 zahlen, 12 ausgegben bekomme ^^

hier der code:

import java.util.Random;
/**
 * dieses Programm zieht zufällige Lottozahlen
 * 
 * @Autor (Patrick)
 * @version (1.0)
 */
public class lotto2
{
    public static void main (String[] args)
    {
            int[] gezogeneZahlen = new int[6];
            Random generator = new Random();
                        
            for (int a=0; a<gezogeneZahlen.length; a++ ) {
                int x = generator.nextInt(49) + 1;
                gezogeneZahlen[a] = x; //Variablen fangen mit Kleinbuchstaben an
            }
            for (int a=0; a<gezogeneZahlen.length; a++ ) {
                System.out.print(gezogeneZahlen[a]);
            }
 
            }
}

ergebniss : 32102352827

 

[Final_Striker]

Schau dir mal die Abbruchbedingung in deiner for-Schleife an.

 

[Paddy123]

also, ich glaube du willst auf ">=" hinaus oder ? so haben wir das in der schule gelernt und da geht das auch, kann das evtl an der version der java plattform (aka: Java DevelopmentKit wenn ich mich nich irre ?) liegen ?

E: ohhh, ja jetzt seh ichs auch xD es sollte wenn dann überhaupt a<=6 sein und nicht a>=6 ^^

 

[Final_Striker]

Genau, die Schleife läuft solange wie die Bedingung [c]a < 6[/c] [c]true [/c]ist.

Bei [c]a >= 6[/c] würde die Schleife gar nicht ausgeführt werden, da sie gleich [c]false [/c]ist.

 

[Paddy123]

das is das was ich an programmierung so hasse, ein kleiner fehler und das ganze programm läuft nicht :-/

 

[XHelp]

Der Grund für die komische Ausgabe ist: du (ich auch durch copy&paste) benutzt

print

. Ersetze es entweder durch

print(... + " ")

oder durch

println

 

[Paddy123]

das

print(...+ " ")

würde ja nur ändern das dann zwischen den zahlen ein leerzeichen ist, was mich aber stört ist das es 12 zahlen sind ...und ich bei gott nich darauf komme warum 12 zahlen gezogen werden ?

 

[XHelp]

Wenn du mit "Zahlen" Zeichen meinst, dann ist es klar. Wenn er z.B. 12 udn 65 generiert, dann sieht es so aus: "1265" (2 Zahlen, aber 4 Zeichen)
Wenn du wirklich Zahlen meinst, dann hast du vermutlich in der 1. Schleife auch noch die Ausgabe stehen.

 

[mariusbopp]

trenne doch erstmal die einzelnen zahlen wie oben schon gesagt und dann weißt du auch wieviel zahlen er ausgibt!! und wenns dann immernoch 12 sind was ich aber eher nicht glaube dann schau weiter:toll:

 

[Paddy123]

sry, das hab ich noch net bedacht, manchmal sitz ich einfach aufm schlauch ^^

E: so hab grad versucht mal zu überprüfen ob eine zahl schon gezogen wurde,

theorie -> man zieht alle 6 zahlen überprüft dann ob eine doppelt vorkommt

public class lotto2
{
        public static void main (String[] args)
        {
              int[] gezogeneZahlen = new int[7];
              Random generator = new Random();
              
              for (int a=0; a<gezogeneZahlen.length; a++){
                gezogeneZahlen[a]=0;
                }
                        
              for (int a=0; a<gezogeneZahlen.length; a++ ) {
                  int x = generator.nextInt(49) + 1;
                  gezogeneZahlen[a] = x;
                  if (a==6){
                    if (gezogeneZahlen[a]==gezogeneZahlen[a-1] | x==gezogeneZahlen[a-2] | x==gezogeneZahlen[a-3]| x==gezogeneZahlen[a-4]| x==gezogeneZahlen[a-5]| x==gezogeneZahlen[a-6]){
                        gezogeneZahlen[a] = x;
                    }
                    }
              }
        }          
   
  
            
}

also da sind noch einige fehler drin aber ich weis grad nicht wie ich weiter machen soll :-/

mfg paddy

 

[timbeau]

Vorne weg, ich würde es anders machen. Wenn wir aber bei dem Array bleiben wollen, brauchen wir eine Methode die prüft ob die Zahl schon enthalten ist. Das machst du aktuell mit deiner :autsch: if-Abfrage.

Besser sowas nutzen:

private boolean istEnthalten(int neueZahl) {

for( ... ) { zahlImArray = neueZahl return true }

return false
}

 if (a==6){

macht keinen sinn, da du ganz am Ende prüfst ob die letzte Zahl identisch ist. Es kann aber schon die 2. mit der ersten identisch sein.

Also Schritte:

Zahl generieren : DONE
Zahl überprüfen ob schon vorhanden : TODO
Wenn Zahl vorhanden neue Zahl für das AKTUELLE Feld im Array generieren: TODO (Index bearbeiten)
Wenn Zahl nicht vorhanden normal einfügen: DONE

Meine Alternative wäre diese und das soll keine Lösung sein. Mach ruhig dein Programm mit dem Array fertig.

 List<Integer> gezogeneZahlen = new ArrayList<Integer>();
              Random generator = new Random();
                        
              for (gezogeneZahlen.size(); gezogeneZahlen.size()<6;) {
                  int x = generator.nextInt(49) + 1;
                  if(!gezogeneZahlen.contains(x)){
                	  gezogeneZahlen.add(x);
                  }
              }

 

[Paddy123]

ich würde es auch lieber ohne array machen, aber unser lehrer will das wir mit arrays arbeiten :-/

naja und befehle wie gezogeneZahlen.size oder ähnliche haben wir bisher im unterricht noch nicht durchgenommen also denke ich wird er evtl. was dagegen haben wenn wir das in der arbeit morgen benutzen 🙁

 

[Paddy123]

also so wie es aussieht hab ich mir völlig umsonst solche sorgen gemacht um die arbeit ^^

lief ganz gut kamen wirklich nur sehr einfache sachen dran, z.b.

erstelle ein programm das ausgibt :
laenge a : 10 cm
laenge b : 12cm
(Rechteck)flaeche : ? cm²

nächste aufgabe war dann...

erstelle ein programm das ausgibt :

Zeile 1: 0 2 4 6
Zeile 2: 8 10 12 14
Zeile 3: 16 18 20 22
Gesamt Summe: ?

arbeite mit arrays und schleifen

wie man sieht recht einfach wenn man die grundbefehle etc. ein wenig kennt 😀...danke an alle für die "Nachhilfe"

 

[mariusbopp]

na also dann muss das ja ne 1 werden mit der hilfe hier^^
:toll: