Baffled by code....

Question

Can anyone help me out here? I've been over this code for the last hour and still can't figure out what's going wrong with it. It's supposed to take integers from 1 to 1000 and find integers that are perfect numbers (A numbers factors added together equal the original number). For some reason it's only outputing the first number and then stops. Any help would be greatly appreciated.
//Java core packages
import java.awt.*;
//Java extension package
import javax.swing.*;
public class Assign3_3097 extends JApplet {
     int FinalSum;     
     public void init()
     //JTextArea to display results
     JTextArea outputArea = new JTextArea();
     //get applet's content pane
     Container container = getContentPane();
     //attach outputArea to container
     container.add( outputArea );
     int result; //store value of call to method
     String output = ""; //String containing results
     for ( int number = 1; number <= 1000; number++)
          result = perfect (number);
          if (result > 0 )
               //append result to String output
               output += result + " is a perfect number\n";          
          }//end for loop
          outputArea.setText( output );//puts results in JTextArea
     } //end method init
     //perfect method declaration
     public int perfect( int x )
          int value;
          for ( int divisor = 1; divisor <= 500; divisor++ ){
          value = x % divisor;
               if ( value == 0 )
                    FinalSum = FinalSum + divisor;
          }//end for loop
          if ( FinalSum == x)
               return x;
          else
               return 0;
     }//end perfect method
} //end class Assign3_3097

Answer 1

hi,
umm, well i can see quite a few mistakes in the code due to which ur program is not working, now as u must be knowing a perfect number is those numbers which are the sum of their positive proper divisors.. for eg
6 = 1 + 2 + 3
now if that is the case, there is a major flaw
for ( int divisor = 1; divisor <= 500; divisor++ ){why would u want to divide the number till 500 ???
should read
for ( int divisor = 1; divisor <  x; divisor++ ){also, the variable
>int FinalSum; which is declared globally must be declared locally for otherwize it will retain itz previous values.
ie
public int perfect( int x )
int value;
int FinalSum = 0;now if you change that ur code will start working fine,
ps: Your program will work faster if you replace the "output" from String type to StringBuffer.
cheerz
ynkrish