Opened 16 months ago

Closed 16 months ago

Last modified 15 months ago

#1045 closed улучшение (fixed)

Слишком сложное условие и слишком много скобок

Reported by: alx Owned by: Denis_N
Priority: minor Component: БД изделий АДС
Keywords: Cc:

Description

В source:base/orders.php@79#L80 в условии после WHERE есть такое условие: (`status` = 'shipped' and (`datetime` >= (NOW() - interval 7 day))) or (`status` != 'shipped') .

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

В этом же условии часть `status` = 'shipped' and тоже не имеет эффекта. Предлагаю ее тоже убрать для улучшения читаемости кода.

Аналогичное условие есть в строке 53...

Change History (2)

comment:1 by Denis_N, 16 months ago

Resolution: → fixed
Status: new → closed

In 103/base:

Улучшение запросов в и-е "Заказы": слишком сложное условие и слишком много скобок
closes #1045
p.s. Спасибо, Алексей

comment:2 by san, 15 months ago

milestone: 1 очередь

Milestone deleted

Note: See TracTickets for help on using tickets.