This week's book giveaway is in the OCAJP 8 forum. We're giving away four copies of OCA Java SE 8 Programmer I Study Guide and have Edward Finegan & Robert Liguori on-line! See this thread for details.
In your else if((p1Choice.equals("paper"))&&(p2Choice.equals("rock"))) you are checking for "paper", "rock", and "scissors" but, you have defined your choices as choice = ("Rock"); with the first letter capitalized "Rock", "Paper" and Scissors". So paper does not equal Paper...; therefore, you never fall into you if statements meaning the counters are never incremented. Also for your draw check you are using if(p1Choice == (p2Choice)) but you should be using if(p1Choice.equals(p2Choice)) since it is the actual string you want to check not the String reference. Hope this gets you going again
David, There are quite a number of things that you could do to make this a better and cleaner program. And although it may work, it simply is not object-oriented. If you're interested, I could show you how to clean it up but I think it would be better to start a new thread in the Refactoring forum. Thanks, Junilu [This message has been edited by JUNILU LACAR (edited June 14, 2001).]