使用函数在向量中查找五的倍数

Question

编写一个名为 findMultipleOfFives 的函数，该函数将整数向量作为参数（作为常量引用传递）。如果向量中的每个数字都是数字 5 的倍数，则您的函数应返回 true。否则，该函数应返回 false。

例如，对于给定的向量变量 vector all_values{10, 20, 30, 40, 50}，函数 findMultipleOfFives 将返回 true。但是，对于给定的向量变量，vector all_values{10, 24, 30, 40, 5}，该函数将返回 false。

我写的代码：

//Name
//This program will determine if a vector has multiples of fives


#include <iostream>
#include <vector>
using namespace std;


string findMultipleOfFives(vector<int> &all_values);

vector<int> all_values{ 10, 20, 30, 40, 50 };

int main()
{
    cout << "This program will determine whether a vector is full of multiples of five." << endl;
    findMultipleOfFives(all_values);
}

string findMultipleOfFives(vector<int>& all_values)
{
    for (int count = 0; count < 5; count++)
    {
        all_values[count] = 5;
        if (all_values[count] % 5 == 0)
        {
            all_values[count]++;
            return "True";
        }
        else
        {
            return "False";
        }
    }
}

在我的main() 函数中调用该函数时出现错误：

避免使用自定义构造和销毁的未命名对象

除了弄清楚该错误的含义以及我做错了什么之外，我还停留在如何阅读向量的每个元素上。我知道使用模数来确定单个值是否是 5 的倍数，但我不知道如何循环并确定每个值的值并返回真或假。

Answer 1

首先，您尝试在循环访问vector 时对其进行修改。您不需要这样做，说明也没有要求您这样做。

其次，指令要求您返回布尔值，而不是字符串。

第三，您的函数没有循环遍历整个向量。 std::vector 有一个 size() 成员来获取元素的实际数量，还有一个 operator[] 来访问元素。但即使使用该大小值，您的循环在第一次迭代时也是 return'ing，因此您只比较第一个元素并忽略其他元素。说明清楚地说明如果 所有 元素都符合条件，则返回 true。

试试这个：

//Name
//This program will determine if a vector has multiples of fives


#include <iostream>
#include <vector>
#include <iomanip>
using namespace std;

bool findMultipleOfFives(const vector<int> &all_values);

int main()
{
    cout << "This program will determine whether a vector is full of multiples of five." << endl;
    vector<int> all_values{ 10, 20, 30, 40, 50 };
    cout << boolalpha << findMultipleOfFives(all_values);
}

bool findMultipleOfFives(const vector<int>& all_values)
{
    if (all_values.empty())
        return false;

    for (size_t count = 0; count < all_values.size(); count++)
    {
        if ((all_values[count] % 5) != 0)
            return false;
    }

    return true;
}

或者，std::vector 可以使用 iterators，例如：

bool findMultipleOfFives(const vector<int>& all_values)
{
    if (all_values.empty())
        return false;

    for (vector<int>::const_iterator iter = all_values.begin(); iter != all_values.end(); ++iter)
    {
        if ((*iter % 5) != 0)
            return false;
    }

    return true;
}

或者，在 C++11 及更高版本中使用基于范围的 for 循环（内部使用 iterators）：

bool findMultipleOfFives(const vector<int>& all_values)
{
    if (all_values.empty())
        return false;

    for (int num : all_values)
    {
        if ((num % 5) != 0)
            return false;
    }

    return true;
}

Answer 2

你错了。

在您的代码中：

string findMultipleOfFives(vector<int>& all_values)  // the exercise call for a function 
                                                     // that returns bool value, 
                                                     // not a string
{
   for (int count = 0; count < 5; count++)  // you are looking for multiples of 5, 
                                            // not loop though the first 5 elements.
   {
       all_values[count] = 5;              // Why are you changing the original array?
       if (all_values[count] % 5 == 0)
       {
           all_values[count]++;            // Why are you still changing the original array?
           return "True";                  // You are returning before the end of the loop.
                                           // all elements must be mults of 5  
                                           // to return true.
       }
       else
       {
           return "False";
       }
    }
}

这是一种写法。

bool AllAreMuliplesOfFive(const vector<int>& v) // Note the use of const, we do not need to change 
                                                  // or destroy the caller's data.
{
   for (size_t i = 0; i < v.size(); ++i)  // looping through all elements in v
                                          // v.size() returns a size_t 
   {
       // test condition
       if (v[i] % 5 != 0)   // multiple of 5? 
          return false;     // no, since we need all to be multiple of 5 for success
                            // return false now.
   }
   return true;  // all elements have passed the test!
}

Answer 3

这应该只是一个算法：

#include <algorithm>

bool findMultipleOfFives(std::vector<int> const& all_values)
{
  return std::none_of(all_values.cbegin(), 
                      all_values.cend(), 
                      [](int v) { return v % 5; });
}

正如@RemyLebeau 指出的那样，将其写成all_of 可能更容易阅读：

bool findMultipleOfFives(std::vector<int> const& all_values)
{
  return std::all_of(all_values.cbegin(), 
                     all_values.cend(), 
                     [](int v) { return v % 5 == 0; });
}