C配列、ポインタのセグメンテーションフォールト

Question

私は最高値のアドレスを検索し、アドレスに値を増やし、配列を作成します。

：

int main() 
{ 
    int i[5] = {2,5,6,5,3}; 
    int *pi = getAdres(i); 
    (*pi)++; 

    printf("%d", i[2]); 
    return 0; 
} 

getAdres()機能は次のようになります

int getAdres(int *i) 
{ 
    int *pi; 
    int higest = 0; 
    for(int j = 0; j < 5; j++) 
    { 
     if(i[j] > higest) 
     { 
      pi = &i[j]; 
      higest = i[j]; 
     } 
    } 
    return pi; 
} 

getアドレス部分を機能に組み込むことなく機能しますが、現在の形式では(*pi)++;は私にセグメンテーションフォールトを与えています。 何が問題になりますか？

Answer 1

getAdresはint *の代わりに返品タイプintです。

いずれにしても、その実装は悪いです。それはマジックナンバー5を使用し、さらに関数に渡される任意の配列は、例えば負の値しか持たない。

例えば関数定義は

int * getMaxElement(const int *a, size_t n) 
{ 
    size_t highest = 0; 

    for (size_t i = 1; i < n; i++) 
    { 
     if (a[highest] < a[i]) 
     { 
      highest = i; 
     } 
    } 

    return (int *)a + highest; 
} 

のように見えることができ、機能は何

int *pi = getMaxElement(i, sizeof(i)/sizeof(*i)); 
++*pi; 

printf("%d\n", *pi); 

Answer 2

あなたは間違った戻り値の型を持って、それを作る：

int * getAddressToMax(int *i); 

また、それが実際に有効なポインタを返すないことを確認してください。配列全体が0の場合、初期化されていないポインタが返されますが、これは本当に悪い考えです。その場合は、最初の要素へのポインタを返すようにしてください：

int *pi = i; 

また、あなたは、一般的に参照してください。その後、

int highest = i[0]; 

と1からループを開始し、このようなケースのために。配列の長さを渡すことは、実際のコードでも非常に一般的です。そのような知識をハードコーディングして汎用関数にすることは、物事を理解しにくくします（もちろん、コードの再利用を殺します）。