我有这个问题,我正在努力解决。基本上,基类具有函数map,它将一个向量作为输入,并在一些映射函数之后输出最终向量,在本例中为-f,已执行。但是,我真的迷失了为什么当我在主函数中打印2 * testVector-test1时,我得到正确的输出,即6,-182等......但是当我打印出2 * testVector时 - 测试2,它仍然是相同的向量。
当我两次创建“DoubleElements”或者只调用两次相同的“DoubleElements”指针(它只执行1个映射)时,会发生这种情况。我从根本上缺少一些理解吗?任何帮助表示赞赏!
#include <iostream>
#include <vector>
using namespace std;
class RecursiveBase {
public:
vector<int> map(vector<int> baseVector) {
static int iter = 0;
// Base case, return the final vector.
if (iter == 5) {
return baseVector;
// Replace the element with the old element mapped to the function.
} else {
baseVector[iter] = this->f(baseVector[iter]);
iter++;
return map(baseVector);
}
}
private:
virtual int f(int value) = 0;
};
class DoubleElements: public RecursiveBase {
private:
int f(int value) {
return 3*value;
}
};
int main() {
vector<int> testVector, o1, o2;
testVector.push_back(3);
testVector.push_back(-91);
testVector.push_back(-42);
testVector.push_back(-16);
testVector.push_back(13);
DoubleElements de;
DoubleElements de1;
RecursiveBase *test1 = &de;
RecursiveBase *test2 = &de1;
o1 = test1->map(testVector);
o2 = test2->map(testVector);
std::cout << "2*testVector - test1" << std::endl;
for (unsigned int iter = 0; iter < o1.size(); iter++) {
std::cout << o1[iter] << std::endl;
}
std::cout << "2*testVector - test2" << std::endl;
for (unsigned int iter = 0; iter < o2.size(); iter++) {
std::cout << o2[iter] << std::endl;
}
}
答案 0 :(得分:1)
static int iter = 0;
除非100%必要,否则应避免在方法中声明局部静态变量。
第一个电话会将iter
增加到5,但在下一个电话iter
,因为它是静态的,不会将其重置为0。
例如,一个简单的程序,如:
void test()
{
static int x = 0;
++x;
cout << x << endl;
}
int main()
{
test();
test();
return 0;
}
将输出
1
2
答案 1 :(得分:1)
静态数据成员不是类的子对象的一部分。
Exception
是iter
。它是static
个对象的class RecursiveBase
NOT 部分。
要解决此问题,请将RecursiveBase
重置为iter
:
0
<强>输出强>
if (iter == 5) {
iter = 0; // reset iter
return baseVector;
}
答案 2 :(得分:1)
您只能按原样拨打RecursiveBase::map
一次,因为iter
是静态的。您还假设您只使用5个元素std::vector<int>
来调用它,此时std::array<int, 5>
是更好的选择。
如果您需要递归解决方案,请将索引作为附加参数传递
public:
std::vector<int> map(std::vector<int> vec) {
return do_map(vec, 0);
}
private:
std::vector<int> do_map(std::vector<int> & vec, std::size_t index) {
if (index == vec.size()) { return vec; }
vec[index] = f(vec[index]);
return do_map(vec, ++index);
}
但这仍然是对递归的无偿使用。更好的解决方案是
public:
std::vector<int> map(std::vector<int> vec) {
std::transform(vec.begin(), vec.end(), vec.begin(), [this](int i) { return f(i); });
return vec;
}
您的主
中也有多余的RecursiveBase *
int main() {
std::vector<int> testVector{3, -91, -42, -16, 13};
DoubleElements de;
DoubleElements de1;
// declare at point of initialisation
// don't need ->
auto o1 = de.map(testVector);
auto o2 = de1.map(testVector);
std::cout << "2*testVector - test1" << std::endl;
for (unsigned int iter = 0; iter < o1.size(); iter++) {
std::cout << o1[iter] << std::endl;
}
std::cout << "2*testVector - test2" << std::endl;
for (unsigned int iter = 0; iter < o2.size(); iter++) {
std::cout << o2[iter] << std::endl;
}
return 0;
}