1

Fazendo uma pesquisa encontrei algo relacionado a este erro. Tenho o seguinte erro abaixo:

ERRO: date.cxx: In member function ‘int CurrentDateTime::GetMonth(const string&)’: date.cxx:125:18: error: cannot convert ‘const Month’ to ‘int’ in return

relacionado a:

int CurrentDateTime::GetMonth(const std::string& input)
{
    for(const auto& itr : monthinfo) 
    {
      if(itr.first == input)
      return static_cast<std::vector<int>itr.second>;
    }
}

onde montinfo vem de:

enum class Month
{
  Jan = 1, Feb, Mar, Apr, May, Jun, Jul, Aug, Sep, Oct, Nov, Dec
};

// Datastructure for string to num conversion in month(.i.e."Mar" Month to 3)
std::array<std::pair<std::string, Month>, 12> monthinfo = 
{
    std::make_pair(std::string("Jan"), Month::Jan),
    std::make_pair(std::string("Feb"), Month::Feb),
    std::make_pair(std::string("Mar"), Month::Mar),
    std::make_pair(std::string("Apr"), Month::Apr),
    std::make_pair(std::string("May"), Month::May),
    std::make_pair(std::string("Jun"), Month::Jun),
    std::make_pair(std::string("Jul"), Month::Jul),
    std::make_pair(std::string("Aug"), Month::Aug),
    std::make_pair(std::string("Sep"), Month::Sep),
    std::make_pair(std::string("Oct"), Month::Oct),
    std::make_pair(std::string("Nov"), Month::Nov),
    std::make_pair(std::string("Dec"), Month::Dec)
};

Li que enum class não converte implicitamente para int por causa de ser proposital, era preciso usar um static_cast no itr.second.

Eu tentei fazer algo assim seria realmente isso?

int CurrentDateTime::GetMonth(const std::string& input)
{
    for(const auto& itr : monthinfo) 
    {
      if(itr.first == input)
      return static_cast<std::vector<int>itr.second>;
    }
}
7
  • Algum motivo pra fazer std::string("Jan")? O que é isto std::vector<int>? De onde surgiu este .second? Acho que tem formas melhores de fazer toda esta estrutura, tem estruturas melhores para fazer desta forma também.
    – Maniero
    Commented 13/09/2017 às 18:18
  • encontrei este código entao resolvi testa-lo, coloquei no pastebin achei o mesmo quando estava pesquisando sobre algumas formas de mecher com vetores de datas e mas ele me retorna este erro e um rapas me disse que eu deveria fazer static_cast para converter enun class em int, mas nem tenho idéia de como fazer isso realmente pastebin.com/MhF0eC71
    – user46920
    Commented 13/09/2017 às 21:35
  • @bigown Qual o problema com o std::string("Jan")? Ele está construindo uma string diretamente no construtor do std::pair<std::string, Month>. Sendo assim, o .second se refere exatamente ao segundo termo deste pair, que neste caso é o Month. Essa estrutura é perfeitamente válida e muito usada em c++. Tanto é que no c++17 é o padrão para o que eles chamam de structured binding, mas nesse caso, eles usam o tuple que é uma generalização do pair
    – user5299
    Commented 13/09/2017 às 22:39
  • @Amadeus não conhecia isto. Este código é ruim demais, eu jogaria fora e começaria um novo :)
    – Maniero
    Commented 13/09/2017 às 22:47
  • @bigown Não sei dizer se o código é bom ou não, mas as estruturas usadas são muito boas e é o jeito idiomático de se fazer
    – user5299
    Commented 13/09/2017 às 22:56

1 Resposta 1

0

Bom, respondendo diretamente a sua pergunta, o modo correto de usar o static_cast, neste caso, é substituir:

return static_cast<std::vector<int>itr.second>;

por:

return static_cast<int>(itr.second);

pois você está convertendo um item do enum class Month para um inteiro.

Agora, sinceramente, não acho que tenha sentido fazer isso. O enum class foi criado exatamente para evitar essas conversões não desejadas. O correto é que na função GetMonth ela retorne um enum class Month. Ou em outras palavras, que ela tenha a seguinte assinatura:

Month GetMonth(const std::string&) const;

Agora, se deseja realmente uma conversão para inteiro, use o enum padrão (sem o class)

10
  • entao eu peguei este código mas tambem nao entendi direito pq o cara que o desenvolveu quis converter para int, pra min o que for mais logico e correto eu só quis saber como consertar o erro dentro deste tipo de conversao..
    – user46920
    Commented 14/09/2017 às 0:25
  • Eu sinceramente nao entendi por que razao ele converte no codigo para int, mas adicionei return static_cast<int>(itr.second); da forma como vc disse e gerou este erro abaixo: /tmp/ccWCXO05.o: In function CurrentDateTime::CurrentDateTime()': date.cxx:(.text+0xb8): undefined reference to vtable for CurrentDateTime' /tmp/ccWCXO05.o: In function main': date.cxx:(.text+0x8ea): undefined reference to CurrentDateTime::~CurrentDateTime()' date.cxx:(.text+0x90e): undefined reference to `CurrentDateTime::~CurrentDateTime()' collect2: error: ld returned 1 exit status
    – user46920
    Commented 14/09/2017 às 0:36
  • Aparentemente o autor tinha o código já feito para uma versão mais antiga do c++ e quis atualiza-la para uma versão mais atual, sem testar o código. Esse erro em particular se refere a ausência da definição do destrutor. Basta defini-lo, por exemplo colocando {}
    – user5299
    Commented 14/09/2017 às 0:42
  • agora correu bem mas de acordo com o autor original do codigo o programa tem que gerar o seguinte resultado abaixo no dia atual da semana: Output Current Day:Tue Current Date:4 Current Month:11 Current Year:2014 Current Hour:10 Current Min:43 Current Second:35 ###################################### portanto ele gerou este: bash-4.4$ ./date Current Day....: Current Date...: 0 Current Month..: 6556000 Current Year...: 0 Current Hour...: 0 Current Min....: 0 Current Second.: 0
    – user46920
    Commented 14/09/2017 às 0:50
  • Bom, agora é com o autor original do programa. A sua questão era sobre como usar o static_cast de forma correta nesse caso. Se o programa está funcionando corretamente ou não, é outro problema
    – user5299
    Commented 14/09/2017 às 0:52

Você deve fazer log-in para responder a esta pergunta.