Использование необъявленного идентификатора в классе

Я создаю класс, в котором конструктор класса принимает строку, представляющую дату. Конструктор должен присвоить месяц, день и год соответствующим элементам данных класса.

До сих пор я написал что-то довольно простое, что предполагает только несколько типов форматов даты.

Моя проблема в том, что я хотел бы использовать строку, которая используется для аргумента конструктора. Я хочу использовать строку в теле класса, но когда я ее использую, я получаю необъявленную ошибку идентификатора, где бы она ни использовалась.

Как я могу предотвратить это?

Код класса:

#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)

person Mars    schedule 06.09.2013    source источник
comment
Что заставляет вас думать, что год, месяц и день должны быть инициализированы в конструкторе, но другие члены могут быть инициализированы так, как вы это делаете? Почему бы вам не инициализировать все остальное в конструкторе?   -  person undu    schedule 06.09.2013


Ответы (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_s все еще может не инициализироваться, когда он используется в инициализаторе indyear. Так что это, возможно, не приведет вас к цели, но это хорошее начало.

person abelenky    schedule 06.09.2013

Вам нужен идентификатор класса:

inline chrono::chrono(std::string s) : year(s.substr(indyear,4)), month(tolower(s).substr(indmonth,lengthmonth)), day(s.substr(inddate,1)) {}
person cpp    schedule 06.09.2013
comment
Спасибо забыл об этом. - person Mars; 06.09.2013

"Я хочу использовать строку, которая используется в качестве аргумента конструктора. Я хочу использовать строку в теле класса"

В вашем конструкторе 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; 
}

Также обратите внимание, что вы определили интегральные элементы year, month и day, но инициализировали их как объекты std::string.

person LihO    schedule 06.09.2013
comment
Разве это не сделало бы вызов конструктора неоднозначным, поскольку у меня уже есть конструктор, который принимает только строку. - person Mars; 06.09.2013
comment
@Comrade: Я имел в виду расширение существующего конструктора. Проверьте мой ответ сейчас. - person LihO; 06.09.2013
comment
@Comrade: я добавил дополнительную информацию, которая может подойти для ваших нужд;) - person LihO; 06.09.2013
comment
Спасибо присматриваюсь. - person Mars; 06.09.2013
comment
Да, это на самом деле из упражнения и указывает, какого типа должны быть члены. Если бы не такое ограничение, я бы сделал это совсем по-другому. - person Mars; 06.09.2013