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 trialSahil Chawla
5,834 PointsError: Are you sure you updated the video title ?
public void fixVideoTitle(Course course, String oldTitle, String newTitle) {
Map<String,Video> videosTitle = new HashMap<>();
videosTitle = videosByTitle(course);
Video temp = videosTitle.remove(oldTitle);
videosTitle.put(newTitle,temp);
}
Please let me know whats wrong in this.
package com.example.model;
import java.util.List;
public class Course {
private String mName;
public List<Video> mVideos;
public Course(String name, List<Video> videos) {
mName = name;
mVideos = videos;
}
public String getName() {
return mName;
}
public List<Video> getVideos() {
return mVideos;
}
}
package com.example.model;
public class Video {
private String mTitle;
public Video(String title) {
mTitle = title;
}
public String getTitle() {
return mTitle;
}
public void setTitle(String title) {
mTitle = title;
}
}
import com.example.model.Course;
import com.example.model.Video;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
public class QuickFix {
public void addForgottenVideo(Course course) {
// TODO(1): Create a new video called "The Beginning Bits"
Video video = new Video("The Beginning Bits");
// TODO(2): Add the newly created video to the course videos as the second video.
course.mVideos.add(1, video);
}
public void fixVideoTitle(Course course, String oldTitle, String newTitle) {
Map<String,Video> videosTitle = new HashMap<>();
videosTitle = videosByTitle(course);
Video temp = videosTitle.remove(oldTitle);
videosTitle.put(newTitle,temp);
}
public Map<String, Video> videosByTitle(Course course) {
Map<String, Video> videosByTitle = new HashMap<>();
List<Video> courseVideos = new ArrayList(course.getVideos());
for ( Video video : courseVideos ) {
videosByTitle.put(video.getTitle(),video);
}
return videosByTitle;
}
}
1 Answer
andren
28,558 PointsYou have changed the mVideos
field from private
to public
which is not something you were meant to do. The Course
class has a getter method called getVideos()
, getters are methods that are used to let other classes access private
fields.
If you change mVideos
back to being private
in the Course
class and simply replace mVideos
with getVideos()
in your code like this:
public void addForgottenVideo(Course course) {
// TODO(1): Create a new video called "The Beginning Bits"
Video video = new Video("The Beginning Bits");
// TODO(2): Add the newly created video to the course videos as the second video.
course.getVideos().add(1, video);
}
Then your code will pass the first task.
Sahil Chawla
5,834 PointsSahil Chawla
5,834 PointsHi Andren,
Thanks for your response, I got it working now. I wasn't actually changing the title of Video Object by using its setter method and yes i will change the private field of Course class to its original state and will use getters.