在C ++中将向量中的数字无序相乘

Question

我叫马特。我是 StackOverflow 的新手，对 C++ 也很陌生。目前正在学习 Lippman 的 C++ Primer。

我在书中做一个练习，任务是将整数读入向量中，然后通过第一个和最后一个、第二个和第二个到最后一个、第三个和第三个等来将这些整数相乘。

我自己做的，没有查任何东西，否则如果我只是复制的话，我几乎学不会......我的程序编译并按预期运行。我的问题是：我做对了吗？有没有更有效的方法？

我不仅想学习如何编写工作代码，而且我想正确地做。提前谢谢！

#include <iostream>
#include <string>
#include <vector>
#include <cctype>

using std::cout; using std::cin; using std::vector; using std::endl;
using std::string;


int main()
{
    vector<int> numbers;
    int usernum = 0;

    cout << "Enter some numbers: ";
    while (cin >> usernum)
    {
        numbers.push_back(usernum);
    }

    unsigned maxElement = numbers.size() - 1;
    unsigned minElement = 0;

    for (auto i : numbers)
    {
        cout << numbers[minElement] << " * " << numbers[maxElement] << " = " << numbers[minElement] * numbers[maxElement] << "\n";
        ++minElement;
        --maxElement;
    }

    return 0;
}

Answer 1

在评论中，你说：

我注意到它实际上并没有完全按预期工作。它通过向量并将所有内容相乘两次，因为 maxElement 一直到开始，minElement 一直到结束。每次只完成一次操作后，不知道如何停止它。

如果您不想重复乘法，则需要稍微更改for 循环。

for ( ; minElement <= maxElement; ++minElement, --maxElement)
{
    cout << numbers[minElement] << " * " << numbers[maxElement] << " = " << numbers[minElement] * numbers[maxElement] << "\n";
}

附言

使用此逻辑时，您需要确保 minElement 和 maxElement 是有符号类型。否则，如果numbers 只有一个元素，就会遇到问题。

Answer 2

1. 让我感到奇怪的第一件事是您使用的命名空间。 而不是这样做：using namespace std::vector; 你可以公平地做using namespace std;，因为无论如何你都调用 std::vector：vector<int> numbers;***。这适用于您使用的任何“使用过的”命名空间。只需一劳永逸地做using namespace std;。 ***我不确定 std::vector/std::cout/... 甚至是一个命名空间。 std - 是一个命名空间。 std::vector 应该是 std 命名空间下的一个类：

namespace std { template<typename T> class vector<T> {...}; }

1. 它怎么会“按预期运行”。我不知道这个循环：while(cin >> usernum)。你怎么知道用户输入何时完成？第一眼（我自己没有编译/运行它）我也期待它：

   • 不编译
   • 在运行时崩溃或出现未定义的行为
   • 无限运行while循环

2. 改用这个： for (int i = 0, end_of_vector = numbers.size(); i < end_of_vector/2; i++) { cout << numbers[i] << " * " << numbers[end_of_vector - 1 - i] << " = " << numbers[i] * numbers[end_of_vector - 1 - i] << "\n"; } 原因：

   • 在这种情况下，您不需要任何特殊变量来存储第一个和最后一个向量索引。
   • 您只遍历数组的一半。
   • 使用for (auto i : numbers)，预计将使用i 作为numbers vector 的元素。但是您不这样做，而是按原样使用数字。因此，这个 for 循环是模棱两可的