Question

Is it a good practice to use ternary operation in for loop condition. I am asking this because i have come across the situation where in ternary operation will resolve my issue in for loop condition.

对于例如 :

for( short i = 0 ; i < count ; i++ ){
   for( short j = 0 ; j < ( ( x[i] < y[i] ) ? x[i] : y[i] ) ; j++ ){ //Here I am using ternary operation at for loop condition place
       //.....
       //.....Some Code Here .......
       //.....
   }
}

Answer 1

我看不出有什么问题

My advice.
1. short change to int.
2. #define ternary(optional).

#define cond(n)  (( x[n] < y[n] ) ? x[n] : y[n] )

for( int a = 0; a < count; a++)
  for( int b = 0; b < cond(a), b++)
 {

  ........code...............

 }

Answer 2

我选 min(x(i),y(i) ,只是因为它更清楚,但我看不出你的代码有什么问题。

Answer 3

我个人会选择 min(x(i),y(i) 。然而,您的可读性问题可以通过预先计算和使用数值来解决:

for( short i = 0 ; i < count ; i++ ){
   int jCount = ( x[i] < y[i] ) ? x[i] : y[i]; // or min equivalent
   for( short j = 0 ; j < jCount ; j++ ){ 

   }
}

这在某种程度上改变了行为, 因为它使状态无法对 x[i] 和 y[i] 和 y[i] 在 j 环中的变化进行修正。我不知道这是否适合您的使用大小写。

Answer 4

如果你同意使用长期操作员一般来说, 那么没有理由不使用它在你的循环状态。 (我喜欢永久操作员自己, 但我知道有些人认为它混淆了)

Answer 5

我想,除了它使代码无法读取之外,使用长期运算符没有问题。但像这样的条件 x(i) < y[i]? x(i) : y[i] 有时会造成问题, 以防您同时修改 x(i) 和 y[i] , 无法用于循环。所以我建议您不使用, 除非您真正知道自己在做什么。

友情链接