Java Homework (using a for loop to fill an array)

Question

I am only in my second semester of java so there is probably a very simple solution to my problem, but I have been pulling my hair our for hours because of this. The following code is being used to collect user input and store instance's info in an array.

There isn't an opportunity to input anything for the album name. The output is as follows.

Enter 5 songs

Title: title
Author: author1
Interpreter: int1
Year released: 2000
Album: File name: //Why doesn't this collect input and place File name: on the next line.

If anyone could point me in the right direction I would greatly appreciate it. I included the two methods below in case they have something to do with my problems. Thanks for any help you can provide.

    public static void main(String[] args){

    Scanner kybd = new Scanner(System.in);

    Song[] songTest = new Song[5];

    System.out.println("Enter 5 songs\n");


    //TEST
    for(Song x:songTest)
    {
        x = new Song();
        System.out.print("Title: ");
        x.setTitle(kybd.nextLine());
        System.out.print("Author: ");
        x.setAuthor(kybd.nextLine());
        System.out.print("Interpreter: ");
        x.setInterpreter(kybd.nextLine());
        System.out.print("Year released: ");
        x.setYearReleased(kybd.nextInt());
        System.out.print("Album: ");
        x.setAlbum(kybd.nextLine()); //this doesn't allow for input.  it prints "File name:" and skips the user input for an album name.  Also, when I comment out Year released, the problem goes away.
        System.out.print("File name: ");
        x.setFileName(kybd.nextLine());
        System.out.print(x);
        System.out.println();
    }


public void setYearReleased(int y)
{
    if (y>0 && y<2013)
        this.yearReleased = y;
    else
    {
        System.out.print ("This song is not that old");
        this.yearReleased = -5;
    }
}

    public void setAlbum(String a)
{
    this.album = a;
}

I love how setting the release year to 2014 results in 'This song is not that old' being printed on standard output. — sehe
– sehe, Commented Sep 23, 2012 at 19:09

Sébastien Le Callonnec · Accepted Answer · 2012-09-23 19:09:08Z

3

You need to do a kybd.nextLine() after kybd.nextInt() so that the following kybd.nextLine() does not capture the newline used when pressing enter for the nextInt().

answered Sep 23, 2012 at 19:09

Sébastien Le Callonnec

26.9k8 gold badges70 silver badges82 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Kevin Goodenough Over a year ago

Thank you so much for the help. There are some very frustrating rookie mistakes that I run into from time to time. Also I removed the restraint that a song needs to be <2013. Also, a good bit of constructive criticism.

Kevin DiTraglia · Accepted Answer · 2012-09-23 19:09:36Z

2

x.setYearReleased(kybd.nextInt());

does not do anything with the end of the line character and will remain on the line giving the behavior you see. A quick fix could be to just add another ktbd.nextLine(); after this to skip anything else on the line after the next int.

System.out.print("Year released: ");
x.setYearReleased(kybd.nextInt());
kybd.nextLine()
System.out.print("Album: ");
x.setAlbum(kybd.nextLine());

answered Sep 23, 2012 at 19:09

Kevin DiTraglia

26.2k20 gold badges95 silver badges145 bronze badges

Collectives™ on Stack Overflow

Java Homework (using a for loop to fill an array)

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related