Skip to main content
adicionou 18 caracteres ao conteúdo
Fonte Link
Augusto Vasques
  • 18,8mil
  • 10
  • 31
  • 62
abstract class Weeks
{

  // Tornei os membros $year, $this_week e $last_week públicos somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Aqui mede o valor de entrada do argumento $date
    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');
    
    // Aqui mede o valor de saída do argumento $date após o código
    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}

...mais precisamente na no método $date->sub. Esse método não só retorna a data modificada como efetivamente modifica o objeto que a evocar no caso o objeto $today que é referenciado por $date dentro do constructor. Para fazer o reparo desse código é simples basta fazer um clone do argumento $date.

abstract class Weeks
{
  // Tornei os membros $year, $this_week e $last_week públicos somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Está qui o reparo do código
    $date = clone $date;

    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');

    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}
abstract class Weeks
{

  // Tornei os membros $year, $this_week e $last_week somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Aqui mede o valor de entrada do argumento $date
    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');
    
    // Aqui mede o valor de saída do argumento $date após o código
    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}

...mais precisamente na no método $date->sub. Esse não só retorna a data modificada como efetivamente modifica o objeto que a evocar no caso o objeto $today que é referenciado por $date dentro do constructor. Para fazer o reparo desse código é simples basta fazer um clone do argumento $date.

abstract class Weeks
{
  // Tornei os membros $year, $this_week e $last_week somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Está qui o reparo do código
    $date = clone $date;

    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');

    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}
abstract class Weeks
{

  // Tornei os membros $year, $this_week e $last_week públicos somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Aqui mede o valor de entrada do argumento $date
    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');
    
    // Aqui mede o valor de saída do argumento $date após o código
    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}

...mais precisamente na no método $date->sub. Esse método não só retorna a data modificada como efetivamente modifica o objeto que a evocar no caso o objeto $today que é referenciado por $date dentro do constructor. Para fazer o reparo desse código é simples basta fazer um clone do argumento $date.

abstract class Weeks
{
  // Tornei os membros $year, $this_week e $last_week públicos somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Está qui o reparo do código
    $date = clone $date;

    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');

    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}
Fonte Link
Augusto Vasques
  • 18,8mil
  • 10
  • 31
  • 62

O problema está ocorrendo é com o argumento $date do constructor public function __construct($date). Mesmo que o padrão php para passagem de argumentos seja a passagem por valor para os tipos primitivos, para tipos complexos como o objeto DateTime essa passagem de argumentos é feita por referencia implicando que qualquer alteração no argumento $date será efetivamente realizada em sua referência que é o seu objeto $today = new DateTime();.

Para fazer a constatação que é um problema com uma modificação não pretendida no objeto referenciado eu alterei o constructor da classe Week de forma a exibir o valores de entrada e de saída de $date:

abstract class Weeks
{

  // Tornei os membros $year, $this_week e $last_week somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Aqui mede o valor de entrada do argumento $date
    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');
    
    // Aqui mede o valor de saída do argumento $date após o código
    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}

Então eu fiz quatro testes:

echo "$training_load_week";
$training_load_week = new Training_Load_Week($today);
echo "$training_load_week1";
$training_load_week1 = new Training_Load_Week($today);
echo "$training_load_last_week";
$training_load_last_week = new Training_Load_Last_Week($today);
echo "$training_load_last_week1";
$training_load_last_week1 = new Training_Load_Last_Week($today);

E resultado foi:

$training_load_week
valor de entrada = 34
valor de saída = 33

$training_load_week1
valor de entrada = 33
valor de saída = 32

$training_load_last_week
valor de entrada = 32
valor de saída = 31

$training_load_last_week1
valor de entrada = 31
valor de saída = 30

Ok isso confirma a modificação indesejada na referência. Agora o próximo passo é identificar o que está causando a modificação.

Analisando a documentação das funções usada no constructor eu encontrei a causa na linha...

$this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');

...mais precisamente na no método $date->sub. Esse não só retorna a data modificada como efetivamente modifica o objeto que a evocar no caso o objeto $today que é referenciado por $date dentro do constructor. Para fazer o reparo desse código é simples basta fazer um clone do argumento $date.

abstract class Weeks
{
  // Tornei os membros $year, $this_week e $last_week somente para facilitar meus testes
  public $year;
  public $this_week;
  public $last_week;

  /**
  * @param $date deverá ser no formato new DateTime().
  **/
  public function __construct($date)
  {
    // Está qui o reparo do código
    $date = clone $date;

    print_r("valor de entrada = ". $date->format('W'). "\n");
    
    $this->year = $date->format('Y');
    $this->this_week = $date->format('W');
    $this->last_week = ($date->sub(new DateInterval('P1W')))->format('W');

    print_r("valor de saída = ". $date->format('W'). "\n");
  }
}

Refiz os mesmos testes e o resultado foi:

$training_load_week
valor de entrada = 34
valor de saída = 33

$training_load_week1
valor de entrada = 34
valor de saída = 33

$training_load_last_week
valor de entrada = 34
valor de saída = 33

$training_load_last_week1
valor de entrada = 34
valor de saída = 33