Java-替换字符串中的字符

Question

don't think I haven't searched online for an answer. Why is it giving me outofbounds error?

I have two 6-chars long strings. One is "daniel" and another is "------". User enters a character. Loop goes through the "daniel" string and checking char by char is they match with the user input. If it matches, it should replace the guessed char with the one in "------". So if you input 'a' it should output "-a----" and the loop continues. Next if you enter 'e' it should output "-a--e-" etc. Code gives no compilation error or any warning and also makes perfect sense. I tried substring and replace but this is simpler and shorter. I tried debugging it but it gives no useful info. I don't know why is it returning outofbounds error.

package hello.world;

import java.util.Scanner;

public class HelloWorld {
    public static void main(String[] args) {
        Scanner in=new Scanner(System.in);
        String word="daniel";
        StringBuilder guess2 = new StringBuilder("------");
        char guess;

        System.out.println("**********************");
        System.out.println("* Welcome to Hangman *");
        System.out.println("**********************");

        for (int i=0;i<10;i++) {
            System.out.print("Enter a letter: ");
            guess=in.nextLine().charAt(0);

            for (int j=0;i<word.length();j++) {
                if (guess==word.charAt(j)) {
                    guess2.setCharAt(word.charAt(j), guess);
                    System.out.print(guess2);
                }
            }
        }
    }
}

Output:

**********************
* Welcome to Hangman *
**********************
Enter a letter: a
Exception in thread "main" java.lang.StringIndexOutOfBoundsException: String index out of range: 97
    at java.lang.AbstractStringBuilder.setCharAt(AbstractStringBuilder.java:380)
    at java.lang.StringBuilder.setCharAt(StringBuilder.java:76)
    at hello.world.HelloWorld.main(HelloWorld.java:22)
Java Result: 1
BUILD SUCCESSFUL (total time: 2 seconds)

Answer 1

Replace

guess2.setCharAt(word.charAt(j), guess);

with

guess2.setCharAt(j, guess);

The first parameter is the index of the character to replace in the StringBuilder , not the character itself.

Also, there seems to be a typo in the for loop using i instead of j .

for (int j=0;i<word.length();j++) {

Answer 2

String.length（）返回从1开始而不是从0开始的字符串的长度。因此，每当您使用String.length（）时，始终使用小于（<）符号。

Answer 3

Instead of using the String builder you can just replace everything using regex + replaceAll method

for (int i=0;i<10;i++) {
        System.out.print("Enter a letter: ");
        guess=in.nextLine().charAt(0);

        word = word.replaceAll("[^"+ guess +"]", "-");
        System.out.println(word);
    }

result:

Enter a letter: a
-a----