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

I am getting an error that says I forgot the TODO's, but there are none in my code.

My code runs fine and there is no error saying that I didn't actually complete a TODO. Why does it say they still exist?

com/teamtreehouse/Main.java
// After you've completed the TODOs locally paste Main.java here
package com.teamtreehouse;

import java.util.Arrays;
import java.util.List;

public class Main {

    public static void main(String[] args) {
    // write your code here

        String story = "Thanks __name__ for helping me out.  You are really a __adjective__ __noun__ and I owe you a __noun__.";
        Template tmpl = new Template(story);
        Prompter prompter = new Prompter();



        prompter.run(tmpl);

    }
}
com/teamtreehouse/Prompter.java
// After you've completed the TODOs locally paste Prompter.java here
package com.teamtreehouse;

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);
        }

        System.out.printf("%s", tmpl.render(results));
    }

    /**
     * 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;
    }


    /**
     * 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 {

        System.out.printf("Enter a %s", phrase);
        String response = mReader.readLine();
        while (mCensoredWords.contains(response)){
            System.out.print("That word is unacceptable, try again: ");
            response = mReader.readLine();
        }
        return response;
    }
}
pseudo-tests.md
#  This is essentially what I am testing 
1.  The user is prompted for a string template (the one with the double underscores in it)

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

UPDATE: I have fixed the while loop in promptForWord to be a do-while and have prompted for a new story in main. However, I am still getting this error message.

2 Answers

I have fixed my code since the posting of this. I added the prompt for a new story instead of using the old one. The program runs fine but i am still getting the same error, anything else you may notice?

Sidney Casagrande
Sidney Casagrande
7,035 Points

I created a new prompt method in the Prompter class (called it promptForPhrase) which prompts the user to key in a phrase and return the phrase as a String. In the main method I then initialised the prompter first sets the story with the return value of my new method (promptForPhrase) and I then continue with setting the template and running the prompter as already done.

Hope that helps :)