SOLVED

remove e.printStackTrace();
you are printing unnecessary stacktraces

 

It is not unnecessary if you want to debug to figure out what is going wrong — the parseInt method throws a NumberFormatException if the string does not contain a parsable integer.

 

@sistem21 you should be doing your own work. What's wrong with the current snippet you posted? Post the error if there is any.

 

Code (Text):

    public boolean isInt(String str)
    {
    try{
          Integer.parseInt(str);
        }catch(NumberFormatException e){
            e.printStackTrace();
          return false;
        }
    return false;
    }

Shouldn't it return true since it is a int. Currently it will always return false.

 

Code (Text):

public static boolean isInteger(String s) {
    try {
        Integer.parseInt(s);
    } catch(NumberFormatException e) {
        return false;
    } catch(NullPointerException e) {
        return false;
    }
    // only got here if we didn't return false
    return true;
}

 

Surely he would be checking the arguments of the commands so it couldn't be null surely?

 

It is about the return statements — his method always returns false, regardless whether the integer successfully parses or not.

 

I pointed that out in my first post I just didn't change I just pointed him/her in the correct direction.

 

Yes, and that would be significantly better than catching a null pointer.

 

What does it do, what should it be doing, any errors that you can post?

 

Debug your code.

 

Log messages to the console, see what code is being executed and what not.

 

Also show what you're currently using.