getter is giving me hard time

Question

sir i think i am doing it correct

Forum.java

public class Forum {
  private String topic;

  // TODO: add a constructor that accepts a topic and sets the private field topic

  public String getTopic() {
    return topic;
  }
    private Forum(String topic){
    topic = "xyz";
  }

  /* Uncomment this when you are prompted to do so
  public void addPost(ForumPost post) {
    System.out.printf("A new post in %s topic from %s %s about %s is available",
            topic,
            post.getAuthor().getFirstName(),
            post.getAuthor().getLastName(),
            post.getTitle()
    );
  }
  */

}

User.java

public class User {
  // TODO: add private fields for firstName and lastName
  private String firstName;
  private String lastName;
  public User(String firstName, String lastName) {
    // TODO: set and add the private fields 
    this.firstName = firstName;
    this.lastName = lastName;
  }

  // TODO: add getters for firstName and lastName
    public getFirstName(){
    return this.firstName;
  }

  public getLastName(){
    return this.lastName;
  }
}

ForumPost.java

public class ForumPost {
  private User author;
  private String title;
  private String description;

  // TODO: add a constructor that accepts the author, title and description

  public User getAuthor() {
    return author;
  }

  public String getTitle() {
    return title;
  }

  public String getDescription() {
    return description;
  }
}

Main.java

public class Main {

  public static void main(String[] args) {
    System.out.println("Beginning forum example");
    if (args.length < 2) {
      System.out.println("Usage: java Main <first name> <last name>");
      System.err.println("<first name> and <last name> are required");
      System.exit(1);
    }
    /* Uncomment this when prompted
    Forum forum = new Forum("Java");
    // TODO: pass in the first name and last name that are in the args parameter
    User author = new User();
    // TODO: initialize the forum post with the user created above and a title and description of your choice
    ForumPost post = new ForumPost();
    forum.addPost(post);

// */
  }

}

Answer 1 · 2019-01-18T22:50:29Z

January 18, 2019 10:50pm

Hi. There are a couple of issues. The first one being with your constructor.

 private Forum(String topic){
    topic = "xyz";
  }

In the first part of the challenge, it says "(in Forum.java.) Add a constructor that accepts a String named topic", which you did correctly:

private Forum(String topic){
 }

But then it says: "Initialize the private field topic in the constructor to the value passed in". The private field topic is

private String topic;

at the top of your class. The value passed in the constructor is String topic. So the constructor should be:

 private Forum(String topic){
     this.topic = topic;
 }

The way you wrote your constructor, you're actually hardcoding the topic of the Forum post, so that any forum post created will have the same "xyz" value, no matter the String passed in the constructor. On a side note, for best practice, the constructor should be at the top of the class, after the field variables, and before the getters/setters.

The second issue is with your getters. You did not provide a return type. They are both returning a String, so they should both start with:

public String ...

Hope that helps :)

Welcome to the Treehouse Community

Looking to learn something new?

Suryapratap Singh Rathore

Suryapratap Singh Rathore

getter is giving me hard time

1 Answer

Lauren Moineau

Lauren Moineau