Jacqueline Turof

On the side where the JUnit bar is, it says there is one error and lists it as initializationError(0.000 s). So I'm assuming I'm not setting up the constructor right at all. I didn't use the @Before in any of my other JUnit attempts when I was able to get them working. Maybe I'm not using it properly?

Thanks for your reply and with that information I was able to make some simple JUnits run and work.
Now, I'm getting stuck on more complex testing. I am trying to test the constructor for my HangmanLogic class. The
constructor is public HangmanLogic(String keyPhrase, int numberOfGuesses) throws InvalidConfigurationException
Just trying to start the testing with making sure the information being passed into the constructor is what it should be. I
keep seeing an initializationError.

import static org.junit.Assert.*; import org.junit.Before; import org.junit.Test; public class HangmanLogicTest { HangmanLogic test; //an instance of the class to test @Before //this should be run before every test public void setUp()throws InvalidConfigurationException { test=new HangmanLogic ("String",10); } @Test public void getNumberofGuesses()//make sure the constructor is working { assertEquals(10,test.getNumberOfGuesses()); } }



/** * @version 0.2, 01/29/2013 * This contains the logic for a Hangman game. * I have chosen to disallow all input except alphabetic characters (disallowing number to make some things easier to manage) */ import java.util.ArrayList; public class HangmanLogic { private String keyPhrase; private int numberOfGuesses; private int numberOfGuessesLeft; private ArrayList<Character> guessedCharacters; /** * Constructor creates this HangmanLogic initial state * @param newPhrase the phrase to be used for the key phrase of the hangman game * @param numGuesses the number of guesses allowed in the hangman game */ public HangmanLogic(String keyPhrase, int numberOfGuesses) throws InvalidConfigurationException { //a better check for valid keyphrases could be made, but that is not the point of this assignment if(keyPhrase.equals("") || keyPhrase.equals(null)) throw new InvalidConfigurationException("Invalid keyphrase, must not be blank or null"); else if(numberOfGuesses < 1) throw new InvalidConfigurationException("Invalid number of guesses, must be one or greater"); this.keyPhrase = keyPhrase; this.numberOfGuesses = numberOfGuesses; this.numberOfGuessesLeft = numberOfGuesses; guessedCharacters = new ArrayList<Character>(); } /** * Accessor method for the keyPhrase field * @return returns the keyPhrase field of this object */ public String getKeyPhrase() { return keyPhrase; } /** * method returns the keyPhrase field in an encoded manner with dashes representing the still unknown letters and letters for the known letters * @return returns the encoded keyPhrase of this object (letters known and dashes) */ public String getKnownKeyPhrase() { char[] encodedKeyPhrase = new char[keyPhrase.length()]; String lowerCaseKeyPhrase = keyPhrase.toLowerCase(); for(int i = 0; i < keyPhrase.length(); i++) { char cur = lowerCaseKeyPhrase.charAt(i); if(guessedCharacters.contains(cur)) encodedKeyPhrase[i] = cur; else encodedKeyPhrase[i] = '-'; } return (new String(encodedKeyPhrase) ); } /** * Accessor method for the numberOfGuesses field * @return returns the numberOfGuesses field of this object */ public int getNumberOfGuesses() { return numberOfGuesses; } /** * Accessor method for the numberOfGuessesLeft field * @return returns the numberOfGuessesLeft field of this object */ public int getNumGuessesLeft() { return numberOfGuessesLeft; } /** * This method returns whether a specified character exists in the keyPhrase field * @param guess a character being checked for in the keyPhrase field * @return returns whether a specified character exists in the keyPhrase field * @throws InvalidInputException if a non-valid character is passed as input to this method * @throws AlreadyGuessedException if a valid character which has already been guessed is passed as input to this method */ public boolean guessCharacter(char guess) throws InvalidInputException, AlreadyGuessedException { if(isValidCharacter(guess)) { //force into one case guess = Character.toLowerCase(guess); if(guessedCharacters.contains(guess) ) { throw new AlreadyGuessedException("" + guess); } else { guessedCharacters.add(guess); if(keyPhrase.contains("" + guess)) return true; else { numberOfGuessesLeft--; return false; } } } else { throw new InvalidInputException("" + guess); } } /** * Returns whether the guessing user is out of guesses in this game of hangman * @return Returns whether the guessing user is out of guesses in this game of hangman */ public boolean isGameOver() { boolean guessed = (getKnownKeyPhrase().equals(keyPhrase.toLowerCase())); return( numberOfGuessesLeft == 0 || guessed); } /** * Returns whether the character parameter is considered valid in this game * @param guess the character being checked for validity * @return returns whether the character parameter is considered valid in this game */ private boolean isValidCharacter(char guess) { return Character.isLetter(guess); } }

public class InvalidConfigurationException extends Exception { public InvalidConfigurationException(String message) { super(message); } }
public class AlreadyGuessedException extends Exception
{
public AlreadyGuessedException(String message)
{
super(message);
}
}
public class InvalidInputException extends Exception { public InvalidInputException(String message) { super(message); } }

Hi,

I'm suppose to be creating a JUnit test and I feel like I don't really understand these JUnit tests at all.
Does anyone know of a good tutorial to look at? I've been looking around and have been unable to find one.
This assignment is due Sunday and at this point I'm not even sure how to begin tackling this problem. Thanks, for any pointers or help.

Ah, thanks I see it now. Sometimes you look at something so many times you stop actually seeing what's there. lol Thanks, again!

Hi,

I'm suppose to be creating two recursive methods. One to figure out fibonaccis numbers and one to figure out if a string is a palindrome. I keep getting the error that my palindrome method must return a boolean. However, I feel like I have done that and don't understand why I'm getting this error message. I just want to test my methods to make sure the logic is correct, but can't even get that far. Any help is appreciated, thanks!

public class RecursionOne { //method for find the fibonanci numbers public static long fib(int fNumber) { //base case if (fNumber<=2) { return 1; } else { return fib(fNumber-1)+fib(fNumber-2); } } public boolean isPalindrome(String palindrome) { palindrome=palindrome.toLowerCase(); int length=palindrome.length(); //base case if (length<=1) { return true; } //get first and last letters in word char first=palindrome.charAt(0); char last= palindrome.charAt(length-1); if(Character.isLetter(first)&& Character.isLetter(last)) { //check to make sure first and last letter are the same if(first==last) { palindrome=palindrome.substring(1,length-1); return true; } } else { return false; } } public static void main(String[] args) { fib(8); String c="civic"; isPalindrome(c); } }

So, I'm back. I've been able to create a board object, but now I'm having a new problem. I'm trying to write a play() method using a board object. I am unable to set up a loop that checks for a win. I've added println statements in the program trying to check the loop and see how its actually working, but I still can't tell what is going on. The reason the loops are set up differently for an X horizontal win was for diagnostic purposes. None of the those testing 1,2,3 print ln statements printed to the console and the loops never ended. If anyone has any advice or insight it would be greatly appreciated. Thanks in advance.

import java.util.Scanner; public class Board{ //declaring the board array and setting to type string static private String [][]board; public Board(String[][]x){ board=x; } //creating a currentBoard() method to return the values of strings line1,line2, and line3 public String currentBoard(){ //declaring our string lines for the currentBoard method String line1= board[0][0]+ board [0][1]+board [0][2]; String line2= board [1][0]+board [1][1]+ board [1][2]; String line3= board [2][0]+ board[2][1]+ board [2][2]; return (line1+"\n"+line2+"\n"+line3); } public void swapX(String[][]x,int a){ if (a==1){ x[0][0]="[X]";} if (a==2){ x[0][1]="[X]"; } if (a==3){ x[0][2]="[X]"; } if (a==4){ x[1][0]="[X]"; } if (a==5){ x[1][1]="[X]"; } if (a==6){ x[1][2]="[X]"; } if (a==7){ x[2][0]="[X]"; } if (a==8){ x[2][1]="[X]"; } if (a==9){ x[2][2]="[X]"; } } public void swapO(String[][]x,int a){ if (a==1){ x[0][0]="[O]";} if (a==2){ x[0][1]="[O]"; } if (a==3){ x[0][2]="[O]"; } if (a==4){ x[1][0]="[O]"; } if (a==5){ x[1][1]="[O]"; } if (a==6){ x[1][2]="[O]"; } if (a==7){ x[2][0]="[O]"; } if (a==8){ x[2][1]="[O]"; } if (a==9){ x[2][2]="[O]"; } } public void play(){ String [][]boardIntial={{"[1]","[2]","[3]"},{"[4]","[5]","[6]"},{"[7]","[8]","[9]"}};//initial board Board myBoard=new Board(boardIntial); boolean win=false; Scanner input=new Scanner(System.in); int turnCounter=0; int playerX=0; int playerO=0; while(win==false) { //checking for and X win horizontially System.out.println("Testing,1,2,3."); if (boardIntial[0][0].equals("[1]")&& boardIntial[0][1].equals("[2]")&& boardIntial[0][2].equals("[3]")){ win=true; } if (boardIntial[1][0].equals("[X]")&& boardIntial[1][1].equals("[X]")&& boardIntial[1][2].equals("[X]")){ win=true; } if (boardIntial[2][0].equals("[X]")&& boardIntial[2][1].equals("[X]")&& boardIntial[2][2].equals("[X]")){ win=true; } ++turnCounter;//set turn counter to add 1 every iteration System.out.println(myBoard.currentBoard()); System.out.println("What the hell is going on?"); //checking for X win vertically if(win==false){ System.out.println("testing 1, 2, 3"); if (boardIntial[0][0].equals("[1]")&& boardIntial[1][0].equals("[2]")&& boardIntial[2][0].equals("[3]")){ win=true; } if (boardIntial[0][1].equals("[X]")&& boardIntial[1][1].equals("[X]")&& boardIntial[2][1].equals("[X]")){ win=true; } if (boardIntial[0][2].equals("[X]")&& boardIntial[1][2].equals("[X]")&& boardIntial[2][2].equals("[X]")){ win=true; } //checking for X win diagonally if (boardIntial[0][0].equals("[X]")&& boardIntial[1][1].equals("[X]")&& boardIntial[2][2].equals("[X]")){ win=true; } if (boardIntial[0][2].equals("[X]")&& boardIntial[1][1].equals("[X]")&& boardIntial[2][0].equals("[X]")){ win=true; } } if ((turnCounter%2)!=0) { System.out.println("Player 1 you are X! Please pick a square."); playerX=input.nextInt(); swapX(boardIntial,playerX); } if ((turnCounter%2)==0) { System.out.println("Player 2 you are O! Please pick a square."); playerO=input.nextInt(); swapO(boardIntial,playerO); //checking for O win horizontially if (boardIntial[0][0].equals("O")&& boardIntial[0][1].equals("O")&& boardIntial[0][2].equals("O")){ win=true; } if (boardIntial[1][0].equals("O")&& boardIntial[1][1].equals("O")&& boardIntial[1][2].equals("O")){ win=true; } if (boardIntial[2][0].equals("O")&& boardIntial[2][1].equals("O")&& boardIntial[2][2].equals("O")){ win=true; } //checking for an O win vertically if (boardIntial[0][0].equals("O")&& boardIntial[1][0].equals("O")&& boardIntial[2][0].equals("O")){ win=true; } if (boardIntial[0][1].equals("O")&& boardIntial[1][1].equals("O")&& boardIntial[2][1].equals("O")){ win=true; } if (boardIntial[0][2].equals("O")&& boardIntial[1][2].equals("O")&& boardIntial[2][2].equals("O")){ win=true; } //checking for an O win diagonally if (boardIntial[0][0].equals("O")&& boardIntial[1][1].equals("O")&& boardIntial[2][2].equals("O")){ win=true; } if (boardIntial[0][2].equals("O")&& boardIntial[1][1].equals("O")&& boardIntial[2][0].equals("O")){ win=true; }} } } }

fred rosenberger wrote:public Board(){ String [][] board= new String [3][3]; for (int i=0; i<board.length;i++){ for (int j=0; j><board[i].length;j++){ board [i][j]= "-"; }} }

Do you actually get to the "board [i][j]= "-"; " line? I would think (and this is a guess) that you get the NPE on line 4 here. you are trying to get the length of an array that doesn't exist.>

Thank you for all your help. I was finally able to get it working.

fred rosenberger wrote:a line like this:
String [][] board= new String [3][3];

creates an array, but it has nothing in it.

Also..remember that java doesn't have multi-dimensional arrays. it has arrays that hold objects - and those objects can be arrays.

So, after this line, you have an array that can hold three things (which have to be arrays of String), but those things don't exist yet. So you can't call a method on them.

I thought I was filling the the array with the board [i][j]= "-"; statement? Thanks for the responses, they've been very informative.

So I've made some changes to my code and I am throwing a NullPointerException and can't figure out what I'm doing wrong.

For my code part of the requirements are that I create a board by making a multidimensional array and that I create a method with the header public String currentBoard() that returns the state of the board as a multi line String. Here is my code so far.

public class Board{ //declaring the board array and setting to type string String [][]board; //declaring our string lines for the currentBoard method String line1= board[0][0]+ board [0][1]+board [0][2]; String line2= board [1][0]+board [1][1]+ board [1][2]; String line3= board [2][0]+ board[2][1]+ board [2][2]; //creating the Board()method to create a board that is a 2 dimensional array of strings public Board(){ String [][] board= new String [3][3]; for (int i=0; i<board.length;i++){ for (int j=0; j<board[i].length;j++){ board [i][j]= "-"; }} } //creating a currentBoard() method to return the values of strings line1,line2, and line3 public String currentBoard(){ return (line1+"\n"+line2+"\n"+line3); } }

I have a checking class with a main method so that I can make sure things are working as they should.
public class Checking { /** * @param args */ public static void main(String[] args) { // TODO Auto-generated method stub Board myBoard= new Board(); System.out.println(myBoard.currentBoard()); } }

Any one have any ideas why I'm getting the error NullPointerException? Any help is appreciated!!!

I'm working on creating a program where I can run two different games. I've just started and I'm stuck on just creating a tic-tac-toe board.
When I test to make sure that the board is set up properly it comes back as null. I'm obviously not setting my values for my array properly and was hoping someone had some advice. Thanks in advance!

import java.util.Arrays; public class Board { public int [][] board; public Board(){ int [][] board= new int[3][3]; for (int i=0;i<board.length;i++) for (int j=0; j<board.length;j++) board [i][j]=1; } public int [][] getboard() { return board; } }


import java.util.Arrays; import java.util.Scanner; public class Games { /** * @param args */ public static void main(String[] args) { /*Setting up our menu. Prompting the user and setting our gameChoice to the user's input * A bunch of if/else statements to deal with user input. * Choice 1=guess the number * Choice 2=tic tac toe * If the user input any other type of data they will receive any error * message. * */ System.out.println("Welcome to the games menu!\n Please choose your game option.\n" + " Press 1 to play guess the number or Press 2 to play tic tac toe."); // trying to create a new board object and test it and print it out Board tictactoe=new Board(); System.out.println(tictactoe.getboard()); Scanner userChoice= new Scanner(System.in); int gameChoice=userChoice.nextInt(); if (gameChoice==1){ GuessGame UserGame=new GuessGame(); } if(gameChoice==2){ } else { System.out.println("Please enter a 1 to play guess the number or a 2 for tic tac toe."); } } }

Thanks for the fast reply and the help. I was able to fix my code and have it working now.

I just started taking programming course in Java and I'm trying to write a guessing game. I would like the user to be able to start the game over again without having to restart the program. Every time I try to set up a loop to do this it never goes through the enter loop again. I'm just not constructing the loop properly, but I'm not sure how I can correct it. Any help or suggestions on how to improve my code would be appreciated. Thanks in advance!

import java.util.Scanner; public class GuessMyCard { public static void main(String[] args) { double myRandomCard= Math.random()* (52-1); int myCard=(int)myRandomCard; System.out.println(); System.out.println("Guess the number! It can be any whole number from 1-52"); Scanner userInput=new Scanner(System.in); //create new scanner object int userInt;//set the user input to a variable boolean win=false;//set the boolean to false int highGuess=52;//set the initial high guess to the max of the range int lowGuess=1;//set the initial low low to the min of the range int userTries=0;//set tries to 0 int playAgain=0; do { while(win==false) { //set up a loop to check if the guess is high, low, or the random value userInt=userInput.nextInt(); userTries++; if(userTries==5){//give the user 5 guesses System.out.println("You've used up all of your guesses! Game Over!"); win=true; break; } if (userInt<myCard){ lowGuess=userInt; System.out.println("Your guess is too low. "); System.out.println("Try again, guess from:"+(userInt+1)+"-"+highGuess); } else if (userInt>myCard){ highGuess=userInt; System.out.println("Your guess is too high. Guess again from:"+lowGuess+"-"+(userInt-1)); } else if (userInt==myCard){ System.out.println("You've guessed the number."); win=true; } } System.out.println("Would you like to play again? \n Press 1 for yes and 2 for no."); playAgain=userInput.nextInt(); }while(playAgain==1); } }