Problem med att skriva ut till fönster

2012-12-12, 13:59 #1

Medlem

Reg: Okt 2012

Inlägg: 32

Hej!

jag förstår inte vad jag gör för fel

Kod:


g.fillRect(r1.getXPostion(),r1.getYPostion(),r1.getBredd(),r1.getHöjd());   

         g.fillRect(r2.getXPostion(),r2.getYPostion(),r2.getBredd(),r2.getHöjd());

Programmet kan inte hitta någon av metoderna getXPostion, getYPostion, getBredd, getHöjd.

Någon snäll person som kan förklara varför?

Koden i sin helhet

Kod:


import javax.swing.*;

import java.awt.*;

/**

 *

 * @author Doh

 */

public class Rektangel4Panel extends JPanel {



 public void paintComponent(Graphics g) {

             super.paintComponent(g);

             

             Rektangel4Panel r1 = new Rektangel4Panel();

             Rektangel4Panel r2 = new Rektangel4Panel();

             

             g.fillRect(r1.getXPostion(),r1.getYPostion(),r1.getBredd(),r1.getHöjd());   

             g.fillRect(r2.getXPostion(),r2.getYPostion(),r2.getBredd(),r2.getHöjd());   

 }

    

    

 

  

    



class Rektangel4{  //variabler och metoder

    

     

           public int bredd;

           public int höjd;

           public int omkrets;

           public int area;

           public int xposition;

           public int yposition;

           

           Rektangel4(){

               bredd = 10;

               höjd = 10;

               xposition = 0;

               yposition = 0;

               

           }

            

            

    public Rektangel4(int b, int h, int x, int y){//Konstruktor

    

        bredd = b;

        höjd = h;

        xposition = x;

        yposition = y; 

                 

    }

            

      //Metoder  

       public int getBredd(int b){return bredd;}

       public int getHöjd(int h){return höjd;}

       public int getOmkrets(){return (bredd + bredd + höjd + höjd);}

       public int getArea(){return (höjd * bredd);}

       public void setXPostion(int x){xposition = x;}

       public void setYPostion(int y){yposition = 20;}

       public int getYPostion(int y){return yposition;}

       public int getXPostion(int x){return xposition;}

}

Citera

2012-12-12, 14:28 #2

Medlem

Reg: Jan 2009

Inlägg: 309

Metoderna ligger ju i din inre klass Rektangel4 och inte i huvudklassen Rektangel4Panel. Förstår inte riktigt varför du har en inre klass där, men ska du kalla metoderna från den får du skriva r1.Rektangel4."metod". Alternativt skapa objekt av den inre klassen istället.

Sen har du även indata till 4 av dina get-metoder som jag gissar måste vara ett fel. De skapar iaf problem då du inte ger någon indata när du kallar dem.

Disclaimer att jag kan vara helt ute och cykla, tag sen jag höll på med Java.

Citera

2012-12-12, 15:11 #3

Medlem

Reg: Jan 2004

Inlägg: 1 593

Rektangel4Panel har ingen metod som heter getXPostion, den finns i klassen Rektangel4.
Fler saker som är helt fel:
Dina get-metoder ska inte ha några argument.
Alla variabelnamn ska vara på engelska.
Du ärver från JPanel, som redan har en hel del liknande metoder.
Klassen Rektangel4Panel skapar två nya Rektangel4Panel-objekt i dess paintComponent, jag tror inte det är vad du vill göra.

Om jag förstått det hela rätt vill du skapa en ny typ av JPanel, som är som en vanlig JPanel med skillnaden att den fyller hela eller delar av panelen med en viss färg (vilket för övrigt kan sättas med tex g.setColor(Color.GREEN); innan g.fillRectangle()

Stämmer detta?

Citera

2012-12-12, 15:22 #4

Medlem

Reg: Okt 2012

Inlägg: 32

Tanken är att det ska bli ett horisontellt stapeldiagram

med 2 st olika rektanglar

under varje stapel ska det dessutom stå rektangelns

"Rektangel objekt r" och omkrets och Area.

Som sagt det är tanken, känns som jag är långt ifrån just nu. ska gå igenom dina ändring och se om jag kommer närmare resultatet

Citera

2012-12-12, 15:46 #5

Medlem

Reg: Jan 2004

Inlägg: 1 593

Citat:

Ursprungligen postat av SagaIsabella

Tanken är att det ska bli ett horisontellt stapeldiagram

med 2 st olika rektanglar

under varje stapel ska det dessutom stå rektangelns

"Rektangel objekt r" och omkrets och Area.

Som sagt det är tanken, känns som jag är långt ifrån just nu. ska gå igenom dina ändring och se om jag kommer närmare resultatet

Okej, den här klassen ritar när man kör upp ett stapeldiagram med fem staplar i olika färger.
Fler staplar kan läggas till i main-metoden genom BarChart-funktionen addColumn som bara behöver veta höjden och färgen. Bredden och höjden på panelen justeras efterhand som fler staplar läggs till eller höjden på en ny stapel är större än den tidigare högsta stapeln.

Kod:


import javax.swing.*; 

import java.awt.*; 

import java.util.ArrayList;



public class BarChart extends JPanel {



    private static final long serialVersionUID = 1L;

    private ArrayList<Column> columns;

    private int maxHeight = 0;

    private static final int COLUMN_WIDTH = 50;

    private static final int COLUMN_MARGIN = 20;



    public static void main(String[] args){

        JFrame frame = new JFrame();

        BarChart r = new BarChart(); 

        r.addColumn(100, Color.GREEN);

        r.addColumn(200, Color.RED);

        r.addColumn(50, Color.BLUE);

        r.addColumn(123, Color.BLACK);

        r.addColumn(80, Color.YELLOW);

        frame.add(r);

        frame.pack();

        frame.setTitle("Stapeldiagram");

        frame.setVisible(true);

    }



    public BarChart(){

        columns = new ArrayList<Column>();

        

    }

    public void paintComponent(Graphics g){

        super.paintComponent(g);

        for (int i = 0; i < columns.size(); i++){

            g.setColor(columns.get(i).c);

            int x = i*(COLUMN_WIDTH + COLUMN_MARGIN);

            int y = maxHeight - columns.get(i).h + COLUMN_MARGIN;

            g.fillRect(x, y, COLUMN_WIDTH, columns.get(i).h);

            g.setColor(Color.BLACK);

            g.drawString(Integer.toString(columns.get(i).h), x, maxHeight + 2*COLUMN_MARGIN);

        }

    } 

    public void addColumn(int h, Color c){

        maxHeight = h > maxHeight ? h : maxHeight;

        columns.add(new Column(h, c));

        setPreferredSize(new Dimension((COLUMN_WIDTH + COLUMN_MARGIN)*columns.size(),maxHeight + 2*COLUMN_MARGIN));

    }



    private class Column{

        private int h;

        private Color c;



        private Column(int h, Color c){

            this.h = h;

            this.c = c;

        }

    }

}

__________________
Senast redigerad av Joonc 2012-12-12 kl. 15:50.

Citera

2012-12-12, 20:46 #6

Medlem

Reg: Apr 2009

Inlägg: 1 576

getHöjd, lol, halva metodnamnet är på engelska medans andra halvan är på svenska

Citera

2012-12-14, 11:22 #7

Medlem

Reg: Maj 2008

Inlägg: 2 692

Citat:

Ursprungligen postat av Un-named

getHöjd, lol, halva metodnamnet är på engelska medans andra halvan är på svenska

Låt TS lärare stå för kritik som TS ej efterfrågar.

Att ställa frågor är att lära sig också, inte få allt serverat på ett fat.

Citera

2012-12-15, 07:58 #8

Medlem

Reg: Aug 2008

Inlägg: 99

Om domänen är på svenska kan man visst blanda in svenska i koden, även på getters. I detta fall var det dock fel.

Citera

2012-12-17, 15:30 #9

Medlem

Reg: Okt 2012

Inlägg: 32

ok skulle uppskatta lite tips och feedback om jag är på rätt väg eller om allt är helt fel...

Klass/konstrutor/metoder

JPanel

får fortfarande fel att den inte hittar rektangel1.paint(g);...

Citera

2012-12-18, 12:40 #10

Medlem

Reg: Jan 2004

Inlägg: 1 593

Citat:

Ursprungligen postat av SagaIsabella

ok skulle uppskatta lite tips och feedback om jag är på rätt väg eller om allt är helt fel...

Klass/konstrutor/metoder

får fortfarande fel att den inte hittar rektangel1.paint(g);...

Det beror på att du inte har en metod som heter paint i klassen Rectangle. Använd inte heller svenska variabelnamn, bara engelska.
Tilldelningen
farg = farg;
gör absolut ingenting. Du säger bara att den ska ha värdet av sig själv, vilket den redan har. Och vill du verkligen representera en färg med en int? Visserligen räcker en int för att representera en färg med åtta bitar för rött, grönt, blått och alpha (transparens), men jag tror det vore bättre att använda java.awt.Color.

Din metod paintComponent är tom.

Du bör inte skapa rektanglar inuti paintComponent-metoden. Gör trianglar tillhörande panelen medlemsvariabler i stället som deklarerar längst upp och instansieras i konstruktorn. Kolla på mitt förra inlägg. Det gör inte precis det du vill göra, men strukturen är ungefär densamma.

Implementera inte hela metoder på en rad som i:

public int getX(int x){return xp;}

skriv i stället

public int getX(){
return xp;
}

Och varför tar denna metoden ett argument? Se get-metoder som en frågeställning; varför skulle någon behöva skicka med ett int-värde när de frågar om ditt fönsters x-koordinat? Det faktum att du inte använder argumentet borde få dig att inse att det inte behövs.

Citera

Problem med att skriva ut till fönster

Skapa ett konto eller logga in för att kommentera

Skapa ett konto

Logga in