| Conditions | 3 | 
| Paths | 4 | 
| Total Lines | 76 | 
| Code Lines | 64 | 
| Lines | 0 | 
| Ratio | 0 % | 
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php  | 
            ||
| 168 | private function outputHtmlXmlComparaison($lang = 'en')  | 
            ||
| 169 |     { | 
            ||
| 170 |         $crawler = $this->client->request('GET', '/example/'.$lang); | 
            ||
| 171 | |||
| 172 | // Read the Curriculum Vitae  | 
            ||
| 173 | $pathToFile = __DIR__.'/../../Resources/data/example.xml';  | 
            ||
| 174 | $this->curriculumVitae = new CurriculumVitae($pathToFile, $lang);  | 
            ||
| 175 | |||
| 176 | $cvXml = array(  | 
            ||
| 177 | 'identity' => $this->curriculumVitae->getIdentity(),  | 
            ||
| 178 | 'lookingFor' => $this->curriculumVitae->getLookingFor(),  | 
            ||
| 179 | 'experiences' => $this->curriculumVitae->getExperiences(),  | 
            ||
| 180 | 'skills' => $this->curriculumVitae->getSkills(),  | 
            ||
| 181 | 'educations' => $this->curriculumVitae->getEducations(),  | 
            ||
| 182 | 'languageSkills' => $this->curriculumVitae->getLanguageSkills(),  | 
            ||
| 183 | 'miscellaneous' => $this->curriculumVitae->getMiscellaneous()  | 
            ||
| 184 | );  | 
            ||
| 185 | // Remove all no visible elements  | 
            ||
| 186 |         switch ($lang) { | 
            ||
| 187 | case 'en':  | 
            ||
| 188 | unset($cvXml['identity']['myself']['birthday']);  | 
            ||
| 189 | break;  | 
            ||
| 190 | default:  | 
            ||
| 191 | // code...  | 
            ||
| 192 | break;  | 
            ||
| 193 | }  | 
            ||
| 194 | unset($cvXml['identity']['myself']['picture']);  | 
            ||
| 195 | unset($cvXml['identity']['address']['street']);  | 
            ||
| 196 | unset($cvXml['identity']['address']['postalcode']);  | 
            ||
| 197 | unset($cvXml['identity']['address']['googlemap']);  | 
            ||
| 198 | unset($cvXml['identity']['contact']['mobile']);  | 
            ||
| 199 | unset($cvXml['experiences']['FirstExperience']['society']['society']['ref']);  | 
            ||
| 200 | unset($cvXml['experiences']['FirstExperience']['society']['siteurl']);  | 
            ||
| 201 | unset($cvXml['experiences']['SecondExperience']['collapse']);  | 
            ||
| 202 | unset($cvXml['experiences']['SecondExperience']['society']['society']['ref']);  | 
            ||
| 203 | unset($cvXml['experiences']['SecondExperience']['society']['siteurl']);  | 
            ||
| 204 | unset($cvXml['experiences']['ThirdExperience']['society']['society']['ref']);  | 
            ||
| 205 | unset($cvXml['experiences']['FourthExperience']['collapse']);  | 
            ||
| 206 | unset($cvXml['skills']['Functional']['lines']['success']['percentage']);  | 
            ||
| 207 | unset($cvXml['skills']['Functional']['lines']['success']['class']);  | 
            ||
| 208 | unset($cvXml['skills']['Functional']['lines']['success']['striped']);  | 
            ||
| 209 | unset($cvXml['skills']['Functional']['lines']['otherSucess']['percentage']);  | 
            ||
| 210 | unset($cvXml['skills']['Functional']['lines']['otherSucess']['class']);  | 
            ||
| 211 | unset($cvXml['skills']['Functional']['lines']['info']['percentage']);  | 
            ||
| 212 | unset($cvXml['skills']['Functional']['lines']['info']['class']);  | 
            ||
| 213 | unset($cvXml['skills']['Functional']['lines']['info']['striped']);  | 
            ||
| 214 | unset($cvXml['skills']['Functional']['lines']['warning']['percentage']);  | 
            ||
| 215 | unset($cvXml['skills']['Functional']['lines']['warning']['class']);  | 
            ||
| 216 | unset($cvXml['skills']['Functional']['lines']['danger']['percentage']);  | 
            ||
| 217 | unset($cvXml['skills']['Functional']['lines']['danger']['class']);  | 
            ||
| 218 | unset($cvXml['skills']['Functional']['lines']['noClass']['percentage']);  | 
            ||
| 219 | unset($cvXml['skills']['OtherSkill']['lines']['success']['percentage']);  | 
            ||
| 220 | unset($cvXml['skills']['OtherSkill']['lines']['success']['class']);  | 
            ||
| 221 | unset($cvXml['skills']['OtherSkill']['lines']['success']['striped']);  | 
            ||
| 222 | unset($cvXml['skills']['OtherSkill']['lines']['info']['percentage']);  | 
            ||
| 223 | unset($cvXml['skills']['OtherSkill']['lines']['info']['class']);  | 
            ||
| 224 | unset($cvXml['skills']['OtherSkill']['lines']['info']['striped']);  | 
            ||
| 225 | unset($cvXml['skills']['OtherSkill']['lines']['warning']['percentage']);  | 
            ||
| 226 | unset($cvXml['skills']['OtherSkill']['lines']['warning']['class']);  | 
            ||
| 227 | unset($cvXml['skills']['OtherSkill']['lines']['warning']['striped']);  | 
            ||
| 228 | unset($cvXml['skills']['OtherSkill']['lines']['danger']['percentage']);  | 
            ||
| 229 | unset($cvXml['skills']['OtherSkill']['lines']['danger']['class']);  | 
            ||
| 230 | unset($cvXml['skills']['OtherSkill']['lines']['danger']['striped']);  | 
            ||
| 231 | unset($cvXml['educations']['HighSchool']['collapse']);  | 
            ||
| 232 | unset($cvXml['educations']['FirstSchool']['collapse']);  | 
            ||
| 233 | unset($cvXml['languageSkills']['French']['icon']);  | 
            ||
| 234 | unset($cvXml['languageSkills']['English']['icon']);  | 
            ||
| 235 | |||
| 236 | $testValue = $this->arrayValuesRecursive($cvXml);  | 
            ||
| 237 |         foreach ($testValue as $value) { | 
            ||
| 238 | $this->assertGreaterThan(0,  | 
            ||
| 239 |                 $crawler->filter('html:contains("'.$value.'")')->count(), | 
            ||
| 240 | 'The value '.$value.' is not diplay for language '.$lang  | 
            ||
| 241 | );  | 
            ||
| 242 | }  | 
            ||
| 243 | }  | 
            ||
| 244 | |||
| 263 |