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

Ruby Ruby Booleans Build a Simple Todo List Program Finding Array Items

Posted by Jesse Van Der Heijden
Jesse Van Der Heijden
3,313 Points

What's wrong with this one?

Don't see what I am missing. Help please!

Sincerely

todo_list.rb 
class TodoList
  attr_reader :name, :todo_items

  def initialize(name)
    @name = name
    @todo_items = []
  end

  def add_item(name)
    todo_items.push(TodoItem.new(name))
  end

  def find_index(name)
  index = 0
  found = false
  todo_items.each do |todo_items|
    if todo_items.name == name
      found = true
      break
    else
      index += 1
    end
  end

  if found 
    return index
  else 
    return nil
  end
end

2 Answers

Jesse Van Der Heijden
Jesse Van Der Heijden
3,313 Points

huh wow, I see that this declares the array index name. But I dont see how this method can say anything about wether the value is found true or not. Strange you also have to delete the lines threehouse gave you. Anyhow it works. Thanks!

Ari Misha
Ari Misha
19,323 Points
Ari Misha
Ari Misha
19,323 Points

Jesse Van Der Heijden Thats the thing Jesse! The question was itself an incomplete question where they provided you some default initial values and methods right? But the behind the scenes, the answer implemented into the challenge is different. I think you nailed the code with respect to what was asked in the challenge. And your code was returning either the index and the value nil. But you see the challenge only accepts booleans.

Ari Misha
Ari Misha
19,323 Points
Ari Misha
Ari Misha
19,323 Points

Hiya there! Phew, that challenge took me a while but eventually i figured it out where the issue lies with this challenge. You've to return true if you find the item in @todo_items, otherwise false. I dunno why wouldnt Treehouse update this challenge. Anyways here is the solution for your reference ( and yes it has nothing to do with variables index and found):

class TodoList
  attr_reader :name, :todo_items

  def initialize(name)
    @name = name
    @todo_items = []
  end

  def add_item(name)
    todo_items.push(TodoItem.new(name))
  end

  def find_index(name)
    @todo_items.index { |item| item.name == name }
  end

end