Cで配列添字を使用しない内積関数

Question

内積用の関数を書き込もうとしましたが、プログラムが正しく機能しません。 私の間違いはどこですか？

#include<stdio.h> 
#include<stdlib.h> 

int inner_product(const int *a, const int *b, int size) 
{ 
    int sum = 0, i; 

    for (i=0; i<size; ++i) 
    sum += *(a+i) * *(b+i); 

    return sum; 
} 


int main() 
{ 
    int n, a, b; 
    printf("How many elements do you want to store? "); 
    scanf("%d",&n); 

    printf("%d",inner_product(&a,&b,n)); 
    system("pause"); 
    return 0; 
} 

Answer 1

だけで起動する：メインで

：機能まず

for (i=0; i<size; ++i) 
sum += *(a+i) * *(b+i); // Accessing the memory out of bound for size>1 behavior is undefined 
// Remember you have only two pointers which you can legally dereference ie a and b 

Answer 2

で

int n, a, b; // a & b are not initialized 
. 
scanf("%d",&n); // Think what is the use of n here. 
. 
printf("%d",inner_product(&a,&b,n)); 
/* You passed address of a and b, this is OK but the value they 
* store is indeterminate as per the standard 
*/ 

を、ポインタa、bが初期化されなければなりませんその作品を作るコードの作業。 "int"値としてそれらを導入します。したがって、これらの「配列」のアイテム数は1です。したがって、合計を計算するためのサイクルは必要ありません。

a[i] 

代わりに：それに加えて

、言う方が良い（とはるかに簡単に読める）である

*(a+i) 

のような複雑な構造を使用しないためにはるかに優れています。

また、オーバーフローの問題がコード内に存在する可能性があります。[int]型よりも大きな合計を生成する2つの値（[i]とb [i] 2^32より）？ 2つの整数の合計をチェックなしで3番目の整数に計算しています。プロダクションソフトウェアでは、セキュリティ上の欠陥につながる可能性があります。