static bool isCrazyMurderinqRobot = false; humans (void){ iffisCrazyMurderinqRobot oppressive lervce / Oppressive Silence :: баг :: фича :: убить всех человеков :: oppressive-silence

+0,7

А разве IDE такое отслеживает? мне казалось это компилятора задача

LcRL

12.08.2016, 08:56

+0,3

Тут нет никаких ошибок с точки зрения компилятора. А с точки зрения IDE может возникнуть замечание, что странное место для присвоения, да и то не факт

DjineyX

12.08.2016, 08:59

+5,0

Нормальный компилятор выдаст ворнинг. Так что не игнорируйте предупреждения, ребята =)
Рекомендую ставить -Werror в релизе ;)

12.08.2016, 09:04

+1,7

Но перед этим
-Wall -Wextra и ещё несколько ключей, которые не входят в all и extra

да и -Werror= и в дебаге не помешает

Gyromitra

12.08.2016, 10:02

0,0

-pedantic

12.08.2016, 10:43

+0,1

-Werror в дебаге порой мешает - то там "лишняя" переменная, то еще что, что тебе скоро понадобится.

-Wall, -Wextra, -pedantic и так далее для каждого языка различны, но чем больше, тем лучше! =)

12.08.2016, 12:07

0,0

По-моему, параметр -pedantic аналогичен картинке ниже.

,Oppressive Silence,баг,фича,убить всех человеков,oppressive-silence

12.08.2016, 12:44

+0,3

Я не говорю делать Werror для всех ворнингов в дебаге, а бить по рукам для особо извращённых случаев.
Например, всегда использую -Werror=return-type -Werror=delete-incomplete, тк считаю подобные косяки слишком опасны, чтобы быть просто ворнингами.

Gyromitra

12.08.2016, 18:40

0,0

Мелочи IDE может отслеживать. Вообще есть и анализаторы всякие, не входящие в IDE для таких вещей.

$1 2 3 4 5 6 7 8 9 10 <?php $b = 0; $isCrazyRobot = false; if ($isCrazyRobot = true) { _________—__________/\______ 6} Assignment in condition more... (Ctrl+Fl),Oppressive Silence,баг,фича,убить всех человеков,oppressive-silence$

Quard_Damage

12.08.2016, 09:19

+1,4

На здравом смысле сэкономили. Булевские переменные сравнивать с true или false бессмысленно.

SpBerkut

12.08.2016, 08:57

+0,0

Побуду кэпом: в if сэкономили на одном знаке равенства и происходит не сравнивание, а присваивание.

MBRUS

12.08.2016, 09:04

+3,0

if (someBooleanVar == true) => if (someBooleanVar) - вот что имел в виду комментатор выше. И совершенно прав в этом.

paragraff

12.08.2016, 09:24

+6,4

Я в курсе. Ошибочка довольно распространённая. Но если бы не пытались сравнить bool с константой, то такого бы не случилось и разрабы могли бы продолжить быдлокодить.

SpBerkut

12.08.2016, 09:26

+2,7

Скорее на кодстайле, все-таки не зря правилом хорошего тона является написание констант слева в условных операторах. Тогда не то что компилятор, любая мало-мальски нормальная ИДЕ начнет очень громко ругаться.

12.08.2016, 09:11

+0,1

Косипоры)

12.08.2016, 08:55

+0,1

12.08.2016, 09:01

-8,7

А ты у нас не программист?
Или, может - не мамкин?

LcRL

12.08.2016, 13:55

+0,0

Лучше в условиях не ставить "== (true/false)"

Lottelion

12.08.2016, 09:03

+0,5

Хорошо, а как тогда сравнить переменные между собой без == ?

12.08.2016, 10:44

-0,4

Так то сравнивание переменных. А в данном случаи компилятор достаточно умен чтоб самому узнать единственная переменная тру или фолс.

Fireball14

12.08.2016, 10:59

+0,0

Как это делают индусы, разумеется:
if (a.toString().length < 5) ...

12.08.2016, 11:36

+0,8

Извини, не знаю я индусский, чтобы объяснить, что сравниваются переменные, а не переменная и константа. Хотел сумничать, та написал бы уж "if (a - b) else doIt();", что равнозначно " if (a == b) doIt();".

12.08.2016, 11:55

-0,4

Ты нервный. Это от процедурного программирования. Сравнивай так:

if (a.compare(b)) ...

А в общем случае "a - b" недопустимо, поскольку для некоторых типов переменных операция +/- не определена.

12.08.2016, 15:23

+0,3

Указатели через приведение типов замечательно вычитаются. Вещественные и целочисленные типы умеют в отрицание. Нельзя сравнить только массивы данных (к которым относятся строки и объекты), но это очевидно. Я что-то пропустил?

12.08.2016, 16:09

-0,1

Строки и объекты нынче встречаются чаще, чем что-либо еще. Но у меня вопрос - ты всерьез собрался сравнивать два вещественных числа через == или сравниваемую с нулем разницу?

12.08.2016, 18:09

0,0

Я написал true/false в скобках для конкретизации: если функция или переменная имеют (возвращают) значение true или false, лучше не писать "==" вообще.

Lottelion

12.08.2016, 12:38

+0,0

Для данного конкретного случая и подобных - согласен.

12.08.2016, 12:46

-0,4

Истину говорю вам, мир погубит быдлокод.

Brenwen

12.08.2016, 09:03

+3,3

Но ведь можно просто
if isCrazyMurderingRobot
kill(humans);

Анус Бонус

12.08.2016, 09:08

-0,5

К else+; и без {}/begin..end отдельная претензия. Зависит от языка.

Dr_Quake

12.08.2016, 09:12

+0,6

Не-не-не надо обязательно писать именно так, а еще вот так while(1==1){...} . Нас так в универе учили(с)
З.Ы. это сарказм если че

12.08.2016, 09:14

+0,1

А можно в стиле perl:
isCrazyMurderingRobot && kill(humans);

12.08.2016, 11:38

0,0

javascript тоже такое сжуёт

achiatma

12.08.2016, 21:36

+0,1

Функция kill есть? Есть. Значит все-таки запрограммировали на это.
А вообще так и надо тем, кто смешивает CamelCase и snake_case

JJack

12.08.2016, 09:11

+4,6

Потому что правильно писать:

if(true == isCrazyMurderingRobot)

Вот тогда IDE точно скажет что-то не так

ridge23

12.08.2016, 10:40

-0,1

Тогда и компилятор заматерится. К сожалению, данный метод работает только при наличии констант или результат функции в условии. С переменными всё равно можно ошибиться.

12.08.2016, 10:46

-0,4

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

12.08.2016, 11:40

+0,1

У опытных разработчиков имеются наборы тестов, которые позволяют частично исключить логические ошибки. Но алгоритмические всё равно останутся незамеченными и всплывут уже при использовании ПО.

12.08.2016, 16:29

0,0

Он все правильно написал, есть вообще правило: КОНСТАНТА при сравнении должна быть слева. А всякие идиоты ебашат справа.

Этому есть одно простое объяснение, если у вас константа слева и вы поставите '=' вместо '=='. Что компилируемый, что интерпретируемый язык даже не запустят программу.

if(true = isCrazyMurderingRobot)

serg.deep

12.08.2016, 11:43

+0,4

Не все читали умные книги про совершенный код.

12.08.2016, 11:56

-0,3

Спасибо магистр Йода, но в языках с нормальными компиляторами (а не как JS/PHP), подобные извращения не нужны. Компилятор выдает предупреждение, этого достаточно.

12.08.2016, 12:10

-0,2

зажрались. компилятор выдаст.
совсем так перестанете головой думать

12.08.2016, 12:18

+0,1

Чем меньше держишь в голове фигни, которую можно в голове не держать, тем больше держишь в голове того, что действительно важно. Ресурсы мозга весьма ограниченны.

12.08.2016, 12:22

+0,1

Посадить бы таких умных писать программы вначале в блокнот (написал с маленькой буквы специально, я не про программу Блокнот).

12.08.2016, 12:47

-0,3

отсутствие знаний про "эту фигня" в итоге и приводит к тому, что оч много "специалистов" вообще не представляют как работают микропроцессорные системы. зато херачат мегабайты кода да еще без комментариев.
а потом все дружно удивляются, чего это 64 процессорный sun загружен под жвах.

13.08.2016, 13:40

0,0

На микропроцессорный уровень реально пофиг сейчас, почти везде.
Но вот как нафигачат O(n^2) там, где не надо, так хоть вешайся =)

14.08.2016, 16:58

0,0

Гм, а вы товарищ рисковый. А если на сборку уходит несколько часов? Будем ждать пока сборка отвалится? А еще при этом всем будете сравнивать переменную с результатом функции и случайно поставите только один знак равенства? Это и компилятор и иде пропустят как валидный код и подвох можно будет заметить только во время тестирования(и будет еще хорошо если автотестами все покрыто). А вот если правильную очередность соблюдать, то можно здорово упростить жизнь себе и своим коллегам.

12.08.2016, 14:29

+0,1

Компилятор это не пропустит, если не забыть про флаги компилятора

vga@lws04:~/code/csandbox$ cat main.cpp finclude <iostream>
int randomNumber() { return 4;
>
int main() {
int i = 0;
if (i = randomNumber()) {
std::cout « "IF TRUE" « std::endl;
> else {
std::cout « "IF FALSE" « std::endl;
>
return 0;
>
vga@lws04:~/code/csandbox$ g++ -Wall -Werror main

12.08.2016, 14:44

0,0

Ну опять таки, компиляция занимает сколько? Пару десятков миллисекунд. А теперь возвращаемся к ситуации когда на компиляцию уходит гораздо больше времени.

12.08.2016, 15:20

0,0

Если компиляция занимает много времени, то это отдельная проблема с которой надо бороться.

12.08.2016, 15:21

0,0

У меня другой вопрос: почему вообще добавлять такую функцию? И почему не добавить идентификацию своих и чужих, если это военный робот? А 3 закона робототехники? Да ну нафиг!

Also is Me

12.08.2016, 15:34

0,0

А писали бы на шарпах, такой фигни не былоб.

unkuse

12.08.2016, 16:31

0,0

Ассемблер рулит. На нём так вообще невозможно ошибиться.

12.08.2016, 16:42

-0,8

асемблер? серьезно? да там яйца можно себе отстрелить

unkuse

12.08.2016, 16:50

+0,6

ну так..точечно да. хорошо использовать (для узких мест).
для клиентского ПО каких нибудь учетных систем он нафиг не нужен.
остаются спецпроцессоры, да критичный по ресурсам и времени функционал.

13.08.2016, 13:46

0,0

Я просто иначе написал фразу unkuse.
Можно ещё так: "А писали бы на асме, такой фигни не былоб" (стилистика сохранена).
А вообще, MenuetOS и ПО к ней.

13.08.2016, 13:59

0,0

ну там вообще все по другому. пришлось тут с год назад вспоминать. аж слышно было как мозги заскрипели.

13.08.2016, 14:15