Method to "uncover"

[Quelsor]

Hi all,
im learning Java atm and got a problem in the following code:
uncover should uncover all characters saved in hidden and Im trying to find a method to get this done. Furthermore, case sensitivity should not matter. Im playing around with char array... but Im stuck atm :)
The previous method turns the confidential term into "_", which works fine so far.
Any help would be highly appreciated.

public class Hangman {
        public static void main(String[] args) {
                String confidential = "Test Test Test ROAR";
                String hidden = createHidden(confidential);
                System.out.println(hidden);
                hidden = uncover(confidential, hidden, 'e');
                System.out.println(hidden);
                hidden = uncover(confidential, hidden, 'R');
                System.out.println(hidden);
                hidden = uncover(confidential, hidden, 'a');
                System.out.println(hidden);
        }
 
        public static String createHidden(String confidential) {
                String result = " ";
               
                for (int i = 0; i < confidential.length(); i++) {
                        if (confidential.charAt(i) == ' ') {
                                System.out.print(" ");
                        } else {
                                System.out.print("_ ");
                        }
                }
               
                return result;
 
        }
 
        public static String uncover(String confidential, String hidden, char character) {
                String result = "";
                                               
                String test1 = new String (hidden.toLowerCase()); //FIXME
                String test2 = new String (test1.toUpperCase()); //FIXME
                
                String all = new String (test1 + test2); //FIXME
                char [] array = all.toCharArray(); //FIXME
                
                for(int i=0; i < confidential.length(); i++) {
                	if (confidential.equals (array)) //FIXME
                		System.out.print(); //FIXME
                
                }
                	
                         
                return result;
        }
 
}

The Output should look like:

_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
_ e _ _ _ e _ _ _ e _ _ _ _ _ _
_ e _ _ _ e _ _ _ e _ _ r _ _ r
_ e _ _ _ e _ _ _ e _ _ r _ a r


Mine so far:

_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

[Fubarable]

Get rid of the System.out.println(...) calls in your uncover and createHidden methods as they don't update the result String which is what you should be doing instead here. So in place of System.out.println, you may do better doing

result += /* something goes here */;

[Quelsor]

Thanks for the fast reply!
Changed my code so far.

Now I want to change the if (...) statement in the second method.
My thought on this was as follows (some pseudo inc ^^)

If confidential.charAt(i) equals a character of array then return the character.

But as I cant do it like this (incomparable types: char and char [])

if (confidential.charAt(i) == (array))

I dont know how to fix it.

[Fubarable]

1) There is no need to call new String(...) anywhere in your code, so get rid of those constructor calls. Instead simply call a method of String, that's it.

2) What are you trying to do here as this code makes little sense to me:

                String test1 = new String (hidden.toLowerCase()); //FIXME
                String test2 = new String (test1.toUpperCase()); //FIXME
                
                String all = new String (test1 + test2); //FIXME
                char [] array = all.toCharArray(); //FIXME

3) You don't want to compare a char to array, again it makes no sense. Instead you want to compare a char with a char. Use the index of the for loop, i, inside the body of the loop to extract your char out of the array.

[Quelsor]

The code lines ending with fixme were just some "farting around". Deleted them.

All right, got rid of the Strings and hopefully i compare chars now:

public static String uncover(String confidential, String hidden, char character) {
                String result = "";
                                               
                char [] array1 = hidden.toCharArray();
                char [] array2 = confidential.toCharArray();

                for(int i=0; i < confidential.length(); i++) {
                	if (array1[i] == array2[i]) 
                		result += (...); //FIXME            
                
                }	
                        
                return result;
        }

Guess im taking a nap now :)

[Fubarable]

don't ignore the char "character". Think on what you want to do with this variable -- perhaps you should be using it in the logic portion of your if block?