АЗ
Ну так-то да
Size: a a a
2020 March 30
АЗ
Если бы это была лара я бы затолкал ее в хэлпер и забыл :D
D
хз.. кажется до максимума упрощено
D
сук :D
SS
можно ===null заменить на is_null(), как то оно приятнее бы смотрелось🤔
E
if ($dateTime1 === null && $dateTime2 === null) {
return true;
} else {
return false;
}return is_null() && is_null()
АЗ
Не люблю is_null за лишнюю функцию в стэке :В
АЗ
Мне строгое сравнение больше нраица
АЗ
В общем это уже вкусовщина
АЗ
Ладно спасибо, оставлю как есть)
АЗ
Посмотрим че на PR скажут)
SS
если лара - можно сократить через
optional()АЗ
Не, тут такое
АЗ
Не лара в общем :)
P
is_null() моветон
J
private function isDatesEqual(?DateTime $dateTime1, ?DateTime $dateTime2): bool
{
if ($dateTime1 === null && $dateTime2 === null) {
return true;
} elseif (($dateTime1 === null && $dateTime2 !== null) || ($dateTime2 === null && $dateTime1 !== null)) {
return false;
} else {
return $dateTime1->getTimestamp() === $dateTime2->getTimestamp();
}
}
Есть вот такая функция. Есть идеи как ее сократить? :)
private function isDatesEqual(?DateTime $dateTime1, ?DateTime $dateTime2): bool
{
if ($this->bothNull($dateTime1, $dateTime2))
return true;
if ($this->oneNull($dateTime1, $dateTime2))
return false;
return $dateTime1->getTimestamp() === $dateTime2->getTimestamp();
}
private function bothNull($first, $second)
{
return is_null($first) && is_null($second);
}
private function oneNull($first, $second)
{
return (is_null($first) && !is_null($second))
|| (!is_null($first) && is_null($second));
}
АЗ
is_null() моветон
+
АЗ
private function isDatesEqual(?DateTime $dateTime1, ?DateTime $dateTime2): bool
{
if ($this->bothNull($dateTime1, $dateTime2))
return true;
if ($this->oneNull($dateTime1, $dateTime2))
return false;
return $dateTime1->getTimestamp() === $dateTime2->getTimestamp();
}
private function bothNull($first, $second)
{
return is_null($first) && is_null($second);
}
private function oneNull($first, $second)
{
return (is_null($first) && !is_null($second))
|| (!is_null($first) && is_null($second));
}
Так кода еще больше стало
АЗ
А условия те же
J
Так читаемо