while loop is prompting one more word than asked in

Question

/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package controller;

import java.util.Scanner;

/**
 *
 * @author Donovan
 */
public class Controller {

    public Controller() {

        sentenceArray();
}

    private void sentenceArray() {
        int size = 0;
        String word = "";
        int i = 0;


        Scanner input = new Scanner(System.in);
            System.out.println("How many words would you like to enter?");
                size = input.nextInt();
                String [] sentence = new String[size];
                System.out.println("please enter a word.");
                word = input.next();



        while( i < size){

            if( i < size){
                sentence[i] = word;
                i++;
                System.out.println("Please enter a word.");

                word = input.next();

           }//end if
        }//end while

        displayArray(sentence);
    } // end sentence array


    public static void displayArray(String []sentence){
                System.out.print( "Your sentence is: " );

        for (String x : sentence){                
            System.out.print( x  + "\t" );
        }

    }//end displayResults


}//end controller

Answer 1

The problem is that you ask for input after i >= size .

Specifically, these lines

i++;
System.out.println("Please enter a word.");

word = input.next();

And this isn't C, so you can declare you variables in-line with the method calls.

public void sentenceArray() {

    Scanner input = new Scanner(System.in);
    System.out.println("How many words would you like to enter?");
    int size = input.nextInt();
    String[] sentence = new String[size];

    int i = 0;
    while (i < size) {
        System.out.println("please enter a word.");
        String word = input.next();
        sentence[i] = word;
        i++;
    }

    displayArray(sentence);
}

Answer 2

do while loop

I would use a do while loop which tests the condition after the loop body, you don't need an if that matches your loop conition, and you also don't need the temporary String . Something like,

private void sentenceArray() {
    Scanner input = new Scanner(System.in);
    System.out.println("How many words would you like to enter?");
    int size = input.nextInt();
    String[] sentence = new String[size];
    int i = 0;
    do {
        System.out.println("Please enter a word.");
        sentence[i] = input.next();
        i++;
    } while (i < size);
    displayArray(sentence);
}

Finally, I think you want a println() after your loop in displayArray . Like,

public static void displayArray(String[] sentence) {
    System.out.print("Your sentence is: ");

    for (String word : sentence) {
        System.out.printf("%s\t", word);
    }
    System.out.println();
}

Answer 3

As my understanding that you are asking for a redundant "Please enter a word"? If yes, please remove these lines above the while:

System.out.println("please enter a word.");
word = input.next();

Please also check the whole code here:

   public class Controller {

    public Controller() {
        sentenceArray();
    }

    private void sentenceArray() {
        int size = 0;
        int i = 0;

        Scanner input = new Scanner(System.in);
        System.out.println("How many words would you like to enter?");
        size = input.nextInt();
        String [] sentence = new String[size];

        while( i < size){
            System.out.println("Please enter a word.");
            sentence[i] = input.next();
            i++;
            //end if
        }//end while

        displayArray(sentence);
    } // end sentence array


    public static void displayArray(String []sentence){
        if(sentence.length > 0) {
            System.out.print( "Your sentence is: " );

            for (String x : sentence){                
                System.out.print( x  + "\t" );
            }
        }

    }//end displayResults

    public static void main(String[] args) {
        Controller controller = new Controller();
    }
}//end controller

Hope this help.