Falsche Methode

[Kitero]

Hey Leute!

Ich bin noch ein ziemlicher Javaneuling und habe in meiner Uni die Aufgabe bekommen, ein Würfelpokerspiel zu erstellen. Ich habe das Problem zwar schon im Hausaufgaben Forum gepostet, aber da dort nur wenige reingucken, versuche ich hier mein Glück :toll:
Ich hoffe das ist nicht schlimm.

Das Programm liefert bei mir zwar das richtige Ergebnis, aber ich habe das Problem, dass unser Prüfprogramm nur eine bestimmte Version davon als richtig anerkennt.
Leider steh ich zurzeit ziemlich am Schlauch und habe keine Ahnung wie ich dieses Programm richtig ändern kann.

Ich muss statt:

public static int bewerteWurf(int[] wurf, int Round, int Player) {

den Code auf:

public static String bewerteWurf (int[] wurf, int Round, int Player) {

umändern... also statt dem int - String verwenden.

Ich hoffe jemand kann mir dabei behilflich sein...
brauche das Programm schon morgen ;(
Wäre euch sehr dankbar

Hier mein ganzer Code:

import java.util.*;
 
 
public class Aufgabe10 {
 
    public static int NrPlayers = 9;
    public static int NrRounds = 2;
 
    public static void main (String[] args) {
 
    int Game[][][];
    Random r = new java.util.Random();
 
    Game = new int[NrPlayers][NrRounds+1][5];
 
    for (int Round=1; Round <= NrRounds; Round++) {
 
        for (int Player = 0; Player < NrPlayers; Player++) {
        Game = rollDice(Game, Round, Player, r);
        Game[Player][0][0] = bewerteWurf(Game[Player][Round], Round, Player);
        } 
 
        processWinner(Game);
    }
 
    } 
 
    public static int[][][] rollDice(int[][][] Game, int Round, int Player, Random r) {
 
    for (int i = 0; i < Game.length; i++) {
        for (int j = 0; j < 5; j++) {
        Game[i][Round][j] = r.nextInt(6)+1;
        } 
    } 
    return(Game);
    }
 
    public static int bewerteWurf(int[] wurf, int Round, int Player) {
    int Win = 0;
    int[] EyesCount = {0, 0, 0, 0, 0, 0}; 
 
    System.out.print("Results for player " + (Player+1) + " in round " + (Round) + ":");
    for (int i = 0; i < Dimes.length; i++) {
        System.out.print(" " + Dimes[i]);
        switch (Dimes[i]) {
        case 1: EyesCount[0] = EyesCount[0] + 1 ;break;
        case 2: EyesCount[1] = EyesCount[1] + 1; break;
        case 3: EyesCount[2] = EyesCount[2] + 1; break;
        case 4: EyesCount[3] = EyesCount[3] + 1; break;
        case 5: EyesCount[4] = EyesCount[4] + 1; break;
        case 6: EyesCount[5] = EyesCount[5] + 1; break;
        default:  break;
        } 
    } 
 
    
    if ( (EyesCount[1]==1) && (EyesCount[2]==1) && (EyesCount[3]==1) && (EyesCount[4]==1) )
        if ((EyesCount[0]==1) || (EyesCount[5]==1)) {
        Win = 1;
        System.out.print(" (Sequence)! ");
        } 
        else  ;
    else {
        for (int j = 0; j < 5 ; j++) {
        switch (EyesCount[j]) {
        case 5: Win = 4; System.out.print(" (Grand)! "); break;
        case 4: Win = 3; System.out.print(" (Poker)! "); break;
        case 3: {
            for (int h = 0; h < 5; h++) {
            if (EyesCount[h]==2) {
                Win = 2;
                System.out.print(" (Full House)! ");
            } 
            } 
            break;
        }
        }
        } 
    } 
    System.out.println("");
    return(Win);
    } 
 
 
    public static void processWinner(int[][][] Game) {
    int Winner = 0;
    int Points = 0;
 
    for (int i = 0; i < Game.length; i++) {
 
        if (Game[i][0][0] >= Points) {
        if (Game[i][0][0] > Points) {
            Winner = i+1;
            Points = Game[i][0][0];
            }
        else
            Winner = 0;
        }
    } 
 
    if(Winner > 0) System.out.println("Der Gewinner ist Spieler " + Winner);
    else  System.out.println("Niemand hat gewonnen!");
    System.out.println("******************************************");
 
    } 
 
}

 

[JUserToto]

Es wäre glaube ich hilfreich wenn wir wüssten, was das Programm eigentlich leisten soll... bzw. was die Methode "bewerteWurf" nun wirklich zurück geben soll.


edit: Variablennamen (game, winner etc.) schreibt man in Java klein

 

[Kitero]

Es wäre glaube ich hilfreich wenn wir wüssten, was das Programm eigentlich leisten soll... bzw. was die Methode "bewerteWurf" nun wirklich zurück geben soll.


edit: Variablennamen (game, winner etc.) schreibt man in Java klein

Man sollte das Ergebnis von 5 Würfeln bewerten.
Als Ausgabe muss "Grand", "Poker", "Full House", "Sequence" zurückgegeben werden.
Verwenden sie dazu:
public static String bewerteWurf (int[] wurf)

das soll wohl bewerteWurf ausgeben...
mehr steht bei unserer Aufgabenstellung nicht.

Und danke wegen den Variablennamen...
hab das übersehn :bloed:

 

[stg]

Zum einen: Es ist echt anstrengend deinen Code zu lesen. Rück doch bitte zusammengehörige Blöcke anständig ein ... :autsch:

Zum anderen: Halte dich doch bitte an die Aufgabenstellung! Du sollst offenbar eine Methode

public static String bewerteWurf (int[] wurf) { ... }

implementieren, die in Abhängigkeit von

wurf

eine der Optionen "Grand", "Poker", "Full House", "Sequence" zurückgibt. Mehr nicht!

Auf den Rest kann man später eingehen, wenn du zunächst einfach mal diese eine Teilaufgabe gelöst hast.

Aber um ehrlich zu sein und nichts schönzureden: Der Code, der hier steht, ist grauenvoll. Den wirst du wohl komplett neu schreiben müssen. :noe: Tu dir selbst den gefallen und versuche nicht den alten Code anzupassen, bis zufälligerweise die Bildschirmausgabe eingiermaßen sinnvoll ist, sondern setzt dich hin, fang komplett vorne an und versuche dabei wirklich genau das zu befolgen, was in der Aufgabenstellung von dir verlangt wird. Diese scheint ja doch recht brauchbar zu sein.