Spiel Hangman programmieren - Ausgabe funktioniert nicht!!!

[Sindbad1983]

Hi!

Ich möcht das Spiel Hangman progammieren!
Nur häng ich jetzt grad an einem Fehler...kann mir bitte jemand helfen!

Ich möcht das Wort von der Konsole einlesen, dann in einem Array speichern und anschließend jeden Buchstaben des Wortes in ein '*' verwandeln..leider hat sich da ein Fehler eingeschlichen!


Es düft wieder irgendein Problem mit den Parametern geben! :autsch:

import java.util.*;
import java.io.*;

public class Hangman{


	char [] word;
	char [] star;
	int size;
	int cnt;
	int errors;
	char actualChar;

	public char [] besetzeArray(String s){ // Wort soll in Array eingefügt werden

		size=s.length();
		word=new char [size];

		for (int i=0;i<word.length;i++){
			word [i]=s.charAt(i);
		}
		return word;
	}

	public void konvertiere (char [] word){ // Buchstaben sollen in **** ausgegeben werden

		for(int i=0;i<word.length;i++){
			word [i]='*';
		}
	}

	public static void main(String [] args){

		Hangman test=new Hangman();
		String s;


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

			System.out.println("Spieler 1, bitte geben sie das Wort ein: ");

			try{
				s=br.readLine();

			}
			catch(IOException e){
				System.out.println(e);
			}

			char [] word=test.besetzeArray(s); // da wirft der Compiler einen Fehler aus!
			test.konvertiere(word);
	}

}

Danke für eure Hilfe!

ciao,Tommy

 

[Guest]

S nicht initialisiert.
Ich bin mir nicht ganz sicher, da ich auch kein Profi bin, aber ich kenne den Fehler.
Gib dem String "S" einfach einen Wert.
Z.B.

String S="";

 

[foobar]

Es ist schon ausreichend den String mit null zu initialisieren:

String s = null;

 

[Sindbad1983]

maa,so ein blöder Fehler!
stimmt!
danke !!!!!

 

[thE_29]

Jop, an dem liegts, weil du das s=br.readLine() in einem try/Catch Block hast und er ja nicht mal beim 1. mal unbedingt was reinlesen muss (vielleicht gibts ja die Datei/tastatur net - blödes Bsp kann aber ja sein) und dann hat s nie einen Wert und das mag er nicht, also musst du entweder sagen String S = ""; oder Stirng S = null;

Nachtrag: Viel zu spät :bae:

 

[Sindbad1983]

soooo..jetzt hab ichs fast geschafft!

Aber die Ausgabe funzt nicht!

:bloed:

ich dürft da wieder mal einen Denkfehler drinnen haben! ???:L
Kann mir bitte nochmal jemand helfen? :bae:

Also das Spiel soll so funktionieren:

Spieler 1 gibt Wort wein, Spieler 2 erfäht Länge des Wortes ****
Spieler 2 gibt Buchstaben ein...und wenn ein Buchstabe richtig ist, z.B:

Kerstin

*******

gibt ein e:
*e*****

ok?

import java.util.*;
import java.io.*;
public class Hangman{


	char [] word;

	int size;
	int cnt;
	int errors;
	char actualChar;

	public char [] besetzeArray(String s){ //nichts neues -> funktioniert

		size=s.length();
		word=new char [size];

		for (int i=0;i<word.length;i++){
			word [i]=s.charAt(i);
		}
		return word;
	}

	public void konvertiere (char [] word){ //nichts neues -> funktioniert

		for(int i=0;i<word.length;i++){
			word [i]='*';
		}

		System.out.println(word);
	}


	public void find(char actualChar, char [] word){ //neu -> Fehler???

		for(int i=0;i<word.length;i++){
			if(word[i]==actualChar){
				word[i]=actualChar;
			}
			else{
				word[i]='*';
			}
		}
		System.out.println(word); //hier passt was nicht!
	}


	public String toString(){ //nichts verändert

		StringBuffer sb=new StringBuffer();

		for(int i=0;i<word.length;i++){
			sb.append(word[i]);
		}

		return sb.toString();
	}




	public static void main(String [] args){

		Hangman test=new Hangman();
		String s="";
		String s2="";
		int cnt=4;

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




			System.out.println("Spieler 1, bitte geben sie das Wort ein: ");

			try{
				s=br.readLine();

			}
			catch(IOException e){
				System.out.println(e);
			}

			char [] word=test.besetzeArray(s);
			test.konvertiere(word);
			System.out.println("Sie haben insgesamt "+cnt+" Versuche, um das Wort zu erraten");
			while(cnt!=0){
			System.out.println("Geben Sie den Buchstaben ein: ");

			try{
				s2=br.readLine();
			}
			catch(IOException e){
				System.out.println(e);
			}

			char actualChar=s2.charAt(0);

			test.find(actualChar, word); //neu!!!!!!!!!!!
			System.out.println("Noch " +cnt+" Versuche");
			cnt--;


		}
	}

}

 

[foobar]

import java.io.*;

public class Hangman
{

	char[] word;
	char[] star;
	int size;
	int cnt;
	int errors;
	char actualChar;

	public void find(char actualChar, char[] word)
	{ //neue Methode!!!!!!!!!!!!!

		for (int i = 0; i < word.length; i++)
		{
			if (word[i] != actualChar)
			{
				word[i] = '*';
			}
		}
		System.out.println(word);
	}

	public String toString()
	{

		StringBuffer sb = new StringBuffer();

		for (int i = 0; i < word.length; i++)
		{
			sb.append(word[i]);
		}

		return sb.toString();
	}

	public static void main(String[] args)
	{

		Hangman test = new Hangman();
		String s = "";
		String s2 = "";

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

		System.out.println("Spieler 1, bitte geben sie das Wort ein: ");

		try
		{
			s = br.readLine();

		}
		catch (IOException e)
		{
			System.out.println(e);
		}

		char[] word = s.toCharArray();

		System.out.println("Geben Sie den Buchstaben ein: ");

		try
		{
			s2 = br.readLine();
		}
		catch (IOException e)
		{
			System.out.println(e);
		}

		char actualChar = s2.charAt(0); //auch neu!!!!!!!!

		test.find(actualChar, word);
	}

}

 

[Sindbad1983]

sensationell!

es geht!
:applaus: Cool!

sind die beiden Methoden, die du rausgelöscht hast völlig unnötig?
er soll ja vorher das gesamte Wort in **** ausgeben, damit man sieht, wieviel Buchstaben es hat!

aber wo war bei mir jetzt der Fehler?

 

[foobar]

er soll ja vorher das gesamte Wort in **** ausgeben, damit man sieht, wieviel Buchstaben es hat!

Oops, hab ich übersehen

Hier ist eine verbesserte Variante:

import java.io.*;

public class Hangman
{
	private char[] word;

	public Hangman()
	{
		this.startGame();
	}

	private void startGame()
	{
		String s 				    =  this.readStdin("Spieler 1, bitte geben sie das Wort ein: ");
		this.word 			    = s.toCharArray();
		this.printShadowedWord();
		String s2 			    =  this.readStdin("Geben Sie den Buchstaben ein: ");
		char actualChar 	= s2.charAt(0); //auch neu!!!!!!!!
		this.find(actualChar);
	}

	private void printShadowedWord()
	{
		for (int i =0; i < this.word.length; i++)
		{
			System.out.print("*");
		}
		System.out.println("");
	}
	
	private String readStdin(String message)
	{
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		System.out.println( message );
		String out = null;
		try
		{
			out= br.readLine();
		}
		catch (IOException e)
		{
			e.printStackTrace();
		}
		return out;
	}

	public void find(char actualChar)
	{
		for (int i = 0; i < word.length; i++)
		{
			if (this.word[i] != actualChar)
			{
				this.word[i] = '*';
			}
		}
		System.out.println(this.word);
	}

	public String toString()
	{
		StringBuffer sb = new StringBuffer();
		for (int i = 0; i < word.length; i++)
		{
			sb.append(this.word[i]);
		}
		return sb.toString();
	}

	public static void main(String[] args)
	{
		new Hangman();
	}
}

 

[foobar]

aber wo war bei mir jetzt der Fehler?

- obwohl du word als Member deklariert hast, wurde die Variable an die einzelnen Methoden übergeben
- Member waren nicht gekapselt
- Redundanter Code beim einlesen der Benutzereingaben
- Unnötige Methoden (besetzeArray). Es gibt eine Methode in der Klasse String die ein char[] zurückliefert
- Stacktrace wurde nicht ausgegeben
- Unnötiger else-Zweig in der Methode find

Das Hauptproblem war aber, daß du in der Methode konvertiere alle chars in Sterne umwandelst.

 

[abollm]

Leute, warum wollt ihr eigentlich das Rad immer neu erfiinden?

Wenn du im Forum, statt nach "hangman" einmal nach "Galgenmännchen" gesucht hättest, hättest du das vielleicht gefunden:

http://www.java-forum.org/de/viewto...E4nnchen&sid=8e0e4013b74d451b99698dff7d85c498

 

[SebiB90]

Leute, warum wollt ihr eigentlich das Rad immer neu erfiinden?

aus übung vllt.

 

[foobar]

Leute, warum wollt ihr eigentlich das Rad immer neu erfiinden?

Ich glaube bei diesem Programm stand der didaktische Wert im Vordergrund.

 

[abollm]

...
Ich glaube bei diesem Programm stand der didaktische Wert im Vordergrund.

Tja, wird wohl stimmen.

Vielleicht sollte man im Forum einmal, unabhängig von der FAQ. über eine Rubrik wie "Lösungen für typische, grundlegende Aufgabenstellungen" oder "Schritt-für-Schritt-Anleitungen" für derartige Aufgabenstellungen, versehen mit ein _wenig_ Theorie, nachdenken.

Wird dann vermutlich auch nicht von allen gelesen, aber macht es insgesamt einfacher, weil man als Antwortender nur darauf verweisen müsste.