更新存儲在映射向量的向量

Question

我在a.h一個以下設計：

class A { 
    virtual void updateCoefficients(std::string /*state*/, std::vector<std::vector<double>>& /*coefs*/, 
            double /*reward*/) {} 
protected: 
    std::map<std::string, std::vector<std::vector<double>>> heuristic; 
} 

class B : public A { 
    virtual void updateCoefficients(std::string state, std::vector<std::vector<double>>& coefs, 
            double reward) override; 
} 

和在a.cc：

void B::updateCoefficients(std::string state, std::vector<std::vector<double>>& coefs, 
            double reward) { 
    for(unsigned i = 0; i < coefs.size(); i++) { 
     coefs[i][0] += 1; 
    } 

    for(unsigned i = 0; i < coefs.size(); i++) { 
     coefs[i][1] += 2; 
    } 
} 

而我請從class A定義的方法updateCoefficients：

void A::foo(std::string state, std::string action, double const& reward) { 
     if (heuristic.count(action)) { 
      updateCoefficients(state, heuristic[action], reward); 
     } else { 
      std::vector<std::vector<double>> coefs(20, std::vector<double>(2, 0.0)); 
      heuristic[action] = coefs; 
    } 
} 

問題是，矢量地圖中的heuristic沒有被更新，但每當調用updateCoefficients時，所有向量都是0，就像它們初始化時一樣。我究竟做錯了什麼？

想法是讓class A作爲一個基礎，它包含所有必須使用的方法和變量，並在繼承class A的基類的子類中定義不同的updateCoefficients方法。

編輯

OK，所以整個代碼可以在這裏找到OfflineHeuristic.h和OfflineHeuristic.cc。類別State,ActionState,THTS中的信息是無關緊要的。我試過了可能的修正，但仍然，地圖heuristic不斷填充零值。從其他類中調用的方法是offlineHeuristic->learn(state, action, reward)。

編輯2

誤差實際上是在另一個地方，在代碼邏輯。上面粘貼的代碼實際上是正確的。如果有人遇到類似問題，我會留下問題。或者，如果這不是一件好事，請讓我知道，所以我刪除了這個問題。

Answer 1

啓發式不更新的原因是因爲您需要兩次調用A :: foo。首先調用它將被初始化爲零，並且只有下一次調用纔會更新它們;

這樣做可能會改變代碼將是一個解決方案：

void A::foo(std::string state, std::string action, double const& reward) { 
     if (!heuristic.count(action)) { 
      std::vector<std::vector<double>> coefs(20, std::vector<double>(2, 0.0)); 
      heuristic[action] = coefs; 
     } 
     updateCoefficients(state, heuristic[action], reward); 
} 

還有，你的代碼的其餘部分 - 數據被更新。您可以try it here

A* instance = new B(); 
instance->foo("aaa", "bbb", 123); 
cout << "First call" << endl; 

for (auto h : instance->heuristic) 
    for (auto b : h.second) 
     for (auto c : b) 
      cout << c << " "; 
cout << endl;    
instance->foo("aaa", "bbb", 123); 
cout << "Second call" << endl; 

for (auto h : instance->heuristic) 
    for (auto b : h.second) 
     for (auto c : b) 
      cout << c << " "; 
cout << endl;    

輸出：

First call 
0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 
Updated 
1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 1 2 

Answer 2

在我看來，您使用的是基類的方法，使純虛以避免錯誤。 virtual void updateCoefficients(...) = 0;

按預期工作。

// Example program 
#include <iostream> 
#include <string> 
#include <vector> 
#include <map> 

class A { 
    virtual void updateCoefficients(std::string /*state*/, std::vector<std::vector<double>>& /*coefs*/, 
            double /*reward*/) = 0; 
protected: 
    std::map<std::string, std::vector<std::vector<double>>> heuristic; 
public: 
    void foo(std::string state, std::string action, double const& reward); 
    virtual ~A() {} 

}; 

void A::foo(std::string state, std::string action, double const& reward) { 
     if (heuristic.count(action)) { 
      updateCoefficients(state, heuristic[action], reward); 
      std::cout << "exist\n"; 
      std::cout << heuristic[action][0][0] << '\n'; 
      std::cout << heuristic[action][0][1] << '\n'; 
     } else { 
      std::vector<std::vector<double>> coefs(20, std::vector<double>(2, 0.0)); 
      heuristic[action] = coefs; 
      std::cout << "not exist\n"; 
    } 
} 

class B : public A { 
    virtual void updateCoefficients(std::string state, std::vector<std::vector<double>>& coefs, 
            double reward) override; 
public: 
    virtual ~B() {} 
}; 

void B::updateCoefficients(std::string state, std::vector<std::vector<double>>& coefs, 
            double reward) { 
    for(unsigned i = 0; i < coefs.size(); i++) { 
     coefs[i][0] += 1; 
     coefs[i][1] += 2; 
    } 
} 

int main() 
{ 
    A* a = new B(); 
    a->foo("aaa", "bbb", 2.0); 
    a->foo("aaa", "bbb", 2.0); 
    a->foo("aaa", "bbb", 2.0); 
    delete a; 
    return 0; 
}