[Java] Probleem met berekening van waardes

[Verwijderd]

Voor het vak informatica ben ik bezig met het maken van een applet waarin je een spelletje kunt spelen. Iedere speler heeft een aantal lucifers. Je raadt in het spelletje het aantal lucifers en speelt tegen de computer en je kunt een ronde winnen door het juiste aantal te raden of door het aantal te raden wat het dicht bij het werkelijke aantal zit.
Ik kom op het volgende probleem. Ik wil dat de computer de overige spelers een random aantal laat raden en dat deze waarde in een array genaamd geradenLucifers komt. Ik heb volgens mij de berekening goed genoteerd, alleen de waarde die steeds wordt toegevoegd aan de array is steeds 0. Hij doet de berekening dus niet goed. Het berekenen van de waarde gebeurt in regel 180. Onderaan de applet zie je een lijst met de waardes uit de array. Je moet bij speler een aantal kiezen, bij rondes en bij in je hand. Dan zou hij het moeten doen. Wie ziet er een fout?

code:

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

public class Knobbelen extends Applet implements ItemListener, ActionListener  
{
    
    int aantalSpelers;
    int aantalLucifersSpeler;
    int aantalLucifers;
    int aantalLucifersTotaal;
    int geradenLucifersSpeler;
    int geradenLucifersComputer;
    int[] geradenLucifers = new int[10];
    int[] verschilLucifers = new int[10];
    String[] spelers = new String[10];
    int[] totaalscore = new int[10];
    // int aantalLucifersMogelijk;
    int startnummer = 0;
    int volgenderonde;
    int totaalnummer;
    int tempSpeler;
    int rondeNr;
    // double geraden;
    
    int minimaalAantalLucifersMogelijk;
    int aantalLucifersMogelijk;
    int totaalLucifersMogelijk;
    
    int breedte1;
    int diameter1;
    int breedte2;
    int diameter2;
    int breedte3;
    int diameter3;
    
    final int MAXLUCIFERS = 3; // final wil zeggen dat de inhoud van de variabele niet veranderd kan worden, de waarde blijft dus altijd 3
       
    Choice selAantalLucifersSpeler = new Choice();
    Choice selAantalSpelers = new Choice();
    Choice selAantalRonden = new Choice();
    Choice selTotaalAantalLucifers = new Choice();
    TextField tekstvak;
    TextField invoervak;
    TextField uitvoervak;
    TextField uitvoervak2;
    Button startknop = new Button( "Start" );
    Button volgenderonde2 = new Button( "Volgende ronde" );
    Button eindberekening = new Button( "Totaal" );
// maak een keuzelijst en een tekstvak
    
    public void init() 
    {
        setLayout( null );
        
        // Stel beginbreedte in op 0 en begindiameter ook op 0
        
        aantalSpelers = 0;
        breedte1 = 0;
        diameter1 = 0;
        breedte2 = 0;
        diameter2 = 0;
        breedte3 = 0;
        diameter3 = 0;
        
        // Aantal spelers
        
        selAantalSpelers.addItem("2");
        selAantalSpelers.addItem("3");
        selAantalSpelers.addItem("4");
        selAantalSpelers.addItem("5");
        selAantalSpelers.addItem("6");
        selAantalSpelers.addItem("7");
        selAantalSpelers.addItem("8");
        selAantalSpelers.addItem("9");
        selAantalSpelers.addItem("10");
        selAantalSpelers.setBounds( 320, 39, 90, 20 );
        selAantalSpelers.addItemListener( this );       // Voeg listener toe aan item van keuze
        
        // Aantal ronden
        
        selAantalRonden.addItem("1");
        selAantalRonden.addItem("2");
        selAantalRonden.addItem("3");
        selAantalRonden.addItem("4");
        selAantalRonden.addItem("5");
        selAantalRonden.addItem("6");
        selAantalRonden.addItem("7");
        selAantalRonden.addItem("8");
        selAantalRonden.addItem("9");
        selAantalRonden.addItem("10");
        selAantalRonden.setBounds( 320, 64, 90, 20 );       // Voeg listener toe aan item van keuze
        selAantalRonden.addItemListener( this );        // Voeg listener toe aan item van keuze
                    
        // Aantal lucifers
        
        selAantalLucifersSpeler.addItem("0");
        selAantalLucifersSpeler.addItem("1");
        selAantalLucifersSpeler.addItem("2");
        selAantalLucifersSpeler.addItem("3");
        selAantalLucifersSpeler.setBounds( 320, 89, 90, 20 );
        selAantalLucifersSpeler.addItemListener( this );            // Voeg listener toe aan item van keuze
        
        // Het raden van het aantal lucifers
        selTotaalAantalLucifers.setBounds( 320, 114, 90, 20 );      
        selTotaalAantalLucifers.addItemListener( this );        // Voeg listener toe aan item van keuze

        startknop.setBounds( 35, 150, 100, 27 );
        startknop.addActionListener (this );
        
        volgenderonde2.setBounds( 155 , 150, 100, 27 );
        volgenderonde2.addActionListener (this );
        
        eindberekening.setBounds( 275, 150, 100, 27 );
        eindberekening.addActionListener (this );
        
        tekstvak = new TextField( 10 );
        tekstvak.setBounds( 25, 239, 90, 20 );
        
        invoervak = new TextField( 10 );
        invoervak.setBounds( 320, 264, 90, 20 );
        invoervak.addActionListener (this );
        
        uitvoervak = new TextField( 10 );
        uitvoervak.setBounds( 25, 200, 90, 20 );
        
        uitvoervak2 = new TextField( 10 );
        uitvoervak2.setBounds( 25, 268, 90, 20 );
        
        
        add( tekstvak );
        add( invoervak );
        add( uitvoervak );
        add( uitvoervak2 );
        add( selAantalLucifersSpeler );
        add( selAantalSpelers );
        add( selAantalRonden );
        add( selTotaalAantalLucifers );
        add( startknop );
        add( volgenderonde2 );
        add( eindberekening );
    }
    
    int totaallucifers() 
    {
        
        aantalLucifersSpeler = Integer.valueOf(selAantalLucifersSpeler.getSelectedItem()); // Haal het aantal lucifers op wat de speler heeft geselecteerd
        double tempTotaalAantalLucifers;
                
        /* 
         * Er wordt een willekeurige waarde berekend tussen 0 en 3 vermenigvuldigd met het aantal 
         * computerspelers minus de speler zelf plus het aantal lucifers dat de speler zelf heeft geselecteerd
        */

        tempTotaalAantalLucifers = ((Math.random() * MAXLUCIFERS) * (aantalSpelers - 1)) + aantalLucifersSpeler; 
        aantalLucifersTotaal = (int) tempTotaalAantalLucifers;
        
        return aantalLucifersTotaal;
    }
    
    //int geradenlucifersSpeler()
    void geradenlucifersSpeler()
    {
        
        geradenLucifers[0] = Integer.valueOf(selTotaalAantalLucifers.getSelectedItem());
        //geradenLucifers[0] = Integer.valueOf(invoervak.getText());    // geradenlucifers van speler 1 worden uit invoervak gehaald    
            
        //return geradenlucifers;
    }
    
    //int geradenlucifersComputer()
    void geradenlucifersComputer()
    {
        double geraden;
        int computerSpelers;
        
         for (computerSpelers = 1; computerSpelers < aantalSpelers; computerSpelers++)
         {
        
            geraden = ((Math.random() * MAXLUCIFERS) * (aantalSpelers ));
            geradenLucifers[computerSpelers] = (int) geraden;
            
            //  return geradenlucifers;
            // geradenlucifers van overige spelers wordt berekend en aan programma teruggegeven
          }
    }
    
    
    // verschil tussen geraden aantal en werkelijke aantal wordt voor iedere speler bepaald
    
    void verschilLucifers()
    {
        int verschil;
        int spelers;
        
        for (spelers = 0; spelers < aantalSpelers; spelers++)
        {
        
            int tempSpelers = spelers;
            verschil = geradenLucifers[spelers] - aantalLucifers; // De waarde van de geraden lucifers wordt per speler uit de array gehaald en het verschil wordt berekend van het werkelijke aantal lucifers
            verschilLucifers[spelers] = Math.abs(verschil); // De absolute waarde wordt berekend en in de array geplaatst
            verschilLucifers[spelers] = tempSpelers + 1;
        }
    }
    
    public void bepaalWinnaar() 
    {
        // pak de eerste waarde van de eerste speler
        // vergelijk deze met allemaal tot aan het einde van de array
        // wanneer vergeleken waarde kleiner is dan de huidige waarde (de kleinste waarde), wordt de vergeleken waarde de kleinste waarde
    
        int spelers;
        int tempVerschil = verschilLucifers[0]; // Pak de eerste waarde
        // int tempSpeler;
        
        for (spelers = 0; spelers < aantalSpelers; spelers++)
        {
            if (verschilLucifers[spelers] < tempVerschil) // Wanneer de waarde van het tijdelijke verschil groter is dan de waarde van het verschil in de array
            {
                tempVerschil = verschilLucifers[spelers]; // vervang dan de tijdelijke waarde met de waarde uit de array
                tempSpeler = spelers + 1;                     // sla de speler waarbij het kleinste verschil hoort
            }   
        }
    
        totaalscore[spelers] = totaalscore[spelers] + 1;     // Tel voor de winnaar er een punt bij
    
    }
    
    
    
    public void paint( Graphics g ) 
    {
        g.setColor( Color.black );
        g.setFont (new Font ("SansSerif",Font.BOLD, 20));
        g.drawString( "Knobbelen", 200, 25);
        g.setFont (new Font ("Dialog",Font.PLAIN, 13));
        g.drawString( "Kies het aantal lucifers dat je in je hand hebt", 25, 100);
        g.drawString( "Kies het aantal spelers", 25, 50);
        g.drawString( "Kies het aantal rondes", 25, 75);
        g.drawString( "Voer het aantal lucifers in dat je raadt", 25, 125);
        g.drawString( "Speler " + tempSpeler + " heeft de " + rondeNr + "e ronde gewonnen!", 40, 300 );
        g.drawString( "geradenlucifers = " + geradenLucifers[0] , 40, 350 );
        g.drawString( "geradenlucifers = " + geradenLucifers[1] , 40, 370 );
        g.drawString( "geradenlucifers = " + geradenLucifers[2] , 40, 390 );
        g.drawString( "geradenlucifers = " + geradenLucifers[3] , 40, 410 );
        g.drawString( "geradenlucifers = " + geradenLucifers[4] , 40, 430 );
        g.drawString( "geradenlucifers = " + geradenLucifers[5] , 40, 450 );
        g.drawString( "geradenlucifers = " + geradenLucifers[6] , 40, 470 );
        g.drawString( "geradenlucifers = " + geradenLucifers[7] , 40, 490 );
        g.setColor( Color.black );
        g.fillRect( 100, 200, breedte1, 50 );
        g.setColor( Color.red );
        g.fillOval( 94, 200, diameter1, diameter1 );
        g.setColor( Color.black );
        g.fillRect( 200, 200, breedte2, 50 );
        g.setColor( Color.red );
        g.fillOval( 194, 200, diameter2, diameter2 );
        g.setColor( Color.black );
        g.fillRect( 300, 200, breedte3, 50 );
        g.setColor( Color.red );
        g.fillOval( 294, 200, diameter3, diameter3 );
    }

    // Teken een lucifer met breedte 'breedte' en diameter van het kopje 'diameter'
    
    public void actionPerformed (ActionEvent e) 
    {
       /*
        
            if (geradenLucifers[0] == aantalLucifersTotaal)  //Integer.valueOf(invoervak.getText()) == aantalLucifersTotaal) 
            {
                invoervak.setText( "goed!,geraden=" + aantalLucifersTotaal );
            }
            
            else 
            {
                    invoervak.setText( "fout, aantal moet zijn " + aantalLucifersTotaal );
            }
            */
            
            if ( e.getSource() == startknop ) {
                startnummer = 1;
                uitvoervak2.setText( "startnummer = " + startnummer );  
                // return (startnummer);
                
            }
            
            if (e.getSource() == volgenderonde2 ) {
                totaallucifers();
                geradenlucifersSpeler();
                geradenlucifersComputer();
                verschilLucifers();
                bepaalWinnaar();
                volgenderonde = 1;
                uitvoervak2.setText( "volgenderonde = " + volgenderonde ); 
                
            }
            
            if (e.getSource() == eindberekening ) {
                totaalnummer = 1;
                uitvoervak2.setText( "eindberekening = " + totaalnummer );
            }
            
    }
    
    
    
    public void itemStateChanged(ItemEvent e) 
    {
        // if ( startnummer == 1)
        // {
        
        // --- Het aantal lucifers ---
        if (selAantalLucifersSpeler.getSelectedItem().toString() == selAantalLucifersSpeler.getItem(1).toString() ) 
        {
            breedte1 = 3;
            diameter1 = 15;
            breedte2 = 0;
            diameter2 = 0;
            breedte3 = 0;
            diameter3 = 0;
            repaint();
                       
        }
        
        // Teken de lucifer overnieuw als de keuze één is en voeg dan 3 toe aan de breedte en 15 aan de diameter
        else if (selAantalLucifersSpeler.getSelectedItem().toString() == selAantalLucifersSpeler.getItem(2).toString() ) 
        {
            breedte1 = 3;
            diameter1 = 15;
            breedte2 = 3;
            diameter2 = 15;
            breedte3 = 0;
            diameter3 = 0;
            repaint();
            
        }
    
        // Teken de lucifers overnieuw als de keuze twee is en voeg dan 3 toe aan de breedte en 15 aan de diameter
        else if (selAantalLucifersSpeler.getSelectedItem().toString() == selAantalLucifersSpeler.getItem(3).toString() ) 
        {
            breedte1 = 3;
            diameter1 = 15;
            breedte2 = 3;
            diameter2 = 15;
            breedte3 = 3;
            diameter3 = 15;
            repaint();
            
        }

        // Teken de lucifers overnieuw als de keuze drie is en voeg dan 3 toe aan de breedte en 15 aan de diameter
        else if (selAantalLucifersSpeler.getSelectedItem().toString() == selAantalLucifersSpeler.getItem(0).toString() ) 
        {
            tekstvak.setText("Fout"); // Nul lucifers mogen toch?
        }


        // --- Het aantal spelers ---

        if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(0).toString() ) 
        {
            tekstvak.setText("2 spelers!");
            aantalSpelers = 2;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
             uitvoervak.setText( "aantal = " + aantalLucifers );
             uitvoervak2.setText("" + 7 ); //verschilLucifers[0] );  
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(1).toString() ) 
        {
            tekstvak.setText("3 spelers!");
            aantalSpelers = 3;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
             uitvoervak.setText( "aantal = " + aantalLucifers );          
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(2).toString() ) 
        {
            tekstvak.setText("4 spelers!");
            aantalSpelers = 4;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
             uitvoervak.setText( "aantal = " + aantalLucifers );          
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(3).toString() ) 
        {
            tekstvak.setText("5 spelers!");
            aantalSpelers = 5;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
             uitvoervak.setText( "aantal = " + aantalLucifers );            
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(4).toString() ) 
        {
            tekstvak.setText("6 spelers!");
            aantalSpelers = 6;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
            uitvoervak.setText( "aantal = " + aantalLucifers );            
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(5).toString() ) 
        {
            tekstvak.setText("7 spelers!");
            aantalSpelers = 7;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
             uitvoervak.setText( "aantal = " + aantalLucifers );          
            
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(6).toString() ) 
        {
            tekstvak.setText("8 spelers!");
            aantalSpelers = 8;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
            uitvoervak.setText( "aantal = " + aantalLucifers );            
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(7).toString() ) 
        {
            tekstvak.setText("9 spelers!");
            aantalSpelers = 9;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
            uitvoervak.setText( "aantal = " + aantalLucifers );            
           
        }
        else if (selAantalSpelers.getSelectedItem().toString() == selAantalSpelers.getItem(8).toString() ) 
        {
            tekstvak.setText("10 spelers!");
            aantalSpelers = 10;
            int aantalLucifers;
            aantalLucifers = totaallucifers();
            uitvoervak.setText( "aantal = " + aantalLucifers );
        }
        setAantalLucifersMogelijk();
        }
        
        // setAantalLucifersMogelijk();
        
    
        
   

    
    
    
    
            
    // Er wordt een aparte class aangemaakt voor het bijhouden van de score

    public void setSpelers()
    {
        spelers[0] = "Speler 1";
        spelers[1] = "Speler 2";
        spelers[2] = "Speler 3";
        spelers[3] = "Speler 4";
        spelers[4] = "Speler 5";
        spelers[5] = "Speler 6";
        spelers[6] = "Speler 7";
        spelers[7] = "Speler 8";
        spelers[8] = "Speler 9";
        spelers[9] = "Speler 10";
    }
    
    public void setTotaalscore()
    {
        
        int spelers;
        
        for (spelers = 0; spelers < aantalSpelers; spelers++)
        {
            totaalscore[spelers] = 0;
        }
    }
    
        
    public void setAantalLucifersMogelijk() 
    {

        selTotaalAantalLucifers.removeAll(); // Leeg de lijst voordat er nieuwe worden toegevoegd

        totaalLucifersMogelijk = aantalSpelers * MAXLUCIFERS;
        minimaalAantalLucifersMogelijk = aantalLucifersMogelijk + aantalLucifersSpeler;
                
        System.out.println("Totaal lucifers mogelijk:" + totaalLucifersMogelijk + ", aantalSpelers " + aantalSpelers + " MAXL " + MAXLUCIFERS);
        
        for (aantalLucifersMogelijk = 0; aantalLucifersMogelijk <= totaalLucifersMogelijk; aantalLucifersMogelijk++)
        {
            selTotaalAantalLucifers.addItem(String.valueOf(aantalLucifersMogelijk)); 
        }
        
    }
}

[RobIII]

Het is hier in [PRG] niet de bedoeling dat we je code gaan zitten debuggen, en zéker geen lappen van bijna 500 regels code. Ik geef je even wat leesvoer:

• Hoe we graag wél een topic zien in [PRG]: Programming Beleid Quickstart
• Handig om zelf je probleem op te lossen: Debuggen: Hoe doe ik dat?
• Dit is wat we vinden van je topic: quickfix

Je bent van harte welkom met een nieuw topic, maar zorg dan op z'n minst dat je enkel relevante code post met een behoorlijke uitleg er bij (en als je je aan onze voorgenoemde Quickstart houdt kan het bijna niet mis). Succes!

Overigens, desalniettemin: welkom op Tweakers!

En regel 180 is leeg

[ Voor 10% gewijzigd door RobIII op 05-06-2007 02:23 ]