和大家分享一个代码，个人认为考验C语言水平

15284083060

• for(TempC = 1; TempC < (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8;  TempC++)
• {
•        VersionInfo[TempD] = RxFrame.Data[TempC];
•        TempD++;
• }
  代码如上述所示，补充：1、ReceiveNum的值是29恒定不变，结构体成员Data的数组大小是8， VersionInfo的数组大小是64。   这个代码有没有问题？

LcwSwust

sfd123 发表于 2023-7-13 12:30
不会啊，TempC在++啊

ReceiveNum的值是29,那么
TempC < (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8
就相当于
TempC < 1  ?  1 :  8
相当于
if(TempC < 1)
   表达式的值＝1；
else
  表达式的值＝8；
这个问号语句的值要么是1要么是8，总之非0，for循环就会一直循环；

相当于
for(TempC = 1; 1;  TempC++)   
或
for(TempC = 1; 8;  TempC++)

可能造成数组越界、死机。
这代码为什么整得这么复杂呢，用if语句多好。

LcwSwust

TempC < (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8


循环条件的值好像一直是非零，那不成死循环了？

sfd123

LcwSwust 发表于 2023-7-13 11:57
TempC < (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8

不会啊，TempC在++啊

sfd123

TempD 这个 要注意一下，不要造成VersionInfo溢出就好

kerongsheng

死循环，TempC的数值从1开始自增，表达式TempC < (ReceiveNum % 7) 永远不成立，8为非0，？表达式的值一直是1

1759817440

LcwSwust 发表于 2023-7-13 14:17
ReceiveNum的值是29,那么
TempC < (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8
就相当于

膜拜大佬的讲解...
请问 ? 1：8是什么意思啊？

jermy_z

1759817440 发表于 2023-7-14 09:31
膜拜大佬的讲解...
请问 ? 1：8是什么意思啊？

C语言表达式    你百度一下问号冒号表达式就知道了

jermy_z

这种代码写出来是被人骂的，不是体现C语言水平的

15284083060

LcwSwust 发表于 2023-7-13 14:17
ReceiveNum的值是29,那么
TempC < (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8
就相当于

这个代码是在项目里面的，ReceiveNum是在动态变化的，所以这么写，答案就是这样的，小于运算符优先级高，所以会先结合，最后导致死循环，我一开始这么写导致死循环都没意识到，后来发现是怎么回事之后，觉得很有意思，就和大家分享一下

LcwSwust

1759817440 发表于 2023-7-14 09:31
膜拜大佬的讲解...
请问 ? 1：8是什么意思啊？

百度 C语言问号表达式

LcwSwust

15284083060 发表于 2023-7-14 10:09
这个代码是在项目里面的，ReceiveNum是在动态变化的，所以这么写，答案就是这样的，小于运算符优先级高， ...

我也老记不住优先级,所以看别的人的码都要查一下,自己写就多加括号.
非必要就不用问号语句了,用if挺好的.

sfd123

LcwSwust 发表于 2023-7-13 14:17
ReceiveNum的值是29,那么
TempC < (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8
就相当于

你是对的！
我是想当然的当成TempC <( (ReceiveNum % 7)  ?  (ReceiveNum % 7)  :  8)