| 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 |