[robgw3]

I have tried to make a method that will produce 3 random numbers which i can then use to compare 3 numbers entered by the user against. However for some reason i cant seem to be able to use the numbers outside of the method, they work fine and generate properly, but i cant access the variables and trying to use them just returns null.

I know i probably wrote it wrong, but i cant see what i did wrong so could someone please take a look at this and explain where i made the mistake?

public void randomNum() {
        randOne = Integer.toString(random.nextInt(9));
        randTwo = Integer.toString(random.nextInt(9));
        randThree = Integer.toString(random.nextInt(9));
        System.out.println("One: " + randOne + " Two: " + randTwo + " Three: " + randThree);
    }

I call it when i click a button by using classname.randomNum();

[pbl]

As this statement does not generate an error

randOne = Integer.toString(random.nextInt(9));

it means that the String randOne, randTwo and randThree are declared outside the method
so they should be visible to other methods

Post your code and your errors

[robgw3]

Well the thing is that it doesnt give me any errors, well not any major ones anyway. All it does is give me null when i try and use them. I guess they would give me errors if i tried to put them back into code, but when they werent working all i did was put them into a System.out.println to show what was happening with them and thats what shows them as null.

As for my code, this is the code from the class that the random method is in:

import java.util.*;

/**
 *
 * @author Robert Wright
 */
public class Fermi {
    
    private String guessOne;
    private String guessTwo;
    private String guessThree;
    public String randOne;
    public String randTwo;
    public String randThree;
    private ArrayList guesses = new ArrayList();
    private Random random;
    public String num1;
    public String num2;
    public String num3;
    
    public Fermi() {
        guesses.clear();
        random = new Random(); 
        num1 = Integer.toString(2);
        num2 = Integer.toString(2);
        num3 = Integer.toString(6);
    }
    
    
    public void randomNum() {
        randOne = Integer.toString(random.nextInt(9));
        randTwo = Integer.toString(random.nextInt(9));
        randThree = Integer.toString(random.nextInt(9));
        System.out.println("One: " + randOne + " Two: " + randTwo + " Three: " + randThree);
    }
    
    
    public void setGuess1 (String guess1) {
        guessOne = guess1;
    }
    
    public String getGuess1 () {
        return guessOne;
    }
    
    public void setGuess2 (String guess2) {
        guessTwo = guess2;
    }
    
    public String getGuess2 () {
        return guessTwo;
    }
    
    public void setGuess3 (String guess3) {
        guessThree = guess3;
    }
    
    public String getGuess3 () {
        return guessThree;
    }
    
    public void checkResults() {
        
        if (guessOne.equals(num1)) {
            guesses.add("Fermi");
        } else 
        if ((guessOne.equals(num2) && !guessTwo.equals(num2)) || (guessOne.equals(num3) && !guessThree.equals(num3))){ 

                
                // 
            guesses.add("Pico");
        } else {
            guesses.add("Nano");
        }
        
        if (guessTwo.equals(num2)) {
            guesses.add("Fermi");
        } else 
        if ((guessTwo.equals(num1) && !guessOne.equals(num1)) || (guessTwo.equals(num3)&& !guessThree.equals(num3))){
            guesses.add("Pico");
        } else {
            guesses.add("Nano");
        }
        
        if (guessThree.equals(num3)) {
            guesses.add("Fermi");
        } else 
        if ((guessThree.equals(num2) && !guessOne.equals(num1)) || (guessThree.equals(num3) && !guessTwo.equals(num2))){
            guesses.add("Pico");
        } else {
            guesses.add("Nano");
        }
        
        System.out.println("Guess1: " + guessOne + " Guess2: " + guessTwo + " Guess3: " + guessThree);
        System.out.println("ROne: " + num1 + " RTwo: " + num2 + " RThree: " + num3);
        //System.out.println("rOne: " + randOne + " rTwo: " + randTwo + " rThree: " + randThree);
        System.out.println(guesses.get(0) + " " + guesses.get(1) + " " + guesses.get(2));
        //System.out.println(guesses.get(1) + " " + guesses.get(2) + " " + guesses.get(3));
    }
    
        
        
}

[pbl]

and where is the main() that drives all that ?

[robgw3]

    public MainForm() {
        initComponents();
        
        Fermi fermi;
        fermi = new Fermi();
        
        fermi.randomNum();
    }

Thats the only part of the main form that actually deals with the random number, i can post the entire code for it if needed though.

[pbl]

that only calls randomNum() that prints the 3 random numbers as String

[robgw3]

How would i go about fixing that?

[pbl]

Depends what you want to do
There is nothing in your code but:
- who is supposed to call checkResults() and under which condition(s) ?
- same thing for all your getter() and setter() who is supposed to call them ? If they are not used, just remove them, it will confuse the next guy, like me, who try to understand your code

[robgw3]

They are used, i just didnt show them because i didnt see how they affected the random numbers. I have a key released event triggered to the text fields so when someone types into the text fields it checks to see if it's a number between 0-9 and if not it tells them to enter a number. It's also supposed to check if the other two boxes are filled and if they are then it's supposed to enable a button but that doesnt work so i removed that for now. As for the check results, thats done through the button i just mentioned. When the button is clicked it gathers the values from the text fields and uses the setter to send the players guesses. The gets are used in the check results, they work at least. So far the only problems i have left are the random numbers not working outside that and not being able to get the text field checker to work right.

Here is the code for the buttons and text fields as well:

private void btnInstructionsMouseReleased(java.awt.event.MouseEvent evt) {                                              
       Instructions instructions;
       instructions = new Instructions();
       instructions.setVisible(true);
        
    }                                             

    private void btnResetMouseReleased(java.awt.event.MouseEvent evt) {                                       
        Fermi fermi;
        fermi = new Fermi();
        
        txtGuess1.setText("");
        txtGuess1.setEnabled(true);
        txtGuess2.setText("");
        txtGuess2.setEnabled(true);
        txtGuess3.setText("");
        txtGuess3.setEnabled(true);
        txtResults.setText("");
        fermi.randomNum();
    }                                      

    // Checks to see what the user entered in textbox 1
    private void txtGuess1KeyReleased(java.awt.event.KeyEvent evt) {                                      
        String guess1 = txtGuess1.getText();
        String guess2 = txtGuess2.getText();
        String guess3 = txtGuess3.getText();
        TextFieldValidator textfield;
        textfield = new TextFieldValidator();
        //textfield.Validate(guess1);
        if (textfield.Validate(guess1)) {
            JOptionPane.showMessageDialog(null, "You must enter a number between 0-9");
            txtGuess1.setText("");
            txtGuess1.requestFocusInWindow();
        } 
    }                                     

    // Checks to see what the user entered in textbox 2
    private void txtGuess2KeyReleased(java.awt.event.KeyEvent evt) {                                      
        String guess1 = txtGuess1.getText();
        String guess2 = txtGuess2.getText();
        String guess3 = txtGuess3.getText();
        TextFieldValidator textfield;
        textfield = new TextFieldValidator();
        //textfield.Validate(guess1);
        if (textfield.Validate(guess2)) {
            JOptionPane.showMessageDialog(null, "You must enter a number between 0-9");
            txtGuess2.setText("");
            txtGuess2.requestFocusInWindow();
        } 
    }                                     

    
    // Checks to see what the user entered in textbox 3
    private void txtGuess3KeyReleased(java.awt.event.KeyEvent evt) {                                      
        String guess1 = txtGuess1.getText();
        String guess2 = txtGuess2.getText();
        String guess3 = txtGuess3.getText();
        TextFieldValidator textfield;
        textfield = new TextFieldValidator();
        //textfield.Validate(guess1);
        if (textfield.Validate(guess3)) {
            JOptionPane.showMessageDialog(null, "You must enter a number between 0-9");
            txtGuess3.setText("");
            txtGuess3.requestFocusInWindow();
        } 
    }                                     

    private void btnCheckGuessMouseClicked(java.awt.event.MouseEvent evt) {                                           
        if (btnCheckGuess.isEnabled()) {
        Fermi fermi;
        fermi = new Fermi();
        
        fermi.setGuess1(txtGuess1.getText());
        fermi.setGuess2(txtGuess2.getText());
        fermi.setGuess3(txtGuess3.getText());
        fermi.checkResults();
        }
    }    

[pbl]

Are you telling me you are creating a new Fermi object everytime the mouse is clicked ?
What for ? Your random numbers will always be different

[robgw3]

pbl, on 26 March 2012 - 08:57 PM, said:

Are you telling me you are creating a new Fermi object everytime the mouse is clicked ?
What for ? Your random numbers will always be different

Cause it was the only way i could get the data from the text field into the Fermi class. I've only been learning java since Jan.

[pbl]

The constructor of Fermi does not receive the JTextField data as parameter so how can building a new Fermi object would fix your problem ?
How do you transfer your JTextField data into your Fermi object ?

[robgw3]

No but i used the setter to send the JTextField data to the Fermi class, if i didnt have that in there then they wont work either.

Fermi fermi;
        fermi = new Fermi();
        
        fermi.setGuess1(txtGuess1.getText());
        fermi.setGuess2(txtGuess2.getText());
        fermi.setGuess3(txtGuess3.getText());
        fermi.checkResults();

Without the fermi = new Fermi(); in there the fermi. in the following 4 lines says "cannot find symbol" and if i remove the fermi from them then setGuess1 says "cannot find symbol".

[pbl]

You should be able to create a Fermi object in your class that extends JFrame and use that object later on
Creating it in your actionEvent() method will make it vanish as soon as the method exits

class Foo extend JFrame {
   Fermi fermi;

   // constructor
   Foo() {
      fermi = new Fermi();

[robgw3]

Sorry i dont really understand what you mean there. I get what your saying about it vanishing, but not the rest of it.