Sha256: c3936a3568b031551dd50a8e99252e742db4f16374ad21923d394c17cce57a0b
Contents?: true
Size: 1.76 KB
Versions: 12
Compression:
Stored size: 1.76 KB
Contents
require 'reek/smells/smell_detector'
require 'reek/smell_warning'
module Reek
module Smells
# Excerpt from:
# http://dablog.rubypal.com/2007/8/15/bang-methods-or-danger-will-rubyist
# since this sums it up really well:
#
# The ! in method names that end with ! means, "This method is dangerous"
# -- or, more precisely, this method is the "dangerous" version of an
# equivalent method, with the same name minus the !. "Danger" is
# relative; the ! doesn't mean anything at all unless the method name
# it's in corresponds to a similar but bang-less method name.
#
# Don't add ! to your destructive (receiver-changing) methods' names,
# unless you consider the changing to be "dangerous" and you have a
# "non-dangerous" equivalent method without the !. If some arbitrary
# subset of destructive methods end with !, then the whole point of !
# gets distorted and diluted, and ! ceases to convey any information
# whatsoever.
#
# Such a method is called PrimaDonnaMethod and is reported as a smell.
#
class PrimaDonnaMethod < SmellDetector
def self.contexts # :nodoc:
[:class]
end
def examine_context(ctx)
ctx.node_instance_methods.map do |method_sexp|
next unless method_sexp.ends_with_bang?
version_without_bang = ctx.node_instance_methods.find do |sexp_item|
sexp_item.name.to_s == method_sexp.name_without_bang
end
next if version_without_bang
SmellWarning.new self,
context: ctx.full_name,
lines: [ctx.exp.line],
message: "has prima donna method `#{method_sexp.name}`"
end.compact
end
end
end
end
Version data entries
12 entries across 12 versions & 1 rubygems