dmytroonopa / rubyhw Goto Github PK

flipper.rb:39:5: C: Style/GuardClause: Use a guard clause (return unless @hunger_indicator > 100) instead of wrapping the code inside a conditional expression.
    if @hunger_indicator > 100
    ^^
flipper.rb:50:5: C: Style/GuardClause: Use a guard clause (return unless @mood > 100) instead of wrapping the code inside a conditional expression.
    if @mood > 100
    ^^
flipper.rb:51:14: C: [Correctable] Layout/SpaceAroundOperators: Operator = should be surrounded by a single space.
      @heals =  100 - 50
             ^
flipper.rb:51:15: C: [Correctable] Layout/ExtraSpacing: Unnecessary spacing detected.
      @heals =  100 - 50
              ^
flipper.rb:62:5: C: Style/GuardClause: Use a guard clause (return unless @fish_wc.negative?) instead of wrapping the code inside a conditional expression.
    if @fish_wc.negative?
    ^^
flipper.rb:72:5: C: Style/GuardClause: Use a guard clause (return unless @fish_mana > 50) instead of wrapping the code inside a conditional expression.
    if @fish_mana > 50
    ^^
flipper.rb:83:5: C: Style/GuardClause: Use a guard clause (return unless @health >= 100) instead of wrapping the code inside a conditional expression.
    if @health >= 100
    ^^
flipper.rb:89:7: C: Naming/MethodName: Use snake_case for method names.
  def DisplayingInformation
      ^^^^^^^^^^^^^^^^^^^^^
flipper.rb:104:7: C: [Correctable] Style/IdenticalConditionalBranches: Move @fish_mana -= 5 out of the conditional.
      @fish_mana -= 5
      ^^^^^^^^^^^^^^^
flipper.rb:106:7: C: [Correctable] Style/IdenticalConditionalBranches: Move @fish_mana -= 5 out of the conditional.
      @fish_mana -= 5
      ^^^^^^^^^^^^^^^
flipper.rb:115:5: C: Style/GuardClause: Use a guard clause (return unless @fish_mana < 0) instead of wrapping the code inside a conditional expression.
    if @fish_mana < 0
    ^^
flipper.rb:115:8: C: [Correctable] Style/NumericPredicate: Use @fish_mana.negative? instead of @fish_mana < 0.
    if @fish_mana < 0
       ^^^^^^^^^^^^^^
flipper.rb:123:7: C: Naming/MethodName: Use snake_case for method names.
  def TimePeriod
      ^^^^^^^^^^
flipper.rb:149:7: C: [Correctable] Layout/CommentIndentation: Incorrect indentation detected (column 6 instead of 7).
      #@health -= 5
      ^^^^^^^^^^^^^
flipper.rb:149:7: C: [Correctable] Layout/LeadingCommentSpace: Missing space after #.
      #@health -= 5
      ^^^^^^^^^^^^^
flipper.rb:150:5: C: [Correctable] Layout/IndentationWidth: Use 2 (not 3) spaces for indentation.
       puts 'Low level mana!'
    ^^^
flipper.rb:163:5: C: Style/GuardClause: Use a guard clause (return unless @health <= 0) instead of wrapping the code inside a conditional expression.
    if @health <= 0
    ^^
flipper.rb:173:1: C: Naming/VariableName: Use snake_case for variable names.
menuSelect = gets.chomp.to_i
^^^^^^^^^^
flipper.rb:174:7: C: Naming/VariableName: Use snake_case for variable names.
while menuSelect != 0
      ^^^^^^^^^^
flipper.rb:176:8: C: Naming/VariableName: Use snake_case for variable names.
  case menuSelect
       ^^^^^^^^^^
flipper.rb:180:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:184:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:188:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:192:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:196:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:200:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:204:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:207:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i
    ^^^^^^^^^^
flipper.rb:210:5: C: Naming/VariableName: Use snake_case for variable names.
    menuSelect = gets.chomp.to_i

while menuSelect != 0
  case menuSelect
  when 1
    fish.feed
  when 2
    fish.pvp
  when 3
    fish.toilet
  when 4
    fish.mana_reg
  when 5
    fish.healing
  when 6
    fish.sleeping
  when 7
    fish.bathe
  when 8
    fish.DisplayingInformation
  when 9
    fish.help
  else
    break
  end
  fish.DisplayingInformation unless [8, 9].include?(menuSelect)
  menuSelect = gets.chomp.to_i
end