[HELP] pls check my JAVA program for errors

[Ryan Sony]

This program is to check if a number is Automorphic (i.e. a number whose square ends in itself. example - 25*25 = 625)

the logic of my program seems to be right to me, but BlueJ does not want to do anything after i enter a number, it does nothing.

(PS ima noob so pls dont overwhelm me with complicated crap)

 

here's the program:

import java.io.*;
public class AutomorphicNumber {  //start of class 
	public static void main(String[] args)throws IOException {  //main method starts
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in)); 
		System.out.println("Enter number to check Automorphic or not : "); 
		int n = Integer.parseInt(br.readLine()); //taking user input
		int temp=n;
		int d=1;
		while(temp>0){    //while loop to count number of digits and form multiple of 10 to do modulus  division
			temp=temp%10; 
			d=d*10;       
		}     //loop ends            
		if ((n*n) % d == n)  //get original number from square
			System.out.println("It is an Automorphic Number.");
		else
			System.out.println("It is not an Automorphic Number.");
	}  //end of method
}  //end of class

 

Edited November 20, 2016 by colonel_mortis
Code tags and cleaned up code formatting

[DXMember]

lol...

Since this is Java, your int temp and int n both have reference to the same memory pointer.

And when you exit the following loop - both temp and n values are at 0 (or less)

 

When you first assign temp value, try this instead.

int temp = Integer.valueOf(n);

Yes Java (and C#) are retarded like that... they are built to get rid of manual memory management and pointers, but then introduce shit like this for God knows what reason....

 

good luck and have fun

 

EDIT:

umm... didn't read the OP careful enough, guess your problem is solved down below, but it's still okay to know about this stuff.

[Castdeath97]

I'm too acustomed to Java, but I would guess standard input got stuck or something.

 

Have you tried using a scanner and a nextint like here:

 

https://www.tutorialspoint.com/java/util/scanner_nextint.htm

[colonel_mortis]

Your problem is this loop

		while(temp>0){    //while loop to count number of digits and form multiple of 10 to do modulus  division
			temp=temp%10; 
			d=d*10;       
		}     //loop ends  

The modulo operator, %, returns the remainder when divided, so 5 % 10 = 5. This means that you have an infinite loop.

To fix it, you need to use / rather than %, so it will divide by 10 each time (and it uses integer division since the arguments are integers, so it rounds down).

[Ryan Sony]

11 minutes ago, DXMember said:

lol...

Since this is Java, your int temp and int n both have reference to the same memory pointer.

And when you exit the following loop - both temp and n values are at 0 (or less)

 

When you first assign temp value, try this instead.

int temp = Integer.valueOf(n);

Yes Java (and C#) are retarded like that... they are built to get rid of manual memory management and pointers, but then introduce shit like this for God knows what reason....

 

good luck and have fun

 

EDIT:

umm... didn't read the OP careful enough, guess your problem is solved down below, but it's still okay to know about this stuff.

yup, its solved. Thanks for answering