Почему такое поведение PHP при сложении чисел?

Question

[psfpro]

Есть код

$article = Article::findOne(['url' => $itemUrl]);
        $article->setViews($article->getViews() + 1);
        $article->save();

Получаю на выходе прибавку на 2 вместо 1.
Далее делаю так

$article = Article::findOne(['url' => $itemUrl]);
        echo $article->getViews();
        $article->setViews($article->getViews() + 1);
        $article->save();

добавляю
echo $article->getViews();
после этого прибавка работает правильно на 1

Методы

/**
     * @return int
     */
    public function getViews()
    {
        return $this->views;
    }

    /**
     * @param int $views
     * @return $this
     */
    public function setViews($views)
    {
        $this->views = (int)$views;
        return $this;
    }

Вот ещё новая информация

Браузер почему-то два раза запросы шлет. Первый раз тип "document", второй раз "text/html".
Если добавляю строку echo $article->getViews();, то второй раз уже не идет запрос.
Браузер 2 раза открывает страницу получается.

Comments

[Радость моя]

стиль похож на Phalcon. Если это так, то почему вы не делаете просто:

$article = Article::findOne(['url' => $itemUrl]);
$article->views++;
$article->save();

Answer 1

[Максим Кудрявцев]

Проверьте тип возвращаемого значения в первую очередь.

echo gettype($article->getViews());

Я думаю у Вас там будет строка и, возможно, проблема растет оттуда.
Далее делаем приведение типов:

$article->setViews((int)$article->getViews() + 1);

Попробуйте

Answer 2

[Евгений Кайдашов]

Что внутри метода $article->getViews()

Answer 3

[Alex Safonov]

Вы уверены что код у вас не выполняется дважды? Редиректы на странице, циклы?

Давайте для полноты картины вы приведете код

$article->getViews();
$article->setViews();

Comments

[psfpro]

В лог записал время, ID и число просмотров. Видно что два раза выполняется
2015-09-18T12:15:55+03:00: 2 - 30
2015-09-18T12:15:57+03:00: 2 - 31
2015-09-18T12:15:59+03:00: 1 - 51
2015-09-18T12:15:59+03:00: 1 - 52
2015-09-18T12:16:06+03:00: 2 - 32
2015-09-18T12:16:09+03:00: 2 - 33

Может nginx не правильно настроен что два раза страница открывается?

[Alex Safonov]

У меня только одна догадка.
Если у вас есть вывод на страницу

echo $article->getViews();

до чего-то вроде

header('Location:' ...)

то PHP вам должно выкинуть warning, что хедеры не принимаются, так как был вывод на страницу и уже поздно чесаться.

Если у вас этого вывода нет (echo) то хедер с редиректом отработает.

Так что давайте просто включите ERROR_REPORTING(E_ALL), и смотрите что за участок кода у вас просит сделать перенаправление.

[psfpro]

Начал распутывать, выяснилось что браузер два раза шлет запросы

[Alex Safonov]

Вот прямо 2 раза, после перехода по ссылке?
Вы это как выяснили? Чем-то вроде веб-прокси?
www.charlesproxy.com

[psfpro]

В хроме открыл и посмотрел заголовки. У меня в коде используется буферизации вывода, думаю возможно из-за этого. Что скажите?

[Alex Safonov]

А попробуйте с выключенным JS.
Меня смущает что такого поведения нет, если у вас есть ECHO.

[psfpro]

Нет, тоже самое. Когда я ECHO делаю, он добавляет до HTML тега текст и в таком случае второй раз не идет запрос.

[Alex Safonov]

A попробуйте другой браузер. Не Хром.

[psfpro]

В хроме два раза, в микрософт эдж один раз, мозила два раза, яндекс браузер тоже два раза.
Интересно, первый раз с таким сталкиваюсь.

[Alex Safonov]

Ну дальше только код смотреть.
У вас самописный движек\фреймворк\cms?
Есть в паблике ссылка на дев-версию страницы?

[psfpro]

Спасибо за участие в вопросе ;)

[psfpro]

Сколько было вариантов, а дело просто в том что я не заполнил тег