[java] MouseHandler - Softwareontwikkeling

vrijdag 12 december 2003 00:00

Acties:

Intrix

Topicstarter

Ik zit nu al 20 minuten dit stukje code op fouten na te lopen en blijf hangen op de MouseHandler, in de constructor van WindingEx blijft de compiler hangen op new MouseHandler. Met de wel bekende error "cannot resolve symbol" nou ben ik dus zoals gezecht al 20 minuten bezig en ik zie de fout niet. Ik heb zelfs alle mogelijk bedenkelijke en zelfs onnodige classes geimporteerd voor de zekerheid maar nog steeds de zelfde fout. Is er nog iemand online die mijn fout in 1x ziet?

(o ja vraag niet het hoe en waarom van de code het is namelijk een oefening uit een boek die ik aan het maken ben)

Java:

import java.applet.Applet;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.awt.geom.*;
import java.io.*;
import java.util.*;

public class WindingEx extends JFrame
{
    myCustomCanvas mc;
    JButton newpath;
    
    public WindingEx()
    {
        super("Winding Examples");
        
        BorderLayout f1 = new BorderLayout();
        Panel uipanel = new Panel();
        newpath = new JButton("new path");
        uipanel.add(newpath);
        
        mc = new myCustomCanvas(this);
        mc.setSize(800,600);
        
        ButtonHandler bhandler = new ButtonHandler(mc);
        MouseHandler mhandler = new MouseHandler(mc);
        
        newpath.addActionListener(bhandler);        
        mc.addMouseListener(bhandler);
        
        this.getContentPane().setLayout(f1);
        this.getContentPane().add(mc,BorderLayout.CENTER);
        this.getContentPane().add(uipanel,BorderLayout.NORTH);
        
        this.setSize(800,600);
        this.show();
        addWindowListener(new WindowEventHandler());
    }// einde constructor
    
    class WindowEventHandler extends WindowAdapter
    {
        public void windowClosing(WindowEvent e)
        {
            system.exit(0);
        }
    }// einde innerclasse eventhandler
    
    public static void main(String[] args)
    {
        new WindingEx();
    }// einde main 
    
}// einde class windingEx

class MouseHandler implements MouseListener
{
    myCustomCanvas mc;
    
    public void mousePressed(MouseEvent e)
    {
        mc.drawPoint(e.getX(),e.getY());
    }
}// einde class mousehandler

class ButtonHandler implements ActionListener
{
    myCustomCanvas mc;
    
    public ButtonHandler(myCustomCanvas mc)
    {
        this.mc = mc;
    }
    
    public void actionPerformed(ActionEvent e)
    {
        mc.generateGp();
    }
}// einde class buttonhandler

class myCustomCanvas extends Canvas
{
    WindingEx wex;
    String insider;
    String even_oddMessage = "click on a point";
    String non_zeroMessage = " ";
     Random r;
    GeneralPath gp;
    
    public myCustomCanvas(WindingEx wex)
    {
        r = new Random();
        this.wex = wex;
        this.setSize(800,600);
        
        generateGp();   
    }// constructor
    
    public void generateGp()
    {
        gp = new GeneralPath();
        gp.moveTo(r.nextInt(this.getSize().width),
                  r.nextInt(this.getSize().height));
        for (int i=1; i<10; i++)
        {
            gp.lineTo(r.nextInt(this.getSize().width),
                      r.nextInt(this.getSize().height));
        }
        
        gp.closePath();
        gp.drawPoint(r.nextInt(this.getSize().width),
                     r.nextInt(this.getSize().height));
        repaint();
    }// nieuw path
    
    public void drawPoint(int x, int y)
    {
        this.x = x;
        this.y = y;
        
        gp.setWindingRule(GeneralPath.WIND_EVEN_ODD);
        even_oddMessage = "EVEN_ODD RULE: ".concat(isInside(x,y));
        gp.setWindingRule(GeneralPath.WIND_NON_ZERO);
        non_zeroMessage = "NON_ZERO RULE: ".concat(isInside(x,y));
        
        repaint();
    }// punt tekenen
    
    public String isInside(int x, int y)
    {
        if (gp.contains(new Point(x,y)))
            insider="INSIDE";
        else
            insider="OUTSIDE";
        
        return insider;
    }// Inside or not in tekst terug
    
    public void paint(Graphics g)
    {
        Graphics2D g2d = (Graphics2D) g;
        g2d.drawString(even_oddMessage,440,80);
        g2d.drawString(non_zeroMessage,440,100);
        g2d.setColor(Color.blue);
        g2d.fill(new Rectangle2D.Double(x,y,5,5));
        g2d.setColor(Color.red);
        
        float dash [] = {5.5f};
        
        BasicStroke stk = new BasicStroke(4.0f,
                                          BasicStroke.CAP_BUTT,
                                          BasicStroke.JOIN_MITER,
                                          10.f, dash, 2.0f);
        g2d.setStroke(stk);
        g2d.draw(gp);
    }
    
    
}

welcome my son, welcome to the machine

vrijdag 12 december 2003 00:16

Acties:

intrix

Intrix

Topicstarter

regel 28 cannot resolve symbol -> als ik die tot commentaar verhef dan gaat
regel 21 logische wijs ook niet door

maar dan

regel 112 daar staat gp.drawPoint( ) dat klopt niet volgens mij (en volgens de compiler) gezien het feit de functie niet in gp staat maar in myCustomCanvas.

na dat gewijzigt te hebben geeft de compiler

regel 118 daar staat this.x = x; daar was ik het dus vanaf ik begon te lezen al niet mee eens gezien x nergens gedefineerd wordt kan iemand mij dit uitleggen waarom de schrijvers van het boek dat dan toch zo willen doen? (of zou er sprake zijn van een drukfout ?)

Na deze fouten zegt de compiler nog dat ik class moushandler abstract moet defineren omdat ik de methode mouseclicked niet overschrijf. Nou dit ga ik dan maar eerst uit zoeken zie toevallig in het boek "..." staan in class mousehandler wat zou betekenen dat daar iets moet staan wat niet meer dan logisch is.

lekker is dat zo'n code die niet helemaal af is / werkt wil je er wat van leren door hem over te tikken en dan regel voor regel leren begrijpen moet je eerst 100 fouten er uit vissen ofzo

[ Voor 30% gewijzigd door intrix op 12-12-2003 00:19 ]

welcome my son, welcome to the machine

vrijdag 12 december 2003 00:19

Acties:

Verwijderd

Ehmmm....je moet die klasse MouseHandler voor de klasse WindingEx kopieren. Tevens staat er dat de klasse MouseHandler MouseListener implementeert, maar je implementeert maar één van de methoden. Zo zou het wel moeten gaan:

code:

import java.applet.Applet; 
import javax.swing.*; 
import java.awt.*; 
import java.awt.event.*; 
import java.awt.geom.*; 
import java.io.*; 
import java.util.*; 

class MouseHandler implements MouseListener 
{ 
    myCustomCanvas mc;

    public void mousePressed(MouseEvent e)
    { 
        mc.drawPoint(e.getX(),e.getY());
    }
    
    public void mouseClicked(MouseEvent e) {}
    public void mouseEntered(MouseEvent e) {}
    public void mouseExited(MouseEvent e) {}
    public void mouseReleased(MouseEvent e) {}
}// einde class mousehandler

public class WindingEx extends JFrame 
{ 
    myCustomCanvas mc; 
    JButton newpath; 

    public WindingEx() 
    { 
        super("Winding Examples"); 
         
        BorderLayout f1 = new BorderLayout(); 
        Panel uipanel = new Panel(); 
        newpath = new JButton("new path"); 
        uipanel.add(newpath); 
         
        mc = new myCustomCanvas(this); 
        mc.setSize(800,600); 
         
        ButtonHandler bhandler = new ButtonHandler(mc); 
        MouseHandler mhandler = new MouseHandler(mc); 
         
        newpath.addActionListener(bhandler);         
        mc.addMouseListener(bhandler); 
         
        this.getContentPane().setLayout(f1); 
        this.getContentPane().add(mc,BorderLayout.CENTER); 
        this.getContentPane().add(uipanel,BorderLayout.NORTH); 
         
        this.setSize(800,600); 
        this.show(); 
        addWindowListener(new WindowEventHandler()); 
    }// einde constructor 
     
    class WindowEventHandler extends WindowAdapter 
    { 
        public void windowClosing(WindowEvent e) 
        { 
            system.exit(0); 
        } 
    }// einde innerclasse eventhandler 
     
    public static void main(String[] args) 
    { 
        new WindingEx(); 
    }// einde main  
     
}// einde class windingEx 


class ButtonHandler implements ActionListener 
{ 
    myCustomCanvas mc; 
     
    public ButtonHandler(myCustomCanvas mc) 
    { 
        this.mc = mc; 
    } 
     
    public void actionPerformed(ActionEvent e) 
    { 
        mc.generateGp(); 
    } 
}// einde class buttonhandler 

class myCustomCanvas extends Canvas 
{ 
    WindingEx wex; 
    String insider; 
    String even_oddMessage = "click on a point"; 
    String non_zeroMessage = " "; 
     Random r; 
    GeneralPath gp; 
     
    public myCustomCanvas(WindingEx wex) 
    { 
        r = new Random(); 
        this.wex = wex; 
        this.setSize(800,600); 
         
        generateGp();    
    }// constructor 
     
    public void generateGp() 
    { 
        gp = new GeneralPath(); 
        gp.moveTo(r.nextInt(this.getSize().width), 
                  r.nextInt(this.getSize().height)); 
        for (int i=1; i<10; i++) 
        { 
            gp.lineTo(r.nextInt(this.getSize().width), 
                      r.nextInt(this.getSize().height)); 
        } 
         
        gp.closePath(); 
        gp.drawPoint(r.nextInt(this.getSize().width), 
                     r.nextInt(this.getSize().height)); 
        repaint(); 
    }// nieuw path 
     
    public void drawPoint(int x, int y) 
    { 
        this.x = x; 
        this.y = y; 
         
        gp.setWindingRule(GeneralPath.WIND_EVEN_ODD); 
        even_oddMessage = "EVEN_ODD RULE: ".concat(isInside(x,y)); 
        gp.setWindingRule(GeneralPath.WIND_NON_ZERO); 
        non_zeroMessage = "NON_ZERO RULE: ".concat(isInside(x,y)); 
         
        repaint(); 
    }// punt tekenen 
     
    public String isInside(int x, int y) 
    { 
        if (gp.contains(new Point(x,y))) 
            insider="INSIDE"; 
        else 
            insider="OUTSIDE"; 
         
        return insider; 
    }// Inside or not in tekst terug 
     
    public void paint(Graphics g) 
    { 
        Graphics2D g2d = (Graphics2D) g; 
        g2d.drawString(even_oddMessage,440,80); 
        g2d.drawString(non_zeroMessage,440,100); 
        g2d.setColor(Color.blue); 
        g2d.fill(new Rectangle2D.Double(x,y,5,5)); 
        g2d.setColor(Color.red); 
         
        float dash [] = {5.5f}; 
         
        BasicStroke stk = new BasicStroke(4.0f, 
                                          BasicStroke.CAP_BUTT, 
                                          BasicStroke.JOIN_MITER, 
                                          10.f, dash, 2.0f); 
        g2d.setStroke(stk); 
        g2d.draw(gp); 
    } 
     
     
}

(Helaas heb ik geen idee hoe jij de sources zo mooi in jouw berichtje krijgt

)

Als alternatief zou je er ook voor kunnen kiezen om MouseHandler MouseAdapter te laten extenden, zodat je niet al die lege methoden hoeft de implementeren.

Edit: effe dat code-gedoe aangepast

[ Voor 4% gewijzigd door Verwijderd op 12-12-2003 00:31 ]

vrijdag 12 december 2003 00:27

Acties:

bigben04

Verwijderd schreef op 12 december 2003 @ 00:19:
(Helaas heb ik geen idee hoe jij de sources zo mooi in jouw berichtje krijgt )

Tussen [ code] en [ /code] zetten (zonder de spaties uiteraard).
Zie ook de FAQ

[ Voor 12% gewijzigd door bigben04 op 12-12-2003 00:28 ]

vrijdag 12 december 2003 00:28

Acties:

intrix

Intrix

Topicstarter

Juist dat had ik dus ook net gedaan

Java:

class MouseHandler implements MouseListener
{
    myCustomCanvas mc;
    
    public void mouseClicked(MouseEvent e) {}
    public void mousePressed(MouseEvent e)
    {
        mc.drawPoint(e.getX(),e.getY());
    }
    public void mouseReleased(MouseEvent e) {}
    public void mouseEntered(MouseEvent e) {}
    public void mouseExited(MouseEvent e) {}
    
}// einde class mousehandler

maar waarom moet moushandler er voor en maakt dat voor buttonhandler enzo niet uit dan?

welcome my son, welcome to the machine

vrijdag 12 december 2003 00:32

Acties:

intrix

Intrix

Topicstarter

Verwijderd schreef op 12 december 2003 @ 00:19:
Ehmmm....je moet die klasse MouseHandler voor de klasse WindingEx kopieren. Tevens staat er dat de klasse MouseHandler MouseListener implementeert, maar je implementeert maar één van de methoden. Zo zou het wel moeten gaan:

Als alternatief zou je er ook voor kunnen kiezen om MouseHandler MouseAdapter te laten extenden, zodat je niet al die lege methoden hoeft de implementeren.

test dus net even maar dat klopt dus niet. Vond het al een beetje c logica om iets voor het ander te zetten en dat het dan ineens zou werken. Die MouseHandler knalt er nog steeds uit!

[ Voor 27% gewijzigd door intrix op 12-12-2003 00:37 . Reden: klopt niet! ]

welcome my son, welcome to the machine

vrijdag 12 december 2003 00:40

Acties:

Verwijderd

Hmm...misschien dat ik me daarin vergist heb

Kan je precieze foutmelding (incl regelnummer enzo) eens effe posten?

vrijdag 12 december 2003 00:57

Acties:

riezebosch

Je hoeft die class MouseHandler niet niet voor de andere class te kopieren, want als innerclass moet ie net zo goed werken. Wat je idd wél moet doen is alle functies die je zegt te implementeren (door het implements MouseListener) ook daadwerkelijk implementeren. Al laat je de functies misschien verder leeg, in het geval je er niks mee wilt doen.

edit:

Zie net dat het in je originele code ook geen innerclass is, maar dat ie er alleen onder staat. Maar dat mag idd ook geen probleem zijn in Java.

En de this.x zit misschien in Canvas, welke dus ook in ook in myCustomCanvas zit door de extends Canvas.

[ Voor 30% gewijzigd door riezebosch op 12-12-2003 01:06 ]

Canon EOS 400D + 18-55mm F3.5-5.6 + 50mm F1.8 II + 24-105 F4L + 430EX Speedlite + Crumpler Pretty Boy Back Pack

vrijdag 12 december 2003 01:05

Acties:

intrix

Intrix

Topicstarter

Verwijderd schreef op 12 december 2003 @ 00:40:
Hmm...misschien dat ik me daarin vergist heb

Kan je precieze foutmelding (incl regelnummer enzo) eens effe posten?

28 wordt geselecteerd en de algemeen bekende error msg is "cannot resolve symbol" 'no help availeble'

(in het forum 27)

je kent hem wel de algemene super fout: zo van "ik ken het niet" "misschien een typefout" "een hoofdletter" ofzo, maar ik heb alles al 10x uit alle hoeken bekeken.

Java:

import java.applet.Applet;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.awt.geom.*;
import java.io.*;
import java.util.*;

public class WindingEx extends JFrame
{
    myCustomCanvas mc;
    JButton newpath;
    
    public WindingEx()
    {
        super("Winding Examples");
        
        BorderLayout f1 = new BorderLayout();
        Panel uipanel = new Panel();
        newpath = new JButton("new path");
        uipanel.add(newpath);
        
        mc = new myCustomCanvas(this);
        mc.setSize(800,600);
        
        ButtonHandler bhandler = new ButtonHandler(mc);
        MouseHandler mhandler = new MouseHandler(mc);
        
        newpath.addActionListener(bhandler);        
        mc.addMouseListener(mhandler);
        
        this.getContentPane().setLayout(f1);
        this.getContentPane().add(mc,BorderLayout.CENTER);
        this.getContentPane().add(uipanel,BorderLayout.NORTH);
        
        this.setSize(800,600);
        this.show();
        addWindowListener(new WindowEventHandler());
    }// einde constructor
    
    class WindowEventHandler extends WindowAdapter
    {
        public void windowClosing(WindowEvent e)
        {
            System.exit(0);
        }
    }// einde innerclasse eventhandler
    
    public static void main(String[] args)
    {
        new WindingEx();
    }// einde main 
    
}// einde class windingEx

class MouseHandler implements MouseListener
{
    myCustomCanvas mc;
    
    public void mouseClicked(MouseEvent e) {}
    public void mousePressed(MouseEvent e)
    {
        mc.drawPoint(e.getX(),e.getY());
    }
    public void mouseReleased(MouseEvent e) {}
    public void mouseEntered(MouseEvent e) {}
    public void mouseExited(MouseEvent e) {}
    
}// einde class mousehandler

class ButtonHandler implements ActionListener
{
    myCustomCanvas mc;
    
    public ButtonHandler(myCustomCanvas mc)
    {
        this.mc = mc;
    }
    
    public void actionPerformed(ActionEvent e)
    {
        mc.generateGp();
    }
}// einde class buttonhandler

class myCustomCanvas extends Canvas
{
    WindingEx wex;
    String insider;
    String even_oddMessage = "click on a point";
    String non_zeroMessage = " ";
     Random r;
    GeneralPath gp;
    int x,y;
    
    public myCustomCanvas(WindingEx wex)
    {
        r = new Random();
        this.wex = wex;
        this.setSize(800,600);
        
        generateGp();   
    }// constructor
    
    public void generateGp()
    {
        gp = new GeneralPath();
        gp.moveTo(r.nextInt(this.getSize().width),
                  r.nextInt(this.getSize().height));
        for (int i=1; i<10; i++)
        {
            gp.lineTo(r.nextInt(this.getSize().width),
                      r.nextInt(this.getSize().height));
        }
        
        gp.closePath();
        drawPoint(r.nextInt(this.getSize().width), r.nextInt(this.getSize().height));
        repaint();
    }// nieuw path
    
    public void drawPoint(int x, int y)
    {
        this.x = x;
        this.y = y;
        
        gp.setWindingRule(GeneralPath.WIND_EVEN_ODD);
        even_oddMessage = "EVEN_ODD RULE: ".concat(isInside(x,y));
        gp.setWindingRule(GeneralPath.WIND_NON_ZERO);
        non_zeroMessage = "NON_ZERO RULE: ".concat(isInside(x,y));
        
        repaint();
    }// punt tekenen
    
    public String isInside(int x, int y)
    {
        if (gp.contains(new Point(x,y)))
            insider="INSIDE";
        else
            insider="OUTSIDE";
        
        return insider;
    }// Inside or not in tekst terug
    
    public void paint(Graphics g)
    {
        Graphics2D g2d = (Graphics2D) g;
        g2d.drawString(even_oddMessage,440,80);
        g2d.drawString(non_zeroMessage,440,100);
        g2d.setColor(Color.blue);
        g2d.fill(new Rectangle2D.Double(x,y,5,5));
        g2d.setColor(Color.red);
        
        float dash [] = {5.5f};
        
        BasicStroke stk = new BasicStroke(4.0f,
                                          BasicStroke.CAP_BUTT,
                                          BasicStroke.JOIN_MITER,
                                          10.f, dash, 2.0f);
        g2d.setStroke(stk);
        g2d.draw(gp);
    }
    
    
}

[ Voor 5% gewijzigd door intrix op 12-12-2003 01:07 . Reden: typevout ]

welcome my son, welcome to the machine

vrijdag 12 december 2003 01:07

Acties:

riezebosch

28 is in mijn ogen lege regel?

edit:

ok, in het forum 27...

[ Voor 37% gewijzigd door riezebosch op 12-12-2003 01:11 ]

Canon EOS 400D + 18-55mm F3.5-5.6 + 50mm F1.8 II + 24-105 F4L + 430EX Speedlite + Crumpler Pretty Boy Back Pack

vrijdag 12 december 2003 01:07

Acties:

intrix

Intrix

Topicstarter

riezebosch schreef op 12 december 2003 @ 01:07:
28 is in mijn ogen lege regel?

dat komt omdat het forum geen blanco openings regel accepteerd

welcome my son, welcome to the machine

vrijdag 12 december 2003 01:08

Acties:

intrix

Intrix

Topicstarter

riezebosch schreef op 12 december 2003 @ 00:57:
Je hoeft die class MouseHandler niet niet voor de andere class te kopieren, want als innerclass moet ie net zo goed werken. Wat je idd wél moet doen is alle functies die je zegt te implementeren (door het implements MouseListener) ook daadwerkelijk implementeren. Al laat je de functies misschien verder leeg, in het geval je er niks mee wilt doen.

edit:

Zie net dat het in je originele code ook geen innerclass is, maar dat ie er alleen onder staat. Maar dat mag idd ook geen probleem zijn in Java.

En de this.x zit misschien in Canvas, welke dus ook in ook in myCustomCanvas zit door de extends Canvas.

maar de compiler zegt dan gewoon "cannot overide not public"

welcome my son, welcome to the machine

vrijdag 12 december 2003 01:12

Acties:

riezebosch

Ehmmm... Volgens mij heb je voor de class MouseHandler nog geen constructor gedefinieerd, en zeker niet een die een myCustomCanvas als argument neemt...

[ Voor 3% gewijzigd door riezebosch op 12-12-2003 01:14 ]

Canon EOS 400D + 18-55mm F3.5-5.6 + 50mm F1.8 II + 24-105 F4L + 430EX Speedlite + Crumpler Pretty Boy Back Pack

vrijdag 12 december 2003 01:14

Acties:

intrix

Intrix

Topicstarter

riezebosch schreef op 12 december 2003 @ 01:12:
Ehmmm... Volgens mij heb je voor de class MouseHandler nog geen constructor gedefinieerd, en zeker niet een die een Canvas als argument neemt...

ga weg?! dat meen je niet was dat alles stomme constructor in MouseHandler zetten?

thnx dat was alles hij werkt damn wat stom zeg!!!

[ Voor 8% gewijzigd door intrix op 12-12-2003 01:16 . Reden: JAHOE ]

welcome my son, welcome to the machine

vrijdag 12 december 2003 11:42

Acties:

Robtimus

me Robtimus no like you

intrix schreef op 12 december 2003 @ 01:14:
[...]

ga weg?! dat meen je niet was dat alles stomme constructor in MouseHandler zetten?

thnx dat was alles hij werkt damn wat stom zeg!!!

Als je alleen een no-arguments constructor wilt die verder niets doet hoef je geen constructor te schrijven. Zodra je echter of speciale initialisaties wilt doen in de constructor of argumenten wilt meegeven tijdens het creeren van een object moet je een constructor schrijven. Eigenlijk vrij basic in OOP.

More than meets the eye
There is no I in TEAM... but there is ME
system specs

vrijdag 12 december 2003 13:43

Acties:

intrix

Intrix

Topicstarter

IceManX schreef op 12 december 2003 @ 11:42:
[...]
Als je alleen een no-arguments constructor wilt die verder niets doet hoef je geen constructor te schrijven. Zodra je echter of speciale initialisaties wilt doen in de constructor of argumenten wilt meegeven tijdens het creeren van een object moet je een constructor schrijven. Eigenlijk vrij basic in OOP.

ja krijg ook java op school dus had moeten weten dat als je
MouseHandler(mc) doet je dat ook in je constructor moet hebben

welcome my son, welcome to the machine

Pagina: 1

Reageer