app/models/manageiq/providers/ibm_cloud/inventory/collector/power_virtual_servers/target_collection.rb
Cyclomatic complexity for parse_targets! is too high. [13/11] Open
Open
def parse_targets!
# `target` here is an `InventoryRefresh::TargetCollection`. This contains two types of targets,
# `InventoryRefresh::Target` which is essentialy an association/manager_ref pair, or an ActiveRecord::Base
# type object like a Vm.
#
- Read upRead up
- Exclude checks
Checks that the cyclomatic complexity of methods is not higher than the configured maximum. The cyclomatic complexity is the number of linearly independent paths through a method. The algorithm counts decision points and adds one.
An if statement (or unless or ?:) increases the complexity by one. An else branch does not, since it doesn't add a decision point. The && operator (or keyword and) can be converted to a nested if statement, and ||/or is shorthand for a sequence of ifs, so they also add one. Loops can be said to have an exit condition, so they add one. Blocks that are calls to builtin iteration methods (e.g. `ary.map{...}) also add one, others are ignored.
def each_child_node(*types) # count begins: 1
unless block_given? # unless: +1
return to_enum(__method__, *types)
children.each do |child| # each{}: +1
next unless child.is_a?(Node) # unless: +1
yield child if types.empty? || # if: +1, ||: +1
types.include?(child.type)
end
self
end # total: 6
Use filter_map
instead. Open
Open
@images ||= references(:miq_templates).map do |ems_ref|
images_api.pcloud_cloudinstances_images_get(cloud_instance_id, ems_ref)
rescue IbmCloudPower::ApiError => err
error_message = JSON.parse(err.response_body)["description"]
_log.debug("ImageID not found: #{error_message}")
- Exclude checks
Use filter_map
instead. Open
Open
@pvm_instances ||= references(:vms).map do |ems_ref|
pvm_instances_api.pcloud_pvminstances_get(cloud_instance_id, ems_ref)
rescue IbmCloudPower::ApiError => err
error_message = JSON.parse(err.response_body)["description"]
_log.debug("PVMInstanceID not found: #{error_message}")
- Exclude checks
Use filter_map
instead. Open
Open
@volumes ||= references(:cloud_volumes).map do |ems_ref|
volumes_api.pcloud_cloudinstances_volumes_get(cloud_instance_id, ems_ref)
rescue IbmCloudPower::ApiError => err
error_message = JSON.parse(err.response_body)["description"]
_log.debug("VolumeID not found: #{error_message}")
- Exclude checks
Use filter_map
instead. Open
Open
@networks ||= references(:cloud_networks).map do |ems_ref|
networks_api.pcloud_networks_get(cloud_instance_id, ems_ref)
rescue IbmCloudPower::ApiError => err
error_message = JSON.parse(err.response_body)["description"]
_log.debug("NetworkID not found: #{error_message}")
- Exclude checks