YaWeather\Models\City

Complexity

Total Complexity

6

Size/Duplication

Total Lines	97
Duplicated Lines	22.68 %

Coupling/Cohesion

Components	1
Dependencies	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getCurrentDay()	7	7	2
A	getListDays()	7	7	2
A	getYesterday()	8	8	2

<?php
/**
 * Created by PhpStorm.
 * User: ignatenkov
 * Date: 21.08.15
 * Time: 1:22
 */

namespace YaWeather\Models;


class City {

    public $city;
    public $country;
    public $region;

    public $fact;
    public $yesterday;

    public $days;


    /**
     * Get current model day
     * @return string
     */
    public function getCurrentDay() {

This method seems to be duplicated in your project.

        $current = date("Y-m-d");
        if(isset($this->days[$current]))
            return $this->days[$current];
        else
            return "not found current day";
    }

    /**
     * Get List models Day
     * @return mixed
     */
    public function getListDays() {

This method seems to be duplicated in your project.

        $current = date("Y-m-d");
        if(isset($this->days[$current]))
            unset($this->days[$current]);

        return $this->days;
    }

    /**
     * Get yesterday model day
     * @return string
     */
    public function getYesterday() {

This method seems to be duplicated in your project.

        $yesterday = date("Y-m-d", time() - 86400);

        if(isset($this->days[$yesterday]))
            return $this->days[$yesterday];
        else
            return "not found yesterday day";
    }




    /*public $listDay = array();

50% of this comment could be valid code. Did you maybe forget this after debugging?

    private $xml;

    public function __construct($fileName) {

        $this->xml = simplexml_load_file($fileName);
        /*$day = new Day();
        $day->p = 0;
        $day->p1 = 1;

        $this->listDay[] = $day;*/
        //echo $fileName;
        //$xml = simplexml_load_file($fileName, "Day");

59% of this comment could be valid code. Did you maybe forget this after debugging?

        //$xml1 = $file = file_get_contents($fileName, true);;

53% of this comment could be valid code. Did you maybe forget this after debugging?

        //var_dump($xml);
        //$xml1 = simplexml_load_string($xml, 'Day');

59% of this comment could be valid code. Did you maybe forget this after debugging?

        //var_dump($xml->day[0]);

80% of this comment could be valid code. Did you maybe forget this after debugging?

        //var_dump($xml);
        //var_dump($this->listDay);

72% of this comment could be valid code. Did you maybe forget this after debugging?

    /*}

50% of this comment could be valid code. Did you maybe forget this after debugging?

    public function getListDay() {




        foreach ($this->xml->day as $day) {
            $this->listDay[] = new Day($day);
            //var_dump($day->sunrise);

        }

        return $this->listDay;

    }*/


    /*public function getListDay() {

56% of this comment could be valid code. Did you maybe forget this after debugging?

        return $this->listDay;
    }*/

}