app/web/view_models/profile.rb
ViewModels::Profile::NullProfile#updated_at doesn't depend on instance state (maybe move it to another class?) Wontfix
Wontfix
def updated_at
- Read upRead up
- Exclude checks
A Utility Function is any instance method that has no dependency on the state of the instance.
ViewModels::Profile#self.build performs a nil-check Wontfix
Wontfix
NullProfile.new if obj.nil?
- Read upRead up
- Exclude checks
A NilCheck
is a type check. Failures of NilCheck
violate the "tell, don't ask" principle.
Additionally, type checks often mask bigger problems in your source code like not using OOP and / or polymorphism when you should.
Example
Given
class Klass
def nil_checker(argument)
if argument.nil?
puts "argument isn't nil!"
end
end
end
Reek would emit the following warning:
test.rb -- 1 warning:
[3]:Klass#nil_checker performs a nil-check. (NilCheck)