创建新数组时出现 C++ 错误：表达式必须具有指向对象类型的指针

Question

我正在尝试创建一个仅取自已创建数组的正值的新数组，当我遍历原始数组时，索引出现错误“表达式必须具有指向对象类型的指针”我尝试进行研究关于错误，每个人在收到此错误时的情况都不同，所以我自己处理这个问题。这是我的代码：

int foo::createNewArray() const {
    int newarray[50];
    int oldarray = oldarray[values];
    int size = (sizeof(oldarray));

    for (int i = 0; i > size; i++){
        if (oldarray[i] > 0)
            newarray[i] = oldarray[i];
    }

上面的“i”是有错误的。 oldarray[values] 在单独的类文件中声明。这是它来自的代码的一小部分。

        class foo{
        int oldarray[1];
        enum unit {values};

        public:
        int createNewArray() const;
};

Answer 1

在这里，您可以使用本地 int 变量来影响 oldarray 数组：

int oldarray = oldarray[values];

从那时起，直到块结束，oldarray 表示一个 int，然后其余代码对此没有多大意义。

Answer 2

这里逐行列出了这段代码的一些问题。

class foo{
    int oldarray[1]; //Do you really want an array of size 1?  Why not just an integer?
    enum unit {values};//An enumeration that only enumerates one thing?  Are you sure you don't really want const int VALUES = 0;  I feel like you don't really even want an enum

    public:
    int createNewArray() const; 
};

int foo::createNewArray() const {
    int newarray[50];  //Magic numbers are bad, what if sizeof(oldarray) > 50?
    int oldarray = oldarray[values];  //Re declaring oldarray as a single integer and assigning it oldarray[values] as its value.
    int size = (sizeof(oldarray));  //is this oldarray an integer or an array of integers???

    for (int i = 0; i > size; i++){  //don't you want i < size??? if size > 0, this loop will never get run.
        if (oldarray[i] > 0) //probably grabbing the correct oldarray(Compilers are smart), but not getting expected values because the array version of oldarray wasn't initialized properly.
            newarray[i] = oldarray[i];
    }

我相信您正在尝试执行以下操作：

int* foo::createNewArray() const {
    const int SIZE = sizeof(oldarray);
    int *newArray = int[SIZE];
    for(int i = 0; i < SIZE; i++) {
        if(oldarray[i] > 0) {
            newArray[i] = oldarray[i];
        } else {
            newArray[i] = 0;//In most environments this is unnecessary, but it is safer and good style
        }
    }

    return newArray;
}

请注意，即使 oldarray 在此代码范围内（不是很好的样式，将其作为参数传递给 createNewArray 会更好，但也可以）并且已正确实例化以便 sizeof(oldarray ) 是数组的大小，而不是整数的大小，或者可能是整数指针，我忘记了。

Answer 3

问题是因为 oldArray 需要是一个 int*，而不仅仅是一个 int。您当前将 oldarray 设置为数组中的第一个值，而不是将其指向数组的根。所以像 int* oldArray = newArray 这样的东西会让你使用索引运算符遍历 oldArray。

class Foo
{
    int* oldArray;
    int size;

public:
    int* CreateNewArray() const
    {
        int* newArray = new int[size];

        int current = 0;
        for( int index = 0; index < size; index++)
        {
            if(oldArray[index] > 0)
            {
                newArray[current] = oldArray[index];
                current++;
            }
        }

        return newArray;
    }
};

对于在没有编译的情况下随意发布此内容，我深表歉意。尽管此解决方案可能比建议的更接近金属，但它仍然是您的问题的有效解决方案，假设在调用此方法之前设置了 oldArray 和 size。