Det er en måde at lave det på - der er mange andre.

Jeg postede noget kode tidligere iidag i http://www.eksperten.dk/spm/361739
(se 08/06-2003 18:10:08).

Koden er ikke "færdig", men den illusterer både abstrakt basis-klasse
og factory.

Hvis du ikke er for skrap til Java bør du måske starte med en
simplere løsning.

F.eks. bare en enkelt klasse og en masse if sætninger til at udregne
porto.

Jeg er ikke helt inde i det, men vil gerne vide lidt mere

Prøv og kig på det kode eksempel jeg refererede til først.

Ok, så vil jeg lige spørge dig hvorfor bruges der private double til længde? Hvorfor skrives der public void? Og hvad betyder det når der står public double?

Længde gemmes som double for at kunne håndtere decimaler f.eks. 29.7 cm.

Længde er private fordi den skal accesses gennem setLængde og
getLængde metoderne (og de er public).

void betyder at metoden ikke returnerer noget.

public betyder at metoden kan kaldes af al kode.

Ok, det vil altså sige at jeg kan godt skrive public int hvis der kun er tale om heltal og ikke andet. 

Hvad nu hvis man ikke ønsker at lave de abstrakte klasser men kun en klasse for brevene er det så også muligt?  Og hvilken løsning er bedst og nemmest at arbejde med?

Ja - hvis det kun er heltal kan du bare erklære den int.

Du kan godt lave kun en klasse der indeholde hele logikken (hvis din opgave
formulering tillader det).

Løsningen med en abstrakt basis klasse og en factory er mere avanceret end
en stor klasse.

Men for at forstå den skal man have rimelig styr på Java.

Har du styr på abstrakte klassser og metoder ?

Hvis ikke tror jeg at du skal vælge en simplere løsning.

Jeg har ikke styr på abstrakte klasser og metoder, derfor vil jeg også gerne vide om det ikke var lettere at lave en klasse hvis man altså kan det.

Hvis man kan hvordan starter man så koden for det?

Det kan du godt og det er nok mindre krævende med hensyn
til Java finesser.

Umiddelbart vil jeg tro at du skal starte med at lave en constructor
med et antal argumenter som beskriver brevet og noget kode der
gemmer dem.

Og så laver du en beregnPorto metode der udfra disse oplysninger
beregner den korrekte porto.

Det må være det allermest simple.

ok, så er problemet bare lige at jeg ikke er sikker på hvordan man laver en constructor og metoder til den.

Constructoren hører den ikke til efter public class {private String destination; osv}. Er du muligt at du kan give mig et hint på koden der skal efterfølge denne class?

en constructor er en metode uden type med samme navn som klassen.

Eksempel:

public class Brev {
  public Brev(int l, int b, int t, int v, String d) {
  }
}

Ok tusind tak for al din hjælp, det var pænt af dig, det kan være jeg vender tilbage på et senere tidspunkt med flere spørgsmål.

Du er velkommen.

Så har lavet et en klasse som vi snakkede om i går, kan du sige om det jeg har gjort indtil videre er rigtigt?


public class Category
{
    // instance variables
    private int length, width, weight;
    private double thickness;   
    private String country;

    /**
    * Constructor for objects of class Category
    */
    public Category(String c, int l, int w, int  we, int t)
    {
        // initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;
    }

Det ser meget fronuftigt ud (bortset fra at jeg tror du mangler at gemme
country).


Men det er nok porto beregningen der er det tricky.

Nå, ja det skal jeg lige huske at gøre. Men skal de forskellige typer af breve ikke også nævnes?

Hvordan går jeg det her?

Hvis det nu stadig skal være meget meget simpelt, kan jeg så ikke lave 2 klasser en hvor jeg beskriver brevene altså standard, large og maxi og en med porto.

Uden at det bliver med de abstrakte klasser du talte om i aftes er det også en mulighed?

Porto beregning er oplagt en metode og ikke en klasse.

Med hensyn til type kunne du jo f.eks. lave en metode:

public String getType()

der beregnede type "Standard", "Stor" eller "Maxi" udfra målene
som du har gemt.

Men der er altså mange måder at gøre det på.

Ja, det er snart klar over :), men som sagt så skal det bare være så simpelt som muligt så jeg har en lille chance for at forstå hvad det er jeg har med at gøre:)

Hvis jeg bruger public string er det så ikke her jeg skal bruge new type = Standard?
Eller hvad det nu er jeg skal have frem.

Næ jeg forestillede mig noget så simpelt som:

public String getType() {
  if(...) {
      return "Standard";
  } else if(...) {
      return "Stor";
  } else {
      return "Maxi";
  }
}

ok tak, det vil jeg prøve det ser noget mere overskueligt ud, men lad os nu om jeg kan få det til at virke:)

Og så er der også porto beregningen !

:-)

Ser det her rigtig ud indtil nu?

public class Category
{
    // instance variables
    private int length, width, weight;
    private double thickness;   
   
   

    /**
    * Constructor for objects of class Category
    */
public Category(int l, int w, int  we, int t)
    {
        // initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;
       
    }
   
    /**
    * Find out which type of letter we are dealing with according to weight,
      width, length and thicknes.
     
      public String getType(){ if(l <= 23) && (w <= 17) && (we <=50) &&
      (t <=0.5)
      {
      return "Standardletter";
      }
      else if {if(l <= 33) && (w <= 23) && (we <= 1000) && (t <= 2)
      {
      return " Largeletter";
      }
      else if(l <= 60> && (l + w + t <= 90) && (we <= 2000)
      {
      return "Maxiletter";
      }
      }


Eller er det helt hen i skove?

Den der med portoberegningen ja, det skal noke blive sjovt:), men det er vel også en masse if else sætninger. Hvor land er inkludereret ikke?

Det oversætter ikke !

Jeg vil anbefale dig løbende at oversætte så du ikke får for mange fejl.

Den her versionb oversætter:

public class Category {
    // instance variables
    private int length, width, weight;
    private double thickness;

    /**
    * Constructor for objects of class Category
    */
    public Category(int l, int w, int we, int t) {
        // initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;

    }

    /**
    * Find out which type of letter we are dealing with according to weight,
    * width, length and thicknes.
    */

    public String getType() {
        if ((length <= 23) && (width <= 17) && (weight <= 50) && (thickness <= 0.5)) {
            return "Standardletter";
        } else if ((length <= 33) && (width <= 23) && (weight <= 1000) && (thickness <= 2)) {
            return " Largeletter";
        } else if ((length <= 60) && (length + width + thickness <= 90) && (weight <= 2000)) {
            return "Maxiletter";
        } else {
            return "Not a letter";
        }
    }
}

Når jeg compiler får jeg følgende fejlmelding her: illegal start of expression public String getType(){ if(l <= 23) && (w <= 17) && (we <=50) &&

(Alt hvad jeg har skrevet indtil videre ser sådan ud.)

public class Category
{
    // instance variables
    private int length, width, weight;
    private double thickness;   
    private string country;
   

    /**
    * Constructor for objects of class Category
    */
public Category(int l, int w, int  we, int t)
    {
        // initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;
        country = c;
    }
   
    //*Find out which type of letter we are dealing with according to weight,width, length and thicknes.*/
     
      public String getType(){ if(l <= 23) && (w <= 17) && (we <=50) &&
      (t <=0.5)
      {
      return "Standardletter";
      }
      else if {if(l <= 33) && (w <= 23) && (we <= 1000) && (t <= 2)
      {
      return " Largeletter";
      }
      else if(l <= 60> && (l + w + t <= 90) && (we <= 2000)
      {
      return "Maxiletter";
      }
      }
   
   
      //* Then we have to make the postage calculation according to weight
        * country.*/

   
       


}

Du mangler parentser omkring hele if udtrykker - det er:

if ( (...) && (...) && (...) ) {

Har sat parenteser nu, men får så følgende fejlmelding: ";" expected
      public String getType()if((l <= 23) && (w <= 17) && (we <=50) && .............

Hvad mener du med ikke oversætter?

"ikke oversætter" = "fejl ved oversættelse"

Nu mangler der vist en "{" mellem "getTtype()" og "if".

ok, Nu får jeg denne fejlmeddelse : Illegal start of expression

      else if((l <= 60>) && (l + w + t <= 90) && (we <= 2000))

Der er en ">" for meget efter 60.

ok er fjernet tak. Nu får jeg denne fejlemelding:

Cannot resolve symbol symbol:     private string country;

Her er det jeg har skrevet:
public class Category
{
    // instance variables
    private int length, width, weight;
    private double thickness;   
    private string country;

Er det fordi jeg ikke har country med i If else?

Nej du har stavet string med lille s (det er String med stort S).

Nu kommer følgende fejlmelding;

cannot resolve symbol
symbol: variable c location: class Category country c;

// initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;
        country = c;
    }

Er c i argument-listen til constructoren ?

Ja jeg har skrevet den med forkortelsen c, det skulle jeg måske ikke

public class Category
{
    // instance variables
    private int length, width, weight;
    private double thickness;   
    private String country;
   

    /**
    * Constructor for objects of class Category
    */
public Category(int l, int w, int  we, int t)
    {
        // initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;
        country = c;

Den er ikke i argument listen:

public Category(int l, int w, int  we, int t)

public Category(int l, int w, int  we, int t, String c)

Nej det kan jeg godt se nu. :)

Hvis jeg skriver den i argumentlisten går det så ikke galt når skriver if, else? Der skal land jo ikke blandes ind det skal jo først ind når jeg skal til at beregne portoen

Nej.

Det er ikke et krav at man skal bruge variable til noget.

Ok, nu får jeg så følgende fejlmelding: cannot resolve symbol symbol: variable l location: class category       if((l <= 23) && (w <= 17) && (we <=50) &&

Alt hvad jeg skrevet til det er her:
public String getType(){
      if((l <= 23) && (w <= 17) && (we <=50) &&
      (t <=0.5))     
      {
      return "Standardletter";
      }
      else if ((l <= 33) && (w <= 23)&&(we <= 1000) && (t <= 2))
      {
      return " Largeletter";
      }
      else if ((l <= 60)&&(l + w + t <= 90)&&(we <= 2000))
      {
      return "Maxiletter";
      }
      else
      {
      return "Not a letter";



Kan alle de fejlmeldinger blive ved? :(

Hvorfor vil den ikke compile når jeg har lavet forkortelser?

Du skal bruge length som er navnet på instans varieblen ikke l som er
navnet på argumentet i constructoren.

Der forstår jeg ikke helt hvad forskellen er?

Argumentet til constructoren kan kun bruges i selve constructoren.

Instans variablen kan bruges i alle metoder.

ok tak igen, du er en MEGET MEGET MEGET stor hjælp. Nu kommer det som jeg så frygter allermest ved opgaven nemlige portoberegning.

Kan jeg godt lave den oveni alt det andet jeg har skrevet?

Ja - du laver bare en ny metode calculatePorto nedenunder
getType metoden.

ok kan jeg så også bruge if-else metoden her eller skal der noget mere avanceret til?

Jeg tror den er umulig at lave uden if else.

ok, jeg prøver at se hvad jeg kan gøre ved den.

Er der andre muligheder end if-else som stadigvæk er simple og forholdsvis nemme at forstå?

Ikke nogen som er relevant til problem-stillingen.

Ok tak, jeg kigger på det og håber jeg kan få det til at lykkes:)

Skulle country ikke også være nævnt et eller andet sted i opgaven ligesom brevetyperne er?

Ikke forstået.

Du gemmer conutry i constructor og kan bruge den i porto beregning.

Det kan jeg simpelthen ikke se logikken i det her. Jeg skal have country, vægt og brevtypen ind i if-else og return skal være postage.

Jeg skulle vist have givet op for længe for længe siden:(

vægt og country er instans variable
brev type kan du beregne med getType

altså har du alt det du skal bruge !

ok jeg prøver og ser hvad jeg kan få ud af det. Atter engang tak.

Jeg har fået sat det her ind til videre er det den rigtige fremgangsmåde?


        public String calculatePostage()
        {
        if ((weight <= 50) && (country.equals("Denmark"))
        {
        return "weight" + Weight + "postage" + postage
        }

Jeg ville nok lave det som:

public double calculatePostage() {
    if ((weight <= 50) && country.equals("Denmark")) {
        return 4.25;
    }
    return -1.0;
}

Hvorfor return -1.0; Og hvad med brevtypen

Java compileren vil brokke sig hvis der ikke altid er en retur værdi.

Jeg troede at du skulel sætte brev type ind, men lad gå:

public double calculatePostage() {
    if (getType().equals("Standardletter") && (weight <= 50) && country.equals("Denmark")) {
        return 4.25;
    }
    return -1.0;
}

du har stadig lidt der mangler inden de 2 andre brev typer er supporteret !

Det skulle jeg også, men jeg kunne ikke se hvor det skulle ind henne, det ser så nemt ud for dem der virkelig kan det her.

Ok, vil det så sige at return altid er -1.0?

kan jeg godt køre if hele vejen ned for først til sidst at skrive else { return no letter}?

Hvorfor får jeg denne fejlmelding return outside method 5.50 ?

Her er hvad jeg har skrevet:
{
        if ((getType().equals("Largeletter")) && (weight <= 50) &&
        country.equals("Denmark"))
      {
      return 5.50;
      }
      return -1.0;
      }

Det lyder som om det står uden for en metode.

Ja, så langt er jeg også med:) Men det står nøjagtig ligesom den med 4.25 og der fik jeg ingen fejl ved compilingen,

Prøv lige og post hele koden.

ok her er den:

public double calculatePostage()
        {
        if ((getType().equals("Standardletter")) && (weight <= 50)
            && country.equals("Denmark"))
            {       
        return 4.25;
        }
        return -1.0;
        }
       
        if ((getType().equals("Largeletter")) && (weight <= 50) &&
        country.equals("Denmark"))
      {
      return 5.50;
      }
      return -1.0;
      }

Der er jo også en parentes for meget.

Prøv med:

public double calculatePostage()
        {
        if ((getType().equals("Standardletter")) && (weight <= 50)
            && country.equals("Denmark"))
            {     
        return 4.25;
        }
     
        if ((getType().equals("Largeletter")) && (weight <= 50) &&
        country.equals("Denmark"))
      {
      return 5.50;
      }
      return -1.0;
      }

Jeg forstår det ikke helt længere er det ikke tilladt at copy paste for det er hvad jeg har gjort med det du lige har skrevet fordi der ikke var nogen fejlmelding, men når jeg paster og retter i det før jeg fejlmeldingen: Illegal start of type if ((getType()equals.("Largeletter")) && (weight <= 100)

Hvad er det jeg gør forkert kan du sige mig det?

Prøv lige og post hele koden.

Her kommer den:

  public double calculatePostage()
        {
        if ((getType().equals("Standardletter")) && (weight <= 50)
            && country.equals("Denmark"))
            {     
        return 4.25;
        }
     
        if ((getType().equals("Largeletter")) && (weight <= 50) &&
        country.equals("Denmark"))
      {
      return 5.50;
      }
      return -1.0;
      }

   
     
        if ((getType().equals("Largeletter")) && (weight <= 100)
            && country.equals("Denmark"))
            {     
        return 6.50;
        }
     
        if ((getType().equals("Largeletter")) && (weight <= 250) &&
        country.equals("Denmark"))
      {
      return 12.00;
      }
      return -1.0;
      }

Der er igen:

    return -1.0;
      }

for meget !

Fjern den første.

Ok tak det vil jeg gøre

STORT PROBLEM IHVERTFALD FOR MIG:)

Når jeg taster ind i selve BLUEJ for create object får jeg følgende fejlmelding:

Error possible loss of precision
found Double
Required int
int_bluej_param3 = 0.5;

Det er når jeg taster thickness ind. Hvordan retter jeg det

Det lyder som du forsøger at putte en double ind i en int.

Det kan du ikke.

Men iøvrigt kender jeg ikke noget til BlueJ.

Jeg har selv rettet fejlen det var i instance variables jeg havde lavet fejlen puha

Kender du slet ikke noget til Blue J???

Ikke spor.

Jeg bruger JBuilder, Eclipse m.v. til min Java programmering.

Når jeg nu kun skal finde frem til porto prisen ved hjælp af destination og vægt hvordan klarer jeg så lige det.

Masser af if sætninger !

Efter portoen er beregnet?

Øh - du bruger if sætningerne til at beregne porto. Du er allerede
startet med din calculatePostage metode. Den gør du færdig.

Og så er din klasse klar til test.

Her er min klasse:

import java.util.*;
/**
* Letter information what kind of letter are we dealing with defined by length,
* width, thickness and weight.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Category
{
    // instance variables
    private int length, width, weight;
    private double thickness;   
    private String country;
   

    /**
    * Constructor for objects of class Category
    */
public Category(int l, int w, int  we, double t, String c)
    {
        // initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;
        country = c;
    }
   
    //*Find out which type of letter we are dealing with according to weight,width, length and thicknes.*/
     
      public String getType(){
      if((length <= 23) && (width <= 17) && (weight <=50) &&
      (thickness <=0.5))     
      {
      return "Standardletter";
      }
      else if ((length <= 33) && (width <= 23)&&(weight <= 1000) && (thickness <= 2))
      {
      return " Largeletter";
      }
      else if ((length <= 60)&&(length + width + thickness <= 90)&&(weight <= 2000))
      {
      return "Maxiletter";
      }
      else
      {
      return "Not a letter";
      }
      }
   
   
   
      //* Then we have to make the postage calculation according to weight and country.*/
       
       
      public double calculatePostage()
        {
if ((getType().equals("Standardletter")) && (weight <= 50)
            && country.equals("Denmark"))
            {     
        return 4.25;
        }
       
       
if ((getType().equals("Standardletter")) && (weight <= 50)
            && country.equals("Europ, The Faroes Island and Greenland"))
            {     
        return 5.50;
        }
       
       
if ((getType().equals("Standardletter")) && (weight <= 50)
            && country.equals("Other foreign countries"))
            {     
        return 6.50;
        } 
       
       
if ((getType().equals("Largeletter")) && (weight <= 50) &&
        country.equals("Denmark"))
      {
      return 5.50;
      }
     
if ((getType().equals("Largeletter")) && (weight <= 100)
            && country.equals("Denmark"))
            {     
        return 6.50;
        }
     
if ((getType().equals("Largeletter")) && (weight <= 250) &&
        country.equals("Denmark"))
      {
      return 12.00;
      }
   
   
if ((getType().equals("Largeletter")) && (weight <= 500) &&
        country.equals("Denmark"))
      {
      return 20.00;
      }
     
if ((getType().equals("Largeletter")) && (weight <= 1000)
            && country.equals("Denmark"))
            {     
        return 28.00;
        }
     
if ((getType().equals("Largeletter")) && (weight <=50) &&
        country.equals("Europe, The Faroes Island and Greenland"))
      {
      return 8.50;
      }
if ((getType().equals("Largeletter")) && (weight <= 100)
            && country.equals("Europe, The Faroes Island and Greenland"))
            {     
        return 12.00;
        }
     
if ((getType().equals("Largeletter")) && (weight <= 250) &&
        country.equals("Europe, The Faroes Island and Greenland"))
      {
      return 19.00;
      }
   
if ((getType().equals("Largeletter")) && (weight <= 500) &&
        country.equals("Europe, The Faroes Island and Greenland"))
      {
      return 33.00;
      }

if ((getType().equals("Largeletter")) && (weight <= 1000)
            && country.equals("Europe, The Faroes Island and Greenland"))
            {     
        return 57.00;
        }
     
if ((getType().equals("Largeletter")) && (weight <=50) &&
        country.equals("Other foreign countries"))
      {
      return 11.50;
      }
   
if ((getType().equals("Largeletter")) && (weight <= 100)
            && country.equals("Other foreign countries"))
            {     
        return 18.00;
        }
     
if ((getType().equals("Largeletter")) && (weight <= 250) &&
        country.equals("Other foreign countries"))
      {
      return 33.00;
      }
   
if ((getType().equals("Largeletter")) && (weight <= 500) &&
        country.equals("Other foreign countries"))
      {
      return 56.00;
      }

if ((getType().equals("Largeletter")) && (weight <= 1000)
            && country.equals("Other foreign countries"))
            {     
        return 95.00;
        }
     
if ((getType().equals("Maxiletter")) && (weight <= 50) &&
        country.equals("Denmark"))
      {
      return 7.00;
      }
     
if ((getType().equals("Maxiletter")) && (weight <= 100)
            && country.equals("Denmark"))
            {     
        return 9.00;
        }
     
if ((getType().equals("Maxiletter")) && (weight <= 250) &&
        country.equals("Denmark"))
      {
      return 15.00;
      }
       
if ((getType().equals("Maxiletter")) && (weight <= 500) &&
        country.equals("Denmark"))
      {
      return 24.00;
      }
     
if ((getType().equals("Maxiletter")) && (weight <= 1000)
            && country.equals("Denmark"))
            {     
        return 32.00;
        }
   
if ((getType().equals("Maxiletter")) && (weight <= 2000)
            && country.equals("Denmark"))
            {     
        return 40.00;
        }
   
if ((getType().equals("Maxiletter")) && (weight <= 50) &&
        country.equals("Europe, The Faroes Island and Greenland"))
        {     
        return 14.00;
        }
   
if ((getType().equals("Maxiletter")) && (weight <= 100)
            && country.equals("Europe, The Faroes Island and Greenland"))
            {     
        return 19.00;
        }
     
if ((getType().equals("Maxiletter")) && (weight <= 250) &&
        country.equals("Europe, The Faroes Island and Greenland"))
      {
      return 26.00;
      }
       
if ((getType().equals("Maxiletter")) && (weight <= 500) &&
        country.equals("Europe, The Faroes Island and Greenland"))
      {
      return 41.00;
      }
     
if ((getType().equals("Maxiletter")) && (weight <= 1000)
            && country.equals("Europe, The Faroes Island and Greenland"))
            {     
        return 65.00;
        }
   
if ((getType().equals("Maxiletter")) && (weight <= 2000)
            && country.equals("Europe, The Faroes Island and Greenland"))
            {     
            return 100.00;
        }
   
if ((getType().equals("Maxiletter")) && (weight <= 50) &&
        country.equals("Other foreign countries"))
        {     
        return 19.00;
        }
   
if ((getType().equals("Maxiletter")) && (weight <= 100)
            && country.equals("Other foreign countries"))
            {     
        return 27.00;
        }
     
if ((getType().equals("Maxiletter")) && (weight <= 250) &&
        country.equals("Other foreign countries"))
      {
      return 41.00;
      }
       
if ((getType().equals("Maxiletter")) && (weight <= 500) &&
        country.equals("Other foreign countries"))
      {
      return 61.00;
      }
     
if ((getType().equals("Maxiletter")) && (weight <= 1000)
            && country.equals("Other foreign countries"))
            {     
        return 105.00;
        }
   
if ((getType().equals("Maxiletter")) && (weight <= 2000)
            && country.equals("Other foreign countries"))
            {     
            return 175.00;
            }
   
            else
            {
              return -1.0;
              }
}
   















}
Den er compilet og det hele men når vil have den til at hente double calculatePostage kommer den op og siger double method result = -1.0
Kan du sige mig hvad jeg har gjort galt her?

Kan du se hvad jeg har gjort galt udover at else return er -1.0, jeg ved bare ikke hvordan jeg skal rette den.

Og så har jeg lige et sidste spørgsmål:) Hvis jeg skal udregne portoen efter vægt og destination hvordan skal jeg så lave if-else koderne og hvor skal de ind henne?

Jeg håber du vil svare på disse ting så skal du nok få fred for mig:)

Jeg gik i seng.

I http://www.eksperten.dk/spm/362755 lyder det som om at de allerede har fundet
løsningen.

Et " " for meget.

Det virker nemlig også hos mig:

/**
* Letter information what kind of letter are we dealing with defined by length,
* width, thickness and weight.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Category {
    // instance variables
    private int length, width, weight;
    private double thickness;
    private String country;

    /**
    * Constructor for objects of class Category
    */
    public Category(int l, int w, int we, double t, String c) {
        // initialise instance variables
        length = l;
        width = w;
        weight = we;
        thickness = t;
        country = c;
    }

    //*Find out which type of letter we are dealing with according to weight,width, length and thicknes.*/

    public String getType() {
        if ((length <= 23)
            && (width <= 17)
            && (weight <= 50)
            && (thickness <= 0.5)) {
            return "Standardletter";
        } else if (
            (length <= 33)
                && (width <= 23)
                && (weight <= 1000)
                && (thickness <= 2)) {
            return "Largeletter";
        } else if (
            (length <= 60)
                && (length + width + thickness <= 90)
                && (weight <= 2000)) {
            return "Maxiletter";
        } else {
            return "Not a letter";
        }
    }

    //* Then we have to make the postage calculation according to weight and country.*/

    public double calculatePostage() {
        if ((getType().equals("Standardletter"))
            && (weight <= 50)
            && country.equals("Denmark")) {
            return 4.25;
        }

        if ((getType().equals("Standardletter"))
            && (weight <= 50)
            && country.equals("Europ, The Faroes Island and Greenland")) {
            return 5.50;
        }

        if ((getType().equals("Standardletter"))
            && (weight <= 50)
            && country.equals("Other foreign countries")) {
            return 6.50;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 50)
            && country.equals("Denmark")) {
            return 5.50;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 100)
            && country.equals("Denmark")) {
            return 6.50;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 250)
            && country.equals("Denmark")) {
            return 12.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 500)
            && country.equals("Denmark")) {
            return 20.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 1000)
            && country.equals("Denmark")) {
            return 28.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 50)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 8.50;
        }
        if ((getType().equals("Largeletter"))
            && (weight <= 100)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 12.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 250)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 19.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 500)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 33.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 1000)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 57.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 50)
            && country.equals("Other foreign countries")) {
            return 11.50;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 100)
            && country.equals("Other foreign countries")) {
            return 18.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 250)
            && country.equals("Other foreign countries")) {
            return 33.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 500)
            && country.equals("Other foreign countries")) {
            return 56.00;
        }

        if ((getType().equals("Largeletter"))
            && (weight <= 1000)
            && country.equals("Other foreign countries")) {
            return 95.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 50)
            && country.equals("Denmark")) {
            return 7.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 100)
            && country.equals("Denmark")) {
            return 9.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 250)
            && country.equals("Denmark")) {
            return 15.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 500)
            && country.equals("Denmark")) {
            return 24.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 1000)
            && country.equals("Denmark")) {
            return 32.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 2000)
            && country.equals("Denmark")) {
            return 40.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 50)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 14.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 100)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 19.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 250)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 26.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 500)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 41.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 1000)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 65.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 2000)
            && country.equals("Europe, The Faroes Island and Greenland")) {
            return 100.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 50)
            && country.equals("Other foreign countries")) {
            return 19.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 100)
            && country.equals("Other foreign countries")) {
            return 27.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 250)
            && country.equals("Other foreign countries")) {
            return 41.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 500)
            && country.equals("Other foreign countries")) {
            return 61.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 1000)
            && country.equals("Other foreign countries")) {
            return 105.00;
        }

        if ((getType().equals("Maxiletter"))
            && (weight <= 2000)
            && country.equals("Other foreign countries")) {
            return 175.00;
        } else {
            return -1.0;
        }
    }
    public static void main(String[] args) {
        Category brev = new Category(25,10,200,1,"Denmark");
        System.out.println(brev.calculatePostage());
    }
}