Использование необъявленного идентификатора в классе
Я строю класс, где конструктор для класса принимает строку, представляющую дату. Конструктор должен назначить месяц, день и год соответствующим членам класса.
До сих пор я написал что-то довольно простое, которое предполагает только несколько типов форматов даты.
Моя проблема в том, что я хотел бы использовать строку, которая используется для аргумента конструктора. Я хочу использовать строку в теле класса, но когда я ее использую, я получаю необъявленную ошибку идентификатора, где бы она ни использовалась.
Как я могу предотвратить это?
Код класса:
#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)) {}
РЕДАКТИРОВАТЬ::
Я отредактировал свой код, используя предложение поместить все инициализации в конструктор. Я думаю, что это по сути то же самое, что и другие методы, которые были предложены.
Код класса:
#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)
Решение
переменная 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_s может все еще не быть инициализирован, когда это используется в indyear инициализатор. Так что, возможно, это не поможет вам, но это хорошее начало.
Другие решения
Вам нужен идентификатор класса:
inline chrono::chrono(std::string s) : year(s.substr(indyear,4)), month(tolower(s).substr(indmonth,lengthmonth)), day(s.substr(inddate,1)) {}
«Я хотел бы использовать строку, которая используется для аргумента конструктора. Я хочу использовать строку в теле класса»
В вашем конструкторе s это временная копия с автоматическим хранением, которая существует только в рамках этого конструктора. Чтобы решить эту проблему, вы можете определить новый член этого класса и инициализировать его, используя параметр, который был передан в конструктор. Также рассмотрите возможность использования #define (или же public static const members) для строк, которые никогда не изменятся во время выполнения:
#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;
}
Также обратите внимание, что вы определили составные элементы year, month а также day, но вы инициализируете их как std::string объекты.