Here are some good questions to ask before coding. Bonus points for you if you have already thought through this!

If the integer‘s last digit is 0, what should the output be? ie, cases such as 10, 100.

Did you notice that the reversed integer might overflow? Assume the input is a 32-bit integer, then the reverse of 1000000003 overflows. How should you handle such cases?

Throw an exception? Good, but what if throwing an exception is not an option? You would then have to re-design the function (ie, add an extra parameter).

 1 public int reverse(int x) {
 2         int right = 0, newInt = 0;
 3         boolean flag = false;
 4         if(x<0)
 5             flag = true;
 6         while(x!=0){
 7             right = x%10;
 8             newInt = newInt*10+right;
 9             x/=10;
10         }
11         // if(newInt<0)
12         //     return -1;
13         if(flag == true)
14             return newInt;
15         else
16             if(newInt<0)
17             return -1;
18         return newInt;
19     }