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 Java Objects Delivering the MVP Forum

Suryapratap Singh Rathore
Suryapratap Singh Rathore
3,012 Points

getter is giving me hard time

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

// */
  }

}

1 Answer

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 :)