使用push_back()在向量中添加项,向量的大小增加但无法从向量中读取值

时间:2013-11-19 14:49:04

标签: c++ vector

我有一个ColorName类和一个ColorNameLookup类。在类ColorNameLookup中,有一个公共成员:vector colorList。 ColorNameLookup类的构造函数是将项添加到vector colorList中。

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

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

我的代码如下所示:

Class 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;
}

Class 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;
  }

1 个答案:

答案 0 :(得分:0)

你需要这个:

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)
{
}