Thread: help with my object and code

Originally Posted by Norm

Can you make a small, simple program that compiles, executes and shows the problem?
I don't understand what you are saying is the problem.

Note: java programming conventions say that variable names should start with lowercase letters. Class names with uppercase letters.
That makes it easier to read and understand code.

thanks for replying sir
attachment is there to download its in net-beans.

it works just want to know about the commented code in FourthActionPerformend() method
thanks for telling me about objects i have changed the name to small letters.
but my concern is why i cant use object it self rather using this in that method.

 
private void FourthActionPerformed(java.awt.event.ActionEvent evt) {                                       
        // TODO add your handling code here:
 
        //FrameExample FrmExmp1 = new FrameExample();
        //FrmExmp1.getContentPane().setBackground(Color.red);
 
        if (counter == 0) {
            this.getContentPane().setBackground(Color.blue);
            counter++;
        } else {
            if (counter == 1) {
                this.getContentPane().setBackground(Color.cyan);
                counter--;
            }
        }
        ShowText.setForeground(Color.red);
        ShowText.setText(inputtext.getText());
        ShowText.setFont(new Font("",Font.BOLD, 24));
 
    }

the commented code dont work why?

Originally Posted by basitdogar

thanks for replying sir
attachment is there to download its in net-beans.

it works just want to know about the commented code in FourthActionPerformend() method
thanks for telling me about objects i have changed the name to small letters.
but my concern is why i cant use object it self rather using this in that method.

 
private void FourthActionPerformed(java.awt.event.ActionEvent evt) {                                       
        // TODO add your handling code here:
 
        //FrameExample FrmExmp1 = new FrameExample();
        //FrmExmp1.getContentPane().setBackground(Color.red);
 
        if (counter == 0) {
            this.getContentPane().setBackground(Color.blue);
            counter++;
        } else {
            if (counter == 1) {
                this.getContentPane().setBackground(Color.cyan);
                counter--;
            }
        }
        ShowText.setForeground(Color.red);
        ShowText.setText(inputtext.getText());
        ShowText.setFont(new Font("",Font.BOLD, 24));
 
    }

the commented code dont work why?

Hi,

The commented-out code doesn't work because it's changing the color of a newly-created FrameExample object instead of the actual FrameExample object that you're seeing on the screen. It's basically making an invisible clone and changing that clone's color behind the scenes where nobody can see it.

In the code that works, 'this.getContentPane()' and simply 'getContentPane()' are equivalent. You only need the 'this' keyword when there's ambiguity (an anonymous sublcass that has a function with the same name which you don't want to call, or a function argument named the same as a class member, things like that.)

The logic simply adds 1 to 'counter', then removes 1 from 'counter', and changes the background depending on whether 'counter' is 1 or 0. Since you appear to only ever have two states (blue or cyan), you might consider re-factoring to use a boolean instead of an int. It'll still work either way, of course, but it's always a good idea to use the right tool for the job.

Since 'ShowText' is a variable, not a class, it should be named 'showText' as Norm suggested. And 'inputtext' should be 'inputText'. It makes no functional difference, but it helps a lot with read-ability.

BinaryDigit09 and Norm
both you are really helping as i am a self taut student i feel really blessed to have people like you.

BinaryDigit09
your reply does make a lot of help to me and am not confused about it any more. the way you tell me to name them or write them. thank you very much.
thanks a lot i will pray for you to be successful in your life. thanks a lot Sir.