配列の値が偶数かどうかをチェックし、真の場合はtrueを返します。

Question

これはこれまでのことですが、配列に奇数または偶数がある場合でも結果は常に1です。私は何をしていますか？ループについて

#include <stdio.h> 
#include <stdbool.h> 

int main(){ 
    int vector[10]={2,4,6,8,10,12,14,16,18,20}; 
    bool function(int i){ 
     for(i=0,i<10,i++){ 
      if(vector[i]%2==0){ 
       return true; 
       } 
      return false; 
      } 
     } 
    } 
} 

Answer 1

エントリない,間;を使用すべきです。あなたは決してループしていません。

for(i=0;i<10;i++) 

...

正直なところ、私はそれがループのための基本の要素が欠落しているとして、あなたのコンパイラはこの文に少なくとも警告を投げることを期待します。

次は、forループまたはそれが最初のエントリを過ぎて反復することはありません外にご

return false; 

を移動する必要があります。

最後にfunctionをmainに定義することは、Cの標準ではありません。それを独自の機能にして、物事はより幸せになれます。

Answer 2

最終的な反復で返すためです。配列全体をチェックしたら、falseを返してください。

また、関数が呼び出されることもなく、C semanticを使用して関数を更新することもできます。

#include <stdio.h> 
#include <stdbool.h> 

bool check_even(int vector[], int size){ 
    int i; 
    for(int i=0; i<size; i++){ 
     if(vector[i] % 2==0) { 
      return true; 
     } 
     // return false; shift this to out of for block 
    } 
    return false; 
} 


int main(){ 
    int vector[10]={2,4,6,8,10,12,14,16,18,20}; 

    bool is_even = check_even(vector, 10); // here function will get called 

    return 0 // always return with main 
} 

Answer 3

C標準に従う場合、別の関数の中で関数を定義することはできません。

しかし、あなたの関数の主な問題（例えば、for文のような構文エラーは別として）は、配列の最初の要素のためにreturn文を奇数か偶数かに独立に使用しています。

プログラムは、以下の方法を見ることができます

#include <stdio.h> 
#include <stdbool.h> 

bool is_even(const int a[], size_t n) 
{ 
    size_t i = 0; 

    while (i < n && a[i] % 2 == 0) i++; 

    return i == n; 
} 

#define N 10 

int main(void) 
{ 
    int vector[N] = { 2, 4, 6, 8, 10, 12, 14, 16, 18, 20 }; 

    printf("All elements are even: %d\n", is_even(vector, N)); 
} 

パラメータなしでC標準関数mainに応じてあなたは常に最初の反復に戻ってきている

int main(void)