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! Add tags to a course

Posted by Enyang Mercy
Enyang Mercy
Courses Plus Student 2,339 Points

Designing UI Fix the addTag and addTags

Stuck again. Please check my code and suggest answers

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

import java.util.List;
import java.util.HashSet;
import java.util.Set;

public class Course {
  private String mTitle;
  private Set<String> mTags;

  public Course(String title) {
    mTags = new HashSet();
    mTitle = title;
    // TODO: initialize the set mTags
  }

  public void addTag(String tag) {
    mTags.add(tag);
    // TODO: add the tag
  }

  public void addTags(List<String> tags) {
    for(String tag : tags) {
      mTags.addTags(tag);

    // TODO: add all the tags passed in
  }

  public boolean hasTag(String tag) {
    // TODO: Return whether or not the tag has been added
    return false;
  }

  public String getTitle() {
    return mTitle;
  }

}

5 Answers

Enyang Mercy
PLUS
Enyang Mercy
Courses Plus Student 2,339 Points
Enyang Mercy
Enyang Mercy
Courses Plus Student 2,339 Points

Something is wrong again i just closed my for loop but it still cant compile think i didnt declare the new string variable

Steve Hunter
Steve Hunter
57,712 Points

Yes, you have - you did that with String tag in the for loop brackets. Have you removed mTags from the line inside the loop? Look at my second code block.

Steve Hunter
Steve Hunter
57,712 Points
Steve Hunter
Steve Hunter
57,712 Points

You haven't closed your for loop.

  public void addTags(List<String> tags) {
    for(String tag : tags) {
      mTags.addTags(tag);
  } // <- add this
    // TODO: add all the tags passed in
  }

Plus, inside the method call addTag(tag) - [edit] your code is calling addTag on mTags. It shouldn't. Just call it straight like:

  public void addTags(List<String> tags) {
    for(String tag : tags) {
      addTags(tag);
    }
  }

The addTag method manages the mTags bit.

Steve.

Enyang Mercy
PLUS
Enyang Mercy
Courses Plus Student 2,339 Points
Enyang Mercy
Enyang Mercy
Courses Plus Student 2,339 Points

For the third question is this ok? if (hasTag.add == mTags) return true

Steve Hunter
Steve Hunter
57,712 Points
Steve Hunter
Steve Hunter
57,712 Points

Hi there,

For the third task you want to return true if the tag is contained in mTags, else false. The List class comes with a handy method called .contains() that does this for you.

You want to use .contains() on mTags using dot notation and pass in the parameter, tag. As this will resolve to a true/false value for you - just return that expression. There's no need for interim variables or if statements:

  public boolean hasTag(String tag) {
    // TODO: Return whether or not the tag has been added
    return mTags.contains(tag);
  }

Make sense

Steve.

Enyang Mercy
PLUS
Enyang Mercy
Courses Plus Student 2,339 Points
Enyang Mercy
Enyang Mercy
Courses Plus Student 2,339 Points

Great !! thanks