[java]probleem met sleep (huiswerk)

[MagicTempest]

Jullie kennen allemaal wel dat spelletje dat je 21 lucifers hebt en dat je om de beurt 1,2 of 3 lucifers van het stapeltje moet pakken, degene die de laatste pakt heeft verloren. Dat moet ik namaken in java. Tot nu toe geen probleem, je kan 1,2 of 3 lucifers pakken en je kan ook niet in de - komen met de lucifers. Als tegenstander moet ik de computer hebben die dus via math.random () telkens een random aantal lucifers van de stapel af neemt. Elke keer als je zelf ingevoerd hebt hoeveel lucifers je wilt pakken gaat de computer direct door, dus als jij er 3 pakt en de computer 2 worden er in 1 keer 5 vanaf gehaald. Je kunt dus niet zien wie wat doet. Nu dacht ik dat op te kunnen lossen met sleep, nu wacht hij wel even, maar alsnog kan je niet zien wie wat pakt, er wordt dus nog steeds in 1x 5 lucifers vanaf gehaald.

code:

package opdracht4;

import java.awt.*;
import java.awt.event.*;
import java.applet.*;

/**
 * <p>Title: opdracht 4f</p>
 * <p>Description: Je gaat een spel voor 2 personen maken. Op tafel ligt een
 * hoopje met 21 lucifers. Om beurten mogen de spelers 1, 2 of 3 lucifers
 * wegnemen. Wie de laatste lucifer pakt heeft verloren. De ene speler ben
 * je zelf en de andere is je programma dat met Math.random() het aantal
 * lucifers bepaalt om weg te nemen. Laat de lucifers die nog over zijn
 * telkens op het scherm zien. Aan het eind meldt het programma wie de
 * winnaar is. </p>
 * <p>Company: team 1 the Magic Circle </p>
 * @author Michael van Kley I102
 * @version 1.0
 */

public class opdracht4f extends Applet
{
  Button ok;
  TextField tekstvakje;
  int paaltjes = 21, stokjes, compu;
  double t;
  public void init ()
  {
    setLayout ( null );

    ok = new Button ("ok");
    ok.addActionListener ( new OkHandler() );
    ok.setBounds ( 200, 10, 20, 20 );

    tekstvakje = new TextField (10);
    tekstvakje.setBounds ( 100, 10, 40, 20 );

    add ( ok );
    add ( tekstvakje );
  }

  public void paint ( Graphics g )
  {
    g.drawString ( "" + paaltjes, 100, 100);
  }

  class OkHandler implements ActionListener
  {
    public void actionPerformed ( ActionEvent e )
    {
      String invoer = tekstvakje.getText ();
      int stokjes = Integer.parseInt ( invoer );

      if ( paaltjes >= 3 )
      {
        if(stokjes >=1)
        {
          if(stokjes <=3)
          {
            paaltjes -= stokjes;
            repaint ();
            try { Thread.sleep(500); }
            catch(InterruptedException f) {}
          }
        }
      }
      else
      {
        if ( paaltjes == 2 )
        {
          if(stokjes >=1)
          {
            if(stokjes <=2)
            {
              paaltjes -= stokjes;
              repaint ();
              try { Thread.sleep(500); }
              catch(InterruptedException f) {}
            }
          }
        }

        else
        {
          if ( paaltjes == 1 )
          {
            if(stokjes ==1)
            {
              paaltjes -= stokjes;
              repaint ();
              try { Thread.sleep(500); }
              catch(InterruptedException f) {}
            }
          }
        }
      }


      if ( paaltjes >= 3 )
      {
        compu = (int) ( 3* Math.random () + 1 );
        paaltjes -= compu;
        repaint ();
      }
      else
      {
        if ( paaltjes == 2 )
        {
          compu = (int) ( 2* Math.random () + 1 );
          paaltjes -= compu;
          repaint ();
        }
        else
        {
          if ( paaltjes == 1 )
          {
            paaltjes -= 1;
            repaint ();
          }
        }
      }
    }
  }
}

Dit is dus mijn code, nu zullen jullie allemaal zeggen dat het op zoveel manieren korter kan, maar ik heb nog maar 4 weken les in java. Kunnen jullie mij misschien helpen hiermee?

[FoggyFreek]

Sorry, geen huiswerk vragen

[^Mo^]

FoggyFreek schreef op 29 september 2002 @ 01:30:
Sorry, geen huiswerk vragen

Welkom in P&W (FAQ-13/7/2002)

[MagicTempest]

mijn vraag voldoet toch aan die eisen, of moet ik er het 1 en ander aan veranderen?

[Eskimootje]

Deze code is zo wel erg onleesbaar hoor. Post is alleen het stukje waar het mis gaat ofzo. Echt OOM is het al helemaal niet.

Je moet de thread.sleep(); wel gebruiken voordat je de computer iets laat doen nu w8 hij als jij iets gedaan hebt tot ie klaar is en vervolgens doet de compu ook meteen wat.

[Eskimootje]

Je kan ook proberen met een hele lange for lus kijken of dat wel werkt. for(int i = 0; i < 1000; i++) {
1+1;
}

[MagicTempest]

als ik een lus gebruik gaat hij meteen helemaal door totdat alle lucifers op zijn.

[Eskimootje]

Je moet wel ff zelf bedenken waar je de lus inzet he dr mag geen code instaan die je nu al hebt.

[TheOneLLama]

Je probleem heeft te maken met de event thread. Als er een event of iets dergelijkt gebeurt wordt de method (in dit geval actionPerformed) aangeroepen vanuit de event thread. Diezelfde thread is echter verantwoordelijk voor het tekenen van al je AWT componenten. Als jij een repaint() geeft gaat repaint() niet gelijk het hele scherm hertekenen. repaint zorgt er alleen voor de event thread dat gaat doen. Jij geeft echter een repaint *vanuit* de event thread en laat de event thread vervolgens een halve seconde wachten. Omdat die daar mee bezig is kan ie niet het scherm hertekenen.

[ Voor 0% gewijzigd door TheOneLLama op 29-09-2002 15:09 . Reden: Het klinkt nog steeds nergens naar maar iets beter ]

[TheOneLLama]

Eskimootje schreef op 29 september 2002 @ 01:52:
Je kan ook proberen met een hele lange for lus kijken of dat wel werkt. for(int i = 0; i < 1000; i++) {
1+1;
}

Ik wou deze graag nomineren voor de top 10 van stomste replies ooit in /14

[Alarmnummer]

TheOneLLama schreef op 29 september 2002 @ 15:11:
[...]


Ik wou deze graag nomineren voor de top 10 van stomste replies ooit in /14

Hij had het inderdaad kunnen optimaliseren.

code:

for(int k=0;k<1000;k++){
    2;
}

aaaaarrgghhh... krijg ik ook weer extra lege regels ertussen.

[MagicTempest]

tnx mensen, ik ga nu het 1 en ander proberen, als het dan nog niet lukt ga ik morgen wel naar me leraar toe, die moet het me uit kunnen leggen.

[wasigh]

Waarom print je niet gewoon de tekst: "de computer pakte er X" ergens in een Label?

[Alarmnummer]

Het probleem aan jouw probleem is dat je gui gaat mixen met logica en daardoor het overzicht gaat verliezen. Ik zou het zelf opzetten mbv MVC. Je gaat dan classes maken waarin de logica van jouw spel zit. Dus Speler, Bord, Steen en methodes getName(), makeMove(...) computerMove()

En dan ga je je gui up to date houden mbv Events. Op deze manier krijg je een veel beter inzicht wat er gebeurd en verder kan je probleem voor probleem alles oplossen ipv deze wanorde van functionaliteit.

Ik vind het beleid van jullie school daarom dus ook niet goed. Jullie moeten helemaal niet bezig gaan met gui. Gui is gewoon vrij complex en je krijgt daardoor geen goeie basis omdat je dus ook problemen moet oplossen die op dit moment nog helemaal buiten jullie kunnen vallen.

[Sick Nick]

Java:

public class Nimb
{
  private int numberOfMatches;
  private int player;

  /**
   * Constructor for objects of class Nimb
  */
  public Nimb(int matches)
  {
    numberOfMatches = 0;
    if (matches > 0) {
      numberOfMatches = matches;
    }
    player = 1;
    System.out.println(numberOfMatches + " on the table");
  }

  /**
   * take some matches off the stack
   */
  public void grab(int matches)
  {
    if (numberOfMatches == 0) {
      System.out.println("Game over");
    }
    else if (matches != 1 && matches != 2 && matches != 3) {
      System.out.println("You can only take 1,2, or 3 matches");
    }
    else if (numberOfMatches - matches < 0) {
      System.out.println("You can only take up to " + numberOfMatches + " matches");
    }
    else {
      numberOfMatches = numberOfMatches - matches;
      if (numberOfMatches == 0) {
        System.out.println("Game over: player " + player + " lost");
      }
      if (player == 1) {
        player = 2;
      }
      else {
        player = 1;
      } 
    }
    System.out.println(numberOfMatches + " matches remaining");
  }
}

Zo heb ik m moeten maken voor mijn opleiding, Informatica & Economie. We werken volgens het boek 'Object first with java, a practical introduction using BlueJ'
Bij deze speel je niet tegen de computer maar met 2 spelers, misschien dat je hier wat mee kan in ieder geval?

[Glimi]

Nog even een leukigheidje. Laat de computer als laatse beginnen, dan wint ie altijd.

Stel je hebt 21 lucifers en persoon X pakt er n ( n =1, n=2, n=3 ) af. Persoon Y pakt er dan 4-n vanaf. Zo worden er elke beurt 4 matches van de stapel afgehaald, waardoor na 5 beurten er 20 vanaf zijn. Persoon X is dan weer aan de beurt en moet de laatste pakken

[MagicTempest]

dat is niet altijd zo volgens mij hoor, want bij mij is het al ingesteld dat de computer als 2e begint, maar toch win ik het wel.

btw, me leraar wist het ook niet, of tenminste hij zei dat hij het wel wist, maar dat dat nog te moeilijk voor me zou zijn.

[Dash2in1]

Probeer het eens op die manier zou ik zeggen MagicTempest..

[J3roen]

MagicTempest schreef op 30 september 2002 @ 22:40:

btw, me leraar wist het ook niet, of tenminste hij zei dat hij het wel wist, maar dat dat nog te moeilijk voor me zou zijn.

Wat een onzin, dat is wel de meest irritante leraar ooit. Als niemand je het ooit uitlegt hoe kan je dan wat leren?

[The - DDD]

Eskimootje schreef op 29 september 2002 @ 01:52:
Je kan ook proberen met een hele lange for lus kijken of dat wel werkt. for(int i = 0; i < 1000; i++) {
1+1;
}

Gadverdamme!!

[mbravenboer]

The - DDD: Gadverdamme!!

Doet me denken aan die hele oude DOS spelletjes die niet meer werken omdat ze hun snelheid op deze manier regelen .

[The - DDD]

En er wordt 1 ding vergeten in dit hele verhaal...

De threading in een Applet. Probeer is de sleep voor de repaint van de computer actie(s) te zetten. Ipv. voor die van de speler. Als ik het zo even snel bekijk moet dat je (niet zo mooie oplossing) werkend maken.

Je bouwsels worden vanzelf mooier en duidelijker, met oefening.