VPF::Assignment in condition - Форум программистов

Gold Dragon

Дата 22.8.2008, 15:04 (ссылка)

(нет голосов)

Загрузка ...

Призрачный

Профиль
Группа: Экс. модератор
Сообщений: 6753
Регистрация: 1.3.2004
Где: Россия, Тамбов

Репутация: 10
Всего: 71

Вот у меня в редакторе выскакивает предупреждение

Цитата

Assignment in condition

Category: Bug

Assignments in a conditional statement are sometimes the result of a typo.In many cases, instead of the assignment operator =, the equality operator == should be used. When comparing a variable and a non-variable expression, it is common practice to put the variable on the right-hand side of the equality operator, so that if the equality operator is erroneously replaced with the assignment operator, PHP will report a parse error immediately. This helps you avoid a big subset ofthis class of bugs.

в этой функции

Код


    function CDatabase($db, $host="localhost", $user="", $pass="") {
        $this->db = $db; $this->host = $host; $this->user = $user; $this->pass = $pass;
        if($this->link = mysql_connect($host,$user,$pass)){
         return mysql_select_db($db, $this->link);
        }else{
         return 0;
        }
    }

я так понимаю что предупреждение возникает потому что я в условии присваиваю значение и одновременно его сравниваю. Но код работает с этим предупреждением. Чем это грозит и стоит ли вообще обращать на это внимание?

--------------------

Нельзя жить в прошлом, оно уже прошло.
Нельзя жить в будущем, оно ещё не наступило.
Нужно жить в настоящем, помня прошлое и думая о будущем!

Sannis

Дата 22.8.2008, 15:11 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 375
Регистрация: 14.8.2007
Где: Москва

Репутация: 6
Всего: 13

Это на случай, если вы случайно вместо == в условии/цикле напишите =.

--------------------

Я Критик Джим, раньше писал моды для IPB 1.3.x-2.3.x, а теперь продвигаю Node.js в массы.

Gold Dragon

Дата 22.8.2008, 15:20 (ссылка)

(нет голосов)

Загрузка ...

Призрачный

Профиль
Группа: Экс. модератор
Сообщений: 6753
Регистрация: 1.3.2004
Где: Россия, Тамбов

Репутация: 10
Всего: 71

ну меня больше интересует на сколько вообще это правильно: присваивать и сравнивать

--------------------

Feldmarschall

Дата 22.8.2008, 15:51 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 2641
Регистрация: 11.12.2007

Репутация: 22
Всего: 32

ну если тебе не нравится, что пишет этот редактор - перепиши все такие места в два действия.
по поводу "насколько это правильно" - это синтаксически корректная конструкция языка. что, в общем, очевидно. какого другого ответа можно ожидать (кроме того, что объяснил Sannis) - непонятно.

Это сообщение отредактировал(а) Feldmarschall - 22.8.2008, 15:59

Mal Hack

Дата 22.8.2008, 23:07 (ссылка)

(нет голосов)

Загрузка ...

Мудрый...

Профиль
Группа: Участник Клуба
Сообщений: 9926
Регистрация: 15.2.2004

Репутация: 122
Всего: 261

Цитата(Gold Dragon @ 22.8.2008, 16:20

)

ну меня больше интересует на сколько вообще это правильно: присваивать и сравнивать

Это нормально, но если надо как правило лучше делать так:
if((a=b)==true) или уж if((a=b)) - две скобки, чтобы присваивание рассматривалось целиком, как логическое выражение... Это из Сей...

Gold Dragon

Дата 24.8.2008, 08:08 (ссылка)

(нет голосов)

Загрузка ...

Призрачный

Профиль
Группа: Экс. модератор
Сообщений: 6753
Регистрация: 1.3.2004
Где: Россия, Тамбов

Репутация: 10
Всего: 71

Цитата(Mal Hack @ 23.8.2008, 00:07

)

или уж if((a=b)) - две скобки, чтобы присваивание рассматривалось целиком, как логическое выражение...

--------------------

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| PHP: Общие вопросы \| Следующая тема »