Thread: nested array loop

okay so i have this nested loop for an array.
when i run it
so i input the first name then the 5 scores but after the 5 scores it skips the second name and goes to the 5 scores again.
I have a picture below when i run it.

import java.util.Scanner;
 
public class test
{
   public static void main(String[] args)
   {
   String[] names = new String[5];
   int[][] scores = new int[5][5];
 
   String name ="";
 
   Scanner keyboard = new Scanner(System.in);
 
   for (int index=0;index<=4;index++)
   {
      System.out.println("enter name");
      name= keyboard.nextLine();
      name += names[index];
 
      for (int index2 = 0; index2<=4;index2++)
   {
      System.out.println("enter test score");
      scores[index][index2] = keyboard.nextInt();
 
   }
   }
 
 
   }
   }

help.jpg

It will be the scanner bug (not really a bug). You can google it.

After you read a number clear the buffer with;
keyboard.next ();

i dont get it

do i put

 System.out.println("enter test score");
      scores[index][index2] = keyboard.nextInt();
     keyboard.next (); //<<<< like this??

Try it

--- Update ---

Okay, I'm at a computer now and not reading off my phone.

There are two things wrong.

name += names[index];

This is the wrong way to assign the input to the names array.

Also, I have given you a slight bum steer on clearing the buffer. Use;
keyboard.nextLine();

sorry i dont really get what you are trying to say. im new to java programming sorry.

like this?

 for (int index=0;index<=4;index++)
   {
      System.out.println("enter name");
      names[index]= keyboard.nextLine();
 
 
      for (int index2 = 0; index2<=4;index2++)
   {
      System.out.println("enter test score");
      scores[index][index2] = keyboard.nextInt();
 
 
   }

Don't apologise, being new is not something to be sorry for. Ask direct questions if you don't understand something. Did you google the 'Scanner Bug'

That new piece of code will assign the name correctly, now you need to clear the scanner buffer either after you've read all the scores or after you read each score using;

keyboard.nextLine();

GOT it thank you so much!

You're welcome. Great work!

Some Tips;

Class files always start with a capital letter.

Rather than set the value for the array size, could you maybe rewrite it so that you use either constant values such as

final int NUM_OF_PEOPLE = 5;
final int NUM_OF_SCORES = 5;

or you could use user inputted values or a combination of both.

How would you use those to create the array objects?

In your for loops, did you know that you can use the array length as an upper limit for the counter?

index < scores.length; // for outer loop
index2 < scores[index].length; // for the inner loop

That way if your array size changes say by user input, your code will still function. The user could say they wanted to put in 100 people with 3 scores each and the loop will still work providing you create the arrays with those values.

Anyway, enough of that. Good luck with Java.