Cで次のリント警告を削除する方法はありますか？

Question

私は、次のコードを持っている：

#define NUM_DAYS 65 
#define NUM_PERSON 33 

int num = 0; 

if(NUM_DAYS % NUM_PERSON) 
{ 
    num = NUM_DAYS/NUM_PERSON; 
} 
else 
{ 
    uum = NUM_DAY/NUM_PERSON + 1; 
} 

num = num - 1; 

while(num > 0) 
{ 
    //do something here 
} 

けど、次のlint警告だ：

Warning 681: Loop is not entered 

何の問題があるとどのようにそれを修正するために？

Answer 1

コードが書き込まれると、ループは入力されません。 NUM_DAYS % NUM_PERSONはtrueと評価されるので、numはNUM_DAYS/NUM_PERSONに等しくなります。 intを扱っているので、65/33は1に等しい。 1 -1は0であるため、while条件は失敗します。

コードが意図したとおりに書かれている場合（これらの定数は、常に使用すると思われる値です）、whileループを削除してください。それは決して使用されません。ただし、NUM_DAYSまたはNUM_PERSONに後で他の値が含まれる場合は、おそらく何も心配はありません。これらの特定の値が重要でない場合は、除算値が1よりも大きい値になるように値を設定してみてください。

Answer 2

は、両方式の値は、他の場合は、コンパイル時に知られている中で、以下のようなので：

他 num == 1で if (num == 0)で

は、ループnum値があるので、前にしながら、どちらか0、（他の）または-1（からif）が0以下であることを意味します。つまり、while - 条件はコンパイル時には常にfalseです。これは、コンパイル時に決して実行されないことを意味します。

コード内の静的な入力はなぜですか？あなたはコンパイル時に価値を与えて、ユーザーから正しいことを尋ねます。

マクロを削除し、以下のような：

int num_days, number_persons; 
scanf("%d", &num_days); 
scanf("%d", &number_persons); 

これは動作しますが、静的にこれらの65、33値を割り当てるノートが問題です！

Answer 3

#define NUM_DAYS 65 
#define NUM_PERSON 33 

int num = 0; 

if(NUM_DAYS % NUM_PERSON) // we go here, since (NUM_DAYS % NUM_PERSON) > 0 
{ 
    num = NUM_DAYS/NUM_PERSON; // so num = 1 now 
} 
else 
{ 
    uum = NUM_DAY/NUM_PERSON + 1; 
} 

num = num - 1; // num = 0 now 

while(num > 0) // num = 0 ! So we don't go in this loop 
{ 
    //do something here 
} 

この警告が表示されます。コンパイラはループが役に立たないと判断しました（現在の#defineの値）。

Answer 4

@patrickvacekが正しいです。私は彼の説明を拡張する。

整数演算では、65を33で割った値はほぼ2ですが、それほど大きくはありません。商は1で残りは32です。C++では商または余剰のいずれかにアクセスできます。商の場合は65/33 == 1です。残りは65 % 33 == 32です。あなたのコードは商を求めるだけで、残りを求めません。したがって、if (NUM_DAYS % NUM_PERSON)は、文字通りif (1)を意味し、これは次にif (true)を意味する。したがって、ifステートメントのelseブランチには決して到達しません。

あなたは残りの部分に従うことができます。さらに、ロジックは比較的単純なので（プリプロセッサやコンパイル時の評価と関連しなければならないその他の理由で）、コンパイラが残ります。したがって、警告。

Answer 5

その可能な値は、それが-1あるintタイプまたはnum = num - 1あるので、0に降格され65/33 or 34こと、およびwhile ((0 or -1) > 0)に値を代入可能性があるのであなたのコンパイラは、すでにnumの値を計算します。そのため、コンパイラが警告します。