Help with replacing char

Question

I have been working with this code for almost a week now and I cannot figure out what I am doing wrong. The program initializes fine, but when I go to use the ActionListener it goes haywire. The purpose of this code is to mask a string and then unmask the characters one letter at a time, sort of like hangman. I got the mask working, unmasking is where I am seriously stuck. Please can I get some help here in figuring out what I am doing wrong. Code is posted below.
   import javax.swing.*;
   import java.awt.*;
   import java.awt.event.*;
   import java.util.*;
    public class JSecretPhrase extends JApplet implements ActionListener
      JLabel title = new JLabel("Secret Phrase Game");
      JLabel intro = new JLabel("Play our game - guess the phrase");
      JLabel enter = new JLabel("Enter one letter");
      //array of Strings for the random phrase
      String[] secretPhrase = {"PAPAS GOT A NEW PAIR OF SHOES",
                         "LIFE IS A BEACH",
                         "DID YOU GET THE MEMO",
                         "LIVE LAUGH AND LOVE",
                         "A PENNY SAVED IS A PENNY EARNED",
                         "ARE YOU GONNA EAT THOSE TOTS",
                         "LIVE FAST AND RIDE HARD",
                         "LIFE IS LIKE A BOX OF CHOCOLATES",
                         "WOULD YOU LIKE FRIES WITH THAT",
                         "HOOAH ITS AN ARMY THING"};
      JLabel phraseMask = new JLabel("");
      JLabel yesOrNo = new JLabel("");
      JButton[] letter = new JButton[26];
      Font font1 = new Font("TimesRoman", Font.BOLD, 34);
      Font font2 = new Font("TimesRoman", Font.BOLD, 20);
      String letters = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
      int rand;
      String phrase, phraseTrue, phraseFix;
      char phraseLtr, buttonLtr, phraseLtrTrue, replaceLtr;
      Container con = getContentPane();
      //random number generator for the selection of the phrase
       public void choosePhrase()
         Random generator = new Random();
         for (int i = 1; i < 11; i++)
            rand = generator.nextInt(10);
       public void init()
         con.setLayout(new FlowLayout());
         con.add(title);
         con.add(intro);
         con.add(enter);
         choosePhrase();
         phraseTrue = secretPhrase[rand];
         phrase = phraseTrue;
         con.add(phraseMask);
         con.add(yesOrNo);
         title.setFont(font1);
         intro.setFont(font2);
         for (int i = 0; i < letters.length(); i++)
            String ltr = Character.toString(letters.charAt(i));
            letter[i] = new JButton(ltr);
            letter.addActionListener(this);
con.add(letter[i]);
for (int p = 0; p < phrase.length(); ++p)
phraseLtr = phrase.charAt(p);
for (int c = 0; c < letters.length(); ++c)
buttonLtr = letters.charAt(c);
if(buttonLtr == phraseLtr)
phrase = phrase.replace(phraseLtr, '*');
phraseMask.setFont(font2);
phraseMask.setText(phrase);
public void actionPerformed(ActionEvent e)
Object source = e.getSource();
for(int s = 0; s < letter.length; ++s)
if(source == letter[s])
buttonLtr = letters.charAt(s);
letter[letters.indexOf(buttonLtr)].setEnabled(false);
for(int t = 0; t < phraseTrue.length(); t++)
phraseLtrTrue = phraseTrue.charAt(t);
if(phraseLtrTrue == buttonLtr)
phraseFix = phrase.replace(phrase.charAt(phraseTrue.indexOf(phraseLtrTrue)), buttonLtr);
phraseMask.setText(phraseFix);
yesOrNo.setText("Correct!");
else
yesOrNo.setText("Sorry - not in the phrase: " + buttonLtr);

Answer 1

corlettk wrote:
TrailorParkKid,
(I love your handle)lol, 30 years old and I still call myself that. Some things you just never grow out of.
>
Yeah, java.lang.* is imported automatically.
The question is what is phrase. Is it a StringBuilder? ... I suspect it's a String.
Cheers. Keith.I started playing with the StringBuilder Class and had a small breakthrough, but I still have a small issue. Instead of unmasking the entire phrase on the first pressed letter that is in the phrase and making them all the same letter, it only unmasks the entire phrase when the last letter in the phrase is the correctly pressed button and shows the phrase as it should read.
I suspect that one of my loops is not working as intended, I just don't know how to determine which one and what to change.
New Code:
import javax.swing.*;
   import java.awt.*;
   import java.awt.event.*;
   import java.util.*;
    public class JSecretPhrase extends JApplet implements ActionListener
      JLabel title = new JLabel("Secret Phrase Game");
      JLabel intro = new JLabel("Play our game - guess the phrase");
      JLabel enter = new JLabel("Enter one letter");
      //array of Strings for the random phrase
      String[] secretPhrase = {"PAPAS GOT A NEW PAIR OF SHOES",
                         "LIFE IS A BEACH",
                         "DID YOU GET THE MEMO",
                         "LIVE LAUGH AND LOVE",
                         "A PENNY SAVED IS A PENNY EARNED",
                         "ARE YOU GONNA EAT THOSE TOTS",
                         "LIVE FAST AND RIDE HARD",
                         "LIFE IS LIKE A BOX OF CHOCOLATES",
                         "WOULD YOU LIKE FRIES WITH THAT",
                         "HOOAH ITS AN ARMY THING"};
      JLabel phraseMask = new JLabel("");
      JLabel yesOrNo = new JLabel("");
      JButton[] letter = new JButton[26];
      Font font1 = new Font("TimesRoman", Font.BOLD, 34);
      Font font2 = new Font("TimesRoman", Font.BOLD, 20);
      String letters = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
      int rand;
      String phrase, phraseTrue, pLtrT, phraseFix;
      char phraseLtr, buttonLtr, phraseLtrTrue, replaceLtr;
      StringBuilder phraseBuilder = new StringBuilder();
      Container con = getContentPane();
      //random number generator for the selection of the phrase
       public void choosePhrase()
         Random generator = new Random();
         rand = generator.nextInt(10);
       public void changeLetter()
         for(int t = 0; t < phraseTrue.length(); t++)
            phraseLtrTrue = phraseTrue.charAt(t);
            pLtrT = Character.toString(phraseLtrTrue);
    @Override
       public void init()
         con.setLayout(new FlowLayout());
         con.add(title);
         con.add(intro);
         con.add(enter);
         choosePhrase();
         phraseTrue = secretPhrase[rand];
         phrase = phraseTrue;
         con.add(phraseMask);
         con.add(yesOrNo);
         title.setFont(font1);
         intro.setFont(font2);
         for (int i = 0; i < letters.length(); i++)
            String ltr = Character.toString(letters.charAt(i));
            letter[i] = new JButton(ltr);
            letter.addActionListener(this);
con.add(letter[i]);
phraseBuilder = new StringBuilder(phrase);
for (int p = 0; p < phrase.length(); ++p)
phraseLtr = phrase.charAt(p);
for (int c = 0; c < letters.length(); ++c)
buttonLtr = letters.charAt(c);
if(buttonLtr == phraseLtr)
phrase = phrase.replace(phraseLtr, '*');
phraseMask.setFont(font2);
phraseMask.setText(phrase);
public void actionPerformed(ActionEvent e)
Object source = e.getSource();
int phraseIndex = 0;
String bLtr = null;
for(int s = 0; s < letter.length; ++s)
if(source == letter[s])
buttonLtr = letters.charAt(s);
bLtr = Character.toString(buttonLtr);
letter[letters.indexOf(buttonLtr)].setEnabled(false);
changeLetter();
phraseIndex = phraseTrue.indexOf(phraseLtrTrue);
if(pLtrT.equals(bLtr))
phraseBuilder.setCharAt(phraseIndex,buttonLtr);
phraseFix = phraseBuilder.toString();
phraseMask.setText(phraseFix);
yesOrNo.setText("Correct!");
else
yesOrNo.setText("Sorry - not in the phrase: " + buttonLtr);