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