Datium\Tools\DayOf

Complexity

Total Complexity

12

Size/Duplication

Total Lines	113
Duplicated Lines	20.35 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	7
Bugs	1	Features	6

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	15	1
A	year()	0	7	1
B	week()	0	26	4
A	persian_day_of_week()	11	11	3
A	islamic_day_of_week()	12	12	3

<?php namespace Datium\Tools;

/**
 * Define Day of current date.
 */
class DayOf {

  protected $date_time;

  protected $month;

  protected $day;

  protected $config;

  protected $result;

  protected $calendar_type;

  protected $geregorian_DayofWeek;

  public function __construct( $date_time, $calendar_type = 'gregorian' ) {

    $this->config = include( 'Config.php' );

    $this->date_time = $date_time;

    $this->calendar_type = $calendar_type;

    $this->day = $this->date_time->format('l');

    $this->geregorian_DayofWeek = $this->date_time->format('w');

    return $this;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

  }

  /**
   * Which day of year is current day.
   * @since Aug, 03 2015
   * @return integer
   */
  public function year() {

    $config = include(  'CalendarSettings/' . ucfirst( $this->calendar_type ) . '.php' );

    return $config[ 'day_of_year' ]( $this->date_time );

  }


  /**
   * Which day of week is current day.
   * @since Aug, 09 2015
   * @return integer
   */
  public function week() {

    switch ( $this->calendar_type ) {

      case 'ir':

      $this->result = $this->persian_day_of_week();

        break;

      case 'gh':

      $this->result = $this->islamic_day_of_week();

       break;

      case 'gr':

      $this->result = date( 'w', strtotime( $this->date_time->format('Y-m-d H:i:s') ) ) + 1;

       break;
    }

    return $this->result;

  }

  /**
   * @since Sept, 14 2015
   * @return integer
   */
  protected function persian_day_of_week() {

This method seems to be duplicated in your project.

    $this->day = str_replace( $this->config['week_days_name']['english'], $this->config['week_days_name']['persian'], $this->day);

    foreach ( $this->config['week_days_name']['persian'] as $key => $value ) {

      if( $value == $this->day ) return $key += 1;

    }

  }

    /**
   * @since Sept, 14 2015
   * @return integer
   */
  protected function islamic_day_of_week() {

This method seems to be duplicated in your project.

    $this->day = str_replace( $this->config['week_days_name']['english'], $this->config['week_days_name']['islamic'][$this->geregorian_DayofWeek], $this->day);

    foreach ( $this->config['week_days_name']['islamic'] as $key => $value ) {

      if( $value == $this->day ) return $key += 1;

    }


  }


}
?>

It is not recommended to use PHP's closing tag ?> in files other than templates.