Class: RailsBestPractices::Reviews::NeedlessDeepNestingReview

Inherits:
Review
  • Object
show all
Defined in:
lib/rails_best_practices/reviews/needless_deep_nesting_review.rb

Overview

Review config/routes.rb file to make sure not to use too deep nesting routes.

See the best practice details here rails-bestpractices.com/posts/2010/07/22/needless-deep-nesting/

Implementation:

Review process:

chech all method_add_block nodes in route file.

it is a recursively check in method_add_block node,

if it is a method_add_block node,
increment @counter at the beginning of resources,
decrement @counter at the end of resrouces,
recursively check nodes in block body.

if the child node is a command_call or command node,
and the message of the node is "resources" or "resource",
and the @counter is greater than @nested_count defined,
then it is a needless deep nesting.

Constant Summary

Constants inherited from Core::Check

Core::Check::ALL_FILES, Core::Check::CAPFILE, Core::Check::CONFIG_FILES, Core::Check::CONTROLLER_FILES, Core::Check::DEPLOY_FILES, Core::Check::GEMFILE_LOCK, Core::Check::HELPER_FILES, Core::Check::INITIALIZER_FILES, Core::Check::MAILER_FILES, Core::Check::MIGRATION_FILES, Core::Check::MODEL_FILES, Core::Check::PARTIAL_VIEW_FILES, Core::Check::ROUTE_FILES, Core::Check::SCHEMA_FILE, Core::Check::SKIP_FILES, Core::Check::VIEW_FILES

Instance Method Summary collapse

Methods inherited from Review

#model_associations, #model_attributes, #models, #remember_variable_use_count, #reset_variable_use_count, #variable, #variable_use_count

Methods inherited from Core::Check

#add_error, debug, debug?, #errors, #is_ignored?, #is_interesting_file?, #method_missing, #parse_file?, #regex_ignored_files, #url, url

Constructor Details

#initialize(options = {}) ⇒ NeedlessDeepNestingReview

Returns a new instance of NeedlessDeepNestingReview.



30
31
32
33
34
35
 
# File 'lib/rails_best_practices/reviews/needless_deep_nesting_review.rb', line 30

def initialize(options = {})
  super(options)
  @counter = 0
  @nested_count = options['nested_count'] || 2
  @shallow_nodes = []
end

Dynamic Method Handling

This class handles dynamic methods through the method_missing method in the class RailsBestPractices::Core::Check