Forum: Ruby Deaf Grandma

Also from the Chris Pine tutorial for beginners:

 • Write a Deaf Grandma program. Whatever you say to grandma (whatever
you type in), she should respond with HUH?!  SPEAK UP, SONNY!, unless
you shout it (type in all capitals). If you shout, she can hear you
(or at least she thinks so) and yells back, NO, NOT SINCE 1938! To
make your program really believable, have grandma shout a different
year each time; maybe any year at random between 1930 and 1950. (This
part is optional, and would be much easier if you read the section on
Ruby's random number generator at the end of the methods  chapter.)
You can't stop talking to grandma until you shout BYE.
Hint: Don't forget about chomp! 'BYE'with an Enter is not the same as
'BYE' without one!
Hint 2: Try to think about what parts of your program should happen
over and over again. All of those should be in your while loop.

• Extend your Deaf Grandma program: What if grandma doesn't want you
to leave? When you shout BYE, she could pretend not to hear you.
Change your previous program so that you have to shout BYE three times
in a row. Make sure to test your program: if you shout BYE three
times, but not in a row, you should still be talking to grandma.


Here is what I came up with:

#Grandma is deaf!

puts "Hey Sonny! It's your lovely Grandmother! How are yeah?"

response = "nope"
bye = 0

while bye < 3
  response = gets.chomp
if response == (response.upcase and "BYE")
  puts "Hmmm... I would prefer..."
  bye = (bye+1)
end
if response != response.upcase
  puts "Huh?! I CAN'T HEAR YOU!"
end
if (response == response.upcase and response != "BYE")
  puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
end
end

Is there an easier way?

#Grandma is deaf!

puts "Hey Sonny! It's your lovely Grandmother! How are yeah?"
response = nil
bye = 0
while bye < 3
  response = gets.chomp
  if response == "BYE"
    puts "Hmmm... I would prefer..."
    bye = (bye+1)
  elsif response == response.upcase
    puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
  else
    puts "Huh?! I CAN'T HEAR YOU!"
  end
end

danielj wrote:
> if response == (response.upcase and "BYE")

response.upcase and "BYE"  evaluates to "BYE" unless response.upcase 
returns
nil, which it won't. So the above is exactly like  if response == "BYE"

Thanks very much for all the help guys.

Programmers seem to be a nice group!

danielj wrote:
> Thanks very much for all the help guys.
> 
> Programmers seem to be a nice group!

I rewrote some of it like this:

#Grandma is deaf!

puts "Hey Sonny! It's your lovely Grandmother! How are you?"

response = gets.chomp
bye = 0

while bye < 1
   if response != response.upcase
     puts "Huh?! I CAN'T HEAR YOU!"
   end

   if (response == response.upcase and response != "BYE")
     puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
   end

   if response == "BYE"
     puts "GOOD BYE, SONNY!"
   bye = (bye+1)
   end

response = gets.chomp
end

I'm a noob, but this seems cleaner to me. I know there is a way to write 
it with out repeating the "if" statements three times but at least it 
works:) The only other thing that bothers me is you have to press enter 
at the end to return to the command prompt. If anyone has an example on 
how to fix it that would be sweet.

Chris Mr. wrote:
> 
>    end
> it with out repeating the "if" statements three times but at least it 
> works:) The only other thing that bothers me is you have to press enter 
> at the end to return to the command prompt. If anyone has an example on 
> how to fix it that would be sweet.

How about:

#Grandma is deaf!

puts "Hey Sonny! It's your lovely Grandmother! How are you?"

while (response = gets.chomp) != "BYE"
    if response != response.upcase
      puts "Huh?! I CAN'T HEAR YOU!"
    end

    if (response == response.upcase)
      puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
    end
  end
  puts "GOOD BYE, SONNY!"

This only uses one entry of response and no flags.

Michael W. Ryder wrote:
>> puts "Hey Sonny! It's your lovely Grandmother! How are you?"
>>      puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
>> I'm a noob, but this seems cleaner to me. I know there is a way to 
> 
> 
> This only uses one entry of response and no flags.

I missed the part about removing the multiple if's in the oringal post.

#Grandma is deaf!

puts "Hey Sonny! It's your lovely Grandmother! How are you?"

while (response = gets.chomp) != "BYE"
    if response != response.upcase
      puts "Huh?! I CAN'T HEAR YOU!"
     else
      puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
    end
end
puts "GOOD BYE, SONNY!"

Michael W. Ryder wrote:
> Michael W. Ryder wrote:
>>> puts "Hey Sonny! It's your lovely Grandmother! How are you?"
>>>      puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
>>> I'm a noob, but this seems cleaner to me. I know there is a way to 
>> 
>> 
>> This only uses one entry of response and no flags.
> 
> I missed the part about removing the multiple if's in the oringal post.
> 
> #Grandma is deaf!
> 
> puts "Hey Sonny! It's your lovely Grandmother! How are you?"
> 
> while (response = gets.chomp) != "BYE"
>     if response != response.upcase
>       puts "Huh?! I CAN'T HEAR YOU!"
>      else
>       puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
>     end
> end
> puts "GOOD BYE, SONNY!"

That one is sweet. Thanks. I guess I was over thinking it, LOL. I didn't 
think of putting the first response in parentheses. This helped me think 
of it in a much easier way.

After reviewing the previous posts, I noticed that many answers did not
incorporate the condition: '...you have to shout BYE three times in a
row...'

This is what I came up with:


puts 'Hey Sonny, it\'s your Grandma! How are you?'

response = nil
bye = 0

while bye < 3
  response = gets.chomp

  if response == 'BYE'
    bye = (bye + 1)

    if bye == 3
      puts 'BYE, SONNY!'
    else
      puts 'HUH?! SPEAK UP, SONNY!'
    end

What do you think?

  elsif response == response.upcase
    puts 'NO! NOT SINCE ' + (1930+rand(21)).to_s + '!'
    bye = 0
  else
    puts 'HUH?! SPEAK UP, SONNY!'
    bye = 0
  end
end

danielj wrote in post #719787:
> Is there an easier way?

Remember to set your bye counter to zero in every case except where 
'BYE' is typed, to ensure three BYE's in a row are required. You can use 
'next' to skip straight to the top of the next iteration, which means 
you can put a single bye=0 at the end of the loop.

Also, I think there is some ambiguity in the original problem 
description: what should happen if what you type contains neither upper 
nor lower case letters? (e.g. empty text or only punctuation 
characters?). In the code below I've let grandma have a snooze. I also 
use the 'case' statement to match the response against a series of 
values and regexp patterns.

puts "Hey Sonny, it's your Grandma! How are you?"
bye = 0
loop do
  case gets.chomp
  when "BYE"
    bye += 1
    break if bye >= 3
    next
  when /[a-z]/
    puts "HUH?! SPEAK UP, SONNY!"
  when /[A-Z]/
    puts "NO, NOT SINCE #{rand(21)+1930}!"
  else
    puts "ZZZ"
  end
  bye = 0
end

But to be honest, I'd say that if...elsif...elsif...end is equally good.

On Mon, Dec 13, 2010 at 3:59 AM, Brian Candler <b.candler@pobox.com> 
wrote:

> what should happen if what you type contains neither upper
> nor lower case letters? (e.g. empty text or only punctuation
> characters?).
>

That's called whispering, deaf grandma won't even bat an eye!

Josh Cheek wrote in post #968096:
> On Mon, Dec 13, 2010 at 3:59 AM, Brian Candler <b.candler@pobox.com>
> wrote:
>
>> what should happen if what you type contains neither upper
>> nor lower case letters? (e.g. empty text or only punctuation
>> characters?).
>>
>
> That's called whispering, deaf grandma won't even bat an eye!

You might be swearing at her though...

I was doing stupid things with arrays and all sorts of dumb crap until I 
came here and read this thread. Super good work. I wanted my Ruby script 
to work with the three goodbyes extension exercise mentioned above. 
Also—I like double quotes and #{} to get things done. In my neophyte 
opinion, it's cleaner and less characters.

puts "TALK TO ME! SO LONELY!\n\n"
bye = 0
while bye < 3
  response = gets.chomp
  if response == "BYE"
    puts "STAY AWHILE!?!"
    bye = (bye+1)
  elsif response == response.upcase
    puts "NO! NOT SINCE #{1910+rand(41)}!"
    bye = 0
  else
    puts "Huh?! I CAN'T HEAR YOU!"
    bye = 0
  end
end

If you make it a single string—it's 263 if you format it to one line and 
strip my OCD new-line characters. It resets "bye" back down to zero if 
it's not said three times straight.

# Deaf Grandma EXTENDED
rep1 = ''
bye = 0
puts 'HELLO DEAR'
while true
rep1 = gets.chomp
  if rep1 == 'BYE' || rep1 == 'BYE!' || rep1 != rep1.upcase || rep1 == 
rep1.capitalize
  if rep1 == 'BYE' || rep1 == 'BYE!'
    bye = bye+1
    if bye == 3
      break
      end
  else bye = 0
  end
    puts 'HUH?! SPEAK UP SONNY'
  elsif rep1 == rep1.upcase
    bye = 0
      while true
      ry1 = rand(51)
      if ry1 >= 30
        break
      end
    end
    puts 'NO, NOT SINCE 19' + ry1.to_s + '!'
  end
  end
 puts 'OH OK SONNY... COME BACK SOON SONY'

puts 'Hi! My name Deaf Grandma'
check = 0
while check <3
if (input = gets.chomp) == input.upcase
                puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
                 if input == 'BYE'
                       check = check + 1
                 else
                       check = 0
                 end

        else
                puts 'HUH?!  SPEAK UP, SONNY!'
        end
end
puts 'Goodbye! See you later!'

Hieu Le wrote in post #1088624:
> puts 'Hi! My name Deaf Grandma'
> check = 0
> while check <3
> if (input = gets.chomp) == input.upcase
>                 puts "NO! NOT SINCE " + (1930+rand(21)).to_s + "!"
>                  if input == 'BYE'
>                        check = check + 1
>                  else
>                        check = 0
>                  end
>
>         else
>                 puts 'HUH?!  SPEAK UP, SONNY!'
>         end
> end
> puts 'Goodbye! See you later!'

I like this one, but you forgot to reset this check to zero, otherwise 
it will end the program if the user inputs 'BYE' 3x non-sequentially 
(i.e 'BYE', 'bye', 'BYE, 'BYE')

else
  puts 'HUH?!  SPEAK UP, SONNY!'
  check = 0  #need this extra reset to zero

For the full iteration of the problem (upcase, random years & triple BYE
in a row requests), I've got a slightly simpler structure, the code
makes sense to me structurally & intuitively:

puts 'Hello there, Sonny. It\'s your grandma!'
reply = gets.chomp

while reply != 'BYE'
  if reply == reply.upcase
    puts 'NO, NOT SINCE ' +(rand(21)+1930).to_s+ '!'
    reply = gets.chomp
  else
    puts 'HUH?! SPEAK UP, SONNY!'
    reply = gets.chomp
  end
if reply == 'BYE'
  puts 'CAN\'T HEAR YOU, DEAR!'
  reply = gets.chomp
  if reply == 'BYE'
    puts 'STILL CAN\'T HEAR YOU, DEAR!'
    reply = gets.chomp
    if reply == 'BYE'
    end
  end
end

On Fri, Feb 8, 2013 at 12:25 PM, Anna Fedorova <lists@ruby-forum.com> 
wrote:
>     reply = gets.chomp
>     if reply == 'BYE'
>     end
>   end
> end
>
> --
> Posted via http://www.ruby-forum.com/.
>

Still LOTS of repetition here. DRY: Don't Repeat Yourself, not only
saves typing, and time, but might also lead to better understanding of
algorithms. See if you can write it with only one gets per loop.

My solution was very similar to @steve_k61.  Any advice from the peanut
gallery on
making this code cleaner? (Pardon all the comments, it helps me figure
things out.)

#Deaf Grandma w/ extension

#Set initial counter
count=0

#Create Loop Conditions
while count<3
response=gets.chomp

  if response==("bye").upcase
  # Grandma ignores but the count increases by 1
    count+=1
    # (puts count
    # this was an internal test)
  elsif response==response.upcase
  # Grandma respond with random year
    puts ("No, not since ").upcase+(rand(21)+1930).to_s
    count=0
  else
  # Grandma can't hear
    puts ("Huh?! Speak up sonny!").upcase
    count=0
  end

  # Grandma says bye after 3 count
  if count==3
  puts "OK.  Fine.  Just leave then."
  end

end

fishina barrel wrote in post #1097688:
> My solution was very similar to @steve_k61.  Any advice from the peanut
> gallery on making this code cleaner? (Pardon all the comments, it helps
> me figure things out.)

Well, without modifying your basic algorithm, you can trim a lot of 
String#upcase calls, and inline a bit of stuff:

  #Deaf Grandma w/ extension

  #Set initial counter
  count=0

  #Create Loop Conditions
  while count < 3
    response=gets.chomp

    if response == 'BYE'
    # Grandma ignores but the count increases by 1
      count+=1
      #puts count  # (this was an internal test)
    elsif response == response.upcase
    # Grandma respond with random year
      puts "NO, NOT SINCE "#{rand(21)+1930}"
      count=0
    else
    # Grandma can't hear
      puts "HUH?! SPEAK UP SONNY!"
      count=0
    end

    # Grandma says bye after 3 count
    if count==3
      puts "OK.  Fine.  Just leave then."
    end

  end

Looks a lot like @steve_k61's code again, huh?

If it was me, I'd probably replace the 'while' statement with 'loop do', 
and after Granny says goodbye (inside the if-end block) I'd add a 
'break'.  I think that just makes it a bit more clear that this is 
actually an infinite loop that can be escaped only when certain criteria 
are met.  Doesn't really make a difference, though, I don't think.

Matthew Kerwin wrote in post #1097694:
> If it was me, I'd probably replace the 'while' statement with 'loop do',
> and after Granny says goodbye (inside the if-end block) I'd add a
> 'break'.  I think that just makes it a bit more clear that this is
> actually an infinite loop that can be escaped only when certain criteria
> are met.  Doesn't really make a difference, though, I don't think.

Sorry, I forgot to mention that you have a slight duplication of logic, 
because you've written 'while count < 3' and 'if count == 3'.  This 
means if Granny turns on her hearing aid you have to remember to change 
the 3 in two places in code.  (Look up the DRY principle.)

To get around this you should consolidate that logic, and I can think of 
two simple ways:

1) what I said above, with a 'loop{}' and 'break' structure, or
2) remove the final 'if-end' block and move the puts statement to the 
bottom of the code, after the while loop:

  count=0
  while count < 3
    response=gets.chomp
    if 'BYE' == response
      count+=1
    elsif response == response.upcase
      puts "NO, NOT SINCE "#{rand(21)+1930}"
      count=0
    else
      puts "HUH?! SPEAK UP SONNY!"
      count=0
    end
  end
  puts "OK.  Fine.  Just leave then."

Even more like @steve_k61's code, huh?

Matthew Kerwin wrote in post #1097695:
> Matthew Kerwin wrote in post #1097694:
>> If it was me, I'd probably replace the 'while' statement with 'loop do',
>> and after Granny says goodbye (inside the if-end block) I'd add a
>> 'break'.  I think that just makes it a bit more clear that this is
>> actually an infinite loop that can be escaped only when certain criteria
>> are met.  Doesn't really make a difference, though, I don't think.
>
> Sorry, I forgot to mention that you have a slight duplication of logic,
> because you've written 'while count < 3' and 'if count == 3'.  This
> means if Granny turns on her hearing aid you have to remember to change
> the 3 in two places in code.  (Look up the DRY principle.)
>
> To get around this you should consolidate that logic, and I can think of
> two simple ways:
>
> 1) what I said above, with a 'loop{}' and 'break' structure, or
> 2) remove the final 'if-end' block and move the puts statement to the
> bottom of the code, after the while loop:
>
>   count=0
>   while count < 3
>     response=gets.chomp
>     if 'BYE' == response
>       count+=1
>     elsif response == response.upcase
>       puts "NO, NOT SINCE "#{rand(21)+1930}"
>       count=0
>     else
>       puts "HUH?! SPEAK UP SONNY!"
>       count=0
>     end
>   end
>   puts "OK.  Fine.  Just leave then."
>
> Even more like @steve_k61's code, huh?


It really is...Thanks for the pointers.  I haven't learned loop do yet, 
so looking forward to that one.  I guess the final conditional statement 
was really unnecessary since the while loop would keep going until 
count==3.  Thanks again.

On Mon, Feb 18, 2013 at 6:18 PM, fishina barrel <lists@ruby-forum.com> 
wrote:
>> means if Granny turns on her hearing aid you have to remember to change
>>   while count < 3
>>   end
> --
> Posted via http://www.ruby-forum.com/.
>

Since we're sharing, here's mine:

puts "HI, SONNY!\nIT'S YOUR GRANDMA."

byes=0

begin
  print "> "
  reply = gets.chomp
  if reply == 'BYE'
    byes += 1
    case byes
    when 1; puts "CAN'T HEAR YOU, DEAR!"
    when 2; puts "STILL CAN'T HEAR YOU, SONNY!"
    else ; puts "OKAY, BYE!"
    end
  else
    byes = 0
    if reply == reply.upcase
      puts "NO, NOT SINCE #{rand(21)+1930}!"
    else
      puts "EHH?? WHAT\'S THAT??"
    end
  end
end until byes >= 3

Outputs:

HI, SONNY!
IT'S YOUR GRANDMA.
> hi gramma
EHH?? WHAT'S THAT??
> HI GRAMMA!
NO, NOT SINCE 1939!
> HOW'S GRAMPA?
NO, NOT SINCE 1934!
> bye
EHH?? WHAT'S THAT??
> BYE
CAN'T HEAR YOU, DEAR!
> really??
EHH?? WHAT'S THAT??
> BYE
CAN'T HEAR YOU, DEAR!
> BYE
STILL CAN'T HEAR YOU, SONNY!
> BYE
OKAY, BYE!