Java-Autorennen

[lucien]

Hi wir haben in der Schule zwei Klassen gestellt bekommen. Auto und Autotest.
Bei mir ist es Auto2 und Auto2Test weil die anderen beiden schon vergeben sind.
Ich will folgende Methode aus Auto aufrufen:

public void fahren1(String name,Graphics g){
     fahrer1=name;
     Random rand = new Random();
     int a = rand.nextInt(10); 
     x=x+a;
     if (x>=680){
         g.setColor(Color.RED);
         g.drawString(fahrer1+"hat gewonnen",50,50);}
    }

Und zwar mit einem Knopf:

else if (e.getSource()==bt2) {
          
            auto.fahren1("Steffen",g);
            auto2.fahren2("Matthias",g);
            auto3.fahren3("Felix",g);
            repaint();
        }

(Der Knopf ist in der Klasse Auto2Test)
Er erkennt als zweites weder Graphics noch Graphics g oder was auch immer an. Das mit den Grafiken war vorgegeben, weil wir noch nicht soweit sind.(ich hab Info nicht mal ein halbes Jahr als Fach)
Wenn ich in der Methode für den Knopf neben ActionEvent e noch Graphics g eingebe kommt da class auto is not abstract and does not override abstract...
Wisst ihr wie ich die Methode aufrufen kann?
Danke im Vorraus!

 

[Flown]

Also mit den Codesnippets kann man dir nicht helfen. Poste doch mal den ganzen Code (und wirklich alles!). Vergiss bitte nicht die

 tags

 

[lucien]

Ok hier ist die komplette Auto2 Klasse:

import java.util.Random;
import java.awt.*;

public class Auto2
{
    int x,y;
    int a;
    int b;
    int c;
    int d=1;
    String fahrer1;
    String fahrer2;
    String fahrer3;
    public Auto2(int a, int b)
    {
        
        x = a; y = b;
    }

    public void zeichneAuto(Graphics g) 
    {
        g.setColor(Color.RED);
        g.fillRect(x,y,100,40);
        g.setColor(Color.BLACK);
        g.fillOval(x+10,y+30,30,30);        
        g.fillOval(x+60,y+30,30,30);
//         x=680;
//         y=0;
//         while(d<34){
//          g.setColor(Color.BLACK);
//          g.fillRect(x,y,10,10);
//          x=x+10;
//          g.setColor(Color.WHITE);
//          g.fillRect(x,y,10,10);
//          y=y+10;
//          x=680;
//          g.setColor(Color.WHITE);
//          g.fillRect(x,y,10,10);
//          x=x+10;
//          g.setColor(Color.BLACK);
//          g.fillRect(x,y,10,10);
//          y=y+10;
//          x=680;
//          d++;
//         }
//         x=100;
    }
    public void fahren1(String name,Graphics g){
     fahrer1=name;
     Random rand = new Random();
     int a = rand.nextInt(20); 
     x=x+a;
     if (x>=680){
         g.setColor(Color.RED);
         g.drawString(fahrer1+"hat gewonnen",50,50);}
    }
    public void fahren2(String name){
     fahrer2=name;
     Random rand = new Random();
     int b = rand.nextInt(20); 
     x=x+b;
     if (x>=680){
         g.setColor(Color.RED);
         g.drawString(fahrer2+"hat gewonnen",50,50);}
    }
    public void fahren3(String name){
     fahrer3=name;
     Random rand = new Random();
     int c = rand.nextInt(20); 
     x=x+c;
     if (x>=680){
         g.setColor(Color.RED);
         g.drawString(fahrer3+"hat gewonnen",50,50);}
    }
}

Und hier die Auto2Testklasse

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class Auto2Test extends JFrame  implements ActionListener
{
    JButton bt1,bt2,bt3,bt4, bt5;
    MyPanel zeichenflaeche;
    Auto2 auto;
    Auto2 auto2, auto3;
    int a=100;
    int b=300;
    int d=0;
    int r=5;
    public static void main(String[] args) {
        new Auto2Test();
    }
    
    public class MyPanel extends JPanel {
        public void paintComponent(Graphics g) {
            g.setColor(Color.gray);
            g.fillRect(0,0,this.getWidth(),this.getHeight());   
            if (auto!=null) auto.zeichneAuto(g);
            if (auto2!=null) auto2.zeichneAuto(g);
            if (auto3!=null) auto3.zeichneAuto(g);
        }
    
    }
    public Auto2Test() {
        initFrame();
    }
    
    public void initFrame() {
        setLayout(new BorderLayout());
        JPanel panel = new JPanel(new GridLayout(1,3));
        zeichenflaeche = new MyPanel();
        bt1 = new JButton("Erzeuge");
        bt2 = new JButton("Fahren");
      
        bt1.addActionListener(this);
        bt2.addActionListener(this);

        panel.add(bt1);
        panel.add(bt2);
        add("South",panel);
        add("Center",zeichenflaeche);
        setSize(800,600);
        setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
        setVisible(true);
    }
  
    public void actionPerformed(ActionEvent e) {
        if (e.getSource()==bt1) {
             auto = new Auto2(100,200);
             auto2 = new Auto2(100,300);
             auto3 = new Auto2(100,450);
            repaint();
        
    }
        else if (e.getSource()==bt2) {
          
            auto.fahren1("Steffen");
            auto2.fahren2("Matthias");
            auto3.fahren3("Felix");
            repaint();
        }
        
       
    }
}

 

[njans]

Wenn ich in der Methode für den Knopf neben ActionEvent e noch Graphics g eingebe kommt da class auto is not abstract and does not override abstract...

Natürlich, weil die Methode durch ein Interface vorgegeben wird. Du kannst da nicht einfach ihre Signatur ändern. Mir ist auch nicht klar, warum du da ein graphics object mitgeben willst.

 

[lucien]

Damit ich bei der fahren1,2,3 Methode das g.drawString aufrufen kann. Sonst kann ich die ja nicht aufrufen.

 

[njans]

 public void paintComponent(Graphics g) {
g.setColor(Color.gray);
g.fillRect(0,0,this.getWidth(),this.getHeight());
if (auto!=null) auto.zeichneAuto(g);
if (auto2!=null) auto2.zeichneAuto(g);
if (auto3!=null) auto3.zeichneAuto(g);
}

Deine paintComponent Methode kümmert sich doch schon um das Zeichnen der Autos, wenn es welche gibt. Du musst doch nur den state deiner Autos (x,y, etc) ändern und dann repaint aufrufen. Warum fahren() überhaupt ein graphics Objekt hat, ist mir weiterhin schleierhaft.

Überlasse das Zeichnen der zeichneAuto() Methode und ändere einfach nur den Zustand deiner Auto-Objekte so, dass das gewünschte gezeichnet wird.

 

[lucien]

Die Fahren Methode hat eine Grafik damit angezeigt wird wer das Rennen gewonnen hat. Das kann ja nicht schon da stehen wenn ich auf Erzeuge klicke. Es geht ja nicht darum, dass das Auto gezeichnet wird sondern um den Text:"Felix hat gewonnen".

 

[JavaMeister]

Und warum soll das Auto wissen, wie man einen Text zeichnet?

Vor allem dass das auto überhaupt gewonnen hat.

Kann es sein, dass hier ein Designfehler vorliegt?

 

[lucien]

Man hat gewonnen wenn x>680 ist. Das macht ja die if-Abfrage. Und dann soll drawString ja ausgeben wer gewonnen hat. Oder geht das auch mit println?

 

[njans]

Hier liegen massive Design-Fehler vor. Das ganze Programm ist unstrukturiert. Das kann man schon daran sehen, dass fahren() und zeichnen() beide sich um das Zeichen des Autos kümmern sollen.

Dir Fehlt hier eine klare Kontrolle, wer für was zuständig ist.
Problem bei dir ist ebenfalls: Wenn ein Auto gewinnt, wissen das die anderen nicht: Das Spiel geht immer noch weiter.
Sinnvoller wäre es eine Zwischenkomponente zu haben, die sich um das Spiel kümmert und der GUI etwas zum Rendern gibt (Bild des Spiels) und überprüft, ob ein Auto im Ziel ist. Wenn das so ist, darauf reagiert und das Spiel beendet.

 

[lucien]

Ok ich hab jetzt meine Auto-Klassse geändert.

import java.util.Random;
import java.awt.*;

public class Auto2
{
    int x,y;
    int a;
    int b;
    int c;
    int d=1;
    int e=1;
    String fahrer1;
    String fahrer2;
    String fahrer3;
    public Auto2(int a, int b)
    {
        
        x = a; y = b;
    }

    public void zeichneAuto(Graphics g) 
    {
        if(e==1){
            g.setColor(Color.RED);
            g.fillRect(x,y,100,40);
            g.setColor(Color.BLACK);
            g.fillOval(x+10,y+30,30,30);        
            g.fillOval(x+60,y+30,30,30);
            //         x=680;
            //         y=0;
            //         while(d<34){
            //          g.setColor(Color.BLACK);
            //          g.fillRect(x,y,10,10);
            //          x=x+10;
            //          g.setColor(Color.WHITE);
            //          g.fillRect(x,y,10,10);
            //          y=y+10;
            //          x=680;
            //          g.setColor(Color.WHITE);
            //          g.fillRect(x,y,10,10);
            //          x=x+10;
            //          g.setColor(Color.BLACK);
            //          g.fillRect(x,y,10,10);
            //          y=y+10;
            //          x=680;
            //          d++;
            //         }
            //         x=100;
            e++;
        }
        g.setColor(Color.YELLOW);
        if (e==3){
            
            g.drawString(fahrer1+"Hat gewonnen",50,50);
        }
         if (e==4){
         g.drawString(fahrer2+"Hat gewonnen",50,50);
        }
         if (e==5){
         g.drawString(fahrer3+"Hat gewonnen",50,50);
        }
    }
    public void fahren1(String name){
     fahrer1=name;
     Random rand = new Random();
     int a = rand.nextInt(20); 
     x=x+a;
     if (x>=680){
         e=3;
        zeichneAuto(g);}
    }
    public void fahren2(String name){
     fahrer2=name;
     Random rand = new Random();
     int b = rand.nextInt(20); 
     x=x+b;
     if (x>=680){
         e=3;
        zeichneAuto(g);}
    }
    public void fahren3(String name){
     fahrer3=name;
     Random rand = new Random();
     int c = rand.nextInt(20); 
     x=x+c;
     if (x>=680){
          e=3;
        zeichneAuto(g);}
    }
}

Problem: Ich kann immer noch nicht die "Grafikmethode" also zeichneAuto aufrufen. Wie gesagt habe ich nicht mal seid einem halbem Jahr Info und ein Teil des Programms wurde von meinem Lehrer vorgegeben.