Question

我试图使用指针获取两个数组的总和，但是当输出全部为零时，我该怎么办？

如果有更好的方法，我想知道

这是代码

更新

#include  < cstdio >  
#include < iostream >

using namespace std;

unsigned i;

int main(){

/* Array`s input */

    short A[3];

    for( i = 0 ; i < 3 ; i++ ){

    printf("Insert number for [A]: ");
    scanf("%hd",&A[i]);

    }

    printf("\n");

    short B[3];

    for( i = 0 ; i < 3 ; i++ ){

    printf("Insert number for [B]: ");
    scanf("%hd",&B[i]);

    }

    short C[3];

// Pointers

    short *punt_A, *punt_B, *punt_C;

    punt_A = &A[0];
    punt_B = &B[0];
    punt_C = &C[0];

// Addition

    for( i = 0 ; i < 3 ; i++ ){

    C[i]=punt_A[i]+punt_B[i];

    }

    printf("\n\nArray addition = { %d, %d, %d }\n", *punt_C, *(punt_C + 1), *(punt_C + 2));



return 0;

}

Answer 1

请记住short A[3];允许您合法访问A [0] .. A [2]

在

for( i = 3; i-- ; )
{
/* i-- is a tricky way of entering the loop
 * This checks i for the condition, and passes (i-1, the current value
 * of i) to the loop
 * Though the method is smart I think it is less readable
 * There is no access violation here, forgive my previous comment :(
 */
printf("Insert number for [A]: ");
scanf("%hd",&A[i]);  // Remember %hd for short.

}

此处的添加可能更好地表示为：

for( i = 0; i<3;i++){ 
/* I changed the forloop structure which may be used 
 * for reading the arrays too. In fact this has no surprises.
 */
C[i]=punt_A[i]+punt_B[i]; // Or *(punt_A+i) + *(punt_B+i)

}

关于，

printf("\n\nArray addition = { %d, %d, %d }\n", *punt_A, *punt_B, *punt_C );

这分别仅打印A[0],B[0],C[0]和您的格式说明符＆＃34;％d＆＃34;不匹配类型短。正确的说明符是＆＃34;％hd＆＃34;。使用循环打印结果数组。

Answer 2

punt_A，punt_B，punt_C都指向一个元素，超过各自数组的末尾，例如punt_A现在指向A[3]。数组沿着字边界分配在堆栈上，因此存在＆＃34;死区＆＃34;在堆栈上的每个数组之后。在Windows调试版本中，未初始化的区域标有0xcccccccc，因此我看到-13108是(short)(0xcccccccc)。在发布版本中，您只需看到之前在该内存地址中留下的内容，在许多情况下只是零。

当我看到for (i = 3; i--; )时，我做了双重拍摄。您已将i--置于通常检查条件的位置。使用for (i = 2; i >= 0; i--; )更清楚。它实际上是偶然的，因为当评估条件i--时，它需要i并检查该条件是否为非零。检查后，i递减并进入循环。当i变为零时，循环终止。所以在循环中你会看到i = 2,1,0。在任何评估之后应用post-decrement运算符，例如int n = 1;然后n + n--;返回2，而不是1. n在计算表达式后递减。

Answer 3

有几件事：

您的数组和指针声明为short，但scanf() / printf()说明符为%d，表示int。通常short是16位整数，int是32位整数。这可能会导致奇怪的效果。
您应该在2而不是3开始循环。 3已经超出界限。
在最后一个循环结束时，所有指针都将指向数组的末尾，但无论如何都会将它们传递给printf()（这实际上是未定义的行为）。
事实上，您不想输出C[]的内容 - 添加结果？你为什么在那里使用punt_A和punt_B？

Answer 4

现在这是你的程序的粗略修复

#include <stdio.h>
#include <iostream>

using namespace std;

unsigned i;

int main(){


/* Array`s inputs */

    int *A = new int[3];

    for( i = 3 ; i-- ; ){

        cout << "Insert number for [A]: " << endl;
        scanf("%d",&A[i]);
    }

    printf("\n");

    int* B = new int[3];

    for( i = 3; i-- ; ){

    cout << "Insert number for [B]: " << endl;
    scanf("%d",&B[i]);

    }

    int* C = new int[3];

// Pointers

    int *punt_A, *punt_B, *punt_C;

    punt_A = A;
    punt_B = B;
    punt_C = C;

// Addition

    for( i = 3; i--; ){

    *punt_C = *punt_A + *punt_B;
    punt_A++;
    punt_B++;
    punt_C++;

    }

    for(int i = 0; i < 3; i++)
    {
        cout << *C << endl;
        C++;
    }

delete [] A;
delete [] B;
delete [] C;
return 0;
}

在您的计划中：

问题是：你初始化了一组短裤。短的大小是2个字节。但是，您正在加载数字，期望整数％d，即4个字节。因此，您加载2个字节，其他两个字节只是省略。我编译它没有-Wall -pedantic标志，仍然得到警告：

main.cpp: In function ‘int main()’:
main.cpp:17:21: warning: format ‘%d’ expects argument of type ‘int*’, but argument 2 has type ‘short int*’ [-Wformat=]
     scanf("%d",&A[i]);
                     ^
main.cpp:28:21: warning: format ‘%d’ expects argument of type ‘int*’, but argument 2 has type ‘short int*’ [-Wformat=]
     scanf("%d",&B[i]);

现在解决这个问题，让它们全部完成。但是你要求更好的解决方案。如果给你两个数组，你必须总结它们，并且因为你使用c ++ 11，对你来说，使用容器可能更容易。

# include <stdio.h>
# include <iostream>

#include <vector>

#define COUNT 3

using namespace std;

unsigned i;

int main()
{
/* Array`s inputs */

    vector<int> A, B;

    int worker;

    for( i = COUNT ; i-- ; )
    {
        cout << "Insert number for [A]: " << endl;

        cin >> worker;
        A.push_back(worker);
    }

    for( i = COUNT ; i-- ; )
    {
        cout << "Insert number for [B]: " << endl;

        cin >> worker;
        B.push_back(worker);
    }

    for(int i = 0; i < COUNT; i++)
    {
        A[i] = A[i] + B[i];
    }
    cout << "summed up" << endl;

    for(vector<int>::iterator it = A.begin(); it != A.end(); it++)
    {
        cout << *it << endl;
    }
return 0;
}

现在注意，我为你的计数做了一个定义。而且，我再次使用Aray A输出，节省空间。问题是，该向量本身对于push_back（）具有O（n）复杂度（除非你重新调整大小，但这需要知道确切的输入数量，未来可能不会这样，你可能会加载到EOF）。为了消除这种情况并使程序更快，可以使用列表来获得更好的性能。

但是如果你想使用指针，请使用迭代器。：

（请注意我的代码没有失败保护，这意味着如果你写了一封信，它会做一些奇怪的事情，但我概述了这个想法：]

需要使用指针添加C ++数组的一些建议

4 个答案: