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

Jordan Botts
Jordan Botts
1,860 Points

I am getting a NullPointerException on my Forum.java code and I am not sure what the solution is.

I am not sure what the NullPointerException error is?

Forum.java
public class Forum {
  private String topic;

  // TODO: add a constructor that accepts a topic and sets the private field topic
  public Forum (String topic) {
    topic = topic;
  }


  public String getTopic() {
    return topic;
  }



  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
    firstName = firstName;
    lastName = lastName;


  }

  // TODO: add getters for firstName and lastName
  public String getFirstName() {
    return firstName;
  }
  public String getLastName() {
    return 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 ForumPost (User author, String title, String description) {
    author = author;
    title = title;
    description = 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);
    }

    Forum forum = new Forum("Java");
    // TODO: pass in the first name and last name that are in the args parameter
    User author = new User(args[0], args[1]);
    // TODO: initialize the forum post with the user created above and a title and description of your choice
    ForumPost post = new ForumPost(author, "This is your forum post title", "This is your forum post description.");
    forum.addPost(post);


  }

}

2 Answers

Alex Bratkovskij
Alex Bratkovskij
5,329 Points

Hi Jordan,

The problem is that in constructor you assign topic to itself. what you want to do is to assign topic to the string in constructor.

public class Forum {
  private String topic;

  // TODO: add a constructor that accepts a topic and sets the private field topic
  public Forum (String topic) {
    this.topic = topic; //  <----- like that for example
  }
Anders Björkland
Anders Björkland
7,481 Points

The problem lies in your constructor. You are assigning the local variable topic to be the same as the local variable topic. You are not assigning it to the member variable of your class. Since both your member variable and parameter of the constructor share the same name, you have to tell it that your member variable of this class should be set to the same as the local variable. Here:

this.topic = topic

Why you would get a NullPointerException is because you could instantiate an object of the Forum class and call its methods where topic is yet to bet instantiated. The compiler notices this and so you get an exception.