class Ameba::Rule::Style::RedundantSelf
- Ameba::Rule::Style::RedundantSelf
- Ameba::Rule::Base
- Reference
- Object
Overview
A rule that disallows redundant uses of self
.
This is considered bad:
class Greeter
getter name : String
def self.init
self.new("Crystal").greet
end
def initialize(@name)
end
def greet
puts "Hello, my name is #{self.name}"
end
self.init
end
And needs to be written as:
class Greeter
getter name : String
def self.init
new("Crystal").greet
end
def initialize(@name)
end
def greet
puts "Hello, my name is #{name}"
end
init
end
YAML configuration example:
Style/RedundantSelf:
Enabled: true
AllowedMethodNames:
- in?
- inspect
- not_nil!
Included Modules
- Ameba::AST::Util
- YAML::Serializable
- YAML::Serializable::Strict
Defined in:
ameba/rule/style/redundant_self.crConstant Summary
-
CRYSTAL_KEYWORDS =
Crystal::Keyword.values.map(&.to_s)
-
MSG =
"Redundant `self` detected"
Constructors
- .new(ctx : YAML::ParseContext, node : YAML::Nodes::Node)
-
.new(config = nil)
A rule that disallows redundant uses of
self
.
Class Method Summary
-
.parsed_doc : String | Nil
Returns documentation for this rule, if there is any.
Instance Method Summary
- #allowed_method_names : Array(String)
- #allowed_method_names=(allowed_method_names : Array(String))
- #description : String
- #description=(description : String)
- #enabled=(enabled : Bool)
- #enabled? : Bool
- #excluded : Array(String) | Nil
- #excluded=(excluded : Array(String) | Nil)
- #severity : Ameba::Severity
- #severity=(severity : Ameba::Severity)
- #since_version : SemanticVersion | Nil
- #since_version=(since_version : String)
- #test(source, node : Crystal::Call, scope : AST::Scope)
- #test(source)
Instance methods inherited from module Ameba::AST::Util
abort?(node)
abort?,
control_exp_code(node : Crystal::ControlExpression, code_lines)
control_exp_code,
dynamic_literal?(node) : Bool
dynamic_literal?,
exit?(node)
exit?,
flow_command?(node, in_loop)
flow_command?,
flow_expression?(node, in_loop = false)
flow_expression?,
literal?(node) : Bool
literal?,
loop?(node)
loop?,
name_end_location(node)
name_end_location,
name_location(node)
name_location,
name_size(node)
name_size,
node_source(node, code_lines)
node_source,
path_named?(node, name) : Bool
path_named?,
raise?(node)
raise?,
source_between(loc, end_loc, code_lines) : String | Nil
source_between,
static_literal?(node) : Bool
static_literal?
Instance methods inherited from class Ameba::Rule::Base
==(other)
==,
catch(source : Source)
catch,
excluded?(source)
excluded?,
group
group,
hash
hash,
name
name,
special?
special?,
test(source : Source, node : Crystal::ASTNode, *opts)test(source : Source) test
Class methods inherited from class Ameba::Rule::Base
default_severity : Ameba::Severity
default_severity
Macros inherited from class Ameba::Rule::Base
issue_for(*args, **kwargs, &block)
issue_for
Macros inherited from module Ameba::Config::RuleConfig
properties(&block)
properties
Constructor Detail
def self.new(config = nil)
#
A rule that disallows redundant uses of self
.
This is considered bad:
class Greeter
getter name : String
def self.init
self.new("Crystal").greet
end
def initialize(@name)
end
def greet
puts "Hello, my name is #{self.name}"
end
self.init
end
And needs to be written as:
class Greeter
getter name : String
def self.init
new("Crystal").greet
end
def initialize(@name)
end
def greet
puts "Hello, my name is #{name}"
end
init
end
YAML configuration example:
Style/RedundantSelf:
Enabled: true
AllowedMethodNames:
- in?
- inspect
- not_nil!
Class Method Detail
def self.parsed_doc : String | Nil
#
Returns documentation for this rule, if there is any.
module Ameba
# This is a test rule.
# Does nothing.
class MyRule < Ameba::Rule::Base
def test(source)
end
end
end
MyRule.parsed_doc # => "This is a test rule.\nDoes nothing."