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

Make TreeMap with Multiple Keys

I am trying to make a Map with a Title as the Key and a Video Object. I can make the Map by using a For Loop to get the Title from each Video and add them to the Map, but it keeps returning only 23 keys instead of 22. I assume this is because there are two videos with the same title. I can only find examples that have a List as the Map Object, so I'm not sure how to check for a repeated title and add it anyway.

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.ArrayList;
import java.util.List;
import java.util.Map;
import java.util.TreeMap;

public class QuickFix {

  public void addForgottenVideo(Course course) {
    Video video = new Video("The Beginning Bits");
    List<Video> videoList = course.getVideos();
    videoList.add(video);
    videoList.set(1,video);

    // TODO(1):  Create a new video called "The Beginning Bits"

    // 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> videosByTitle = new TreeMap<>();
    List<Video> videoList = course.getVideos();
    for(Video video : videoList) {
      String title = video.getTitle();      
      //List<Video> videos = videosByTitle.get(title);
     // titles.add(video.getTitle());
      //if(videos == null){
      //  videos = new ArrayList<>();
      //  videos.add(video);
      //}
      videosByTitle.put(title,video);  
    }

    return videosByTitle;

  }

}

2 Answers

Jon Kussmann
PLUS
Jon Kussmann
Courses Plus Student 7,254 Points

Hi Abby,

When you are calling the lines:

    videoList.add(video);
    videoList.set(1,video);

You are adding your new video to the end of the list of videos as well as setting the video in the second position to the video list. That is how you are getting the same video twice in one list.

I hope that helps, if you need more clarification let me know.

Thank you!