Jump to content

    

Проблема с условием

Добрый день!

 

Я переношу чужую программу с AVR на STM32 (под HAL Keil).

 

m = m+I|P == I ? m : 0;

 

Эта строка выдает предупреждение: "| имеет более низкий приоритет, чем ==; == будет оцениваться сначала"

Когда пишу m = (m+I|P) == I ? m : 0; предупреждение исчезает, но я не знаю можно ли так писать, не нарушит ли эта запись условие.

 

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

 

if (x+r-y|u&32|p > 2&(p-4|j-7 || b[G=x+3^r>>1&7]-k-6 || b[G^1]|b[G^2])) t += p<5; else F = y;

 

Warning: '&' within (внутри) '|'

Warning: & has lower precedence than >; > will be evaluated first

Warning: '&' within (внутри) '|'

 

Подскажите ссылку где можно поучится этим условиям в контексте моей проблемы.

Share this post


Link to post
Share on other sites
Подскажите ссылку где можно поучится этим условиям в контексте моей проблемы.

Гуглите "c operator precedence".

 

Надо заметить, что это -

if (x+r-y|u&32|p > 2&(p-4|j-7 || b[G=x+3^r>>1&7]-k-6 || b[G^1]|b[G^2])) t += p<5; else F = y;

- просто прекрасно. В жизни не видел такой красоты в реальных проектах.

Share this post


Link to post
Share on other sites

Воу. Если не секрет, то что это за проект такой? А так, Вам необходимо просто почитать о приоритетах операторов.

Share this post


Link to post
Share on other sites

Дык, обфускация.

Share this post


Link to post
Share on other sites

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

Edited by segment

Share this post


Link to post
Share on other sites
m = m+I|P == I ? m : 0;

 

Эта строка выдает предупреждение: "| имеет более низкий приоритет, чем ==; == будет оцениваться сначала"

Когда пишу m = (m+I|P) == I ? m : 0; предупреждение исчезает, но я не знаю можно ли так писать, не нарушит ли эта запись условие.

Скобки вы расставили неправильно даже несмотря на подсказку компилятора. Если автор исходного кода знал приоритеты операций (а судя по всему он их знал), то скобки должны стоять так: m = ((m + I) | (P == I)) ? m : 0.

 

P.S. В последнее время компиляторы что-то уж больно часто стали ругаться на очевидные выражения. Неужели настолько упал уровень основной массы современных программистов, что они не в состоянии запомнить приоритет операторов? Жду - не дождусь, когда компилятор начнет требовать скобки в выражении a*x+b.

Share this post


Link to post
Share on other sites
Дык, обфускация.

Для обычной обфускации это как-то слишком. Тут что-то болезненное просматривается.

 

P.S. В последнее время компиляторы что-то уж больно часто стали ругаться на очевидные выражения...

GCC еще и замены предлагает в гугл-стиле: "а не имели ли Вы, уважаемый, в виду A, а не B?"

А ведь кто-то просто последует такой рекомендации не вникая.

Share this post


Link to post
Share on other sites
Скобки вы расставили неправильно даже несмотря на подсказку компилятора. Если автор исходного кода знал приоритеты операций (а судя по всему он их знал), то скобки должны стоять так: m = ((m + I) | (P == I)) ? m : 0.

 

P.S. В последнее время компиляторы что-то уж больно часто стали ругаться на очевидные выражения. Неужели настолько упал уровень основной массы современных программистов, что они не в состоянии запомнить приоритет операторов? Жду - не дождусь, когда компилятор начнет требовать скобки в выражении a*x+b.

 

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

 

А по теме - врядли в IAR и gcc разный приоритет операторов, если проект у вас был рабочий - оставьте как есть, просто отключите эти ворнинги. Если хотите улучшить читаемость - лучше ее отдельно улучшать, после того как перенесенный проект заработает в таком виде

 

Share this post


Link to post
Share on other sites
Воу. Если не секрет, то что это за проект такой?

Это не секрет, это шахматная программа micro-Max.

Весь код помещается на одном листе А4, притом, что программа знает даже такое правило, как "взятие пешки на проходе".

Мне этот алгоритм очень понравился, я когда-то с друзьями в турнир играл, четырех обыграл, а эта програмка меня взувает (сделал шахматный калькулятор на ATmega48).

Тут верно заметили, что код писал математик.

Спасибо за советы.

Edited by ave!

Share this post


Link to post
Share on other sites
if (x+r-y|u&32|p > 2&(p-4|j-7 || b[G=x+3^r>>1&7]-k-6 || b[G^1]|b[G^2])) t += p<5; else F = y;

Типичный говнокод, а не обфускация. Ибо приводить тип bool к численному - дурная манера, которая может вылезти боком при переходе на другой компилятор. И нормальный обфускатор такое делать не станет.

Да и просто это ведёт к неоптимальному коду.

Share this post


Link to post
Share on other sites
Вот ни разу не очевидное выражение
Приоритет операций известен. Да, оно сложнее, чем a+b, но ничего неоднозначного и тем более неочевидного в нем нет. В a*x+b тоже будем скобки расставлять?

 

Плюс что-то кажется мне, что тут правильней не бинарное или, а логическое.
С этим согласен.

Share this post


Link to post
Share on other sites
Тут верно заметили, что код писал математик.

Код писал извращенец человек, цель которого была совсем не оптимальность и не решение практической задачи, а цель - вычурность кода.

Ибо: вместо t += p<5 простой программист использует if, а более продвинутый заменит на что-то типа: t -= (int)(p - 5) >> 31;

Share this post


Link to post
Share on other sites
Скобки вы расставили неправильно даже несмотря на подсказку компилятора. Если автор исходного кода знал приоритеты операций (а судя по всему он их знал), то скобки должны стоять так: m = ((m + I) | (P == I)) ? m : 0.

 

Сергей Борщ, я скопировал все 7 предупреждений.

Может у вас будет настроение расставить все скобки.

Буду вам очень благодарен.

Чувствую, что даже после изучения приоритетов я все-равно где-то, что-то не так сделаю.

Программа скомпилируется и даже работать будет, но это шахматный алгоритм, там можно не понять что что-то не так работает.

 

(1) while (d++<n||d<3||z&K==I&&(N<T&d<98||(K=X,L=Y&~M,d=3)))

 

(2) while (r=p>2&r<0?-r:-o[++j])

 

(3) if (t&k|p<3&!(y-x&7)-!t)

 

(4) v-=p-4|R>29?0:20;

 

(5) Y=y|S&F;

 

(6) if (x+r-y|u&32|p>2&(p-4|j-7||b[G=x+3^r>>1&7]-k-6||b[G^1]|b[G^2]))

 

(7) m=m+I|P==I?m:0;

Исправление готово: m = ((m+I)|(P==I)) ? m : 0;

 

Мой случай как раз яркий пример для статьи на хабре:

Как вы пишете условия в СИ-подобных языках? Со скобками в условиях или без?

 

Share this post


Link to post
Share on other sites
Исправление готово: m = ((m+I)|(P==I)) ? m : 0;

Эта конструкция скомпилится в большее число команд, чем простое, читаемое и аналогичное по результату: if (!(m + I) && P != I) m = 0;

И в других местах - аналогично.

Поэтому единственная цель такого поделия - сделать вычурный код.

 

Share this post


Link to post
Share on other sites
Поэтому единственная цель такого поделия - сделать вычурный код.

ну собственно так там и написано

My original aim was to write a chess program smaller than 1024 characters

 

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now