Applet not quite working correctly, need second opinion

Question

I'm sure you guys are getting tired of seeing me post questions, but I have another one. The following code is a simple Card Bet game(High Low). The problem is the score is not accumulating correctly. Sometimes it add when with should subtract, and vice versa. I got tired of trying to figure out my problems with SecretPhrase so I figured I would try another part of my project, only to run into more problems.
Code:
   import javax.swing.*;
   import java.awt.*;
   import java.awt.event.*;
   import java.util.*;
    public class JCardBet extends JApplet implements ActionListener
      JLabel title = new JLabel("** Bet the Card **");
      JLabel compCard = new JLabel("");
      JLabel playerCard = new JLabel("");
      int compRand, playerRand;
      int total = 10;
      JLabel totalMoney = new JLabel("");
      JLabel gameOver = new JLabel("");
      JButton fiveHigh = new JButton("$5 Higher");
      JButton tenHigh = new JButton("$10 Higher");
      JButton fiveLow = new JButton("$5 Lower");
      JButton tenLow = new JButton("$10 Lower");
      JButton ok = new JButton("OK");
      Font font1 = new Font("Helvetica", Font.BOLD, 34);
      Font font2 = new Font("Helvetica", Font.BOLD, 20);
      String[] deck = {"2 of Spades", "2 of Hearts", "2 of Diamonds", "2 of Clubs",
                             "3 of Spades", "3 of Hearts", "3 of Diamonds", "3 of Clubs",
                        "4 of Spades", "4 of Hearts", "4 of Diamonds", "4 of Clubs",
                             "5 of Spades", "5 of Hearts", "5 of Diamonds", "5 of Clubs",
                           "6 of Spades", "6 of Hearts", "6 of Diamonds", "6 of Clubs",
                        "7 of Spades", "7 of Hearts", "7 of Diamonds", "7 of Clubs",
                             "8 of Spades", "8 of Hearts", "8 of Diamonds", "8 of Clubs",
                        "9 of Spades", "9 of Hearts", "9 of Diamonds", "9 of Clubs",
                        "10 of Spades", "10 of Hearts", "10 of Diamonds", "10 of Clubs",
                        "Jack of Spades","Jack of Hearts", "Jack of Diamonds", "Jack of Clubs",
                        "Queen of Spades", "Queen of Hearts", "Queen of Diamonds", "Queen of Clubs",
                        "King of Spades", "King of Hearts", "King of Diamonds", "King of Clubs",
                        "Ace of Spades", "Ace of Hearts", "Ace of Diamonds", "Ace of Clubs"};
       public void cardGenerator()
         Random generator = new Random();
         compRand = generator.nextInt(52);
         playerRand = generator.nextInt(52);
       public void cardSort()
         Arrays.sort(deck);
       public void init()
         Container con = getContentPane();
         setLayout(new FlowLayout());
         con.add(title);
         title.setFont(font1);
         cardGenerator();
         cardSort();
         con.add(compCard);
         compCard.setText("Computer: " + deck[compRand]);
         compCard.setFont(font2);
         con.add(fiveHigh);
         fiveHigh.addActionListener(this);
         con.add(tenHigh);
         tenHigh.addActionListener(this);
         con.add(fiveLow);
         fiveLow.addActionListener(this);
         con.add(tenLow);
         tenLow.addActionListener(this);
         con.add(playerCard);
         playerCard.setFont(font2);
         con.add(totalMoney);
         totalMoney.setText("Starting with $" + total);
         con.add(ok);
         ok.addActionListener(this);
         ok.setEnabled(false);
         con.add(gameOver);
       public void actionPerformed(ActionEvent e)
         Object source = e.getSource();
         if(source == fiveHigh)
            cardGenerator();
            playerCard.setText("Your card: " + deck[playerRand]);
            cardSort();
            ok.setEnabled(true);
            fiveHigh.setEnabled(false);
            tenHigh.setEnabled(false);
            fiveLow.setEnabled(false);
            tenLow.setEnabled(false);
            if(playerRand > compRand)
               total = total + 5;
               totalMoney.setText("Your new total is $" + total);
            else if(playerRand < compRand)
               total = total - 5;
               totalMoney.setText("Your new total is $" + total);
         else if(source == tenHigh)
            cardGenerator();
            playerCard.setText("Your card: " + deck[playerRand]);
            cardSort();
            ok.setEnabled(true);
            fiveHigh.setEnabled(false);
            tenHigh.setEnabled(false);
            fiveLow.setEnabled(false);
            tenLow.setEnabled(false);
            if(playerRand > compRand)
               total = total + 10;
               totalMoney.setText("Your new total is $" + total);
            else if(playerRand < compRand)
               total = total - 10;
               totalMoney.setText("Your new total is $" + total);
         else if(source == fiveLow)
            cardGenerator();
            playerCard.setText("Your card: " + deck[playerRand]);
            cardSort();
            ok.setEnabled(true);
            fiveHigh.setEnabled(false);
            tenHigh.setEnabled(false);
            fiveLow.setEnabled(false);
            tenLow.setEnabled(false);
            if(playerRand < compRand)
               total = total + 5;
               totalMoney.setText("Your new total is $" + total);
            else if(playerRand > compRand)
               total = total - 5;
               totalMoney.setText("Your new total is $" + total);
         else if(source == tenLow)
            cardGenerator();
            playerCard.setText("Your card: " + deck[playerRand]);
            cardSort();
            ok.setEnabled(true);
            fiveHigh.setEnabled(false);
            tenHigh.setEnabled(false);
            fiveLow.setEnabled(false);
            tenLow.setEnabled(false);
            if(playerRand < compRand)
               total = total + 10;
               totalMoney.setText("Your new total is $" + total);
            else if(playerRand > compRand)
               total = total - 10;
               totalMoney.setText("Your new total is $" + total);
         else if(source == ok)
            cardGenerator();
            compCard.setText("Next computer card :" + deck[compRand]);
            cardSort();
            ok.setEnabled(false);
            fiveHigh.setEnabled(true);
            tenHigh.setEnabled(true);
            fiveLow.setEnabled(true);
            tenLow.setEnabled(true);
         if(total <= 0 || total >= 100)
            gameOver.setText("Game Over!");
            gameOver.setFont(font1);
            ok.setEnabled(false);
            fiveHigh.setEnabled(false);
            tenHigh.setEnabled(false);
            fiveLow.setEnabled(false);
            tenLow.setEnabled(false);
   }

Answer 1

See thats just the thing, I don't think my sort() is working correctly, because if it was thensort() is a library method. Whatever it is documented to do, you can rest assured it is doing it correctly.
playerRand and compRand are ints. You assign to them in the cardGenerator() method with the lines
compRand = generator.nextInt(52);
playerRand = generator.nextInt(52);and there is no reason why they should not be equal.
I don't know if this is the source of the problem or not - basically I don't know what "the score is not accumulating correctly" means.
Since you have random numbers being generated it can be difficult both to reproduce and describe problems. One way to lessen this difficulty would be to make generator an instance variable and initialise it in the constructor. One of the Random constructors lets you provide a seed value. The effect of this that the numbers generated are still "random", but are also completely reproducible. This is a good thing for testing.
Also you could split the huge actionPerformed() method - perhaps having a number of action listeners each with a more moderate sized actionPerformed() method whose intent can be documented and whose implemention can be tested.
Edited by: pbrockway2 on Jul 20, 2008 2:13 PM
Why do you sort the array of strings anyway? And why, having sorted it, do you periodically sort it again (since at that point it is already sorted)?