String array out of bounds

Question

I am trying to make an array of type string of which the size and strings are inputted by the user and i keep getting error ArrayIndexOutOfBoundsException There is similar threads i found pertaining to this but none of which have solved this error. in my for statement if i take out the equal to and just do less than i receive no errors but i can only input one name less than the number i input. If i leave the code as is everything appears to work as it should minus the error. I understand i am getting this fault because my array is going out of bounds but i cant figure out why. please help! thank you!

package sales;
import compare.*;  
import java.util.*;

public class Sales {
   public static void main(String[] args) {

        Scanner scan=new Scanner(System.in);
        System.out.println("Enter number of employees to compare:");
        int numEmp = scan.nextInt();

        while(numEmp < 2){
            System.out.println("Has to be at least 2 employees:");
            numEmp = scan.nextInt();
        }

        String[] names = new String[numEmp];

        System.out.println("Enter employees name:");
        for(int i=0;i<=names.length;i++){
            names[i]=scan.nextLine();    
        }
    }
}

This particular scenario is precisely explained by the accepted answer in the duplicate. — Oliver Charlesworth, May 14 '17 at 20:29
i am not seeing it there, what solved my issue was adding scan.nextLine(); before my for statement. that is not in the topic you marked as duplicate. — Zachary, May 14 '17 at 20:47
If that works, that's by chance. Accessing `names[names.length]` is invalid - that is the real problem in your code. — Oliver Charlesworth, May 14 '17 at 20:48
even if i change names.length to numEmp i get the same error — Zachary, May 14 '17 at 20:51

dumbPotato21 · Accepted Answer · 2017-05-14T20:30:26.650

0

Change the condition of the for to i<names.length instead of i<=names.length.

scan.nextLine(); //add this
for(int i=0;i<names.length;i++){
    names[i]=scan.nextLine();
}

edited May 14 '17 at 20:30

answered May 14 '17 at 20:26

dumbPotato21

5,669
5
21
34

if i do this my program does not work correctly. If i input number of employees = 2 then when i go to enter names i can only enter one name. – Zachary May 14 '17 at 20:29
@Zachary see my answer now – dumbPotato21 May 14 '17 at 20:30
Awesome, thank you! – Zachary May 14 '17 at 20:32

String array out of bounds

1 Answers1