public  void readArguments(String[]args ) {
         
         
        String [] readArray = new String[ 10 ]; 
        Scanner readscanner=new Scanner(System.in); 
         
        // read values from keyboard into array 
        System.out.println("Enter your arguments or type Exit to exit the loop");
         for ( int i = 0 ; i <  readArray.length ; i ++) { 
                 
                             
                 if(readArray[i] =="Exit" ) break;                                    
                 readArray[i] = readscanner.nextLine();
                 
                 
          }              
              
   //Determine if the number of Arguments is less or greater than 5
               if(readArray.length<=4 ){
                  
                  System.out.println("There are less than 5 arguments passed");
                }
                 else {
                       System.out.println("More than 5 arguments passed");         
              
              
             
                 
        
       } 
    }

First, when you do "if(readArray =="Exit" ) break;", you don't actually check the value of the string, rather the position in the memory. To check the actual value, use readArray.equals("Exit").

Second, you need to place your if statement after your input. As it is now, at your if statement, readArray is null, since you haven't done anything with it yet. So your if-statement will never be true.

commented: Very helpful answer to beginner question. +4

Wow!! thank you very much. its working now.

Member Avatar for hfx642

Another option would be to use a while loop.

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.