java - Matdorspil - Grafisk programmering - Udvikleren.dk

Tags: java

<< < 1 2 3 > >>

Bruger #15575 @ 17.01.10 15:09

Hej

Så er jeg gået i stå med mit Java igen.

Jeg havde tidligere lavet et matadorspil (med hjælp fra bogen), og nu består opgaven så i at lave det grafisk.

Jeg har fået alle felterne til at blive vist grafisk, men jeg vil gerne have at der står under feltet hvem der ejer det. Jeg synes jeg har prøvet alt, men kan ikke få det til at virke.

I klassen Grund2, har jeg sat nogle udskrivninger ind, for at se hvad der sker. Det viser at metoden bliver kørt igennem, men den ikke vil tegne ejeren til grunden.

Håber i kan hjælpe.

Programmet er delt op i mange klasser, og jeg poster de mest relevante for at spillet kan fungere. Dog i rækkefølge efter relevans. Jeg er næsten sikker på at problemet ligger i MatadorGrafikpanel, Felt og Grund2.

Grafikpanelet:

Kode

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

public class MatadorGrafikpanel extends JPanel{
    
    Matadorspil spil = new Matadorspil();
    
    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);

     
        for(int i = 0; i<spil.felter.size(); i++)
        {
            Felt felt = spil.felter.get(i);

            double v = Math.PI * 2 * i / spil.felter.size(); //Vinkel i radianer
            int x = 300 + (int) (250 * Math.cos(v)+1.5*Math.PI);
            int y = 300 + (int) (250 * Math.sin(v));

            g.setColor(Color.BLACK);
            felt.tegn(g, x, y);
        }
    }
}

Felt:

Kode

import java.awt.*;


public class Felt {

    String navn; //Feltets navn
    
    //Kaldes når en spilelr passerer feltet
    public void passeret(Spiller sp)
    {
        sp.besked("Du passerer "+navn);
    }

    //kaldes når en spiller lander på feltet
    public void landet(Spiller sp)
    {
    }

    public void tegn(Graphics g, int x, int y)
    {
        g.drawString(navn, x, y);
    }
}

Grund2:

Kode

import java.awt.Graphics;


public class Grund2 extends Felt{

    Spiller ejer;
    double pris;
    double grundleje;


    public Grund2 (String navn, double pris, double grundleje)
    {
        this.navn = navn;
        this.pris = pris;
        this.grundleje = grundleje;
    }

    public double beregnLeje()
    {
        return grundleje;
    }

    public void landet(Spiller sp)
    {
        {
        sp.besked("Du er landet på "+navn);

        if(sp == ejer) //Spilleren ejer allerede grunden
        {
            sp.besked("Det er din egen grund");
        }
        else if (ejer == null) //Ingen ejer grunden, og spilleren får tilbudt den
        {
            if(sp.konto >= pris) //Hvis spilleren har penge nok til grunden
            {
                if(sp.spørgsmål("købe "+navn+" for "+pris)) //Vil spilleren købe?
                {
                    sp.transaktion(-pris); //Spilleren betaler for grunden
                    ejer = sp;
                }
            }
            else sp.besked("Du har ikke råd til at købe "+navn);
        }
        else    //Feltet ejes af anden spiller
        {
            grundleje = beregnLeje();
            sp.besked("Leje: "+grundleje);
            sp.betal(ejer, grundleje);      //Spilleren betaler til ejeren
        }
    }
    }

    public void tegn(Graphics g, int x, int y)
    {
        g.drawString(navn, x, y);
        if(ejer != null) 
        {
            g.drawString(ejer.navn, x, y+15);
            System.out.println("Test1");
        }
        System.out.println("Test2");
    }
}

Gade2:

Kode

import java.awt.Graphics;

public class Gade2 extends Grund2{

    int antalHuse;
    double huspris;

    public Gade2(String navn, double pris, double grundleje, double huspris)
    {
        super(navn, pris, grundleje);
        this.huspris = huspris;
        antalHuse = 0;
    }

    public double beregnLeje()
    {
        return grundleje+antalHuse*grundleje;
    }

    public void landet(Spiller sp)
    {        
        if(sp == ejer)
        {
            sp.besked("Du er landet på "+navn);
            if(antalHuse<5 && sp.konto>huspris && sp.spørgsmål("købe hus for "+huspris))
            {
                ejer.transaktion(-huspris);
                antalHuse++;
                sp.besked("Du bygger hus på "+navn+" for "+huspris);
            }
        }
        else super.landet(sp);
        
    }

    public void tegn(Graphics g, int x, int y)
    {
        super.tegn(g, x, y);
        if(antalHuse > 0) g.drawString(antalHuse+" huse", x, y+15);
    }
}

Spiller:

Kode

import java.awt.*;


public class Spiller {

    String navn; //Navn på spiller
    double konto; //Penge på spillerens konto
    int feltnr; //Hvad feltnumret er
    int slag;


    //Opretter en spiller
    public Spiller(String navn, double konto)
    {
        this.navn = navn;
        this.konto = konto;
        this.feltnr = 0;
        this.slag = (int) (Math.random()*6)+1;
    }


    //Besked til spilleren
    public void besked(String besked)
    {
        System.out.println(navn+": "+besked);
    }

    //Ja/nej spørgsmål. Hvis brugeren svarer Ja, returnerer true, ellers false
    public boolean spørgsmål(String spørgsmål)
    {
        String spm = navn+": Vil du "+spørgsmål+"?";
        String svar = javax.swing.JOptionPane.showInputDialog(spm, "Ja");
        System.out.println(spm+" "+svar);
        if(svar != null && svar.equals("Ja")) return true;
        else return false;
    }

    //Modtag penge
    public void transaktion(double kr)
    {
        konto = konto + kr;
        System.out.println(navn+"s konto lyder nu på "+konto+" kr");
    }

    //Overfør penge til anden spiller
    public void betal(Spiller modtager, double kr)
    {
        System.out.println(navn+" betaler "+modtager.navn+": "+kr+" kr");
        modtager.transaktion(kr);
        transaktion(-kr);
    }

    public void tegn(Graphics g, int x, int y)
    {
        g.drawString(navn, x, y-15);
    }

}

Matadorspil:

Kode

import java.util.*;
import javax.swing.*;

public class Matadorspil extends JPanel{


    ArrayList<Felt> felter = new ArrayList<Felt>(); //Indeholder alle felter

    ArrayList<Spiller> spillere = new ArrayList<Spiller>(); // Indeholder alle spillere

    int spillersTur = 0;


    public Matadorspil()
    {
        felter.add(new Start(3000));
        felter.add(new Gade2("Andevej", 18000, 900, 1800));
        felter.add(new Gade2("Ællingevej", 18000, 900, 1800));
        felter.add(new Helle(10000));
        felter.add(new Rederi2("Stenaline", 20000, 4500));       
        felter.add(new Bryggeri2("Carlsberg", 20000, 1000));
        felter.add(new Gade2("Nygade", 15000, 700, 1500));
        felter.add(new Gade2("Gammelgade", 15000, 700, 1500));
        felter.add(new Gade2("Slåenhaven", 17000, 800, 1700));
        felter.add(new Gade2("Kærsangervænget", 17000, 800, 1700));
        felter.add(new Bryggeri2("Coca Cola", 30000, 1200));
        felter.add(new Gade2("Rødovrevej", 10000, 400, 1000));
        felter.add(new Gade2("Hvidovrevej", 10000, 400, 1000));
        felter.add(new Rederi2("Maersk", 17000, 4200));
        felter.add(new Gade2("Trianglen", 12000, 500, 1200));
        felter.add(new Gade2("Bredgade", 12000, 500, 1200));
        felter.add(new Gade2("Frederiksberg Allé", 20000, 1000, 2000));
        felter.add(new Gade2("Rådhuspladsen", 25000, 1200, 2500));        
    }
}

BenytMatadorspil:

Kode

import javax.swing.*;


public class BenytMatadorspil extends JPanel{


    public static void main(String[] args) {



        Matadorspil spil = new Matadorspil();

        //Opret spillere
        spil.spillere.add(new Spiller("Kristian", 300000));
        spil.spillere.add(new Spiller("Signe", 300000));
        spil.spillere.add(new Spiller("Sofie", 300000));

        MatadorGrafikpanel panel = new MatadorGrafikpanel(); // opret panelet

        JFrame vindue = new JFrame("MatadorGrafikpanel"); // opret et vindue på skærmen
        vindue.add(panel); // vis panelet i vinduet
        vindue.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); // reagér på luk
        vindue.setSize(650,650); // sæt vinduets størrelse
        vindue.setLocation(550, 150); //Placering på skærmen
        vindue.setVisible(true); // åbn vinduet

        spil.spillersTur=0;
        //20 runder (40 ture)
        while (spil.spillere.size()>1)
        {
            vindue.repaint();
            System.out.println("Repaint");

            //Skifter mellem spillerne           
            Spiller sp = spil.spillere.get(spil.spillersTur % spil.spillere.size());

            //Kast terning
            sp.slag = (int) (Math.random()*6)+1;
            
            System.out.println("*****************************");
            System.out.println(sp.navn+" er på felt "+sp.feltnr+" og slår "+sp.slag);

            
            for(int i=1; i<=sp.slag; i++)
            {
                //Gå til næste felt. Hvis vi kommer over antal felter, så start forfra (0)
                sp.feltnr = (sp.feltnr + 1) % spil.felter.size();
                Felt felt = spil.felter.get(sp.feltnr);

                if(i<sp.slag) felt.passeret(sp); //Kald passeret på passerede felter
                else felt.landet(sp);       // Kald landet på sidste felt

                if (sp.konto <0)
                {
                    int plads = spil.spillere.indexOf(sp);
                    spil.spillere.remove(plads);
                    System.out.println(sp.navn+" er ude, da personen gik konkurs");
                }

                if (spil.spillere.size()==1)
                {
                    spil.spillersTur++;
                    sp = spil.spillere.get(spil.spillersTur % spil.spillere.size());
                    System.out.println("*****************************");
                    System.out.println(sp.navn+" har vundet spillet!");
                    System.out.println("*****************************");
                }

                try { Thread.sleep(300); } catch (Exception e) {} //Vent 0.3 sekunder
            }
            try { Thread.sleep(3000); } catch (Exception e) {} //Vent 3 sekunder

            spil.repaint();
            spil.spillersTur++;
        }
    }
}

23 svar postet i denne tråd vises herunder
2 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #14645 @ 24.01.10 02:04

806

Hvis du bare gerne vil have det til at virke burde du kunne gøre følgende. Her referer jeg til den kode du postede fra starten af:

Du skal kun have et Matatadorspil nemlig den du laver i BenytMatadorspil. Dvs. der ændres kun i en under alle omstændigheder. Men dit grafikpanel kender ikke noget til den. Derfor kan du give den med i "constructoren".

Ændrer Grafikpanelet til:

Kode

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

public class MatadorGrafikpanel extends JPanel{
    
    //Du fortæller at klassen MatadorGrafikPanel her et matadorspil men !!! du laver ikke et her.
    Matadorspil spillet;
    
    //Constructoren, denne køres når du laver din klasse. Den kan tage argumenter ligesom en metode. (hvilket den vidst nok også er)
    public MatadorGrafikpanel(Matadorspil spil) {
   spillet = spil;
}

    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);

     
        for(int i = 0; i<spil.felter.size(); i++)
        {
            Felt felt = spil.felter.get(i);

            double v = Math.PI * 2 * i / spil.felter.size(); //Vinkel i radianer
            int x = 300 + (int) (250 * Math.cos(v)+1.5*Math.PI);
            int y = 300 + (int) (250 * Math.sin(v));

            g.setColor(Color.BLACK);
            felt.tegn(g, x, y);
        }
    }
}

Så når du laver dette grafikpanel skal du huske at give den Matadorspillet når du laver panelet:

Kode

import javax.swing.*;


public class BenytMatadorspil extends JPanel{


    public static void main(String[] args) {


        //Dette er spillet. Der skal kun være en. For ellers får du det problem at der er foskellige spil forskellige steder i koden.
        Matadorspil spil = new Matadorspil();

        //Opret spillere
        spil.spillere.add(new Spiller("Kristian", 300000));
        spil.spillere.add(new Spiller("Signe", 300000));
        spil.spillere.add(new Spiller("Sofie", 300000));

        //Når du laver panelet giver du "spillet" videre til constructoren ligesom med en normal metode
        MatadorGrafikpanel panel = new MatadorGrafikpanel(spil); // opret panelet

        JFrame vindue = new JFrame("MatadorGrafikpanel"); // opret et vindue på skærmen
        vindue.add(panel); // vis panelet i vinduet
        vindue.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); // reagér på luk
        vindue.setSize(650,650); // sæt vinduets størrelse
        vindue.setLocation(550, 150); //Placering på skærmen
        vindue.setVisible(true); // åbn vinduet

        spil.spillersTur=0;
        //20 runder (40 ture)
        while (spil.spillere.size()>1)
        {
            vindue.repaint();
            System.out.println("Repaint");

            //Skifter mellem spillerne           
            Spiller sp = spil.spillere.get(spil.spillersTur % spil.spillere.size());

            //Kast terning
            sp.slag = (int) (Math.random()*6)+1;
            
            System.out.println("*****************************");
            System.out.println(sp.navn+" er på felt "+sp.feltnr+" og slår "+sp.slag);

            
            for(int i=1; i<=sp.slag; i++)
            {
                //Gå til næste felt. Hvis vi kommer over antal felter, så start forfra (0)
                sp.feltnr = (sp.feltnr + 1) % spil.felter.size();
                Felt felt = spil.felter.get(sp.feltnr);

                if(i<sp.slag) felt.passeret(sp); //Kald passeret på passerede felter
                else felt.landet(sp);       // Kald landet på sidste felt

                if (sp.konto <0)
                {
                    int plads = spil.spillere.indexOf(sp);
                    spil.spillere.remove(plads);
                    System.out.println(sp.navn+" er ude, da personen gik konkurs");
                }

                if (spil.spillere.size()==1)
                {
                    spil.spillersTur++;
                    sp = spil.spillere.get(spil.spillersTur % spil.spillere.size());
                    System.out.println("*****************************");
                    System.out.println(sp.navn+" har vundet spillet!");
                    System.out.println("*****************************");
                }

                try { Thread.sleep(300); } catch (Exception e) {} //Vent 0.3 sekunder
            }
            try { Thread.sleep(3000); } catch (Exception e) {} //Vent 3 sekunder

            spil.repaint();
            spil.spillersTur++;
        }
    }
}

Prøv lige og teste om det så ikke virker.

Bruger #12836 @ 22.01.10 10:40

857

Hej Kristian,

Med kun den kode som du har vist der, bliver pladen ikke vist når jeg kører det. Istedet vises kun et tomt Jpanel.

Jeg har været nødt til at udkommentere nogle ikke virkende felter i BenytMatadorspil klassen.

Det tyder på at der er noget galt i måden tingene tegnes og opdateres på. Men har ikke kunnet gå i dybden med det konkrete problem endnu.

Jeg har leget midt med koden for at få den tiol at tegne noget, jeg tror virkelig at du skal se på din programstruktur - den er ikke super veldesignet, selvom den stadig er ok

Jeg har set værre.

MEN!!!
Du skal ikke have 2 JPanel's. Henret klassen MatadorGrafikpanel - den funktionalitet skal ligge i Matadorspil.java.

Samtidig skal du KUN lave den instans af Matadorspil.java, som laves i starten af BenytMatadorspil - så ligger grund ejerne der, og de gemmes og vises faktisk (da jeg først fik den fiflet til at vise grafikken).

Held og lykke med det

Med venlig hilsen
Ieet

Indlæg senest redigeret d. 22.01.2010 11:03 af Bruger #12836

Bruger #14645 @ 17.01.10 15:17

806

Jeg har kun kigget på de 3 klasser du nævner. I grafikpanelet tegner du alle felter. Felter tegner deres eget navn. Informationen om ejeren ligger i Grund2. Den tegner du aldrig.

Bruger #15575 @ 17.01.10 15:20

Jeg har kun kigget på de 3 klasser du nævner. I grafikpanelet tegner du alle felter. Felter tegner deres eget navn. Informationen om ejeren ligger i Grund2. Den tegner du aldrig.

Grund2 forlænger (extends) jo Felt, så det bliver vel overført?

Eller er jeg helt forkert på den?

Bruger #14645 @ 17.01.10 15:23

806

Ups havde jeg ikke lige set extends der.

Bruger #14645 @ 17.01.10 15:56

806

Har debugget og problemet ligger i at "ejer" ikke bliver gemt ordentlig af en eller anden årsag. Ejer variablen virker fint i metoden "landet" men underligt nok er den altid null i tegn.

Det er nok en trådningsproblem. Du har main metoden som sætter ejer i main tråden. Og så er der GUI tråden som bruger den til at tegne. De har nok hver deres version af variablen. At sætte ejer til volatile gjorde det dog ikke for mig.

Indlæg senest redigeret d. 17.01.2010 16:13 af Bruger #14645

Bruger #15575 @ 17.01.10 17:45

Ja kan jeg godt se nu. Ejer er altid null, da min
if(ejer != null) metode aldrig bliver kørt.

Kan det have noget at gøre med at ejer er en lokal variabel i metoden "landet"?

Og hvordan kan man så lave det?

Bruger #14645 @ 17.01.10 18:09

806

Ja kan jeg godt se nu. Ejer er altid null, da min
if(ejer != null) metode aldrig bliver kørt.

Kan det have noget at gøre med at ejer er en lokal variabel i metoden "landet"?

Og hvordan kan man så lave det?

Ejer er lokal til instansen af klassen, ikke til metoden.

Bruger #15575 @ 17.01.10 18:57

Ja men den bliver da først initialiseret i metoden?

Indlæg senest redigeret d. 17.01.2010 18:58 af Bruger #15575

Bruger #14645 @ 17.01.10 19:01

806

Ja men den bliver da først initialiseret i metoden?

Ja men den kan "ses" i hele klassen ikke kun i metoden. Du initialiserer den i metoden, men du deklarerer den i klassen. Hvis du lavede en ny variabel, "String ejer;" inde i metoden ville den være "lokal til metoden" og overskygge klassens "ejer". Men det har du ikke så det burde ikke være noget problem.

Er 100% sikker på at problemet er noget med at Swing GUI tråden og main tråden ikke ser den samme.

Indlæg senest redigeret d. 17.01.2010 19:02 af Bruger #14645

<< < 1 2 3 > >>

Matdorspil - Grafisk programmering

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler