使用push_back（）在向量中添加项目，向量的大小会增加，但无法从向量中读取值

Question

我有一个类ColorName和一个类ColorNameLookup。 在ColorNameLookup类中，有一个公共成员：矢量colorList。 ColorNameLookup类的构造函数是将项目添加到向量colorList中。

我将140个项目添加到向量colorList中。 我检查了向量的大小是否正确，但是我无法从向量中读取任何值。

例如，我初始化变量ColorNameLookup findColor ，并使用函数findColor.colorList[0].getR()获取第一个元素的R值。 返回值是未初始化的整数值。

我的代码如下所示：

类ColorName：

#include "ColorName.h"
ColorName::ColorName(std::string name, int r, int g, int b)
{
    r = r;
    g = g;
    b = b;
    name = name;
}
ColorName::ColorName(void)
{
}
ColorName::~ColorName(void)
{
}

int ColorName::computeMSE(int pixR, int pixG, int pixB) {
      return ((pixR-r)*(pixR-r) + (pixG-g)*(pixG-g) + (pixB-b)*(pixB-b)/3);
}

int ColorName::getR() {
      return r;
}

int ColorName::getG() {
      return g;
}

int ColorName::getB() {
      return b;
}

std::string ColorName::getName() {
      return name;
}

类ColorNameLookup：

ColorNameLookup::ColorNameLookup(void)
{
    colorList.push_back(ColorName("AliceBlue",0xF0,0xF8,0xFF));
    //...139 push_back
    colorList.push_back(ColorName("YellowGreen",0x9A,0xCD,0x32));
}


ColorNameLookup::~ColorNameLookup(void)
{
}

std::string ColorNameLookup::getColorName(int r, int g, int b) {
    ColorName closestMatch; // = NULL;
    int findFlag = 0;
    int minMSE = std::numeric_limits<int>::max();
    int mse;
    for (ColorName c : colorList) {
      mse = c.computeMSE(r, g, b);
      if (mse < minMSE) {
          findFlag = 1;
        minMSE = mse;
        closestMatch = c;
        //printf("Find color!\n");
      }
    }
    if (findFlag ==1){
        return closestMatch.getName();
    }
    else
        return NULL;
  }

Answer 1

您需要这个：

ColorName::ColorName(std::string name, int r, int g, int b)
{
    this->r = r;
    this->g = g;
    this->b = b;
    this->name = name;
}

r = r只会为其自身分配相同的局部r ，而成员变量未初始化。

为了初始化这些成员，最好使用一个初始化列表：

ColorName::ColorName(std::string name, int r, int g, int b) : 
    r(r),
    g(g),
    b(b),
    name(name)
{
}