array med objekter

har glemt at skrive....når jeg kalder på data[i].beregnData()
står det sådan her i main
Data[] data = new Data[5];
data[0] = new Data[par 1, par 2]
osv...

for (int i = 0; i < 5; i++)
{
System.out.print(data[i].beregnData());
}

Uhmm... noget i stil med...

public int findMaxData(Object[] pObj) {
int max = 0;
for (int i=0; i<data.length; i++) {
  if (data[i] > max)
    max = data[i];
}
return max;
}


.... nu ved jeg jo ikke hvad der ligger i dit object array, men hvis det er rigtige object\'er med metoder til at hente ting og sager, så husk at du skal cast\'e dem så du kan bruge dem.

Er det svar nok?

I din klasse med beregnData, har du metoden findMaxData:
public Data findMaxData(Data[] data) {
    Data resultat = data[0];
    for (int i=1; i<data.length(); i++) {
        if (data[i].større(resultat)) {
            resultat = data[i];
        }
    }
    return resultat;
}

Klassen Data skal så have metoden boolean større(Data data), som du kan implementere så den passer til klassens data.

Eksempel: klassen Data modellerer et rektangel med data bredde og højde. Det rektangel du vil finde, er det der har det største areal. Så implementerer du større() således:

public class Data {
    int bredde;
    int højde;
    ...
    // constructor og metoder udeladt
    ...
    boolean større(Data data) {
      if ((this.bredde * this.højde) > (data.bredde * data.højde)) {
          return true;
      } else {
          return false;
      }
  }
}

Jeg er gået ud fra at linien
data[0] = new Data[par 1, par 2]
skulle have været
data[0] = new Data(par1, par2);

er det korrekt?

Når du fra main vil udskrive det største Data objekt, er standarden, at du i Data implementerer metoden toString().

nej, det er ().....
jeg sender lige koden

Her er main:

public class Pakke
{
    public static void main (String []args)
    {
        Pakker[] pak = new Pakker[5];

        pak[0] = new Pakker(1.5 , 0);
        pak[1] = new Pakker(2 , 0);
        pak[2] = new Pakker(5 , 1);
        pak[3] = new Pakker(1 , 1);
        pak[4] = new Pakker(3.7 , 0);

        for(int i = 0; i < 5; i++)
       
        System.out.println(\"Portoen for pakke \" + i + \" er på \" + pak[i].beregnPorto() + \"kr.\");
       
    }
}

og her er klassen med metoden:

class Pakker
{
    public Pakker()
    {
        maxporto = 0;
        porto = 0;
    }

    public Pakker (double avægt, int axpress)
    {
        vægt = avægt;
        xpress = axpress;
    }


    public double beregnPorto()
    {

        if(vægt <= 1 && xpress == 0)
            porto = 20;

        else if (vægt <= 1 && xpress == 1)
            porto = 30;

        else if(vægt <= 2 && xpress == 0)
            porto = 35.5;

        else if(vægt <= 2 && xpress == 1)
            porto = 45.5;

        else if(vægt <= 3.5 && xpress == 0)
            porto = 60.25;

        else if(vægt <= 3.5 && xpress == 1)
            porto = 70.25;

        else if(vægt > 3.5 && xpress == 0)
            porto = 100;
   
        else if(vægt > 3.5 && xpress == 1)
            porto = 110;
   
        return porto;
   
    }
    private double porto;
    private double maxporto;
    private double vægt;
    private int xpress;
}


og så skal jeg bruge en metode som beregne den porto som har højeste værdi.....og udskrive det i main

kresten......kan du ikke kigge på koden.....
jeg er helt forvirret.og kan ikke få det til at virke

kan du bruge min hjælp?

class Pakker
{
    public Pakker()
    {
        maxporto = 0;
        porto = 0;
    }

    public Pakker (double avægt, boolean axpress)
    {
        vægt = avægt;
        xpress = axpress;
    }


    public double beregnPorto()
    {

        if(vægt <= 1 && !xpress)
            porto = 20;

        else if (vægt <= 1 && xpress)
            porto = 30;

        else if(vægt <= 2 && !xpress)
            porto = 35.5;

        else if(vægt <= 2 && xpress)
            porto = 45.5;

        else if(vægt <= 3.5 && !xpress)
            porto = 60.25;

        else if(vægt <= 3.5 && xpress)
            porto = 70.25;

        else if(vægt > 3.5 && !xpress)
            porto = 100;
   
        else if(vægt > 3.5 && xpress)
            porto = 110;
   
        return porto;
   
    }

    public boolean større(Pakker pakke) {
        if (this.porto > pakke.porto) {
            return true;
        } else {
            return false;
        }
    }

    private double porto;
    private double maxporto;
    private double vægt;
    private boolean xpress;
}

Her har jeg lagt større() ind i klassen Pakker, og så har jeg også lavet xpress om til en boolean.

public class Pakke
{
    public static void main (String []args)
    {
        Pakker[] pak = new Pakker[5];

        pak[0] = new Pakker(1.5 , false);
        pak[1] = new Pakker(2 , false);
        pak[2] = new Pakker(5 , true);
        pak[3] = new Pakker(1 , true);
        pak[4] = new Pakker(3.7 , false);

        for(int i = 0; i < 5; i++)
       
        System.out.println(\"Portoen for pakke \" + i + \" er på \" + pak[i].beregnPorto() + \"kr.\");

        Pakker dyreste = findDyreste();

        System.out.println(\"Den dyreste pakke koster \" + dyreste.getPorto();
       
    }

    Pakker findDyreste() {
        Pakker dyreste = pak[0];
        for (int i=1; i<5; i++) {
            if (pak[i].getPorto()>dyreste.getPorto()) {
                dyreste = pak[i];
            }
        }
        return dyreste;
    }
}

Hey

Du har jo næsten selv lavet det i dit for-loop, her udskriver du jo selv portoen på hver enkelt pakke. Det eneste du skal gøre så at putte det ind i en metode som tager dit array som argument.

Mit tidlige svar, omskrevet lidt...

public void skrivMaxPorto(Object[] mitArray) {
Pakker maxPakke = Pakke(0.0, 0);
Pakker traverseringsPakke = null;

for (int i=0; i<mitArray.length; i++) {
  traverseringsPakke = (Pakke)mitArray[i];
  if (traverseringsPakke.beregnPorto() > maxPakke.beregnPorto())
    maxPakke = traverseringsPakke;
}
System.out.println(\"Den dyreste pakke koster \" + maxPakke.beregnPorto());
}

.... uhm, eller noget i den stil. Det er IKKE compilet så jeg ved ikke om det virker!?!?? Men jeg håber du kan se ideen med det.

En anden ting, prøv at lade være med at poste så meget pstric okay?

kresten>> Hvad mener du?

pstric>> Dine svar er gode nok, men kunne du ikke sende dem i 1 posting... vælter jo ind med mail :)

Ellers en god løsning du har, nemmere at læse/forstå end min :-P

sandra26>> Ups, der var en fejl i findDyreste()

Pakker findDyreste() {
    Pakker dyreste = pak[0];
    for (int i=0; i<5; i++) {
        if (pak[i].større(dyreste)) {
            dyreste = pak[i];
        }
    }
    return dyreste;
}

og Pakker skal have en get-metode til portoen
double getPorto() {
    return porto;
}

pstric: selvfølgelig.....*ss*
har bare udtrykt mig forkert
kigger lige på det

jeg kan ikke finde ud af det....
pstric.....det kan godt være din løsning fungerer..men metoden skal ligge i samme fil som den anden metode
kresten: når jeg compiler den, viser den 3-4 fejl med konverteringen

og der er meget stor chance for at jeg i morgen kommer op i sådan et spørgsmål og skal kunne lave det sidste......

sandra26>> Mener du, at metoden findMax() (eller hvad du nu vil kalde den) skal ligge i klassen Pakker?

Ja....det skulle det gerne.....:-)

forresten....det skal helst være sådan noget kersten har lavet....med at den henter værdierne fra den anden metode som gemmes i en array og så kan jeg sagtens klare resten....
men jeg kan ikke finde ud af at hente værdierne fra den anden metode

kresten:...sorry, ved godt at det vælter ind med mails....*ss*

sandra26>> Kunne du ikke prøve at sende opgaveformuleringen?

den har jeg ikke......men ham der har været op i den her opgave har skrevet.....at man skulle udregne samlet porto i alt, porto for en enkelt pakke, og hvilken pakke der var dyrest

og alle de ting skulle hentes fra instansmetoder

det kan godt være at det også kan laves i en classe for sig selv...men det skal være en instans metode

sandra26>> det giver ikke megen mening, at skulle hente værdien af den dyreste pakke fra en bestemt instans af Pakker. Den kender jo ikke noget til de andre instanser.

så det eneste løsning er at lave det i  main?

sandra26>> Jeg ville nok lave gennemløbet i main(). Så ville jeg have to variabler maxPorto og samletPorto, som jeg opdaterer under gennemløbet.

Sig til, hvis du gerne vil have et lille eksempel på, hvordan jeg ville gøre det.

pstric...har lige lavet det. og det virker....
        double best = 0;
                double sum  = 0;
        for(int i = 0; i < 5; i++)
        {
        System.out.println(\"Portoen for pakke \" + (i+1) + \" er på \" + pak[i].beregnPorto() + \"kr.\");
           
                sum += pak[i].beregnPorto();
               
                best = pak[0].beregnPorto();
                for (int j = 0; j < pak.length; j++)
            {
                double max = pak[j].beregnPorto();
                if (max > best)
                    best = max;
            }
           
            }
                System.out.println(\"Samlet porto er: \" + sum);
                System.out.println(\"Den dyreste pakke koster: \" + best);

      }


pstric....har du lidt tid, jeg har brug for lidt hjælp....

sandra26>> Jeg har tid. Det du har lavet, ser meget godt ud. Der er nu ingen grund til at lave løkken, der beregner max-portoen inde i den anden løkke. Vent lidt, så sender jeg lige et andet forslag.

der er to opgaver hvor jeg ikke kan løse sidste sprøgsmål.....den første opgave går ud på at lave en todim array med random værdier og så skal man udskrive den, udskrive samlet gennemsnit og når man har indtastet nummer på et bestemt række skal den udskrive summen for den række.....og det har jeg lavet..det hele skulle være statiske metoder

men,(det står ikke i opgaven) kan man lave en statisk metode som beregner summen af alle rækker hver for sig og udskriver dem i en endim tabel?

sandra26>> Her er en ny udgave. Jeg har omdøbt klassen Pakke til PortoBeregning. Du burde nok også omdøbe Pakker til Pakke. Klassen modellerer jo én pakke.

public class PortoBeregning {
   
    private final int antal = 5;
   
    private Pakke[] pakker;
   
    private double maxPorto;
    private double samletPorto;
   
    public PortoBeregning() {
        pakker = new Pakker[antal];

        pakker[0] = new Pakker(1.5 , false);
        pakker[1] = new Pakker(2 , false);
        pakker[2] = new Pakker(5 , true);
        pakker[3] = new Pakker(1 , true);
        pakker[4] = new Pakker(3.7 , false);

        maxPorto = 0.0;
        samletPorto = 0.0;
    }
   
    private udskrivVærdier() {
        for (int i=0; i<antal; i++) {
            System.out.println(\"Den \" + (i+1) + \". pakke koster \" + pakker[i].beregnPorto());
            if (pakker[i].getPorto() > maxPorto) {
                maxPorto = pakker[i].getPorto();
            }
            samletPorto += pakker[i].getPorto();
        }
        System.out.println(\"Den dyreste pakke koster \" + maxPorto);
        System.out.println(\"Samlet pris for pakkerne \" + samletPorto);
    }
   
    public static void main(final String args) {
        PortoBeregning beregning = new PortoBeregning();
       
        beregning.udskrivVærdier();
    }
}

class Pakker
{
    public Pakker()
    {
        porto = 0;
    }

    public Pakker (double avægt, int axpress)
    {
        vægt = avægt;
        xpress = axpress;
    }


    public double beregnPorto()
    {

        if(vægt <= 1 && xpress == 0)
            porto = 20;

        else if (vægt <= 1 && xpress == 1)
            porto = 30;

        else if(vægt <= 2 && xpress == 0)
            porto = 35.5;

        else if(vægt <= 2 && xpress == 1)
            porto = 45.5;

        else if(vægt <= 3.5 && xpress == 0)
            porto = 60.25;

        else if(vægt <= 3.5 && xpress == 1)
            porto = 70.25;

        else if(vægt > 3.5 && xpress == 0)
            porto = 100;
   
        else if(vægt > 3.5 && xpress == 1)
            porto = 110;
   
        return porto;
   
    }
    private double porto;
    private double vægt;
    private int xpress;
}

I klassen Pakker har jeg også fjernet maxporto. Den ligger nu i PortoBeregning.

Jeg kigger lige på dit sidste problem, og så vender jeg nok tilbage med nogle spørgsmål :-)

Ups, jeg havde ikke compilet det her. Det er fyldt med fejl, så vent lidt med at bruge det.

Så, nu kan det compile.

public class PortoBeregning {
   
    private final int antal = 5;
   
    private Pakker[] pakker;
   
    private double maxPorto;
    private double samletPorto;
   
    public PortoBeregning() {
        pakker = new Pakker[antal];

        pakker[0] = new Pakker(1.5 , false);
        pakker[1] = new Pakker(2 , false);
        pakker[2] = new Pakker(5 , true);
        pakker[3] = new Pakker(1 , true);
        pakker[4] = new Pakker(3.7 , false);

        maxPorto = 0.0;
        samletPorto = 0.0;
    }
   
    public void udskrivVærdier() {
        for (int i=0; i<antal; i++) {
            System.out.println(\"Den \" + (i+1) + \". pakke koster \" + pakker[i].beregnPorto());
            if (pakker[i].getPorto() > maxPorto) {
                maxPorto = pakker[i].getPorto();
            }
            samletPorto += pakker[i].getPorto();
        }
        System.out.println(\"Den dyreste pakke koster \" + maxPorto);
        System.out.println(\"Samlet pris for pakkerne \" + samletPorto);
    }
   
    public static void main(final String[] args) {
        PortoBeregning beregning = new PortoBeregning();
       
        beregning.udskrivVærdier();
    }
}

class Pakker
{
    public Pakker()
    {
        porto = 0;
    }

    public Pakker (double avægt, boolean axpress)
    {
        vægt = avægt;
        xpress = axpress;
    }


    public double beregnPorto()
    {

        if(vægt <= 1 && !xpress)
            porto = 20;

        else if (vægt <= 1 && xpress)
            porto = 30;

        else if(vægt <= 2 && !xpress)
            porto = 35.5;

        else if(vægt <= 2 && xpress)
            porto = 45.5;

        else if(vægt <= 3.5 && !xpress)
            porto = 60.25;

        else if(vægt <= 3.5 && xpress)
            porto = 70.25;

        else if(vægt > 3.5 && !xpress)
            porto = 100;
   
        else if(vægt > 3.5 && xpress)
            porto = 110;
   
        return porto;
   
    }
   
    public double getPorto() {
        return porto;
    }
   
   
    private double porto;
    private double vægt;
    private boolean xpress;
}

Ang, dit sidste spørgsmål.

Ja, hvis du allerede har en metode til beregning af summen af en enkelt række, så kan du bare bruge den.

Men det virker næsten for let. Er der noget, der forhindrer det i at virke?

´her er hvordan jeg udregner summen for en bestemt række...men jeg ved ikke hvordan jeg får den til at tage næste række

public static int sum (int[][] table, int r)
    {
            int sum = 0;
            for (int j = 0; j < table[r].length; j++)
                                 
                sum = sum + table[r][j];
                return sum;
    }   
   

Kan du ikke bare kalde den fra en løkke?

int[] summer = new int[table.length];

for (int i=0; i<table.length; i++) {
    summer[i] = sum(table, i);
}

jeg ved det ikke.....kan ikke finde ud af det...har prøvet i lang tid

Jamen, så send din kode, så finder vi nok ud af det.

jeg får det her....
I@dd0253b3

[I@da1653b3

[I@da2653b3

og det skal være en statisk metode ligesom andre hvis muligt.....:-)

import java.util.Random;

public class Opgave2
{  public static void main(String[] args)
  { 
      Random generator = new Random();
        final int COLUMN_WIDTH = 10;
       
        int[][] tabel = new int[10][8];
        for (int i = 0; i < tabel.length; i++)
            for (int j = 0; j < tabel[i].length; j++)
                    tabel[i][j] = generator.nextInt(100);
                    printTable(tabel, COLUMN_WIDTH);
                   
        ConsoleReader console = new ConsoleReader(System.in);
       
        System.out.println(\"Indtast nummeret på den række du vil summere:\");
        int raekke = console.readInt() - 1;
       
        System.out.println(sum(tabel, raekke));
        System.out.println(average(tabel));
               
    }
   
    public static int sum (int[][] table, int r)
    {
            int sum = 0;
            for (int j = 0; j < table[r].length; j++)
                                 
                sum = sum + table[r][j];
                return sum;
    }   
   

    public static double average (int[][] table)
    {
        int gennemsnit = 0;
      int sum = 0;
      for (int i = 0; i<table.length; i++)
      { 
      for (int j = 0; j<table[i].length; j++)
        sum += table[i][j];
 
          gennemsnit = sum/table[i].length;
      }
      return gennemsnit;
    }


         
  public static void printTable(int[][] table, int width)
  {  for (int i = 0; i < table.length; i++)
      {  for (int j = 0; j < table[i].length; j++)
        {  System.out.print(format(table[i][j], width));
        }
        System.out.println();
      }
  }
 
 
  public static String format(int n, int width)
  {  String nstr = \"\" + n;

      // pad with spaces

      while (nstr.length() < width)
        nstr = \" \" + nstr;

      return nstr;
  }
}

I@dd0253b3

[I@da1653b3

[I@da2653b3

Er objekt-referencer. Det skriver den hvis man fx forsøger at udskrive arrayet uden at angive index numre

System.out.println (tabel);

eller
System.out.println (tabel[0]);
når det er 2-dimensionelt. Det sker vist ikke i den kode du har sendt. Er udsrivningerne fra den?

Her er en rettet udgave:

import java.util.Random;

public class Opgave2
{  public static void main(String[] args)
  { 
      Random generator = new Random();
        final int COLUMN_WIDTH = 10;
       
        int[][] tabel = new int[10][8];
        for (int i = 0; i < tabel.length; i++)
            for (int j = 0; j < tabel[i].length; j++)
                    tabel[i][j] = generator.nextInt(100);
        printTable(tabel, COLUMN_WIDTH);

        int summer[] = new int[tabel.length];
        for (int i=0; i<tabel.length; i++) {
            summer[i] = sum(tabel, i);
        }
        for (int i=0; i<tabel.length; i++) {
            System.out.print(summer[i] + \" \");
        }
        System.out.println();
        System.out.println(average(tabel));
               
    }
   
    public static int sum (int[][] table, int r)
    {
            int sum = 0;
            for (int j = 0; j < table[r].length; j++)
                                 
                sum = sum + table[r][j];
                return sum;
    }   
   

    public static double average (int[][] table)
    {
        int gennemsnit = 0;
      int sum = 0;
      for (int i = 0; i<table.length; i++)
      { 
      for (int j = 0; j<table[i].length; j++)
        sum += table[i][j];
 
      }
          gennemsnit = sum/(table[0].length * table.length);
      return gennemsnit;
    }


         
  public static void printTable(int[][] table, int width)
  {  for (int i = 0; i < table.length; i++)
      {  for (int j = 0; j < table[i].length; j++)
        {  System.out.print(format(table[i][j], width));
        }
        System.out.println();
      }
  }
 
 
  public static String format(int n, int width)
  {  String nstr = \"\" + n;

      // pad with spaces

      while (nstr.length() < width)
        nstr = \" \" + nstr;

      return nstr;
  }
}

Din gennemsnitsberegning var forkert.

Hejsa codemon...
nej det var noget jeg havde prøvet.
Der var 9 fra min klasse op til eksamen i dag og 4 af dem er dumpet......

pstric......ups....*ss*....takker

Jamen, så held og lykke med eksamen i morgen.

pstric: forresten.....Jeg har altid en masse spørgsmål...
hvis jeg vil lave en brugergrænseflade til den første opgave....så man selv kan indtaste oplysningeren om pakkerne...hvad er det bedste at gøre(.lave 3 * 5 felter..for 5 pakker) eller?

tak pstric....*s*
det bliver spændende....

En bedre løsning ville nok være, at lave en liste, hvor brugeren kan se de pakker der er indtastet. Ved at klikke på en \'Ny pakke...\' knap kunne brugeren så komme til en dialog, hvor oplysninger om en ny pakke kunne indtastes, og når brugeren trykkede på OK, skulle den nye pakke så tilføjes til listen. Et statusområde på samme vindue skulle så vise prisen på den dyreste og den samlede pris.

Men så skulle pakkerne heller ikke længere ligge i et array, men en container, der var mere velegnet til formålet.

vi er kun lige gået i gang med GUI...og det eneste vi skal kunne er at lave inputfelterne, knapper og felt hvor man kan vise resultat...men jeg ved ikke om man overhovedet kan bruge det når det er en array

Jamen, det kan du da sagtens. Den grænseflade, du selv foreslog ville være udemærket til formålet.

nå okay...
det ville aldrig lykkedes mig at få det til at virke..
i hvert fald ikke i aften...
*s*
tak....:-)