guessing game help

[yasmin k]

yyyyyyyyyy

[zweibieren]

As written, the declarations of min and max are inside the actionPerformed() method.
Their value will be initialized as the declarations are evaluated every time actionPerformed() is called.

They will only be initialized once if the declarations are moved outside, but still within class Action:

static class Action implements ActionListener {

[COLOR="Navy"]    int max, min, guess;
    boolean guessed;
    guessed = false;
    max = 1000;
    min = 0;
    guess = (0 + 1000) / 2;[/COLOR]

    public void actionPerformed(ActionEvent e) {
        ...

[yasmin k]

thank you for your suggestion but
i get the three errors in the follwoing areas highlighted in red,

i tried fixing it by adding open curly braces after "boolean guessed; {" and close curly bracket after " guess = (0 + 1000) / 2;}" which removes all the errors but the functionality does not work properly.

any suggestion again :D



buttonCorrect.addActionListener(a);
panel2.add(label);

}
});

}

static class Action implements ActionListener {

int max, min, guess;
boolean guessed;
guessed = false;
max = 1000;
min = 0;
guess = (0 + 1000) / 2;

public void actionPerformed(ActionEvent e) {


if (e.getSource() == buttonHigh) {
min = guess;

[zweibieren]

Perhaps I shouldn't try to write these things in the middle of the night. :(

Of course it won't compile; I should not have put assignment statements among the class declarations.
Your fix was on the right track, but did not go far enough;
all four assigments would have to be inside the {...}
Another approach is initializers; they are allowed even though they look almost like assignments:

static class Action implements ActionListener {
    int max = 1000, min = 0;     [COLOR=Green]// initializers are legal[/COLOR]
    int guess = (min+max) / 2;
    boolean guessed = false;

    // max = 1000; min = 0;    [COLOR=Red]// assignments are illegal[/COLOR]

    public void actionPerformed(ActionEvent e) {
        ...

[yasmin k]

Thank you soo much for your time and suggestions, you have helped me greatly. ;)