Getting Error on Non Number

[Steve Dyke]

Try it figure out why I get this error message:

java.lang.NumberFormatException: For input string: ""

On this code:

    editQualityControlPlanItemListDataModelParms.setOverridepercent(Integer.parseInt((request.getParameter("overridepercent") == null || 
    request.getParameter("overridepercent").isEmpty())?"0":request.getParameter("overridepercent").replaceAll(GenericPrams.QMS_ALLOWED_CHARS_NUMBERS, "")));

[Tim Cooke]

What is GenericPrams.QMS_ALLOWED_CHARS_NUMBERS? A regex expression I assume? What is its value?

[Tim Cooke]

And to ask my next question ahead of time.... What are you expecting getParameter("overridepercent") to yield?

[Tim Cooke]

And finally, what Integer value are you expecting to get out of the whole thing?

[Norm Radder]

What is the value returned by this expression:  request.getParameter("overridepercent")
Add a print statement before the posted statements to show its value.  Be sure to add delimiters around the value in the print statement so blanks will show:  

System.out.println(">" + request.getParameter("overridepercent") + "<");

[Steve Dyke]

Tim Cooke wrote:What is GenericPrams.QMS_ALLOWED_CHARS_NUMBERS? A regex expression I assume? What is its value?

public final static String QMS_ALLOWED_CHARS_NUMBERS = "[^0-9]";

[Steve Dyke]

Tim Cooke wrote:And to ask my next question ahead of time.... What are you expecting getParameter("overridepercent") to yield?

Any number or combination of numbers 0-9

[Steve Dyke]

Tim Cooke wrote:And finally, what Integer value are you expecting to get out of the whole thing?

A number representing a percent.

[Steve Dyke]

I think I have posted this in error.
I just realized I made some adjustments to my code base after the error was generated so the line of code the error says it happened in is not what the current line of code is at now.
Sorry.

[Tim Holloway]

In the interest of readability:

    editQualityControlPlanItemListDataModelParms
                    .setOverridepercent(
                          Integer.parseInt((
                             request.getParameter("overridepercent") == null
                   || 
                    request
                       .getParameter("overridepercent")
                           .isEmpty()) ? "0"
                              :request.getParameter("overridepercent")
                                   .replaceAll(GenericPrams.QMS_ALLOWED_CHARS_NUMBERS, ""
                   )));

I'm afraid that the original not only is likely not to fit on smaller displays, but also our formatter tend to obscure critical text.

Note that if you use an IDE's automated pretty-printer, some (depending on settings) may put the "." operators at the end of the line, not the beginning, but I find the operator-at-the-beginning more informative to me.

As for general nit-picking, I'm certainly one for complex statements, but this is a bit much. Also, there are two reasons why breaking this up into simpler statements is a good idea (indentation or not).

First, if you have to step through the code with a debugger, debuggers step per-statement so a LOT of details will be more or less invisible in a complex statement.

Secondly, if you break the code into smaller sequences, it's easier to capture the intermediate values in case you want to print them out to a debug log.

And, if you are REALLY paranoid, you can always put assertions into the code, although I've never seen anyone use the Java assert the way it is in C/C++. Partly, I think because Java's assert is overly complex. Instead we prefer unit-test assertion methods.

Last, but not least, as I've mentioned elsewhere, I prefer to document the allowable data values that go into code. We could, based on context, assume that the percentage can be blank or a number from 0 to 100, but maybe you work for a firm that demands 100%. Or gets negative percentages.

The statement also seems to try to sanitize data. I'm not so kind. If they cram in non-numeric characters into a numeric field, I'm going to reject the whole request. Who knows what else they messed up?

[Liutauras Vilda]

Tim Holloway wrote:The statement also seems to try to sanitize data. I'm not so kind. If they cram in non-numeric characters into a numeric field, I'm going to reject the whole request. Who knows what else they messed up?

Totally.

@OP
i.e. abc1def4. If possible in your use case, you may can to return 400 Bad Request.

Try this version of formatting (note, I extracted some code to variables and inverted some logic, for better readability(?)):

String overrideValue = request.getParameter("overridepercent");
boolean overrideValueProvided = overrideValue != null && !overrideValue.isEmpty();

editQualityControlPlanItemListDataModelParms.setOverridepercent(
        Integer.parseInt(
                overrideValueProvided
                        ? overrideValue.replaceAll(GenericPrams.QMS_ALLOWED_CHARS_NUMBERS, "")
                        : "0"
        )
);

[Steve Dyke]

Liutauras Vilda wrote:

Try this version of formatting (note, I extracted some code to variables and inverted some logic, for better readability(?)):

Okay I changed my code to:

String overrideValue = request.getParameter("overridepercent");
boolean overrideValueProvided = overrideValue != null && !overrideValue.isEmpty();
		 
editQualityControlPlanItemListDataModelParms.setOverridepercent(
	Integer.parseInt(
		overrideValueProvided
		? overrideValue.replaceAll(GenericPrams.QMS_ALLOWED_CHARS_NUMBERS, "")
		: "0"
	)
);

I am still getting this:

java.lang.NumberFormatException: For input string: ""

This is my client code:

else if($('#overridepercent').val() == null || $.trim($('#overridepercent').val()) == ""){
	return [false,"ATTENTION - Record was not saved. Sample Size Override% cannot be blank(zero is accaptable)."];
}

[Ron McLeod]

What is the value of overrideValue after this is executed?

String overrideValue = request.getParameter("overridepercent");

[Ron McLeod]

Try printing/logging the value:

String overrideValue = request.getParameter("overridepercent");

if (overrideValue == null) {
    System.out.println("overrideValue is null");
} else {
    System.out.println("overrideValue length: " + overrideValue.length());
    System.out.println("overrideValue value: " + "|" + overrideValue + "|");
}

[Steve Dyke]

Ron McLeod wrote:What is the value of overrideValue after this is executed?

String overrideValue = request.getParameter("overridepercent");

The default value for the input is a zero.
It is a space. The user is wiping out the zero using the space bar.
Seems like my client code reads a space as allowed.
How do I check for a space and not allow it?
Would I use a regex in lieu of the jQuery trim method?

else if($('#overridepercent').val() == null || $.trim($('#overridepercent').val()) == ""){
	return [false,"ATTENTION - Record was not saved. Sample Size Override% cannot be blank(zero is accaptable)."];
}

[Ron McLeod]

Steve Dyke wrote:How do I check for a space and not allow it?
Would I use a regex in lieu of the jQuery trim method?

Try using isBlank() rather than isEmpty():

boolean overrideValueProvided = overrideValue != null && !overrideValue.isBlank();

[Steve Dyke]

This does not seem to work either:

else if($('#overridepercent').val() == null || $('#overridepercent').val().replace(/(\S)/g,'') == ""){
	return [false,"ATTENTION - Record was not saved. Sample Size Override% cannot be blank(zero is accaptable)."];
}

[Steve Dyke]

Ron McLeod wrote:

Try using isBlank() rather than isEmpty():

boolean overrideValueProvided = overrideValue != null && !overrideValue.isBlank();

isBlank() does not seem to be a method of Java String class

[Ron McLeod]

On the client side, couldn't you just use the JS trim() method?

[Steve Dyke]

Sorry, I am using Java 8

[Ron McLeod]

Steve Dyke wrote:Sorry, I am using Java 8

overrideValue.trim().isEmpty()

[Steve Dyke]

Ron McLeod wrote:On the client side, couldn't you just use the JS trim() method?

This does not prevent spaces either:

else if($('#overridepercent').val() == null || $('#overridepercent').val().trim() == ""){
    return [false,"ATTENTION - Record was not saved. Sample Size Override% cannot be blank(zero is accaptable)."];
}

[Steve Dyke]

Ron McLeod wrote:
overrideValue.trim().isEmpty()

I will give this a try. Thanks.

[Ron McLeod]

Steve Dyke wrote:This does not prevent spaces either

trim() does work, but I don't know about the jQuery bits through or what else is in your if-else block> var overridepercent = "   "; < undefined > console.log(overridepercent.length); < 3 > console.log(overridepercent.trim().length); < 0 > console.log(overridepercent.trim() == ""); < true edit: added check for == ""

[Tim Holloway]

I have never done any of these. Too complicated/too much overhead.

I do a a simple Integer.valueOf(string) type conversion. If it throws a NumberFormatConversion exception, I reject it. Otherwise I consider it good enough.

If I was really obsessive, I could do a character-by-character isDigit() test, but again, it's not worth the overhead to me.