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 
   
    shree sri
540 Pointshelp me to resolve
def repeat(string, times) fail "times must be 1 or more" if times < 1 counter = 0 loop do string = "shree" puts string if counter == 1 break end end end
def repeat(string, times)
  fail "times must be 1 or more" if times < 1
  counter = 0
  loop do
    string = "shree"
    puts string
    if counter == 1 
    break
    end
  end
end
2 Answers
 
    elk6
22,916 PointsHi Shree,
Couple of things wrong:
- You are not printing
- You are inputting an incorrect string
- You are not incrementing counter, so it will stay 0 forever
- You are not comparing counter with times ( both vars ) to break out of the loop
Try your code like this
def repeat(string, times)
  fail "times must be 1 or more" if times < 1
  counter = 0
  loop do
    print string
    counter += 1
    if counter == times
      break    
    end
  end
end
 
    andynguyen9
1,629 PointsWhy there are 3 ends?
What is "fail" here? Search the net revealed that it is a Kernel method like puts=> don't understand.
fail "times must be 1 or more" if times < 1 => don't understand.
Thank you.