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 trialSuryapratap Singh Rathore
3,012 Pointsgetter is giving me hard time
sir i think i am doing it correct
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()
);
}
*/
}
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;
}
}
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;
}
}
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
Lauren Moineau
9,483 PointsHi. 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 :)