Hvordan laver jeg en metode (throw)

Læs dine hint. det er faltisk givet



void throwDie() {
kast = r.nextInt(6) + 1;
    }

jeg tænkte om man ikke bare danner en modulo (%) ?

void throwDie() {
    kast  =  (int)(Math.random() * 6 + 1);
    }

Det er hvad jeg har skrevet:

/**
* Write a description of class Die here.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Die
{
   
    private int die;
    private int eye;
    private int increment;
    private java.util.Random r;
   
    /**
    * Constructor for objects of class Die
    */
    public Die()
    {
    eye = 0;
    r = new java.util.Random();
    }
 
    /**
    * obtain a new number of eyes for this die.
    */
    public void throwDie() {
    eye = 6;
    }
   
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    if(0 < eye && eye < 7) {return eye;}   
    return eye;
    }
   
}

Hvor er mine fejl?

Sådan:

/**
* Write a description of class Die here.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Die
{
    // private int die; - SLETTES, BRUGES IKKE
    private int eye;
    private int increment;
    private java.util.Random r;
   
    /**
    * Constructor for objects of class Die
    */
    public Die()
    {
        r = new java.util.Random(); - FLYTTET OP

        // eye = 0; - SLETTES OG ERSTATTES MED:
        throwDie();
    }
 
    /**
    * obtain a new number of eyes for this die.
    */
    public void throwDie() {
        // if((0 <= eye && eye <= 6)) {r.nextInt(6);} - NEJ, SOM NEDENFOR:
        eye = r.nextInt(6) + 1;
    }
   
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
        // if(0 < eye && eye < 7) {return eye;} - HVORFOR TJEKKE?
        return eye;
    }
}

Denne:

private int increment;

kan også slettes - bruges ikke.

Hvordan afgrænses den så? Hvis man slår med en terning kan man jo ik få 11. Den skal ligge mellem 1-6.

Jeg håber, at du vil hjælpe mig.

Mon ikke svaret allerede er givet i linien:

    eye = r.nextInt(6) + 1;

r.nextInt(6) giver et tal mellem 0 og 5 derfor +1 (1-6)

http://java.sun.com/javase/6/docs/api/java/util/Random.html#nextInt(int)

Det gør throwDie() allerede helt af sig selv. Prøv at genlæse Vink 2 i dit link.

Nårh ja, det er eye = r.nextInt(6) + 1;

Nu skal jeg lave det med to terninger. Hvordan gør jeg det?

Hvad mener du præcit med at du skal lave det med to terninger?

Jeg har gjort følgende:

/**
* Write a description of class Diecup here.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Diecup
{


    private int eye1;
    private int eye2
    private java.util.Random r; 

    public void throwDie() {
    eye1 = r.nextInt(6) +1;
    eye2 = r.nextInt(6) +1;
    }
   
  /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return eye1 + eye2;

    }
   

}

Hvad er der galt?

Her er den tilsvarende.

/**
* Write a description of class Diecup here.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Diecup
{


    private int Die d1; ////Når jeg compiler siger den ; expected
    private int Die d2;
    private java.util.Random r; 

    public void throwDice() {
    Die d1 = r.nextInt(6) +1;
    Die d2 = r.nextInt(6) +1;
    }
   
  /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return "Die d1" + "Die d2";

    }
   

}

Opgaven lyder helt eksakt:
Lav en klasse DieCup, der repræsenterer et raflebæger indeholdende to terninger. Der skal være to metoder i klassen, throwDice og getEyes, der giver mulighed for dels at slå med bægeret (dvs. med begge terninger), dels at inspicere hvor mange øjne der blev slået i alt.
        --------------------
        |      DieCup      |
        |------------------|
        | Die d1          |
        | Die d2          |
        --------------------
        | void throwDice() |
        | int getEyes()    |
        -------------------- 
Hint: Implementer klassen med to feltvariabler af typen Die uden iøvrigt at spekulere på hvordan klassen Die er implementeret.

Lav en constructor efter samme opskrift som i et-terning tilfældet.

Sludder ...

Du skal ikke lave endnu et Random objekt - det har du jo allerede i Die klassen, og den genbruges bare. Men du skal lave en constructor.

Er den constroctor god nok:
public Diecup() {
   
        r = new java.util.Random();
        throw(Die d1 + Die d2);
    }

Nej ... som sagt skal du ikke oprette en Random for sådan en har du allerede i den Die-klasse.

Din constructor skal faktisk bare indeholde dette:

    public Diecup() {
        d1 = new Die();
        d2 = new Die();
    }

Og i din throwDice() og din getEyes(} metoder skal du udnytte at de to Die objekter d1 og d2 hver især har sådan nogle metoder. Brug dem.

public void throwDice() {
    Die d1 = r.nextInt(6) +1;
    Die d2 = r.nextInt(6) +1;
    }
    Her er de som to metoder, men vi skal jo have summen ik?

Ikke sådan - sådan:

    public void throwDice() {
        d1.throwDice();
        d2.throwDice();
    }

Er det rigtig:
public class Diecup
{


    private int d1;
    private int d2;
    private java.util.Random r; 

   
    public Diecup() {
   
        d1 = new Die();
        d2 = new Die();
    }
    }
   
    public void throwDice() {
    d1 = throwDice();
    d2.throwDice();
    }
   
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return "new Die" + "new Die";

    }
   

}

Nej:

1) Genlæs lige 23/09-2007 16:22:00
2) Din getEyes() vil slet ikke kompilere - du bør spørge dog selv om hvorfor.
3) Din getEyes() bør udnytte det faktum at d1 og d2 også har getEyes() funktioner.

public int getEyes() {
    return d1();
    return d2;

    }

Hvordan elles? Jeg er tabt..

Nej, du skal

a) find antal øjne på d1
b) find antal øjne på d2
c) læg de to tal sammen
d) returner summen

Det kan klares med 4 linier, hver svarende til et af disse 4 punkter. Det kan også løses på een linie, men tænk over det senere.

Altså 3 void metoder og en get-metode?

Hvad mener du?

eller er det det hele i get-metoden

/**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return d1();
    return d2();
    return d1 + d2

    }

Kan du forklare mig hvordan du håndterer

  a) find antal øjne på d1

Jeg kan ikke se et eneste sted i din kode, hvor du prøver det??

Den er her hele koden:

/**
* Write a description of class Diecup here.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Diecup
{


    private int Die d1;
    private int Die d2;
    private java.util.Random r; 

   
    public Diecup() {
   
        d1 = new Die();
        d2 = new Die();
    }
    }
   
    public void throwDice() {
    d1 = throwDice();
    d2.throwDice();
    }
   
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return d1();
    return d2();
    return d1 + d2

    }
   

}

23/09-2007 17:20:32> Nej.

Når du har en "return" så stopper funktionen *der*. Med din første "return d1();" så udføres de to sidste linjer slet *ikke*.

I øvrigt er det forkert med:

return d1();

da din getEyes() jo skal returnere en int og din d1 er objekt af typen Die.

Jeg er ligeglad med hele din kode. Jeg har ikke bedt om den. Vi kigger på getEyes-metoden i Diecup-klassen. Ikke andet. Hvordan vil du løse punkt a) i den liste jeg har givet dig?

Jeg skal finde antallet af øjne på d1, men hvordan gør jeg det?

Det er jo et godt spørgsmål, hvordan du vil gøre det. Du har en klasse Die. Den har du "selv" lavet. Mon ikke den skal bruges.

Jeg kan kun se, at jeg har lavet en void-metode, som giver værdierne for de to terninger.

Så skal jeg returnere værdien for summen af de to terninger. Det kan jeg ikke løse..

Jeg kan regne ud du går på 1. semester på Datalogi på Århus Universitet. Du har haft programmering i et par uger eller så'n. Det er kun rimeligt at være lidt forvirret og ikke helt vide hvad der er hvad. Men både i opgaverne, og i de indlæg du læser her, er der helt klare og utvetydige vejledninger til hvad du skal gøre. Jeg kan ikke se at du læser dem, eller forstår dem.

Skal du gennemføre et studium på universitet, så skal du være i stand til at læse.

Når du kommer til efterårsferien, så sæt dig ned og find ud af med dig selv hvad du rent faktisk har lært og kan klare selv. Brug noget tid på det. Vær ærlig med dig selv.

Er konklusionen at du ikke har lært noget, og ikke kan noget, så er der vel to muligheder:
  1) Gi' op
  2) Tage fat på studiet på en anden måde.

Du kan ikke gennemføre ved at 3-4 personer hjælper dig igennem hver eftermiddag - og det gider vi nok heller ikke... ;)

Du skal slet ikke tænke på noget med summen endnu. Jeg beder dig overveje hvordan du finder ud af punkt a) på min liste: find antal øjne på d1.

Okay, men hvordan kan jeg blive bedre til det?
Jeg ved ikke, hvordan jeg finder antallet af øjne..

Du har en klasse Die. Den har du "selv" lavet. Mon ikke den skal bruges.

ER nået frem til dette:

/**
* Write a description of class Die here.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Diecup
{
   

    private int eye1;
    private int eye2;
    private java.util.Random r;
   
   
    /**
    * Constructor for objects of class Die
    */
    public Diecup()
    {
        r = new java.util.Random();
        throwDie();
    }
 
    /**
    * obtain a new number of eyes for this die.
    */
    public void throwDie() {
    eye1 = r.nextInt(6) +1;
    eye2 = r.nextInt(6) +1;
    }
   

   
   
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return eye1 + eye2;
    }
   
}

Jeg har nu testet objectet men den retunerer altid 5. hvordan kan det være?

Læs: det er i direkte modstrid med opgaven, som du skriver den i 23/09-2007 15:50:53

Men kan jeg ikke bare sætte den til det samme som den første opgave bare med (12) istedet for (6) ?

Du skal gøre som der står i opgaveformuleringen.

Du får -3 (er det ikke sådan nu...) hvis du bare laver en løsning, der virker.

Du skal gøre det på den måde, der bliver bedt om.

Du kunne jo starte med at spørge (os) om præcist hvad du ikke forstår.

Okay, jeg har lavet to metoder, som står i koden, men vil du forklare mig, hvad der er galt i koden? Jeg forstår det ikke helt.
Jeg har prøvet at forstå næsten hele dagen

Koden i 23/09-2007 17:43:43 var nu ellers meget god da den viser at du faktisk kan tilpasse en anden løsning som dur. Og den giver forresten ikke 5 hele tiden.

Men, erikjacobsen har ret, det er jo *ikke* den korrekte løsning på den opgave du har fået stillet.

"Du kunne jo starte med at spørge (os) om præcist hvad du ikke forstår."  - bemærk ordet "præcist".

Det ville være så let for os bare at give dig løsningen, hente nogle point, og så komme videre med vore liv - men det er dig der skal lære det.

Lad os så alligevel snakke lidt om det.

A) Du bliver bedt om at lave en klasse Die - den står et sted deroppe. Du er velkommen til at vise os hvordan din ser ud nu.

B) Du bliver bedt om at labe en klasse DieCup, som bruger den klasse du har lavet i A). Klassen DieCup er ligegyldig - i denne sammenhæng - hvis du ikke bruger klassen Die. Den er der her ovenfor i en slags foreløbig udgave, men den sidste du viser bruger ikke klassen Die.

Rigtigt, nielle 'det er jo *ikke* den korrekte løsning'  -  og da det er en skole-opgave, så er den ikke bare ikke-korrekt - den er forkert. Selv om den så måske endda virker.

Jeg ved det, men det er nu stadig en positiv ting at kunne tilpasse anden kode på en korrekt måde :^)

Det lange svar på a):

d1 er et objekt af klassen Die. Die-klassen har en metode som hedder getEyes() og den returnere antallet af øjne. Du får derfor antallet af øjne ved dette: d1.getEyes(). Den værdi bliver du nødt til at gemme, hvis du skal bruge den sernere. Og det skal du jo skal hvis du vil udregne summen mellem d1- og d2-terningerne. Den bedste løsning på det problem er at gemme den i en variabel specielt til formålet. Da getEyes() returnere en int-type skal variablen også være en int.


Derfor:

Det korte svar på a):

int eye1 = d1.getEyes();


Derfor:

public class Diecup
{
    private int Die d1;
    private int Die d2;
    private java.util.Random r; 
   
    public Diecup() {
        d1 = new Die();
        d2 = new Die();
    }

    // ...
   
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
        int eye1 = d1.getEyes();
        // b) find antal øjne på d2
        // c) læg de to tal sammen
        // d) returner summen
    }
}

Og så mangler vi en frivillig til at rette linien

      private int Die d1;

;)

public class Diecup
{
   

    private int Die d1;
    private int Die d2;
    private java.util.Random r;
   
   
    /**
    * Constructor for objects of class Die
    */
    public Diecup()
    {
        r = new java.util.Random();
        throwDie();
    }
 
    /**
    * obtain a new number of eyes for this die.
    */
    public void throwDie() {
    Die d1 = r.nextInt(6) +1;
    Die d2 = r.nextInt(6) +1;
    }
 
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return Die d1 + Die d2;
    }
}

Den anden er :

/**
* Write a description of class Die here.
*
* @author (your name)
* @version (a version number or a date)
*/
public class Die
{
   

    private int eye;
    private java.util.Random r;
   
   
    /**
    * Constructor for objects of class Die
    */
    public Die()
    {
        r = new java.util.Random();
        throwDie();
    }
 
    /**
    * obtain a new number of eyes for this die.
    */
    public void throwDie() {
    eye = r.nextInt(6) +1;
    }
   
    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    return eye;
    }
   
}

Og hvad skal vi bruge

    private java.util.Random r; 

til ??

Vil du forklare mig, hvad jeg gør forkert? Jeg ved det ikke selv, og jeg prøver virkelig at se det!

Skal    private java.util.Random r;  kun fremhæves i klassen "Die"?

23/09-2007 18:06:55> korrekt :^)

Ole:  Die d1 ...  er en erklæring af en variabel. Det er forkert (her) at erklære d1 een gang til nede i metoden.

Ole: hvad skal du bruge  private java.util.Random r;    til i DieCup ?

"Vil du forklare mig, hvad jeg gør forkert?"  - nej, jeg vil ikke. Det er den helt forkerte indstilling, hvis du skal lære noget. DU skal finde spørgsmålene - de præcise spørgsmål. Eet af gangen.

Er det ikke en nødvendig feltvariabel?

Hvad skal du bruge den til i DieCup?

Den skal kun bruges i Die så

Ja, altså hvis jeg skal fortolke det du siger som at du ikke ved eller mener at den skal  bruges i DieCup, som jeg spurgte efter, så er det nok rigtigt at den ikke skal være i DieCup.

Kan du fortælle mig, Ole, hvad der sker i denne linie:

    private int Die d1;

Vink: den er ikke rigtig!

Spørgsmål: " Der skal være to metoder i klassen, throwDice og getEyes, der giver mulighed for dels at slå med bægeret (dvs. med begge terninger), dels at inspicere hvor mange øjne der blev slået i alt. " Hvordan slår man med to på én gang?

Den ser rigtig ud. Skal det være private int(Die d1);

Nej, både "int" og "Die" er en type - der skal kun være en type. En af dem er altså for meget.

Skal "int" fjernes?

Kan du i det mindste argumentere hvorfor det skulle være int som skal fjernes?

Nej egentlig ikke, fordi jeg plejer at skrive "int" i de andre feltvariabler..

d1 (og d2) skal være et objekt af klassen Die. De er ikke heltal (hvilket ville være int) - de er terninger.

når okay. det forstår jeg

Prøv så at nærlæse 23/09-2007 18:05:25

/**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    int eye1 = d1.getEyes();
    int eye2 = d2.getEyes();
    Die = int eye1 + int eye2;
    return Die;
    }

De to sidste linjer er forkerte - når eye1 og eye2 er heltal, må summen nødvendigvis også være et heltal.

Hvordan formuleres det

`så
int Die  = int eye1 + int eye2;
    return int Die;

Prøv at køre den - værktøjerne plejer at fortælle hvad der er galt.

class extpected siger den

ved    int Die = int eye1 + int eye2;

Jeg har skrevet dette:

    /**
    * = the number of eyes of thus die.
    */
    public int getEyes() {
    int eye1 = d1.getEyes();
    int eye2 = d2.getEyes();
    int Die = int eye1 + int eye2;
    return int Die;
    }

Ergo er der nok fejl i den linje.

(og faktisk er der hele 3 - hvoraf 2 dog er ens).

Hvordan ser den så ud? Jeg kan ikke se det

Jeg synes at du skulle kigge i din bog - der må være mindst et eksempel på hvordan man lægger to variable sammen ... meget tæt på starten.

Vil du ik fortælle mig det i det her tilfælde, det er det eneste jeg mangler

Nej.

Det her er *meget* fundamentale spørgsmål du stiller ... hvis du ikke selv kan svare på dem efter at have læst de første par sider i jeres lærenbog ... så skal du læse den igen!

Jeg vil dog soge så meget som at der ikke skal stå int Die på venstresiden af =-tegnet. "Die" er allerede brugt til et andet formål, og kan derfor ikke bruges.

Hvordan lærer man programmering bedst?

Hvorfor vil I ikke hjælpe?

Det er dig som er ved at studerer datalogi.

Hvis du kan spørge om helt elemæntære ting efter at have læst bogen ... ja, så behøver du altså at læse bogen igen! At vi giver dig svaret, vil det bare være at gøre dig en rigtig dårlig tjeneste.

så læser jeg bogen

Fandt du ud af hvordan det skal se ud?

Hvad med noget respons?