Hangman

[Feriscool]

I'm trying to create a hangman game, but the problem is, how would I make it so that if the person presses "J", "A", and "V" on the keyboard, it moves onto word2();?

public static void WORD1() { //Java
    char key = ' ';
    
    c.setFont(new Font("arial", Font.PLAIN, 50));
    c.setColor(Color.BLACK);
    c.drawString("_", 251, 472);
    c.drawString("_", 287, 472);
    c.drawString("_", 322, 472);
    c.drawString("_", 357, 472);
    
    while(key != 'J' || key != 'A' || key != 'V' || key != 'j' || key != 'a' || key != 'v')
    {
      key = c.getChar();

      if (key == 'J' || key == 'j')
      {
        c.setFont(new Font("arial", Font.BOLD, 50));
        c.setColor(Color.BLACK);
        c.drawString("J", 250, 470);
      }
      else if (key == 'A' || key == 'a')
      {
        c.setFont(new Font("arial", Font.BOLD, 50));
        c.setColor(Color.BLACK);
        c.drawString("A", 285, 470);
        c.drawString("A", 355, 470);
      }
      else if (key == 'V' || key == 'v')
      {
        c.setFont(new Font("arial", Font.BOLD, 50));
        c.setColor(Color.BLACK);
        c.drawString("V", 320, 470);
      }
      else if(key != 'J' || key != 'A' || key != 'V' || key != 'j' || key != 'a' || key != 'v')
      {
        COUNTER();
        
        if(counter == 11)
        {
          c.clear();
          c.setFont(new Font("arial", Font.PLAIN, 25));
          c.setColor(Color.BLACK);
          c.drawString("You          him! Try the next word!", 125, 250);
          c.setColor(Color.RED);
          c.drawString("killed", 174, 250);
          
          c.setFont(new Font("arial", Font.PLAIN, 15));
          c.setColor(Color.BLACK);
          c.drawString("The next word will automatically start in 5 seconds.", 150, 275);
          
          WAIT();
          c.clear();
          counter = 0;
          STAND();
          WORD2();
        }
      }
    }
  }

[Norm]

A quick observation on a line of code:
(key != 'J' || key != 'A' || key != 'V' || key != 'j' || key != 'a' || key != 'v')

If key is 'A' it is NOT 'J'. In other words this if will always be true
However being at the end of the if/else if chain it could be OK in this instance.

A GOOD IDEA - Always add a final else at the end of a chain of if/else if statements that prints out when all of the above if/else if statements were false.

Nesting the call to WORD2 inside of the WORD1 method could be a problem. Have WORD1 return a value to a higher level method that would call WORD2

[Feriscool]

Alright, I'm not sure how I'd go around that though.

[Norm]

It will probably take some thought and a redesign.

[Feriscool]

Probably. :p

[Feriscool]

Would something like this work?

if(key == 'j' && key == 'a' && key == 'v')
{
      WORD2();
}

[Norm]

How can the variable key have three different values at the same time? You need a different way to store the 3 letters so that you can test if they are J, A and V. For example: Either concatenate then in a string or store them in an array.

[Feriscool]

Okay, if I were to store them in an array, how would I call upon it so that if they pressed those three keys, move to WORD2()?

[Norm]

Using something like this:

if(key[0] == 'j' && key[1] == 'a' && key[2] == 'v')

[Feriscool]

I'm a beginner, so please bear with me. I've tried doing this. It works but it only moves onto the next word (WORD2()) if I click a different letter on the keyboard.

public static void WORD1() { //Java
    [b][COLOR="Red"]char[] JAVA = {'j','a','v'};[/COLOR][/b]
    char key = ' ';
    
    c.setFont(new Font("arial", Font.PLAIN, 50));
    c.setColor(Color.BLACK);
    c.drawString("_", 251, 472);
    c.drawString("_", 287, 472);
    c.drawString("_", 322, 472);
    c.drawString("_", 357, 472);
    
    while(key != 'J' || key != 'A' || key != 'V' || key != 'j' || key != 'a' || key != 'v')
    {
      key = c.getChar();
      
      if (key == 'J' || key == 'j')
      {
        c.setFont(new Font("arial", Font.BOLD, 50));
        c.setColor(Color.BLACK);
        c.drawString("J", 250, 470);
      }
      else if (key == 'A' || key == 'a')
      {
        c.setFont(new Font("arial", Font.BOLD, 50));
        c.setColor(Color.BLACK);
        c.drawString("A", 285, 470);
        c.drawString("A", 355, 470);
      }
      else if (key == 'V' || key == 'v')
      {
        c.setFont(new Font("arial", Font.BOLD, 50));
        c.setColor(Color.BLACK);
        c.drawString("V", 320, 470);
      }
      [b][COLOR="Red"]else if(JAVA[0] == 'j' && JAVA[1] == 'a' && JAVA[2] == 'v')
      {
        c.clear();
        counter = 0;
        STAND();
        WORD2();
      }[/COLOR][/b]
      else if(key != 'J' || key != 'A' || key != 'V' || key != 'j' || key != 'a' || key != 'v')
      {
        COUNTER();
        
        if(counter == 11)
        {
          c.clear();
          c.setFont(new Font("arial", Font.PLAIN, 25));
          c.setColor(Color.BLACK);
          c.drawString("You          him! Try the next word!", 125, 250);
          c.setColor(Color.RED);
          c.drawString("killed", 174, 250);
          
          c.setFont(new Font("arial", Font.PLAIN, 15));
          c.setColor(Color.BLACK);
          c.drawString("The next word will automatically start in 5 seconds.", 150, 275);
          
          WAIT();
          c.clear();
          counter = 0;
          STAND();
          WORD2();
        }
      }
    }  
  }

[Norm]

(key != 'J' || key != 'A' || key != 'V' || key != 'j' || key != 'a' || key != 'v')
This condition will always be true.

[Feriscool]

(key != 'J' || key != 'A' || key != 'V' || key != 'j' || key != 'a' || key != 'v')
This condition will always be true.

I don't quite understand. Should I remove the while loop?

[Norm]

If key is equal to 'J' then key != 'A' is true.
Since the comparisons are connected by ORs if any one is true then true is returned for the full conditional expression. The condition as you coded it will always be true.

What is the logic for the WORD1 method? Is it supposed to loop on some conditions? If so you need a loop of some kind. If not, remove the while loop. You need to have a design for the method before you code it.

[Feriscool]

There is a loop and all that's there is:

public static void main(String[] args) {
    
    STAND();
    WORD1();
    WORD2();
    WORD3();
    WORD4();
    WORD5();
    
  }

[Norm]

I have no idea what the code you posted is supposed to show or do.
The code shows 6 methods being called one after the other. No logic or loops.

[Feriscool]

I have it all working fine, the only puzzle piece missing is getting it from one word to the other when the word is complete.

[Norm]

getting it from one word to the other when the word is complete

Not sure what this means. Your terminology in confusing.
Do you mean that when one method (eg WORD1) finishes
you want the execution to go to the next method( WORD2)?

With the main() method as you show it, if the WORD1 method returns, then the WORD2 method would be called. When it returns, the WORD3 method would be called, etc

[Feriscool]

Yes, that is what I mean, I'll try returning but I've never used them. :/

[Norm]

Time to learn how.

See Cross-post at:
http://www.daniweb.com/software-deve...threads/364323

[Feriscool]

Arg, I don't have much time to finish this project. :/ I've been trying but can't seem to get it working.