Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Java Local Development Environments Advanced Tooling Finishing TreeStory

Bruno Aldo Lunardi
Bruno Aldo Lunardi
15,795 Points

Seems to be not working, my code is correct, working on my IDE

keep returning this message over and over, my validation algorithm seems to be working just fine....i dunno what's going on...

"Bummer! Whoops looks like you didn't censor the words (2a). mCensoredWords contains 'dork', but it made it past somehow."

com/teamtreehouse/Main.java
package com.teamtreehouse;
import java.util.Arrays;
import java.util.List;

public class Main {

    public static void main(String[] args) {
    // write your code here
        Prompter prompter = new Prompter();
        String story = prompter.promptForPhrase();
        Template tmpl = new Template(story);
        prompter.run(tmpl);

    }
}
com/teamtreehouse/Prompter.java
package com.teamtreehouse;

import com.sun.org.apache.xpath.internal.SourceTree;

import java.io.*;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.ArrayList;
import java.util.HashSet;
import java.util.List;
import java.util.Set;


public class Prompter {
    private BufferedReader mReader;
    private Set<String> mCensoredWords;

    public Prompter() {
        mReader = new BufferedReader(new InputStreamReader(System.in));
        loadCensoredWords();
    }

    private void loadCensoredWords() {
        mCensoredWords = new HashSet<String>();
        Path file = Paths.get("resources", "censored_words.txt");
        List<String> words = null;
        try {
            words = Files.readAllLines(file);
        } catch (IOException e) {
            System.out.println("Couldn't load censored words");
            e.printStackTrace();
        }
        mCensoredWords.addAll(words);
    }

    public void run(Template tmpl) {
        List<String> results = null;
        try {
            results = promptForWords(tmpl);
        } catch (IOException e) {
            System.out.println("There was a problem prompting for words");
            e.printStackTrace();
            System.exit(0);
        }
        String words = tmpl.render(results);
        System.out.printf("Your TreeStory:%n%n%s", words);
    }

    /**
     * Prompts user for each of the blanks
     *
     * @param tmpl The compiled template
     * @return
     * @throws IOException
     */
    public List<String> promptForWords(Template tmpl) throws IOException {
        List<String> words = new ArrayList<String>();
        for (String phrase : tmpl.getPlaceHolders()) {
            String word = promptForWord(phrase);
            words.add(word);
        }
        return words;
    }

    public String promptForPhrase() {
        String phrase = null;
        System.out.printf("Enter with the phrase that you want to complete, remember to use double underscores: %n");
        try {
            phrase = mReader.readLine();
        } catch (IOException e) {
            e.printStackTrace();
        }
        return phrase;
    }

    /**
     * Prompts the user for the answer to the fill in the blank.  Value is guaranteed to be not in the censored words list.
     *
     * @param phrase The word that the user should be prompted.  eg: adjective, proper noun, name
     * @return What the user responded
     */
    public String promptForWord(String phrase) throws IOException {
        String word;
        boolean isAcceptable = false;
        do {
            System.out.printf("Enter the %s%n", phrase);
            word = mReader.readLine();
            isAcceptable = mCensoredWords.contains(word);
            if (isAcceptable)
                System.out.printf("The word %s is not valid, watch your mouth big  boy!!%n ", word);
        } while (isAcceptable);
        return word;
    }
}
pseudo-tests.md
#  This is essentially what I am testing 
1.  The user is prompted for a new string template (the one with the double underscores in it).

  a. The prompter class has a new method that prompts for the story template, and that method is called.

2.  The user is then prompted for each word that has been double underscored.

   a. The answer is checked to see if it is contained in the censored words.
      User is continually prompted until they enter a valid word

3.  The user is presented with the completed story

2 Answers

Alexander Nikiforov
seal-mask
.a{fill-rule:evenodd;}techdegree seal-36
Alexander Nikiforov
Java Web Development Techdegree Graduate 22,175 Points

Your code is awesome! The problem is with testing. Craigs testing sees the 'censored' word, even if you've filtered it and gives an error.

TL;DR change this line:

            if (isAcceptable)
                System.out.printf("The word %s is not valid, watch your mouth big  boy!!%n ", word);

to

            if (isAcceptable)
                System.out.printf("The word is not valid, watch your mouth big  boy!!%n ");

For explanation, you can check here.

Bruno Aldo Lunardi
Bruno Aldo Lunardi
15,795 Points

Awesome! simple like that! It worked....thanks for the tip... :D