配列検索はポインタを作成するだけで動作します

Question

この関数を使って配列を検索しても機能しませんでした。配列を渡すことと何らかの関係があると思いました。だから私は別のものを試していた。配列へのポインタを作成するだけで作業が始まりましたが、何もしませんでした。それはなぜですか？あなたがバインドされたアクセスのうち配列であるarr[size+1]にアクセスしようとしたところ

void search(int a[],int n,int find); 

int main(void){ 
    int size; 
    int arr[]={345,767,232,45,768,2342,78,-89,-354,-234}; 

    int *p=arr; 

    size=sizeof(arr)/sizeof(int); 
    search(arr[size+1],size,-89); 
    return 0; 
} 
void search(int a[],int len,int find){ 
    int spot,i,FOUND; 
    spot=i=FOUND=0; 

    for(i=0;i<len;i++) 
     if (a[i]==find){ 
      FOUND=1; 
      break; 
     } 
     if (FOUND==1){ 
      printf("%d found at element %d\n",find, i); 
     } 
     else 
      printf("Number: %d \tNOT FOUND",find); 
    } 
} 

Answer 1

このコードの基本的な問題は

search(arr[size+1],size,-89); 

です。これによりundefined behaviorが呼び出されます。サイズnを有するアレイ、arr[n]に有効に結合されたアクセスの外にもあるよう

FWIW、Cは、アレイの0ベースのインデックスを使用します。有効なアクセスはarr[0]〜arr[n-1]になります。

arrは、関数の引数として渡されたとき自動的に最初の要素へのポインタに減衰する配列名を渡す必要があります。あなたが一方

、

int *p=arr; 

とp、あなたが効果的にpに配列の最初の要素へのポインタを格納し、渡しているを渡すことで機能、（へ期待どおり）、うまく動作します。

、

size=sizeof(arr)/sizeof(int); 

を作るために

size=sizeof(arr)/sizeof(arr[0]); 

ように書き換えることができるだけの提案は、より堅牢である、と述べました。