在类中使用未声明的标识符

时间:2013-09-06 15:41:41

标签: c++ constructor initialization lifetime class-members

我正在构建一个类,其中类的构造函数采用表示日期的字符串。构造函数应将月,日和年分配给类的相应数据成员。

到目前为止,我已经编写了一些非常基本的内容,它只假设几种类型的日期格式。

我的问题是我想使用用于构造函数参数的字符串。我想在类的主体中使用字符串,但是当我使用它时,无论在何处使用它都会得到未声明的标识符错误。

我该如何防止这种情况?

班级代码:

#ifndef CHRONO_H
#define CHRONO_H
#include <iostream>
#include <string>
class chrono {
public:
    inline chrono(std::string s);
    unsigned year;
    unsigned month;
    unsigned day;
    std::string numyear{"0123456789"};
    std::string alph{"abcdefghijklmnopqrstuvwxyz"};
    std::string punc{",/"};
    std::string::size_type indyear = s.find_first_of(punc);
    std::string::size_type indmonth = s.find_first_of(alph);
    std::string::size_type indmonthend = s.find_last_of(alph);
    std::string::size_type lengthmonth = indmonthend - indmonth;
    std::string::size_type inddate = s.find_first_of(numyear);
    std::string::iterator begin = s.begin();
    std::string::iterator end = s.end();
};
#endif

构造函数代码:

#include <iostream>
#include <string>
#include "chrono.h"
inline chrono(std::string s) : year(s.substr(indyear,4)), month(tolower(s).substr(indmonth,lengthmonth)), day(s.substr(inddate,1)) {}

EDIT ::

我使用建议编辑了我的代码,将所有初始化放在构造函数中。我认为这与提出的其他方法基本相同。

班级代码:

#ifndef CHRONO_H
#define CHRONO_H
#include <iostream>
#include <string>
class chrono;
class chrono {
public:
    inline chrono(std::string s);
    std::string numyear{"0123456789"};
    std::string alph{"abcdefghijklmnopqrstuvwxyz"};
    std::string punc{",/"};
    std::string::size_type indyear, indmonth, indmonthend, lengthmonth, inddate;
    std::string::iterator begin, end;
    unsigned year;
    unsigned month;
    unsigned day;
};
#endif

构造函数代码:

#include <iostream>
#include <string>
#include "chrono.h"
inline chrono::chrono(std::string s) : indyear(s.find_first_of(punc)), indmonth(s.find_first_of(alph)), indmonthend(s.find_last_of(alph)), lengthmonth(indmonthend - indmonth), inddate(s.find_first_of(numyear)), begin(s.begin()), end(s.end()), year(stoi(s.substr(indyear,4))), month(stoi(s.substr(indmonth,lengthmonth))), day(stoi(s.substr(inddate,1))) {}

主:

#include <iostream>
#include <string>
#include "chrono.h"
int main()
{
    std::string st;
    std::cout << "Enter a date" << std::endl;
    std::cin >> st;
    chrono today(st);
    std::cout << "Month " << today.month << std::endl;
    std::cout << "Day " << today.day << std::endl;
    std::cout << "Year " << today.year << std::endl;
    return 0;
}

我收到以下错误:

Undefined symbols for architecture x86_64:
  "chrono::chrono(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >)", referenced from:
      _main in ex9_51-JhoQAx.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

3 个答案:

答案 0 :(得分:3)

变量s是构造函数的参数,在该构造函数中只有范围和生命周期。

然后,您尝试在构造函数外部访问它,如:

std::string::size_type indyear = s.find_first_of(punc);

如果你想保留s,你需要将它存储在一个成员变量中:

class chrono {
public:
    inline chrono(std::string s);
    unsigned year;
    unsigned month;
    unsigned day;
    std::string  member_s;   // Here's the member-variable.
};

inline chrono(std::string s) :
    year(s.substr(indyear,4)),
    month(tolower(s).substr(indmonth,lengthmonth)),
    day(s.substr(inddate,1)),
    member_s(s) // Here we store the local-variable s in the member-variable
   { }

最后,您需要引用member_s而不是参数s

std::string::size_type indyear = member_s.find_first_of(punc);

注意我认为这不会解决您的所有问题,因为我认为member_sindyear初始值设定项中使用时可能仍未初始化。所以这可能不会让你一直到那里,但它是一个良好的开端。

答案 1 :(得分:1)

您需要班级标识符:

inline chrono::chrono(std::string s) : year(s.substr(indyear,4)), month(tolower(s).substr(indmonth,lengthmonth)), day(s.substr(inddate,1)) {}

答案 2 :(得分:0)

“我想使用用于构造函数参数的字符串。我想在类的主体中使用字符串”

在构造函数中,s是具有自动存储持续时间的临时副本,该副本仅存在于此构造函数的范围内。要解决此问题,您可以定义此类的新成员,并使用已传递给构造函数的参数对其进行初始化。还要考虑将#define(或public static const成员)用于在运行时永远不会更改的字符串:

#define CHRONO_ALPH "abcdefghijklmnopqrstuvwxyz"
#define CHRONO_DIGITS "0123456789"
#define CHRONO_PUNC ",/"

class chrono {
public:
    chrono(std::string s_) :
        s(s_),
        indyear(s.find_first_of(CHRONO_PUNC)),
        indmonth(s.find_first_of(CHRONO_ALPH)),
        inddate(s.find_first_of(CHRONO_DIGITS)),
        indmonthend(s.find_last_of(CHRONO_ALPH)),
        lengthmonth(indmonthend - indmonth),
        year(s.substr(indyear,4)),
        month(tolower(s).substr(indmonth,lengthmonth)),
        day(s.substr(inddate,1)) { }

    std::string s, year, month, day;
    size_t indyear, indmonth, inddate, indmonthend, lengthmonth; 
}

另请注意,您已定义了完整成员yearmonthday,但您将其初始化为std::string个对象。