没有得到所需的输出（带数组的结构）C++ [关闭]

Question

这是我的代码：

#include <iostream>

using namespace std;

struct galaxy
{
    int x_pixel;
    int y_pixel;
};

galaxy get_space_ship(int size)
{
    galaxy UFO[size];
    for(int i=0;i<size;i++)
    {
        UFO[i].x_pixel = 10;
        UFO[i].y_pixel = 10;
    }
    return UFO[size];
}

galaxy update(galaxy UFO[], int size)
{
    for(int i=0;i<size;i++)
    {
        UFO[i].x_pixel += 100;
        UFO[i].y_pixel += 100;
    }
    return UFO[size];
}

int main()
{
    galaxy space_ship[10];
    space_ship[10] = get_space_ship(10);
    for(int i=0;i<10;i++)
    {
        cout << space_ship[i].x_pixel << endl;
        cout << space_ship[i].y_pixel << endl;
    }
    space_ship[10] = update(space_ship,10);
    for(int i=0;i<10;i++)
    {
        cout << space_ship[i].x_pixel;
        cout << space_ship[i].x_pixel;
    }
}

在这里，我正在尝试为一个数组创建一个具有两个字段（x_pixel，y_pixel）的结构的程序，然后初始化它们并更新它们的值并打印它们但没有得到预期的输出。我的代码没有任何编译错误。请检查我的代码，让我知道我错在哪里，并包括一些提示，因为我是新手。 谢谢

Answer 1

C 中的数组不像你想象的那样工作。他们不是一等公民，因此您不能按价值传递或归还他们。数组会衰减为指向其第一个元素的指针。这是一件好事，因为在您的示例中，您不想来回复制整个数组这么多次。这是一个可以满足您要求的工作示例：

struct galaxy
{
    int x_pixel;
    int y_pixel;
};

void get_space_ship(galaxy* arr, int size)
{
    for (int i = 0; i < size; i++)
    {
        arr[i].x_pixel = 10;
        arr[i].y_pixel = 10;
    }
}

void update(galaxy* arr, int size)
{
    for (int i = 0; i < size; i++)
    {
        arr[i].x_pixel += 100;
        arr[i].y_pixel += 100;
    }
}

int main()
{
    galaxy space_ship[10];
    get_space_ship(space_ship, 10);
    for (int i = 0; i < 10; i++)
    {
        cout << space_ship[i].x_pixel << endl;
        cout << space_ship[i].y_pixel << endl;
    }
    update(space_ship, 10);
    for (int i = 0; i < 10; i++)
    {
        cout << space_ship[i].x_pixel;
        cout << space_ship[i].x_pixel;
    }
}

但是，由于这些原因，C 数组被认为是一种非常低级且容易出错的构造。更好的方法是使用 std::vector 或 std::array 并通过引用传递它们以保存副本。

std::vector<galaxy> get_space_ship(int size)
{
    std::vector<galaxy> ships;

    for (int i = 0; i < size; ++i)
        ships.push_back(galaxy{ 10, 10 });

    return ships;
}

void update(std::vector<galaxy>& ships) // note reference
{
    for (auto & ship : ships)
    {
        ship.x_pixel += 100;
        ship.y_pixel += 100;
    }
}

int main()
{
    auto ships = get_space_ship(10);

    for (const auto & ship : ships)
    {
        cout << ship.x_pixel << endl;
        cout << ship.y_pixel << endl;
    }

    update(ships);

    for (const auto & ship : ships)
    {
        cout << ship.x_pixel;
        cout << ship.x_pixel;
    }
}

请注意，您不再需要多次声明大小，因为您使用的是知道其大小的更高级别的抽象。

Answer 2

您对以下 c++ 结构有误解：

• space_ship[10] 并不意味着“所有十艘宇宙飞船”。它表示索引为 10 的宇宙飞船（=第十一艘宇宙飞船）。你没有那么多的宇宙飞船，你有十个。
• 如果您不熟悉指针，请避开普通的 C++ 数组，它们会出现一些令人惊讶的行为，迫使您使用指针。特别是很难将它们作为一个整体进行处理。如果您完全不熟悉 C/C++，我建议您使用 std::vector 或 std::array（对于固定数量的元素），这对您来说更容易预测。
• 如果您想继续使用常规 [] 数组，请注意 galaxy_update 不会收到数组的副本，它会直接修改数组。不需要返回修改后的数组。

建议：一定要正确缩进你的代码。一个合适的 IDE 将帮助您。缩进可能看起来很麻烦，但它的成本/收益比很高。