Identify shapes

[Yossry Helal]

thats what I'm trying to do
If user inputs 3, onscreen message should say "A triangle has 3 sides."  
*     If user inputs 4, onscreen message should say "A rectangle has 4 sides."
*     If user inputs 5, onscreen message should see "A pentagon has 5 sides."

and here is my java code

import java.util.Scanner;

/*
 * 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.
 */

/**
 *
 * @author Yossry
 */
public class week2 {

    /**
     * @param args the command line arguments
     */
   public static void main(String[] args) {
               
        String userInputStringOfAngles;  // Declare a variable of type String to capture user input
        int numberOfAngles; // Declare a variable of type int to hold the converted user input

        Scanner myInputScannerInstance = new Scanner(System.in); // Recognize the keyboard
        System.out.print("Please type the integer press Enter: "); // Prompt the user
        userInputStringOfAngles= myInputScannerInstance.next(); // Capture user input as string
        numberOfAngles = Integer.parseInt(userInputStringOfAngles); // Convert the string to a number in case this will be useful later
        
      
       
       } 
         int numberOfAngles=3;// LINE 1. CODE TO DETERMINE WHETHER USER INPUT IS OUT OF BOUNDS GOES HERE
         String userInputStringOfAngles;
            switch (numberOfAngles){
           case '1' : userInputStringOfAngles="3"
               break;
           case '2': userInputStringOfAngles="4"
               
               break;
           case '5':userInputStringOfAngles="5"
              
               break;
         default :
            System.out.println("Invalid input");
        
        // LINE 2. SWITCH CODE TO PRINT CORRECT "SHAPE" MESSAGE BASED ON USER INPUT GOES HERE
        
           
      }
    }
if (userInputStringOfAngles==3){
 System.out.println("A triangle has 3 sides. ");


};

else if (userInputStringOfAngles==4){
System.out.println("A rectangle has 4 sides");

};
else if (userInputStringOfAngles==5){
 System.out.println("A pentagon has 5 sides");
}
else  System.out.println("Invalid input");
}};

[Carey Brown]

         int numberOfAngles=3;// LINE 1. CODE TO DETERMINE WHETHER USER INPUT IS OUT OF BOUNDS GOES HERE
         String userInputStringOfAngles;
            switch (numberOfAngles){
           case '1' : userInputStringOfAngles="3"
               break;
           case '2': userInputStringOfAngles="4"
                
               break;
           case '5':userInputStringOfAngles="5"
               
               break;
         default :
            System.out.println("Invalid input");
         
        // LINE 2. SWITCH CODE TO PRINT CORRECT "SHAPE" MESSAGE BASED ON USER INPUT GOES HERE
         
            
      }

Why do you have 1, 2, and 5 when your requirements state 3, 4, 5?

[Yossry Helal]

I have changed it into 3,4,5 but still didnt work.

[Carey Brown]

// LINE 1. CODE TO DETERMINE WHETHER USER INPUT IS OUT OF BOUNDS GOES HERE

Assuming that this is part of your requirements, this calls for an 'if' not a 'switch'. Do you know what the correct range is?

[Daniel Demesmaecker]

How weird... When using a switch why would you check if the users input is out of bounds? Add a default, when the user is out of bounds the default message is displayed.
Like Carey already pointed out you're using '1', '2' and '5' in your switch while it should be "3" "4" and "5" and you're comparing a number to a string, so it should be case 3, 4 and 5.
Your if statements are totally useless and use the userInputStringOfAngle which is a string yet they compare it to a number.
To compare strings don't use == but use the .equals("") method. The .equals("") method compares the value of the string, when using == java compares the objects, not the value.
So:

String s = "3"; 
if(s == s) -> true;
if(s == "3") -> false;
if(s.equals("3") -> true

And finally, your variablenames are verry clear, I give you that, but still a bit long for my taste, for example: myInputScannerInstance, whats wrong with input or simple scanner.

To demonstrate what I ment with the default:

Switch(userInputStringOfAngles){
    case "3" : 
        System.out.println("A triangle has 3 sides");
        break;
    case "4":
        System.out.println("A rectangle has for shapes");
        break;
    case "5":
        System.out.println("A pentagon has 5 sides");
        break;
    default:
        System.out.println("You gave an invalid input");
}

I also see you taking in a string just to parse it to a number, if you're sure the user will always put in a number you could take in the int directly.

int nrOfAngles = myInputScannerInstance.nextInt();
//Don't forget to flush the new line afterwords.
myInputScannerInstance.nextLine();

[Campbell Ritchie]

Welcome to the Ranch

Please always tell everybody what the question is about in its title.
Please have a look at your variable names, “week2” is a bad name for a class and it should start with a CapitalLetter, too. I think the name for your Scanner is too long: maybe “input” would be better, but you can probably think of a better name.
Why mess around with parseInt() when your Scaanner can read an int directly. There is an example in its documentation.

[Fred Kleinschmidt]

If you parse numberOfAngles as an int, then your switch statement should use ints, not Strings:
For example,

case "3":

should just be

case 3: