infos.php ➔ ShowProductionTable() - Code Metrics - Inspection of "CODE-misc-41a52.67: Fixed EntityModel/KeyModel con..." - supernova-ws/SuperNova - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — work-fleets ( fc0000...961997 )

by SuperNova.WS

created 2016-09-05 12:06 UTC

infos.php ➔ ShowProductionTable() F

↳ Parent: Project

Complexity

Conditions	13
Paths	312

Size

Total Lines	113
Code Lines	86

Duplication

Lines	12
Ratio	10.62 %

Importance

Changes

Metric	Value
cc	13
eloc	86
c	0
b	0
f	0
nc	312
nop	4
dl	12
loc	113
rs	3.7737

How to fix Long Method Complexity

<?php

/**
 * infos.php
 *
 * Information about every aspect of in-game objects: buildings, officiers, techs etc
 *
 * @version 1.1st Security checks & tests by Gorlum for http://supernova.ws
 * @version 1.1
 * @copyright 2008 By Chlorel for XNova
 */
include('common.' . substr(strrchr(__FILE__, '.'), 1));

function ShowProductionTable($CurrentUser, $CurrentPlanet, $BuildID, $Template) {
  $config_resource_multiplier = game_resource_multiplier();
  $config_resource_multiplier_plain = game_resource_multiplier(true);

  $CurrentBuildtLvl = mrc_get_level($CurrentUser, $CurrentPlanet, $BuildID);

  $BuildLevel = ($CurrentBuildtLvl > 0) ? $CurrentBuildtLvl : 1;

  $modifiers = sn_get_groups('modifiers');

  $Prod[STRUC_MINE_METAL] = floor(mrc_modify_value(
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
    $CurrentUser,
    $CurrentPlanet,
    $modifiers[MODIFIER_RESOURCE_PRODUCTION],
    $config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_METAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)

  ));
  $Prod[STRUC_MINE_CRYSTAL] = floor(mrc_modify_value(
    $CurrentUser,
    $CurrentPlanet,
    $modifiers[MODIFIER_RESOURCE_PRODUCTION],
    $config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_CRYSTAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
  ));
  $Prod[STRUC_MINE_DEUTERIUM] = floor(mrc_modify_value(
    $CurrentUser,
    $CurrentPlanet,
    $modifiers[MODIFIER_RESOURCE_PRODUCTION],
    $config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_DEUTERIUM]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
  ));
  $Prod[STRUC_MINE_SOLAR] = floor(mrc_modify_value(
    $CurrentUser,
    $CurrentPlanet,
    $modifiers[MODIFIER_RESOURCE_PRODUCTION],
    $config_resource_multiplier_plain * $unit_data[P_UNIT_PRODUCTION][RES_ENERGY]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
  ));

  $ActualProd = floor($Prod[$BuildID]);
  if($BuildID != STRUC_MINE_FUSION) {

    $ActualNeed = floor($Prod[STRUC_MINE_SOLAR]);
  } else {
    $ActualNeed = floor($Prod[STRUC_MINE_DEUTERIUM]);
  }

  $BuildStartLvl = $CurrentBuildtLvl - 2;
  if($BuildStartLvl < 1) {
    $BuildStartLvl = 1;
  }
  $Table = '';
  $ProdFirst = 0;
  for($BuildLevel = $BuildStartLvl; $BuildLevel < $BuildStartLvl + 10; $BuildLevel++) {
    if($BuildID != STRUC_MOON_PHALANX) {
      $Prod[STRUC_MINE_METAL] = floor(mrc_modify_value(
        $CurrentUser,
        $CurrentPlanet,
        $modifiers[MODIFIER_RESOURCE_PRODUCTION],
        $config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_METAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
      ));
      $Prod[STRUC_MINE_CRYSTAL] = floor(mrc_modify_value(
        $CurrentUser,
        $CurrentPlanet,
        $modifiers[MODIFIER_RESOURCE_PRODUCTION],
        $config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_CRYSTAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
      ));
      $Prod[STRUC_MINE_DEUTERIUM] = floor(mrc_modify_value(
        $CurrentUser,
        $CurrentPlanet,
        $modifiers[MODIFIER_RESOURCE_PRODUCTION],
        $config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_DEUTERIUM]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
      ));
      $Prod[STRUC_MINE_SOLAR] = floor(mrc_modify_value(
        $CurrentUser,
        $CurrentPlanet,
        $modifiers[MODIFIER_RESOURCE_PRODUCTION],
        $config_resource_multiplier_plain * $unit_data[P_UNIT_PRODUCTION][RES_ENERGY]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
      ));

      $bloc['build_lvl'] = ($CurrentBuildtLvl == $BuildLevel) ? "<font color=\"#ff0000\">" . $BuildLevel . "</font>" : $BuildLevel;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
      if($ProdFirst > 0) {
        if($BuildID != STRUC_MINE_FUSION) {
          $bloc['build_gain'] = "<font color=\"lime\">(" . pretty_number(floor($Prod[$BuildID] - $ProdFirst)) . ")</font>";
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
        } else {
          $bloc['build_gain'] = "<font color=\"lime\">(" . pretty_number(floor($Prod[STRUC_MINE_SOLAR] - $ProdFirst)) . ")</font>";
        }
      } else {
        $bloc['build_gain'] = '';
      }
      if($BuildID != STRUC_MINE_FUSION) {
        $bloc['build_prod'] = pretty_number(floor($Prod[$BuildID]));
        $bloc['build_prod_diff'] = pretty_number(floor($Prod[$BuildID] - $ActualProd), true, true);
        $bloc['build_need'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR]), true, true);
        $bloc['build_need_diff'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR] - $ActualNeed), true, true);
      } else {
        $bloc['build_prod'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR]));
        $bloc['build_prod_diff'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR] - $ActualProd), true, true);
        $bloc['build_need'] = pretty_number(floor($Prod[STRUC_MINE_DEUTERIUM]), true, true);
        $bloc['build_need_diff'] = pretty_number(floor($Prod[STRUC_MINE_DEUTERIUM] - $ActualNeed), true, true);
      }
      if($ProdFirst == 0) {

        if($BuildID != STRUC_MINE_FUSION) {
          $ProdFirst = floor($Prod[$BuildID]);
        } else {
          $ProdFirst = floor($Prod[STRUC_MINE_SOLAR]);
        }
      }
    } else {
      // Cas particulier de la phalange
      $bloc['build_lvl'] = ($CurrentBuildtLvl == $BuildLevel) ? "<font color=\"#ff0000\">" . $BuildLevel . "</font>" : $BuildLevel;
      $bloc['build_range'] = ($BuildLevel * $BuildLevel) - 1;
    }
    $Table .= parsetemplate($Template, $bloc);
  }

  return $Table;
}

function eco_render_rapid_fire($unit_id) {
  $classLocale = classLocale::$lang;

  $unit_data = get_unit_param($unit_id);
  $unit_durability = $unit_data['shield'] + $unit_data['armor'];

  $str_rapid_from = '';
  $str_rapid_to = '';
  foreach(sn_get_groups(array('fleet', 'defense_active')) as $enemy_id) {
    $enemy_data = get_unit_param($enemy_id);
    $enemy_durability = $enemy_data['shield'] + $enemy_data['armor'];

    $rapid = floor($unit_data['attack'] * (isset($unit_data['amplify'][$enemy_id]) ? $unit_data['amplify'][$enemy_id] : 1) / $enemy_durability);
    if($rapid >= 1) {
      $str_rapid_to .= "{$classLocale['nfo_rf_again']} {$classLocale['tech'][$enemy_id]} <font color=\"#00ff00\">{$rapid}</font><br>";
    }

    $rapid = floor($enemy_data['attack'] * (isset($enemy_data['amplify'][$unit_id]) ? $enemy_data['amplify'][$unit_id] : 1) / $unit_durability);
    if($rapid >= 1) {
      $str_rapid_from .= "{$classLocale['tech'][$enemy_id]} {$classLocale['nfo_rf_from']} <font color=\"#ff0000\">{$rapid}</font><br>";
    }
  }

  if($str_rapid_to && $str_rapid_from) {
    $str_rapid_to .= '<hr>';
  }

  return array('to' => $str_rapid_to, 'from' => $str_rapid_from);
}

$unit_id = sys_get_param_id('gid');
if($unit_id == RES_DARK_MATTER) {
  sys_redirect('dark_matter.php');
}

if($unit_id == RES_METAMATTER) {
  sys_redirect('metamatter.php');
}

lng_include('infos');
if(!$unit_id || (!get_unit_param($unit_id) && !isset(classLocale::$lang['info'][$unit_id]))) {
  sys_redirect('index.php?page=techtree');
}

$template = gettemplate('novapedia', true);

$unit_data = get_unit_param($unit_id);
$unit_type = $unit_data['type'];

if($unit_type == UNIT_SHIPS) {
  $template_result['UNIT_IS_SHIP'] = true;

  $ship_data = get_ship_data($unit_id, $user);

  $template_result += array(
    'BASE_SPEED'         => pretty_number($ship_data['speed_base']),
    'ACTUAL_SPEED'       => pretty_number($ship_data['speed']),
    'BASE_CONSUMPTION'   => pretty_number($ship_data['consumption_base']),
    'ACTUAL_CONSUMPTION' => pretty_number($ship_data['consumption']),

    'BASE_CAPACITY'   => pretty_number($unit_data['capacity']),
    'ACTUAL_CAPACITY' => pretty_number($ship_data['capacity']),
  );

  $engine_template_info = array();
  foreach($unit_data['engine'] as $unit_engine_data) {
    $unit_engine_data = get_engine_data($user, $unit_engine_data);

    $engine_template_info[] = array(
      'NAME'               => classLocale::$lang['tech'][$unit_engine_data['tech']],
      'MIN_LEVEL'          => $unit_engine_data['min_level'],
      'USER_TECH_LEVEL'    => mrc_get_level($user, null, $unit_engine_data['tech']),
      'BASE_SPEED'         => pretty_number($unit_engine_data['speed_base']),
      'BASE_CONSUMPTION'   => pretty_number($unit_engine_data['consumption_base']),
      'ACTUAL_SPEED'       => pretty_number($unit_engine_data['speed']),
      'ACTUAL_CONSUMPTION' => pretty_number($unit_engine_data['consumption']),
    );
  }
  $template_result['.']['engine'] = $engine_template_info;

}


$sn_data_group_combat = sn_get_groups('combat');
if(in_array($unit_id, $sn_data_group_combat)) {
  $template_result['UNIT_IS_COMBAT'] = true;

  $unit_durability = $unit_data['shield'] + $unit_data['armor'];

  $volley_arr = $rapid_to = $rapid_from = array();
  $str_rapid_from = '';
  $str_rapid_to = '';
  foreach($sn_data_group_combat as $enemy_id) {
    $enemy_data = get_unit_param($enemy_id);
    $enemy_durability = $enemy_data['shield'] + $enemy_data['armor'];

    $rapid = $unit_data['attack'] * (isset($unit_data['amplify'][$enemy_id]) ? $unit_data['amplify'][$enemy_id] : 1) / $enemy_durability;
    if($rapid >= 1) {
      $volley_arr[$enemy_id]['TO'] = floor($rapid);
    }

    $rapid = $enemy_data['attack'] * (isset($enemy_data['amplify'][$unit_id]) ? $enemy_data['amplify'][$unit_id] : 1) / $unit_durability;
    if($rapid >= 1) {
      $volley_arr[$enemy_id]['FROM'] = floor($rapid);
    }
  }
  foreach($volley_arr as $enemy_id => &$rapid) {
    $rapid['ENEMY_ID'] = $enemy_id;
    $rapid['ENEMY_NAME'] = classLocale::$lang['tech'][$enemy_id];
  }
  $template_result['.']['volley'] = $volley_arr;

  $template_result += array(
    'BASE_ARMOR'  => pretty_number($unit_data['armor']),
    'BASE_SHIELD' => pretty_number($unit_data['shield']),
    'BASE_WEAPON' => pretty_number($unit_data['attack']),

    'ACTUAL_ARMOR'  => pretty_number(mrc_modify_value($user, false, array(MRC_ADMIRAL, TECH_ARMOR), $unit_data['armor'])),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    'ACTUAL_SHIELD' => pretty_number(mrc_modify_value($user, false, array(MRC_ADMIRAL, TECH_SHIELD), $unit_data['shield'])),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    'ACTUAL_WEAPON' => pretty_number(mrc_modify_value($user, false, array(MRC_ADMIRAL, TECH_WEAPON), $unit_data['attack'])),
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
  );

}

if(classLocale::$lang['info'][$unit_id]['effect']) {
  $template_result['UNIT_EFFECT'] = classLocale::$lang['info'][$unit_id]['effect'];
}

if($unit_data['bonus']) {
  $unit_bonus = !$unit_data['bonus'] || $unit_data['bonus_type'] == BONUS_ABILITY ? '' : (
    ($unit_data['bonus'] >= 0 ? '+' : '') . $unit_data['bonus'] . ($unit_data['bonus_type'] == BONUS_PERCENT ? '%' : '')
  );
  $template_result['UNIT_BONUS'] = $unit_bonus;
}

$template_result += array(
  'PAGE_HEADER' => classLocale::$lang['wiki_title'],

  'UNIT_ID'          => $unit_id,
  'UNIT_NAME'        => classLocale::$lang['tech'][$unit_id],
  'UNIT_TYPE'        => $unit_type,
  'UNIT_TYPE_NAME'   => classLocale::$lang['tech'][$unit_type],
  'UNIT_DESCRIPTION' => classLocale::$lang['info'][$unit_id]['description'],
);

$template_result['.']['require'] = unit_requirements_render($user, $planetrow, $unit_id);

$template->assign_recursive($template_result);
display($template);


1		<?php
2
3		/**
4		* infos.php
5		*
6		* Information about every aspect of in-game objects: buildings, officiers, techs etc
7		*
8		* @version 1.1st Security checks & tests by Gorlum for http://supernova.ws
9		* @version 1.1
10		* @copyright 2008 By Chlorel for XNova
11		*/
12		include('common.' . substr(strrchr(__FILE__, '.'), 1));
13
14		function ShowProductionTable($CurrentUser, $CurrentPlanet, $BuildID, $Template) {
15		$config_resource_multiplier = game_resource_multiplier();
16		$config_resource_multiplier_plain = game_resource_multiplier(true);
17
18		$CurrentBuildtLvl = mrc_get_level($CurrentUser, $CurrentPlanet, $BuildID);
19
20		$BuildLevel = ($CurrentBuildtLvl > 0) ? $CurrentBuildtLvl : 1;
21
22		$modifiers = sn_get_groups('modifiers');
23
24		$Prod[STRUC_MINE_METAL] = floor(mrc_modify_value(
		0 ignored issues – show Coding Style Comprehensibility introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report `$Prod` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$Prod = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
25		$CurrentUser,
26		$CurrentPlanet,
27		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
28		$config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_METAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
		0 ignored issues – show Bug introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report The variable `$unit_data` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
29		));
30		$Prod[STRUC_MINE_CRYSTAL] = floor(mrc_modify_value(
31		$CurrentUser,
32		$CurrentPlanet,
33		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
34		$config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_CRYSTAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
35		));
36		$Prod[STRUC_MINE_DEUTERIUM] = floor(mrc_modify_value(
37		$CurrentUser,
38		$CurrentPlanet,
39		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
40		$config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_DEUTERIUM]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
41		));
42		$Prod[STRUC_MINE_SOLAR] = floor(mrc_modify_value(
43		$CurrentUser,
44		$CurrentPlanet,
45		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
46		$config_resource_multiplier_plain * $unit_data[P_UNIT_PRODUCTION][RES_ENERGY]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
47		));
48
49		$ActualProd = floor($Prod[$BuildID]);
50	View Code Duplication	if($BuildID != STRUC_MINE_FUSION) {
		0 ignored issues – show Duplication introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
51		$ActualNeed = floor($Prod[STRUC_MINE_SOLAR]);
52		} else {
53		$ActualNeed = floor($Prod[STRUC_MINE_DEUTERIUM]);
54		}
55
56		$BuildStartLvl = $CurrentBuildtLvl - 2;
57		if($BuildStartLvl < 1) {
58		$BuildStartLvl = 1;
59		}
60		$Table = '';
61		$ProdFirst = 0;
62		for($BuildLevel = $BuildStartLvl; $BuildLevel < $BuildStartLvl + 10; $BuildLevel++) {
63		if($BuildID != STRUC_MOON_PHALANX) {
64		$Prod[STRUC_MINE_METAL] = floor(mrc_modify_value(
65		$CurrentUser,
66		$CurrentPlanet,
67		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
68		$config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_METAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
69		));
70		$Prod[STRUC_MINE_CRYSTAL] = floor(mrc_modify_value(
71		$CurrentUser,
72		$CurrentPlanet,
73		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
74		$config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_CRYSTAL]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
75		));
76		$Prod[STRUC_MINE_DEUTERIUM] = floor(mrc_modify_value(
77		$CurrentUser,
78		$CurrentPlanet,
79		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
80		$config_resource_multiplier * $unit_data[P_UNIT_PRODUCTION][RES_DEUTERIUM]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
81		));
82		$Prod[STRUC_MINE_SOLAR] = floor(mrc_modify_value(
83		$CurrentUser,
84		$CurrentPlanet,
85		$modifiers[MODIFIER_RESOURCE_PRODUCTION],
86		$config_resource_multiplier_plain * $unit_data[P_UNIT_PRODUCTION][RES_ENERGY]($BuildLevel, 100, $CurrentUser, $CurrentPlanet)
87		));
88
89		$bloc['build_lvl'] = ($CurrentBuildtLvl == $BuildLevel) ? "<font color=\"#ff0000\">" . $BuildLevel . "</font>" : $BuildLevel;
		0 ignored issues – show Coding Style Comprehensibility introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report `$bloc` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$bloc = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
90		if($ProdFirst > 0) {
91		if($BuildID != STRUC_MINE_FUSION) {
92		$bloc['build_gain'] = "<font color=\"lime\">(" . pretty_number(floor($Prod[$BuildID] - $ProdFirst)) . ")</font>";
		0 ignored issues – show Bug introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report The variable `$bloc` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
93		} else {
94		$bloc['build_gain'] = "<font color=\"lime\">(" . pretty_number(floor($Prod[STRUC_MINE_SOLAR] - $ProdFirst)) . ")</font>";
95		}
96		} else {
97		$bloc['build_gain'] = '';
98		}
99		if($BuildID != STRUC_MINE_FUSION) {
100		$bloc['build_prod'] = pretty_number(floor($Prod[$BuildID]));
101		$bloc['build_prod_diff'] = pretty_number(floor($Prod[$BuildID] - $ActualProd), true, true);
102		$bloc['build_need'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR]), true, true);
103		$bloc['build_need_diff'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR] - $ActualNeed), true, true);
104		} else {
105		$bloc['build_prod'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR]));
106		$bloc['build_prod_diff'] = pretty_number(floor($Prod[STRUC_MINE_SOLAR] - $ActualProd), true, true);
107		$bloc['build_need'] = pretty_number(floor($Prod[STRUC_MINE_DEUTERIUM]), true, true);
108		$bloc['build_need_diff'] = pretty_number(floor($Prod[STRUC_MINE_DEUTERIUM] - $ActualNeed), true, true);
109		}
110	View Code Duplication	if($ProdFirst == 0) {
		0 ignored issues – show Duplication introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
111		if($BuildID != STRUC_MINE_FUSION) {
112		$ProdFirst = floor($Prod[$BuildID]);
113		} else {
114		$ProdFirst = floor($Prod[STRUC_MINE_SOLAR]);
115		}
116		}
117		} else {
118		// Cas particulier de la phalange
119		$bloc['build_lvl'] = ($CurrentBuildtLvl == $BuildLevel) ? "<font color=\"#ff0000\">" . $BuildLevel . "</font>" : $BuildLevel;
120		$bloc['build_range'] = ($BuildLevel * $BuildLevel) - 1;
121		}
122		$Table .= parsetemplate($Template, $bloc);
123		}
124
125		return $Table;
126		}
127
128		function eco_render_rapid_fire($unit_id) {
129		$classLocale = classLocale::$lang;
130
131		$unit_data = get_unit_param($unit_id);
132		$unit_durability = $unit_data['shield'] + $unit_data['armor'];
133
134		$str_rapid_from = '';
135		$str_rapid_to = '';
136		foreach(sn_get_groups(array('fleet', 'defense_active')) as $enemy_id) {
137		$enemy_data = get_unit_param($enemy_id);
138		$enemy_durability = $enemy_data['shield'] + $enemy_data['armor'];
139
140		$rapid = floor($unit_data['attack'] * (isset($unit_data['amplify'][$enemy_id]) ? $unit_data['amplify'][$enemy_id] : 1) / $enemy_durability);
141		if($rapid >= 1) {
142		$str_rapid_to .= "{$classLocale['nfo_rf_again']} {$classLocale['tech'][$enemy_id]} <font color=\"#00ff00\">{$rapid}</font><br>";
143		}
144
145		$rapid = floor($enemy_data['attack'] * (isset($enemy_data['amplify'][$unit_id]) ? $enemy_data['amplify'][$unit_id] : 1) / $unit_durability);
146		if($rapid >= 1) {
147		$str_rapid_from .= "{$classLocale['tech'][$enemy_id]} {$classLocale['nfo_rf_from']} <font color=\"#ff0000\">{$rapid}</font><br>";
148		}
149		}
150
151		if($str_rapid_to && $str_rapid_from) {
152		$str_rapid_to .= '<hr>';
153		}
154
155		return array('to' => $str_rapid_to, 'from' => $str_rapid_from);
156		}
157
158		$unit_id = sys_get_param_id('gid');
159		if($unit_id == RES_DARK_MATTER) {
160		sys_redirect('dark_matter.php');
161		}
162
163		if($unit_id == RES_METAMATTER) {
164		sys_redirect('metamatter.php');
165		}
166
167		lng_include('infos');
168		if(!$unit_id \|\| (!get_unit_param($unit_id) && !isset(classLocale::$lang['info'][$unit_id]))) {
169		sys_redirect('index.php?page=techtree');
170		}
171
172		$template = gettemplate('novapedia', true);
173
174		$unit_data = get_unit_param($unit_id);
175		$unit_type = $unit_data['type'];
176
177		if($unit_type == UNIT_SHIPS) {
178		$template_result['UNIT_IS_SHIP'] = true;
179
180		$ship_data = get_ship_data($unit_id, $user);
181
182		$template_result += array(
183		'BASE_SPEED' => pretty_number($ship_data['speed_base']),
184		'ACTUAL_SPEED' => pretty_number($ship_data['speed']),
185		'BASE_CONSUMPTION' => pretty_number($ship_data['consumption_base']),
186		'ACTUAL_CONSUMPTION' => pretty_number($ship_data['consumption']),
187
188		'BASE_CAPACITY' => pretty_number($unit_data['capacity']),
189		'ACTUAL_CAPACITY' => pretty_number($ship_data['capacity']),
190		);
191
192		$engine_template_info = array();
193		foreach($unit_data['engine'] as $unit_engine_data) {
194		$unit_engine_data = get_engine_data($user, $unit_engine_data);
195
196		$engine_template_info[] = array(
197		'NAME' => classLocale::$lang['tech'][$unit_engine_data['tech']],
198		'MIN_LEVEL' => $unit_engine_data['min_level'],
199		'USER_TECH_LEVEL' => mrc_get_level($user, null, $unit_engine_data['tech']),
200		'BASE_SPEED' => pretty_number($unit_engine_data['speed_base']),
201		'BASE_CONSUMPTION' => pretty_number($unit_engine_data['consumption_base']),
202		'ACTUAL_SPEED' => pretty_number($unit_engine_data['speed']),
203		'ACTUAL_CONSUMPTION' => pretty_number($unit_engine_data['consumption']),
204		);
205		}
206		$template_result['.']['engine'] = $engine_template_info;
207
208		}
209
210
211		$sn_data_group_combat = sn_get_groups('combat');
212		if(in_array($unit_id, $sn_data_group_combat)) {
213		$template_result['UNIT_IS_COMBAT'] = true;
214
215		$unit_durability = $unit_data['shield'] + $unit_data['armor'];
216
217		$volley_arr = $rapid_to = $rapid_from = array();
218		$str_rapid_from = '';
219		$str_rapid_to = '';
220		foreach($sn_data_group_combat as $enemy_id) {
221		$enemy_data = get_unit_param($enemy_id);
222		$enemy_durability = $enemy_data['shield'] + $enemy_data['armor'];
223
224		$rapid = $unit_data['attack'] * (isset($unit_data['amplify'][$enemy_id]) ? $unit_data['amplify'][$enemy_id] : 1) / $enemy_durability;
225		if($rapid >= 1) {
226		$volley_arr[$enemy_id]['TO'] = floor($rapid);
227		}
228
229		$rapid = $enemy_data['attack'] * (isset($enemy_data['amplify'][$unit_id]) ? $enemy_data['amplify'][$unit_id] : 1) / $unit_durability;
230		if($rapid >= 1) {
231		$volley_arr[$enemy_id]['FROM'] = floor($rapid);
232		}
233		}
234		foreach($volley_arr as $enemy_id => &$rapid) {
235		$rapid['ENEMY_ID'] = $enemy_id;
236		$rapid['ENEMY_NAME'] = classLocale::$lang['tech'][$enemy_id];
237		}
238		$template_result['.']['volley'] = $volley_arr;
239
240		$template_result += array(
241		'BASE_ARMOR' => pretty_number($unit_data['armor']),
242		'BASE_SHIELD' => pretty_number($unit_data['shield']),
243		'BASE_WEAPON' => pretty_number($unit_data['attack']),
244
245		'ACTUAL_ARMOR' => pretty_number(mrc_modify_value($user, false, array(MRC_ADMIRAL, TECH_ARMOR), $unit_data['armor'])),
		0 ignored issues – show Documentation introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
246		'ACTUAL_SHIELD' => pretty_number(mrc_modify_value($user, false, array(MRC_ADMIRAL, TECH_SHIELD), $unit_data['shield'])),
		0 ignored issues – show Documentation introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
247		'ACTUAL_WEAPON' => pretty_number(mrc_modify_value($user, false, array(MRC_ADMIRAL, TECH_WEAPON), $unit_data['attack'])),
		0 ignored issues – show Documentation introduced 2016-03-23 16:50 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
248		);
249
250		}
251
252		if(classLocale::$lang['info'][$unit_id]['effect']) {
253		$template_result['UNIT_EFFECT'] = classLocale::$lang['info'][$unit_id]['effect'];
254		}
255
256		if($unit_data['bonus']) {
257		$unit_bonus = !$unit_data['bonus'] \|\| $unit_data['bonus_type'] == BONUS_ABILITY ? '' : (
258		($unit_data['bonus'] >= 0 ? '+' : '') . $unit_data['bonus'] . ($unit_data['bonus_type'] == BONUS_PERCENT ? '%' : '')
259		);
260		$template_result['UNIT_BONUS'] = $unit_bonus;
261		}
262
263		$template_result += array(
264		'PAGE_HEADER' => classLocale::$lang['wiki_title'],
265
266		'UNIT_ID' => $unit_id,
267		'UNIT_NAME' => classLocale::$lang['tech'][$unit_id],
268		'UNIT_TYPE' => $unit_type,
269		'UNIT_TYPE_NAME' => classLocale::$lang['tech'][$unit_type],
270		'UNIT_DESCRIPTION' => classLocale::$lang['info'][$unit_id]['description'],
271		);
272
273		$template_result['.']['require'] = unit_requirements_render($user, $planetrow, $unit_id);
274
275		$template->assign_recursive($template_result);
276		display($template);
277

supernova-ws / SuperNova

Push — work-fleets ( fc0000...961997 )

infos.php ➔ ShowProductionTable() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Duplication Side-by-Side

Filter issues like