Hangman problem please help

joe hinder

i got given this problem for homework here it is : create a simple hang man game with a set of related words in an array. these words should be selected randomly from the array and the correct number of characters in the selected word should be indicated on the screen. the player guesses letters from the word and, if these are in the word they are placed in the correct postion, otherwise one life is lost. the player as six lives.

the system should:

record the letters that have been used by the player and not accept these again during the play.
record the number of lives left after the word has been guessed and use these as a score.
allow an end user to enter new sets of words.

the things that i have underlined i cannot do please help below is my code in a windows form application :-

public Form1()
{
InitializeComponent();
}

private void label2_Click(object sender, EventArgs e)
{

}

string word = "";
List labels = new List();
int number = 7;
int amount = 0;
string[] array = new string[1];


void makelabels()
{
word = getrandomword();
char[] chars = word.ToCharArray();
int between = 330 / chars.Length - 1;
for (int i = 0; i < chars.Length - 1; i++)
{
labels.Add(new Label());
labels[i].Location = new Point((i * between) + 10, 80);
labels[i].Text = "_";
labels[i].Parent = groupBox1;
labels[i].BringToFront();
labels[i].CreateControl();

}
label1.Text = "word length:" + (chars.Length - 1).ToString();

}

string getrandomword()
{

}

private void Form1_Load(object sender, EventArgs e)
{
makelabels();
}

private void button1_Click(object sender, EventArgs e)
{
}

private void button1_Click_1(object sender, EventArgs e)
{
char letter = textBox2.Text.ToCharArray()[0];
if (!char.IsLetter(letter))
{
MessageBox.Show("you can only enter letters!", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
return;
}
if (word.Contains(letter))
{
char[] letters = word.ToCharArray();
for (int i = 0; i < letters.Length; i++)
{
if (letters[i] == letter)
labels[i].Text = letter.ToString();
}
foreach (Label l in labels)
if (l.Text == "_") return;
MessageBox.Show("you have won");
resetgame();
}

else
{
MessageBox.Show("the letter you guessed is wrong!", "sorry");
label2.Text += " " + letter.ToString() + ",";
amount++;
if (amount == 6)
{
MessageBox.Show("sorry you lost the word was: " + word);
resetgame();
}

}

}

void resetgame()
{
label2.Text = "Missed: ";
textBox1.Text = "";
}

private void button2_Click(object sender, EventArgs e)
{
if (textBox2.Text == word)
{
MessageBox.Show("you won");
resetgame();
}
else
{
MessageBox.Show("you guessed wrong");
}
}

private void label3_Click(object sender, EventArgs e)
{

}
}
}

19 Replies

Know the answer? Post it — somebody with the same question will find it here.

Frogleg14y ago
Yes, that's one thing that did'nt cross my mind
joe hinder14y ago
In your origanal zip how could you not allow the user not to enter intergers as well.
Frogleg14y ago
V2
Frogleg14y ago
Change 1 to -1
The code probably needs changing because if you use all the lives, the program exits
On your previous question - look at this application
joe hinder14y ago
on the bit where you put while (lives > 1); when you run the program it only allows you to enter letters until the lives = 1 but i need it to be able to run until the lives = 0 please respond
joe hinder14y ago
What do the static voids do and do i need them.

also thank you very much for all your help
Frogleg14y ago
Well at least you are trying
joe hinder14y ago
i done what you suggested and tried doing the whole program and I was successful on most parts of it but couldnt do some things this is my code :-

{
Int32 lives = 6;
string letterguess;
Int32 counter;
string words;
Int32 randomindex;
string setwords;

string[] wordarray = new string[25];

Random Randomnumbergen = new Random();
randomindex = Randomnumbergen.Next(0, 24);

wordarray[0] = "cat";
wordarray[1] = "goose";
wordarray[2] = "elk";
wordarray[3] = "bear";
wordarray[4] = "grasshopper";
wordarray[5] = "falcon";
wordarray[6] = "deer";
wordarray[7] = "albatross";
wordarray[8] = "dolphin";
wordarray[9] = "bison";
wordarray[10] = "parrot";
wordarray[11] = "hippo";
wordarray[12] = "rat";
wordarray[13] = "pig";
wordarray[14] = "liger";
wordarray[15] = "tiger";
wordarray[16] = "lion";
wordarray[17] = "rhino";
wordarray[18] = "mouse";
wordarray[19] = "lizard";
wordarray[20] = "mole";
wordarray[21] = "worm";
wordarray[22] = "ant";
wordarray[23] = "bat";
wordarray[24] = "dog";

words = wordarray[randomindex];

char[] chars = words.ToCharArray();


counter = words.Length;

Console.Write("The words length is : " + counter);

Console.WriteLine("");

for (int i = 0; i < 6; i++)
{
Console.WriteLine("please enter a letter: ");
letterguess = Console.ReadLine();

char[] guesss = letterguess.ToCharArray();



if (letterguess != words)
{
Console.WriteLine("you guessed wrong");
lives--;
Console.WriteLine("lives : " + lives);
}
else
{
Console.WriteLine("you guessed correct");
}

Console.WriteLine("these are your guesses " + letterguess + ",");
/* foreach (string value in wordarray)
{
Console.WriteLine("you cannot enter the same letter twice");
}
*/
}
string[] newwords = new string[15];

setwords = newwords.ToString();
Console.WriteLine("please eneter a new set of words");
words = Console.ReadLine();
Console.Read();


}
}
}

these are the things i cannot get to work

allow user to enter new sets of words at end
record letters and not accept again
show letters in correct postion on screen if guessed correct

Please help
Frogleg14y ago
C# operators
Equal to a == b

Not equal to a != b

Greater than a > b
Less than a < b
Greater than or equal to a >= b
Less than or equal to a <= b
Add x = 1 + 2;
joe hinder14y ago
what do you mean //two ==
Frogleg14y ago
char[] guesss = letterguess.ToCharArray();

if (letterguess == guesss)//two ==

            {

                Console.WriteLine("you cannot type in the same letter twice");

            }

            else

            {

            }
joe hinder14y ago
to check if the letter entered has been used before i tryed doing this below why wont it work please help.

char[] guesss = letterguess.ToCharArray();
if (letterguess = guesss)
{
Console.WriteLine("you cannot type in the same letter twice");
}
else
{

}
Frogleg14y ago
You have to zip(compress) the file to upload
Do as much as you can and I'll help you through it
joe hinder14y ago
Could you help me with this code same problem but in c# console application

i got given this problem for homework here it is : create a simple hang man game with a set of related words in an array. these words should be selected randomly from the array and the correct number of characters in the selected word should be indicated on the screen. the player guesses letters from the word and, if these are in the word they are placed in the correct postion, otherwise one life is lost. the player as six lives.
the system should:
- record the letters that have been used by the player and not accept these again during the play.
- record the number of lives left after the word has been guessed and use these as a score.
- allow an end user to enter new sets of words.
- it wouldnt let me put it into a file sorry.
  my code:
  
  Int32 lives = 6;
  
  string letterguess;
  
  Int32 counter;
  
  Int32 randomindex;
  
  string words;
  
  string[] wordarray = new string[25];
  
  Random Randomnumbergen = new Random();
  
  randomindex = Randomnumbergen.Next(0, 24);
  
  wordarray[0] = "cat";
  
  wordarray[1] = "goose";
  
  wordarray[2] = "elk";
  
  wordarray[3] = "bear";
  
  wordarray[4] = "grasshopper";
  
  wordarray[5] = "falcon";
  
  wordarray[6] = "deer";
  
  wordarray[7] = "albatross";
  
  wordarray[8] = "dolphin";
  
  wordarray[9] = "bison";
  
  wordarray[10] = "parrot";
  
  wordarray[11] = "hippo";
  
  wordarray[12] = "rat";
  
  wordarray[13] = "pig";
  
  wordarray[14] = "liger";
  
  wordarray[15] = "tiger";
  
  wordarray[16] = "lion";
  
  wordarray[17] = "rhino";
  
  wordarray[18] = "mouse";
  
  wordarray[19] = "lizard";
  
  wordarray[20] = "mole";
  
  wordarray[21] = "worm";
  
  wordarray[22] = "ant";
  
  wordarray[23] = "bat";
  
  wordarray[24] = "dog";
  
  words = wordarray[randomindex];
  
  char[] chars = words.ToCharArray();
  
  counter = words.Length;
  
  Console.Write("The words length is : " + counter);
  
  Console.WriteLine("");
  
  for (int i = 0; i < 6; i++)
  
  {
  
  Console.WriteLine("please enter a letter: ");
  
  letterguess = Console.ReadLine();
  
  if (letterguess != words)
  
  {
  
  Console.WriteLine("you guessed wrong");
  
  lives--;
  
  Console.WriteLine("lives : " + lives);
  
  }
  
  else
  
  {
  
  Console.WriteLine("you guessed correct");
  
  }
  
  }
  
  Console.Read();
  
  }
  
  }
  
  }
Frogleg14y ago
You won't learn anything if i do it for you
Do as much as you can, then upload your project so we can help - but not do it all for you
joe hinder14y ago
Could u put that into code this is first time doing windows form so very hard to get my head round it please thank you.
Frogleg14y ago
First off, as the user inputs a char, you will add that to an array - 26 letters long ie - inputChar[26]
Then create an integer called "missed" that increments each time an wrong letter choice is made
If the char matches the first letter of the word, then label1 will display that letter. You can have an integer say index =0
As correct letters are added to the label, index would increment - so the program knows where it is.
The index would also align with your chars array.
Second
At the next char input, you check if inputChar contains the new char,then the index to check the input char to see if chars[1]
is a match.
Increment index or missed as required
See how you go from there
Also a tip, upload your project rather than pasting code as it makes it
far easier to spot problems
joe hinder14y ago
How can i get that also to work even if they get the letter wrong
Frogleg14y ago
As it is home work you will have to do most of the coding
To get the char pressed
1: Set form1 keyPreview to true
2:use

char input;
private void Form1_KeyPress(object sender, KeyPressEventArgs e)
         {
             if (Char.IsLetter(e.KeyChar))
             {
                 input = e.KeyChar;

            }

        }

You can put the chars in an array, then with each key press
check if array contains the pressed char