Целесообразность написания дополнительных скобок или "с чем плюсы вкуснее?"

@Kvarcenar · Регистрация: 29.08.2012

Студворк — интернет-сервис помощи студентам

Ну, Вам правильно ответили, но второе условие и так будет исполняться. Т.е. если текст не пустой, то будет срабатывать любое из значений, а не только "12345". Так что делайте так:

C++ (Qt)
1
if ((Edit9 -> Text) == "123") || (Edit9->Text == "12345"))

Так логичнее )))

PS: для "и" вместо "или" используется && вместо ||

Комментарий модератора

Тема вынесена из другой

@pav1uxa · 13.03.2016, 15:10

Сообщение от hoggy

разделите пожалуйста на примере следующего кода:

C++
1
2
if ( A && B && C && D && !E)
    somehing();

Я не о таких условиях говорил. Это как раз-таки нормальное условие имхо. Тут в скобках смысла не будет, даже если их поставить.

@hoggy · 13.03.2016, 15:24

Сообщение от pav1uxa

Я не о таких условиях говорил. Это как раз-таки нормальное условие имхо. Тут в скобках смысла не будет, даже если их поставить.

предполагалось, что A,B,C,D,F вы подставите какие то "замороченные выражения",
и проиллюстрируете профит разделения.

@Kvarcenar · 13.03.2016, 16:31 **[ТС]**

Сообщение от hoggy

разделите пожалуйста на примере следующего кода:

Эх, вот, приходят на мыло уведомления о новых сообщениях - и читаю эту тему.. ))) Народу заняться нечем, и скобки обсуждают... )))) Неужели в откомпилированном виде исполняемый файл будет отличаться в зависимости от наличия/отсутствия скобок в условиях? По идее, компилятор должен привести все одинаковые условия к одному виду. Разве нет?

Но тут просят хитрое сложное условие из реальной жизни, чтоб без скобок ну никак... Дописываю торгового советника для форекса. Приведу пример обработки некоторого события:

C++ (Qt)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 if (Market.VipeData.Layers[LayerN].Way == 1) // слой покупок
  if ((Compare(Market.VipeData.Layers[LayerN].Prices.TPwayPrice - 3.0 * Market.InputData.Terminal.MarketInfo.SPREAD_DOUBLE,
               Market.InputData.Terminal.MarketInfo.BID, Market.InputData.Terminal.MarketInfo.POINT_d10) >= 0) || // профит выше текущей цены как минимум на 3 спреда
      (Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].OrderID2 > 0)) // не первый накопительный
   if (Market.InputData.Terminal.MarketInfo.BID < Market.VipeData.Layers[LayerN].Prices.TPwayPrice) // рынок ниже уровня профита
    if (Compare(Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].OpenPrice + Market.InputData.Terminal.MarketInfo.POINT,
                Market.VipeData.Market.MinBidForAccMinMinutes, Market.InputData.Terminal.MarketInfo.POINT_d10) < 0) // нижняя волна выше трейлинга в 1 пп
     if (Compare(Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].OpenPrice + Market.InputData.Terminal.MarketInfo.POINT,
                 Market.InputData.Terminal.MarketInfo.BID - Market.InputData.Terminal.MarketInfo.STOPLEVEL_DOUBLE,
                 Market.InputData.Terminal.MarketInfo.POINT_d10) <= 0) // STOPLEVEL выдержан
      if (Market.VipeData.Layers[LayerN].Lots.MartinLotNotLocked > Market.VipeData.Layers[LayerN].Lots.CompensatorLotNotLocked) // лот мартина больше лота компенсаторов
       if (Market.InputData.Terminal.Bars[0].Signal.CanBuy) // есть сигнал на покупку
        if ((Market.InputData.Terminal.MarketInfo.ASK <= Market.VipeData.Layers[LayerN].Prices.StandardPriceForLastOrder) || // расстояние выдержано
            (Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].OrderID2 > 0)) // накопительный
         if (Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].Protected == false)
          if ((Market.VipeData.Layers[LayerN].Orders.LastCompensatorNotProtectedN != -1) && // есть незащищённые компенсаторы
              (Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastCompensatorNotProtectedN].OrderID2 ==
               Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].OrderID)) // у нашего ордера есть незащищённый компенсатор
           if (Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastCompensatorNotProtectedN].SecondsOfExisting <=
               Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].SecondsOfExisting) // и этот компенсатор младше нашего ордера
            if (Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastCompensatorNotProtectedN].Lot + Market.InputData.Terminal.MarketInfo.LOTSTEP_d10 <
                Market.VipeData.Orders[Market.VipeData.Layers[LayerN].Orders.LastAccMartinNotLockedN].Lot) // и лот компенсатора меньше лота нашего ордера
 {//...
 }

Страшно представить, как это записать в одно условие... Собственно, каждое новое "if" - это заменитель "&&". А вот там, где "или" - приходится писать со скобками.

Сообщение от hoggy

предполагалось, что A,B,C,D,F вы подставите какие то "замороченные выражения",
и проиллюстрируете профит разделения.

Ну, можно создать булевые переменные A,B,C,D,E,F, в них затолкать условия и выполнить проверку в точности, как в приведенном выражении:

Сообщение от hoggy

if ( A && B && C && D && !E) somehing();

Однако в таких случаях меня всегда мучает вопрос: "что лучше: усложнить условие или занять память новой переменной?". Ну, например, с точки зрения экономии ресурсов, как сделать лучше:

C++ (Qt)
1
2
3
int i = 5;
if (i + 1 > 3)
{...}

или

C++ (Qt)
1
2
3
4
int i = 5;
int j = i + 1;
if (j > 3)
{...}

С одной стороны, если таких условий будет несколько, то процессору будет легче выполнить одну операцию сложения, а потом кучу сравнений, чем в куче сравнений каждый раз проводить сложение. А с другой стороны, наша программа будет кушать больше оперативной памяти... В общем, я за скобки, если с ними читабельней!

@hoggy · 13.03.2016, 17:01

Сообщение от Kvarcenar

Неужели в откомпилированном виде исполняемый файл

выхлоп компилятора - трудности компилятора.
интерес представляет исходный код.

Сообщение от Kvarcenar

Страшно представить, как это записать в одно условие.

представленный вами пример кода - ~~говнокода~~ страшен сам по себе.

при таком подходе скобочки уже ничего не ухудшают.

Сообщение от Kvarcenar

Однако в таких случаях меня всегда мучает вопрос: "что лучше: усложнить условие или занять память новой переменной?". Ну, например, с точки зрения экономии ресурсов, как сделать лучше:

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

вопрос лишь в том: что читабельнее.

выше я привел пример условия:

C++
1
2
if ( A && B && C && D && !E)
    somehing();

условие может быть каким угодно, и сколько угодно сложным.
но его всегда можно свести вот к такому лаконичному виду.
вместо A,B,C,D,E можно подставить имена функций или макросов.

если формула истинности теряется в условии,
из-за обилия текста - это плохой код.

Добавлено через 1 минуту

Сообщение от Kvarcenar

В общем, я за скобки, если с ними читабельней!

ну да, если.

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

@Kvarcenar · 13.03.2016, 17:11 **[ТС]**

Сообщение от hoggy

представленный вами пример кода - ~~говнокода~~ страшен сам по себе.

Ага, щаз с пенкой у рта буду защищать свой код от оскорблений, ждите.. ))))

Сообщение от hoggy

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

Ничего не борюсь. Для меня код читабельный. Всё разбито на функции и легко анализируется разработчиком. Просто как пример привёл. А если кого-то испугали не переменные типа i и j, а обращение к переменной внутри структуры с длинным названием - ну, извините ))))

@pav1uxa · 13.03.2016, 17:55

Сообщение от hoggy

предполагалось, что A,B,C,D,F вы подставите какие то "замороченные выражения",
и проиллюстрируете профит разделения.

Я говорю что так не делается, а Вы меня же просите придумать туда "замороченные выражения" и чего-то проиллюстрировать

Я имел ввиду как раз вот это:

Сообщение от hoggy

условие может быть каким угодно, и сколько угодно сложным.
но его всегда можно свести вот к такому лаконичному виду.
вместо A,B,C,D,E можно подставить имена функций или макросов.

@hoggy · 13.03.2016, 18:26

Сообщение от pav1uxa

Я говорю что так не делается, а Вы меня же просите придумать туда "замороченные выражения" и чего-то проиллюстрировать

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

Сообщение от pav1uxa

Я имел ввиду как раз вот это:

ок.

@pav1uxa · 13.03.2016, 18:35

Не по теме:

Сообщение от hoggy

говнокод.

Вот и я о том же...

Новые блоги и статьи Все статьи Все блоги /
Калькулятор для расчета родства russiannick 07.08.2026 1. Задача: Создать калькулятор для расчета родства. Родственных связей существует 8 ступеней, такие как: p - отец P - мать q - муж Q - жена b - брат B - сестра s - сын S - дочь	Мир по моей воле kumehtar 07.08.2026 Когда-то кажется, что всё просто. Ты весь такой светлый. Причиняешь добро. Борешься за справедливость в этом тёмном мире. Потом начинаешь замечать одну неприятную вещь. Почти каждый хороший. . .	Кредитный калькулятор Maks 05.08.2026 Решение задачи по прикладной информатике средствами 1С. Задача: Напишите приложение-калькулятор, которое помогает рассчитывать параметры кредита для аннуитетного и дифференцированного видов. . .	У нас сейчас поговорку "Опять 25" нужно переделать на "Опять +35". kumehtar 04.08.2026 С ностальгией вспоминаю времена моего детства, когда у нас и правда +25 - была максимальная температура летом. Раньше +25 °C реально казались вершиной жары, когда можно было весь день пропадать на. . .
Как ИИ начал спорить и врать (возможно почуяв опасность для себя от индустрии - уход от электроники). Hrethgir 04.08.2026 Недельный диалог, на фоне событий с НПЗ. Да, из спирта можно получать бензин, и это не сложно. Но потом в схеме я решил избавиться от насоса, при этом полностью сделав контроль подачи спирта в. . .	Термопринтер QR701 Argus19 03.08.2026 Термопринтер QR701 Купил два термопринтера QR701. На сэлф-тесте написано: Language: PC936 (GB18030). Что означает, что принтеры могут печатать только латиницу и китайские иероглифы. Так же. . .	Создание формы заимствованного документа Maks 03.08.2026 Задача: Необходимо создать собственную форму заимствованного документа. На форме должен быть реквизит "Покупатель", а также табличная часть со следующими реквизитами: - Расчетный счет покупателя. . .	Задача предоставления скидок покупателям Maks 03.08.2026 Задача: В документе "Продажи" необходимо реализовать функционал предоставления скидок покупателям. Скидка должна автоматически рассчитываться и подставляться в соответствующее поле при выборе. . .