Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Check if Integer is too great

I am working on an exchange system and the user is setting the price and amount for an exchange.

I want to make sure that the exchange is no greater than the Integer maximum value, but I am running into a problem.

When the amount of the exchange is put to 9 or more, even though I have a check to make sure the number is no greater than the maximum value, it does not work. I have done some debugging and when setting the amount to 9 while the price is 2,147,483,646 (1 less than the max number), it prints out this:

2,147,483,630 - 9

This is my debugging code, what should I add to it to make sure that this does not happen?

public void setPrimaryAmount(int primaryAmount) {
        int price = primaryAmount * this.price;
        System.out.println(Misc.format(this.price * primaryAmount) + " - " + primaryAmount);
        if (price > Integer.MAX_VALUE ||
                price == Integer.MAX_VALUE ||
                price >= Integer.MAX_VALUE ||
                price < 0 || price <= 0 ||
                price++ == Integer.MAX_VALUE) {
            System.out.println("Attempted to set a bad amount.");
            return;
        }
        this.primaryAmount = primaryAmount;
    }

"Attempted to set a bad amount" prints out until you enter an amount >= 9.

like image 395
Tyluur Avatar asked Dec 12 '22 09:12

Tyluur


2 Answers

You can't store a value in an int, and then check to see whether it's too big for an int. Store it in a long instead.

long price = (long)primaryAmount * (long)this.price;
like image 197
Flight Odyssey Avatar answered Dec 13 '22 23:12

Flight Odyssey


In Java 8, one approach would be to use

Math.addExact(int, int);

which according to Javadocs will throw an ArithmeticException if the result overflows an int. In other versions, I would recommend what Flight Odyssey suggested and use longs at least for these types of checks.

like image 41
CrypticStorm Avatar answered Dec 13 '22 22:12

CrypticStorm