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 Data Structures - Retired Efficiency! Changing Course

I'm not sure what I'm doing wrong

Can someone help me?

com/example/model/Course.java
package com.example.model;

import java.util.List;

public class Course {
  private String mName;
  private 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;
  }

}
com/example/model/Video.java
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;
  }

}
QuickFix.java
import com.example.model.Course;
import com.example.model.Video;
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"
List<Video> videoList = course.getVideos();
    Video video = new Video ("The Beginning Bits");

    course.getVideos();
    // TODO(2):  Add the newly created video to the course videos as the second video.

  }

  public void fixVideoTitle(Course course, String oldTitle, String newTitle) {

  }

  public Map<String, Video> videosByTitle(Course course) {
      Map<String, Video> videoMap = new TreeMap<>();
      for (Video video : videoMap) {
        videoMap.put(video.getTitle(), video);
      }
    return videoMap;
  }

}

1 Answer

Christopher Augg
Christopher Augg
21,223 Points

Emily,

       public void addForgottenVideo(Course course) {
       // TODO(1):  Create a new video called "The Beginning Bits"
        List<Video> videoList = course.getVideos(); // This is ok.
        Video video = new Video ("The Beginning Bits"); //This is ok but it requires import for List.

          course.getVideos(); //You do not need to call this again. Remove.
         // TODO(2):  Add the newly created video to the course videos as the second video.

              // You have videoList with the list of videos. You need to add video to videoList.
              // List documentation shows us that we want to use the add(int index, e element) method to do so.
              videoList.add(???,???); //what goes here?
       }

An easier way.....

       public void addForgottenVideo(Course course) {

            // TODO(1):  Create a new video called "The Beginning Bits"
                Video video = new Video("The Beginning Bits");  // you got this right!

           // TODO(2):  Add the newly created video to the course videos as the second video.
               // You do not need to assign a list variable. You can just call the course.getVideos() method and chain
               // the list.add(index, element) method.
               course.getVideos().add(???, ???); //what goes here?
       }

Hope this helps.

Regards,

Chris

thank you for helping me but I still don't really get what is supposed to go before the course.getVideos and what goes inside.

wait never mind I get it can you help me with the next one?