Berkeley SaaS class HW1 Part 4: Basic OOP

gistfile1.rb

# (a) Create a class Dessert with getters and setters for name and calories. Define
# instance methods healthy?, which returns true if a dessert has less than 200
# calories, and delicious?, which returns true for all desserts.
#
# (b) Create a class JellyBean that extends Dessert, and add a getter and setter for
# flavor. Modify delicious? to return false if the flavor is black licorice (but delicious?
# should still return true for all other flavors and for all non-JellyBean desserts).

class Dessert
  attr_accessor :name
  attr_accessor :calories
  
  def initialize(name, calories)
    @name=name
    @calories=calories
  end
  
  def healthy?
     @calories<200 
  end

  def delicious?
    true
  end
end

class JellyBean < Dessert
  attr_accessor :flavor
  def initialize(name, calories, flavor)
    @name=name 
    @calories=calories
    @flavor=flavor
  end
  
  def delicious?
    if @flavor== "black licorice"
      false
    else
      true
    end
  end
end

a=JellyBean.new("a",232,"asd")
p a.delicious?
p a.healthy?
b=Dessert.new("asd",190)
p b.delicious?
p b.healthy?

def delicious? in JellyBean could be written shorter by doing this

def delicious?
return @flavor=="black licorice"
end

I think you can also ommit the word return

FerPerales, I agree that it can be written more concisely, but I think you accidentally reversed the logic.

def delicious?
@flavor != "black licorice"
end