Pytanie odnośnie dostępu do składowych klasy

0

Bawię się kodem z tutoriala Suna odnośnie rysowania w Javie ( http://download.oracle.com/javase/tutorial/uiswing/painting/refining.html )

Chciałbym zmienić przykładowo wartość xPos na naciśnięcie przycisku:

class RedSquare{

    private int xPos = 50;
    private int yPos = 50;
    private int width = 20;
    private int height = 20;

    public void setX(int xPos){
        this.xPos = xPos;
    }

[...]

  private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {
  redSquare.setX(5);
    }

Niestety netbeans wpluwa błąd, nie znajdując obiektu redSquare. Sam redSquare jest tworzony tak:

class MyPanel extends JPanel {

    RedSquare redSquare = new RedSquare();
[...]

MyPanel to klasa po której dziedziczy mój jPanel tworzony już metodą drag&drop w okienku design formy NewJFrame, więc wydawałoby się, że redSquare zostanie stworzony razem z GUI czyli tu:

 public static void main(String[] args) {
     new NewJFrame().setVisible(true);
    }

Niestety tak jak mówiłem netbeans nie znajduje redSquare. Gdzie w toku mojego myślenia znajduje się błąd?

edit. Oczywiście można rozwiązać problem tworząc X jako zmienną statyczną i usunąć metodę setX ale to rozwiązanie dość nieeleganckie.

0
  private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {
  redSquare.setX(5);
    }

Ty masz ten kawałek kodu wewnątrz RedSquare? poyebało?

0

Wydawało mi się tak oczywiste, że tego tam ma nie być, że z lenistwa nie zrobiłem osobnego bloku code. Ten fragment kodu jest tam gdzie tworzy go netbeans, Red Square jest w osobnym pliku. Kod w założeniu bardzo podobny do pierwszego posta:

package javaapplication11;
public class Main {

    public static void main(String[] args) {
         new NewJFrame().setVisible(true);
    }
} 
package javaapplication11;
public class zmienna {
    int x = 1;
}
package javaapplication11;
public class NewJFrame extends javax.swing.JFrame {
     zmienna var = new zmienna();
    /** Creates new form NewJFrame */
    public NewJFrame() {
        initComponents();
    }
[.......]                   
    private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {                                         
        // TODO add your handling code here:
      var.x = 2;
    }  
     
 

Działa bez problemu.

EDIT:

Zadam pytanie inaczej :D Jak mając kod z oficjalnego tutoriala zmienić wartość xPos w redSquare?:

package painting;
import javax.swing.SwingUtilities;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.BorderFactory;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseMotionListener;
import java.awt.event.MouseMotionAdapter;

public class SwingPaintDemo4 {

    public static void main(String[] args) {

        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
               createAndShowGUI();
                
            }
        });
    }

   private static void createAndShowGUI() {
        System.out.println("Created GUI on EDT? "+
        SwingUtilities.isEventDispatchThread());
       JFrame f = new JFrame();

        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.add(new MyPanel());
        //f.setSize(250,250);
        f.setVisible(true);
    }

}

class MyPanel extends JPanel {

    RedSquare redSquare = new RedSquare();

    public MyPanel() {

        setBorder(BorderFactory.createLineBorder(Color.black));

        addMouseListener(new MouseAdapter(){
            public void mousePressed(MouseEvent e){
                moveSquare(e.getX(),e.getY());
            }
        });

        addMouseMotionListener(new MouseAdapter(){
            public void mouseDragged(MouseEvent e){
                moveSquare(e.getX(),e.getY());
            }
        });

    }

    private void moveSquare(int x, int y){

        // Current square state, stored as final variables
        // to avoid repeat invocations of the same methods.
        final int CURR_X = redSquare.getX();
        final int CURR_Y = redSquare.getY();
        final int CURR_W = redSquare.getWidth();
        final int CURR_H = redSquare.getHeight();
        final int OFFSET = 1;

        if ((CURR_X!=x) || (CURR_Y!=y)) {

            // The square is moving, repaint background
            // over the old square location.
            repaint(CURR_X,CURR_Y,CURR_W+OFFSET,CURR_H+OFFSET);

            // Update coordinates.
            redSquare.setX(x);
            redSquare.setY(y);

            // Repaint the square at the new location.
            repaint(redSquare.getX(), redSquare.getY(),
                    redSquare.getWidth()+OFFSET,
                    redSquare.getHeight()+OFFSET);
        }
    }

    public Dimension getPreferredSize() {
        return new Dimension(250,200);
    }

    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.drawString("This is my custom Panel!",10,20);

        redSquare.paintSquare(g);
    }
}

class RedSquare{

    private int xPos = 50;
    private int yPos = 50;
    private int width = 20;
    private int height = 20;

    public void setX(int xPos){
        this.xPos = xPos;
    }

    public int getX(){
        return xPos;
    }

    public void setY(int yPos){
        this.yPos = yPos;
    }

    public int getY(){
        return yPos;
    }

    public int getWidth(){
        return width;
    }

    public int getHeight(){
        return height;
    }

    public void paintSquare(Graphics g){
        g.setColor(Color.RED);
        g.fillRect(xPos,yPos,width,height);
        g.setColor(Color.BLACK);
        g.drawRect(xPos,yPos,width,height);
    }
}

Wybaczcie jeżeli pytanie nadaje się do przedszkola ale Javą bawie się od niedawna :P

0
redSquare.setX(...);
0

Jeszcze jedno, jeżeli nie chcesz ustalać rozmiaru okna (f.setSize(250,250);), to zastąp tę instrukcję przez f.pack();.

0
bo napisał(a)
redSquare.setX(...);

I właśnie tu pojawia się problem. Jeżeli stworzę nowy JFrame o nazwie NewJFrame i zastąpię:

JFrame f = new JFrame();

na
<

NewJFrame f = new NewJFrame();

NewJForm zawiera oprócz panelu typu MyPanel() przycisk który ma wykonać redSquare.setX(...);, i właśnie w miejscu gdzie ustalam akcje przycisku wywala błąd że package redSquare does not exist.

0

Znam tylko fragmentu Twojego kodu, więc zgaduję co zrobiłeś źle.
W klasie NewJFrame zdefiniuj pole typu MyPanel, utwórz panel w taki sposób

   panel=new MyPanel();
//a w obsłudze przycisku
   panel.redSquare.setX(...);
0

Zrobiłem w ten sposób, a przynajmniej tak mi się wydaje. Kliknąłem na mój panel w oknie design->customize code, wybieram custom creation i wpisuje:

jPanel1 = new MyPanel();

a w przycisku

 private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {                                         
                   jPanel1.redSquare.setXpos(5);
    }               

I nic. Dalej nie wie co to jest redSquare.

wrzucam cały kod pliku w którym trzymam GUI. Reszta kodu jest po prostu skopiowana z tutoriala z linka w pierwszym poście, tylko JFrame domyślny zmieniłem na NewJFrame którego kod wrzucam.

package painting;
public class NewJFrame extends javax.swing.JFrame {
   // RedSquare redSquare = new RedSquare();

    /** Creates new form NewJFrame */
    public NewJFrame() {
        initComponents();
    }
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        jPanel1 = new MyPanel();
        jButton1 = new javax.swing.JButton();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jPanel1.setBackground(new java.awt.Color(255, 255, 255));

        javax.swing.GroupLayout jPanel1Layout = new javax.swing.GroupLayout(jPanel1);
        jPanel1.setLayout(jPanel1Layout);
        jPanel1Layout.setHorizontalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 423, Short.MAX_VALUE)
        );
        jPanel1Layout.setVerticalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 282, Short.MAX_VALUE)
        );

        jButton1.setText("jButton1");
        jButton1.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                jButton1ActionPerformed(evt);
            }
        });

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, layout.createSequentialGroup()
                .addGap(20, 20, 20)
                .addComponent(jButton1)
                .addGap(48, 48, 48)
                .addComponent(jPanel1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                .addGap(37, 37, 37))
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGap(44, 44, 44)
                .addComponent(jPanel1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                .addGap(59, 59, 59))
            .addGroup(layout.createSequentialGroup()
                .addGap(72, 72, 72)
                .addComponent(jButton1)
                .addContainerGap(290, Short.MAX_VALUE))
        );

        pack();
    }// </editor-fold>                        
    private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {                                         
                   jPanel1.redSquare.setX(5);

    }                                        
    // Variables declaration - do not modify                     
    private javax.swing.JButton jButton1;
    private javax.swing.JPanel jPanel1;
    // End of variables declaration                   

}
0

Jaja sobie robisz?

   // RedSquare redSquare = new RedSquare();

I się dziwisz, że redSquare jest nieznane?

0

redSquare w tutorialu jest tworzony tu:

class MyPanel extends JPanel {

    RedSquare redSquare = new RedSquare();

Jak go przeniosę tam, gdzie Ty go znalazłeś, cały MyPanel robi się czerwony.

0

Przed chwilą namieszałem.

   private javax.swing.JPanel jPanel1;

Czy w klasie JPanel jest pole typu RedSquare? Nie ma.
Musi być

   private MyPanel jPanel1;

Nie pytaj mnie, jak to uzyskać. Rzadko korzystam z NetBeans, nigdy nie korzystam z graficznego projektanta wyglądu.

0

Wielkie dzięki. Możesz mi jeszcze powiedzieć po co ta zmienna:

 private javax.swing.JPanel jPanel1;

skoro obiekt typu jpanel1 jest tworzony tu?

 jPanel1 = new MyPanel();
0

Każda zmienna musi być zadeklarowana. Mógłbyś w initComponents() napisać

  MyPanel jPanel1=new MyPanel(); //zamiast
  jPanel1=new MyPanel();

i zrezygnować z deklaracji pola: private MyPanel jPanel1. Ale wtedy jPanel1 byłoby zmienna lokalną metody initComponents() i nie mógłbyś tej zmiennej użyć w żadnej innej metodzie, w szczególności w metodzie jButton1ActionPerformed.

1 użytkowników online, w tym zalogowanych: 0, gości: 1