Final help with this stupid program

Question

so here is the code of the final greekNumbers program
import java.math.*;
import java.util.*;
import javax.swing.JOptionPane;
public class GreekNumbers {
     private String userInput;
     private int supliedNumber;
     private int numberFactor;
     private int generatedNumber;
     public ArrayList factors;
     private int counter = 1;
     private String message = "Your Number was ";
     private int addedFactors;
     public void Factoring () {
     factors = new ArrayList(1);
          userInput =
               JOptionPane.showInputDialog (null, "Please enter your number.");
               supliedNumber = Integer.parseInt(userInput);
               message += (supliedNumber + ". And " + supliedNumber + " is ");
                    do {
                              numberFactor = supliedNumber / counter;
                              if (numberFactor % 1 == 0){
                                   factors.add (counter);
                                   factors.add (numberFactor);
                                   counter++;
                              else {counter++;}
                    }while (counter <= supliedNumber);
                    counter = 0;
                    do {
                            int a = (Integer) factors.get (counter); 
                              counter++;
                              int b = (Integer) factors.get (counter);
                              addedFactors = a + b;
                              counter++;
                    }while (counter <= factors.size ());
     public void Perfect () { // sees if the number abblied is perfect
          if (addedFactors == supliedNumber){
               message += "Perfect ";
     public void Abundant () { // sees if the number supplied is abundant
          if (addedFactors >= (supliedNumber * 2)){
                         message += "Abundant ";
     public void Deficient () {  // sees if the number supplied is deficient
          if (addedFactors <= (supliedNumber * 2)){
                         message += " Deficient";
     public String sendGreekNumber (){
         return message;// sends main method the greek number analysis.
}and all it does is run directly through the program and pops up with a box that contains my message string instance?
what am i doing wrong here?

Answer 1

DJ-Xen wrote:
here is the main thread
import javax.swing.JOptionPane;
     public class GreekNumbersDriver{
          public static void main (String [] args) {
               GreekNumbers number  = new GreekNumbers ();
               (new GreekNumbers()).sendGreekNumber ();
                    String finalNumber = number.sendGreekNumber ();
                    JOptionPane.showMessageDialog(null, finalNumber);
          }is the problem in there somewhere?Yes, you never call the Factoring method of your GreekNumber object.
Plus you are creating two instances of GreekNumber, why I don't know. There is no need to do this.