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

sabath rodriguez
sabath rodriguez
11,055 Points

I don't understand what's wrong with this. please help

Im doing a project in the Java track and im stuck, the "preview" is saying that i need to define a return type for the constructors, which doesn't make sense.

Forum.java
public class Forum {
  private String topic;

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

  public getForum(String topic) {
    this.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
    this.firstName = firstName;
    this.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 getUserInfo(User author, String title, String description) {
    this.author = author;
    this.title = title;
    this.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("Sabath", "Rodriguez");
    //TODO: initialize the forum post with the user created above and a title and description of your choice
    ForumPost post = new ForumPost(author, "SomeTitle", "SomeDescription");
    forum.addPost(post);

  }

}

1 Answer

andren
andren
28,558 Points

A constructor has to have the exact same name as the class it is placed in. If it doesn't then it will be treated as a regular method. You have called the constructor for the Forum class getForum and the constructor for the ForumPost class getUserInfo.

So if you rename getForum to Forum and getUserInfo to ForumPost (remember that the names are case-sensitive) then that error will go away. There is another minor issue in your code though, which is the fact that the challenge asks you to create the User object using the arguments passed to the program. You have passed in your own name instead.

To retrieve values that have been passed as arguments you simply need to extract values from the args object. The first value that has been passed to the program will be in args[0], the second in args[1], and so on. So the line that creates the User object in Main.java should look like this:

User author = new User(args[0], args[1]);
Jialong Zhang
Jialong Zhang
9,821 Points

yeah, you can't change the name of a constructor.

//class name is Forum
 public getForum(String topic) { // constructor: same name as class name. if this is method, must return something or write void after public.
    this.topic = topic;
  }