関数の引数としてのポインタ

Question

私は以前も同様のコードを書いていましたが、何らかの理由でこれは実行されませんでしたが、何が残っているのか間違っているのか分かりません。

int fn(int *pArg) 
{ 
    if(*pArg!=NULL) 
     return *pArg+fn(pArg+1); 
    else 
     return 0; 
} 

int main() 
{ 
    int nArray[128],i,nLenght; 
    std::cin>>nLenght; 
    for(i=1;i<=nLenght;i++) 
     std::cin>>nArray[i]; 
    std::cout<<"The sum of the array is: "<<fn(nArray); 
    system("PAUSE"); 
    return 0; 
} 

Answer 1

このプログラムは

*pArg!=NULL 

がヌルポインタ定数でintを比較することであるそのうちの一つ、多くの点で欠陥があります。代わりにpArg != NULLを実行してください。

（およびfnは欠陥があることに注意してください、それはポインタがNULLにラップアラウンドするまで再帰しようとしますが、おそらくがオーバーフロー長くそれが起こる前にスタックします。）

Answer 2

std::cin>>a[i]; 

は、この行は次のようになります。

std::cin>>nArray[i];