Hilfe bei Umänderung von Java Code

[SpaghettiBolo]

Hallo zusammen,

ich habe diesen Code für einen Adventskalender und muss folgendes umändern, doch komme ich selbst nicht auf die Lösung:

• nicht das ganze Türchenarray in jedem Türchen speichern - einmal im Adventskalender und im Türchen einzelne Sprüche, diese dann bei Erstellung mitgeben.
• die Eingabe in Scanner umwandeln

[CODE lang="java" title="Main"]import java.io.BufferedReader;
import java.io.InputStreamReader;

public class App {
static Adventskalender adventskalender = new Adventskalender();

public static void main(String[] args) throws Exception {
System.out.println(adventskalender);

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

while(true) { // Dauerhafte Schleife
int input = Integer.parseInt(reader.readLine());
if(input < 1 || input > 24) {
break; // Unpassende Eingabe bricht Schleife ab
}

System.out.println(); // Platz machen
if(!adventskalender.isTuerchenOffen(input)) {
adventskalender.oeffneTuerchen(input);
System.out.println(Tuerchen.getSpruchByNummer(input));
} else {
System.out.println("Türchen wurde schon geöffnet!");
}
System.out.println();
System.out.println(adventskalender);
}
}
}[/CODE]

[CODE lang="java" title="Adventskalender"]import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.Random;

public class Adventskalender {
Random random = new Random(); // Zufallsobjekt
List<Tuerchen> tuerchen = new ArrayList<>();

public Adventskalender() {
for(int i = 1; i <= 24; i++) {
tuerchen.add(new Tuerchen(i));
}

Collections.shuffle(tuerchen);
}

public List<Tuerchen> getTuerchen() {
return this.tuerchen;
}

public boolean isTuerchenOffen(int nummer) {
for(int i = 1; i <= 24; i++) {
if(tuerchen.get(i - 1).nummer == nummer) {
return tuerchen.get(i - 1).offen;
}
}
return false;
}

public void oeffneTuerchen(int nummer) {
for(int i = 1; i <= 24; i++) {
if(tuerchen.get(i - 1).nummer == nummer) {
tuerchen.get(i - 1).offen = true;
}
}
}

public String toString() {
StringBuilder string = new StringBuilder();
int tempNummer;

for(int i = 1; i <= 24; i++) {
tempNummer = tuerchen.get(i - 1).nummer;
if(tempNummer < 10 && !tuerchen.get(i - 1).offen) {
string.append(" "); // Padding damit der Kalender rechteckig bleibt
}

if(!tuerchen.get(i - 1).offen) {
string.append(tempNummer);
} else {
string.append("xx");
}

if(i % 6 == 0) {
string.append("\n");
} else {
string.append(" ");
}
}

return string.toString();
}
}
[/CODE]

[CODE lang="java" title="Tuerchen"]import java.util.Arrays;
import java.util.List;

public class Tuerchen {
int nummer;
boolean offen;
String spruch;
private static final List<String> sprueche = Arrays.asList(
"Spruch 1",
"Spruch 2",
//....
"Weihnachten "
);

public Tuerchen(int nummer) {
this.nummer = nummer;
this.offen = false;
}

public static String getSpruchByNummer(int nummer) {
return sprueche.get(nummer - 1);
}
}
[/CODE]

Vielen Dank für Eure Hilfe!!

 

[jari]

nicht das ganze Türchenarray in jedem Türchen speichern - einmal im Adventskalender und im Türchen einzelne Sprüche, diese dann bei Erstellung mitgeben

Hä?

 

[SpaghettiBolo]

Hä?

vielleicht hilft dir das:

- nicht das ganze Türchenarray in jedem Türchen speichern, es reicht einmal im Adventskalender und im Türchen dann einzelne Sprüche.
- Diese kannst du dann dem Türchen bei Erstellung mitgeben --> Dazu kannst du auch denselben Index wie für die Nummer nutzen. Sprich "new Tuerchen(i, spruch)"

 

[Mart]

woran scheitert es wenn du es anderen erklären kannst?

 

[SpaghettiBolo]

woran scheitert es wenn du es anderen erklären kannst?

Ich muss genau das ändern und weiß nicht wie, aber habe oben mein ursprüngliches Problem ungünstig ausgedrückt

 

[Jw456]

die Eingabe in Scanner umwandeln

dann schaue doch mal wie du bis jetzt im alten Code die Eingabe machst.
Tipp das konnte sich hinter den BufferedReader und der While schleife verbergen.

nicht das ganze Türchenarray in jedem Türchen speichern - einmal im Adventskalender und im Türchen einzelne Sprüche, diese dann bei Erstellung mitgeben.

Was in der Klasse Tuerchen ist den dein Array ? Eigentlich hast du eine Liste und kein Array.
Und die soll eben nicht in der Klasse sein , sondern in der Adventskalender Klasse das hast du ja auch schon gemacht.

Wo genau hängt es den jetzt.

 

[SpaghettiBolo]

Hallo zusammen,

ich habe diesen Code für einen Adventskalender und muss folgendes umändern, doch komme ich selbst nicht auf die Lösung:

• nicht das ganze Türchenarray in jedem Türchen speichern - einmal im Adventskalender und im Türchen einzelne Sprüche, diese dann bei Erstellung mitgeben.
• die Eingabe in Scanner umwandeln

[CODE lang="java" title="Main"]import java.io.BufferedReader;
import java.io.InputStreamReader;

public class App {
static Adventskalender adventskalender = new Adventskalender();

public static void main(String[] args) throws Exception {
System.out.println(adventskalender);

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

while(true) { // Dauerhafte Schleife
int input = Integer.parseInt(reader.readLine());
if(input < 1 || input > 24) {
break; // Unpassende Eingabe bricht Schleife ab
}

System.out.println(); // Platz machen
if(!adventskalender.isTuerchenOffen(input)) {
adventskalender.oeffneTuerchen(input);
System.out.println(Tuerchen.getSpruchByNummer(input));
} else {
System.out.println("Türchen wurde schon geöffnet!");
}
System.out.println();
System.out.println(adventskalender);
}
}
}[/CODE]

[CODE lang="java" title="Adventskalender"]import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.Random;

public class Adventskalender {
Random random = new Random(); // Zufallsobjekt
List<Tuerchen> tuerchen = new ArrayList<>();

public Adventskalender() {
for(int i = 1; i <= 24; i++) {
tuerchen.add(new Tuerchen(i));
}

Collections.shuffle(tuerchen);
}

public List<Tuerchen> getTuerchen() {
return this.tuerchen;
}

public boolean isTuerchenOffen(int nummer) {
for(int i = 1; i <= 24; i++) {
if(tuerchen.get(i - 1).nummer == nummer) {
return tuerchen.get(i - 1).offen;
}
}
return false;
}

public void oeffneTuerchen(int nummer) {
for(int i = 1; i <= 24; i++) {
if(tuerchen.get(i - 1).nummer == nummer) {
tuerchen.get(i - 1).offen = true;
}
}
}

public String toString() {
StringBuilder string = new StringBuilder();
int tempNummer;

for(int i = 1; i <= 24; i++) {
tempNummer = tuerchen.get(i - 1).nummer;
if(tempNummer < 10 && !tuerchen.get(i - 1).offen) {
string.append(" "); // Padding damit der Kalender rechteckig bleibt
}

if(!tuerchen.get(i - 1).offen) {
string.append(tempNummer);
} else {
string.append("xx");
}

if(i % 6 == 0) {
string.append("\n");
} else {
string.append(" ");
}
}

return string.toString();
}
}
[/CODE]

[CODE lang="java" title="Tuerchen"]import java.util.Arrays;
import java.util.List;

public class Tuerchen {
int nummer;
boolean offen;
String spruch;
private static final List<String> sprueche = Arrays.asList(
"Spruch 1",
"Spruch 2",
//....
"Weihnachten "
);

public Tuerchen(int nummer) {
this.nummer = nummer;
this.offen = false;
}

public static String getSpruchByNummer(int nummer) {
return sprueche.get(nummer - 1);
}
}
[/CODE]

Vielen Dank für Eure Hilfe!!

Hallo nochmal!

ich habe das mit den Türchen jetzt gelöst bekommen, dennoch hänge ich immer noch mit der Scanner Eingabe und der Überprüfung, ob der eingegebene Wert eine ganzzahlige zahl ist.
Hier mein Ansatz:

import java.util.Scanner;

public class App {
    
        Scanner scanner = new Scanner(System.in);
        String eingabe = scanner.nextLine();

        public static void main(String[] args) {

            String[] sprueche = new String[] { "Spruch1", "Spruch2", "Spruch3", "Spruch4", "Spruch5", "Spruch5", "Spruch7", "Spruch8", "Spruch9", "Spruch10",
                    "Spruch11", "Spruch12", "Spruch13", "Spruch14", "Spruch15", "Spruch16", "Spruch17", "Spruch18", "Spruch19", "Spruch20",
                    "Spruch21", "Spruch22", "Spruch23", "Schöne Weihnachten!"};

            Adventskalender kalendar = new Adventskalender(sprueche);
            int nummer = 0;
            String spruch = null;
            System.out.println(kalendar);
            do {
                nummer = eingabeZuInt("Bitte gebe eine Zahl für ein Türchen ein:");
                spruch = kalendar.offneTuerchen(nummer);
                System.out.println(kalendar);
                printSpruch(spruch);
            } while (spruch != null);
        }

        private static boolean eingabeZuInt(String ueberschrift) {
            boolean ok = false;
            this.eingabe = eingabe;
            if(eingabe.matches("-?\\d+")){
                int b = Integer.parseInt(eingabe);
                return ok= true;
            }else{
                System.out.println("Bitte nur ganze Zahlen eingeben");
            }
            while (!ok);
            return ok;
        }

Danke euch!

 

[Mart]

was ist mit scanner.nextInt() ?ist das kaputt?

 

[SpaghettiBolo]

was ist mit scanner.nextInt() ?ist das kaputt?

Wie meinst du?
Ich bin leider absoluter Anfänger..

 

[Jw456]

Scanner scanner = new Scanner(System.in);
String eingabe = scanner.nextLine();

etweder in die main. oder static

 

[SpaghettiBolo]

Scanner scanner = new Scanner(System.in);
String eingabe = scanner.nextLine();

etweder in die main. oder static

Vielen Dank!

Die eingabeZuInt Methode benötige ich dann nicht mehr, oder?

 

[Jw456]

doch du host ja einen String und kein int.
dann frage doch gleich nach einen int.

und hier solltest du die Eingabe holen und nicht im Klassenrumpf.

 

[Jw456]

Wie meinst du?
Ich bin leider absoluter Anfänger..

scanner.nextInt() liest die Zeichen von der Tastatur als int ein wenn es einen Int entspricht sonst gibt es einen Fehler.

Da ihr Fehlerbehandlung wohl noch nicht hattet habt ihr wohl erstmal das NextLine() gezeigt bekommen.

 

[SpaghettiBolo]

  Adventskalender kalendar = new Adventskalender(sprueche);
            int nummer = 0;
            String spruch = null;
            System.out.println(kalendar);
            do {
                nummer = eingabeZuInt("Bitte gebe eine Zahl für ein Türchen ein:"); //gibt mir hier einen fehler an
                spruch = kalendar.offneTuerchen(nummer);
                System.out.println(kalendar);
                printSpruch(spruch);
            } while (spruch != null);
        }

        private static boolean eingabeZuInt(String ueberschrift) {
            Scanner scanner = new Scanner(System.in);
            String eingabe = scanner.nextLine();

            boolean ok = false;

            if(eingabe.matches("-?\\d+")){
                int value = Integer.parseInt(eingabe);
                return true;
            }else{
                System.out.println("Bitte nur ganze Zahlen eingeben");
            }
            while (!ok);
            return ok;
        }

ich habe es jetzt mal so umgeändert, aber es gibt mir hier einen Fehler aus

 

[Jw456]

Welcher Fehler kommt denn ?
Und was soll das While am ende ohne Do "while (!ok)"

bei deinem Regex bin ich mir nicht sicher das dies Passt.

 

[SpaghettiBolo]

Welcher Fehler kommt denn ?
Und was soll das While am ende ohne Do

Wie gesagt ich hab eigentlich keine ahung was ich da mache haha

er sagt dass boolean cannot be convertet to int

 

[Jw456]

nummer = eingabeZuInt("Bitte gebe eine Zahl für ein Türchen ein:"); //gibt mir hier einen fehler an

Welchen Datentyp gibt die Methode zurück.
und von welchen Datentyp ist nummer.


Das sind Fragen du dir selber stellen solltest, ich kenne die Antwort du sollst selber drauf kommen.
Müsstet dir die fragen eigentlich selber beantworten können.