Various questions

[benn22]

Hi, I am making a movie manager program and I have some questions to ask.

1. Multiple commands

		System.out.println("Enter the three main casts of the movie:");
		movieCast = in.nextLine();
		for (int i = 1; i<3; i++){
			System.out.println(movieCast);
		}

I want to have three main casts for the movie, but this input will only take one name and repeat it three times. For example, if I type "cast1", it will print:

cast1
cast1
cast1

2. Skipping an input (2 Parts)

                System.out.println("Enter the genre of the movie:");
		System.out.println("Action/Drama/Science Fiction/Horror/Martial Arts");
		System.out.println("Note: Press 'Enter' to cancel this option.");
		movieGenre = in.nextLine();
		movieGenre = movieGenre.toLowerCase();
		movieGenreChar = movieGenre.charAt(0);
		switch (movieGenreChar){
			case 'a':
				movieGenre = "Action";
				break;
			case 'd':
				movieGenre = "Drama";
				break;
			case 's':
				movieGenre = "Science Fiction";
				break;
			case 'c':
				movieGenre = "Comedy";
				break;
			case 'h':
				movieGenre = "Horror";
				break;
			case 'm':
				movieGenre = "Martial Arts";
				break;
			case 'o':
				movieGenre = "Other";
				break;
		}

Here, I would like the user to skip this input by pressing the 'Enter' key. I've tried various methods with a do-while loop, but none of them seemed to work well for me. Similarily:

		System.out.println("Enter the rating of the movie (1-5 stars):");
		System.out.println("Note: Enter '-1' to cancel this option.");
		do{
			try{
				movieRating = in.nextInt();
			}catch (InputMismatchException e){
				System.out.println("Please enter an integer from 1-5:");
				in.next();
			}
		}while (movieRating < 1 || movieRating > 5);

In this case, the user should just type '-1' to skip this input. However, even if the user types '89', for instance, the program will just keep on asking for inputs (with white-space).
i.e.

89

67
23
sadas
Please enter an integer from 1-5:
sa
Please enter an integer from 1-5:
34

67

etc.

Thank you.
ben

[Zman3359]

Ok for the first one you are telling it to input one name, and then print that name three times. To fix that move the input inside the for loop. Like this:
Note: i < 3; changes to i <= 3 because we want the loop to execute three times not 2

for(int i = 1; i <= 3 i ++)
{
    System.out.println("Enter the three main casts of the movie:");
    movieCast = in.nextLine();
    System.out.println(movieCast);
}

For the second one what does it do after you press ENTER? The code looks like it should work.

For the third I think that this might work better for the while part of your do-while loop:

while(movieRating >= 1 && movieRating <= 5)

This above method allows the input to be any int from 1 to 5, and it will exit the loop if it is anything else. Also InputMismatchException is only thrown if the type of the input is wrong, so if you put in an 88 it will just exit the loop and go on. You could do something like this to catch numbers that are greater than 5:

System.out.println("Enter the rating of the movie (1-5 stars):");
System.out.println("Note: Enter '-1' to cancel this option.");
do
{
    try
    {
        movieRating = in.nextInt();
    }
    catch (InputMismatchException e)
    {
        System.out.println("Please enter an integer from 1-5:");
	in.next();
    }
    if(moveRating > 5)
    {
        System.out.println("Please enter an integer from 1-5:");
        in.next();
    }
}while (movieRating >= 1);

[Junky]

1. Look closely at your code. I''l pseudocode it for you.

Prompt user
Get input
Loop three times (actually your code only loops twice i = 1 and i = 2)l

2. a) If user enters nothing then you input will be a zero length string. As such this line will generate an IndexOutOfBoundsException.

movieGenreChar = movieGenre.charAt(0);

Perhaps you should check if user input was nothing before doing anything.

2. b) Check the condition in your while loop. Since 89 satisfies the second condition the loop will continue.