C++, Função strcpy_s

[embreve]

Que estranho, faça o seguinte, me encaminhe o script aqui no tópico para analisarmos ele completamente, creio eu que você utilizando OGOCX LUA?

Não! Estou trabalhando com Plugin. 99.60T.

strcpy_s(this->PatentStruct[this->_Count].NamePatent, sizeof(this->PatentStruct[this->_Count].NamePatent), (Section.Rows[i].GetStringPtr(2)));

void cPatent::HandlePatentCommand(int aIndex)
{
int RankIndex = this->GetRankIndex(aIndex);


if (RankIndex == -1)
{
Function.MsgUser(aIndex, 1, "| Você não possui uma Patent.");
return;
}


CUSTOM_RANKUSER_DATA CurrentPatent = this->PatentStruct[RankIndex];


bool isMaxPatent = (CurrentPatent.Max == -1);


if (isMaxPatent)
{
Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);
Function.MsgUser(aIndex, 1, "| Você já atingiu a Patent máxima.", CurrentPatent.NamePatent);
}
else
{
CUSTOM_RANKUSER_DATA nextPatent = this->PatentStruct[RankIndex + 1];
int PointsNeeded = nextPatent.Min - Custom[aIndex].Patent;


Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);


Function.MsgUser(aIndex, 1, "| Próxima Patent: %s", nextPatent.NamePatent);


Function.MsgUser(aIndex, 1, "| Pontos Necessários: %d.", PointsNeeded);
}
}

[Ravoc]

Agora, quando você processar uma string como "Diamante I", ela será copiada inteira, incluindo o espaço e o "I". Além disso, se algo der errado com a string de origem, o código não quebra e coloca um valor padrão.

const char* sourceString = Section.Rows[i].GetStringPtr(2);

if (sourceString != nullptr && strlen(sourceString) > 0) {
    strncpy_s(this->PatentStruct[this->_Count].NamePatent, 
              sizeof(this->PatentStruct[this->_Count].NamePatent), 
              sourceString, 
              _TRUNCATE);
} else {
    strcpy_s(this->PatentStruct[this->_Count].NamePatent, 
             sizeof(this->PatentStruct[this->_Count].NamePatent), 
             "Desconhecida");
}


void cPatent::HandlePatentCommand(int aIndex)
{
    int RankIndex = this->GetRankIndex(aIndex);


    if (RankIndex == -1)
    {
        Function.MsgUser(aIndex, 1, "| Você não possui uma Patent.");
        return;
    }


    CUSTOM_RANKUSER_DATA CurrentPatent = this->PatentStruct[RankIndex];


    bool isMaxPatent = (CurrentPatent.Max == -1);


    if (isMaxPatent)
    {
        Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Você já atingiu a Patent máxima.");
    }
    else
    {
        CUSTOM_RANKUSER_DATA nextPatent = this->PatentStruct[RankIndex + 1];
        int PointsNeeded = nextPatent.Min - Custom[aIndex].Patent;


        Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Próxima Patent: %s", nextPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Pontos Necessários: %d.", PointsNeeded);
    }
}

[embreve]

Agora, quando você processar uma string como "Diamante I", ela será copiada inteira, incluindo o espaço e o "I". Além disso, se algo der errado com a string de origem, o código não quebra e coloca um valor padrão.

const char* sourceString = Section.Rows[i].GetStringPtr(2);

if (sourceString != nullptr && strlen(sourceString) > 0) {
    strncpy_s(this->PatentStruct[this->_Count].NamePatent, 
              sizeof(this->PatentStruct[this->_Count].NamePatent), 
              sourceString, 
              _TRUNCATE);
} else {
    strcpy_s(this->PatentStruct[this->_Count].NamePatent, 
             sizeof(this->PatentStruct[this->_Count].NamePatent), 
             "Desconhecida");
}


void cPatent::HandlePatentCommand(int aIndex)
{
    int RankIndex = this->GetRankIndex(aIndex);


    if (RankIndex == -1)
    {
        Function.MsgUser(aIndex, 1, "| Você não possui uma Patent.");
        return;
    }


    CUSTOM_RANKUSER_DATA CurrentPatent = this->PatentStruct[RankIndex];


    bool isMaxPatent = (CurrentPatent.Max == -1);


    if (isMaxPatent)
    {
        Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Você já atingiu a Patent máxima.");
    }
    else
    {
        CUSTOM_RANKUSER_DATA nextPatent = this->PatentStruct[RankIndex + 1];
        int PointsNeeded = nextPatent.Min - Custom[aIndex].Patent;


        Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Próxima Patent: %s", nextPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Pontos Necessários: %d.", PointsNeeded);
    }
}

Muito obrigado pela sua contribuição! Assim que eu chegar do trabalho irei testar e retornar o feedback.

1 32 minutes:------- Atualizado -------

Agora, quando você processar uma string como "Diamante I", ela será copiada inteira, incluindo o espaço e o "I". Além disso, se algo der errado com a string de origem, o código não quebra e coloca um valor padrão.

const char* sourceString = Section.Rows[i].GetStringPtr(2);

if (sourceString != nullptr && strlen(sourceString) > 0) {
    strncpy_s(this->PatentStruct[this->_Count].NamePatent, 
              sizeof(this->PatentStruct[this->_Count].NamePatent), 
              sourceString, 
              _TRUNCATE);
} else {
    strcpy_s(this->PatentStruct[this->_Count].NamePatent, 
             sizeof(this->PatentStruct[this->_Count].NamePatent), 
             "Desconhecida");
}


void cPatent::HandlePatentCommand(int aIndex)
{
    int RankIndex = this->GetRankIndex(aIndex);


    if (RankIndex == -1)
    {
        Function.MsgUser(aIndex, 1, "| Você não possui uma Patent.");
        return;
    }


    CUSTOM_RANKUSER_DATA CurrentPatent = this->PatentStruct[RankIndex];


    bool isMaxPatent = (CurrentPatent.Max == -1);


    if (isMaxPatent)
    {
        Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Você já atingiu a Patent máxima.");
    }
    else
    {
        CUSTOM_RANKUSER_DATA nextPatent = this->PatentStruct[RankIndex + 1];
        int PointsNeeded = nextPatent.Min - Custom[aIndex].Patent;


        Function.MsgUser(aIndex, 1, "| Patent Atual: %s", CurrentPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Próxima Patent: %s", nextPatent.NamePatent);
        Function.MsgUser(aIndex, 1, "| Pontos Necessários: %d.", PointsNeeded);
    }
}

Ravoc, o problema persiste! Que estranho... Mais eu te agradeço pelo seu tempo de ensino! Vou continuar quebrando a cabeça... Preciso corrigir isso.

[Ravoc]

Vish, consegue preparar um log mais detalhado?

[embreve]

Vish, consegue preparar um log mais detalhado?

Boa tarde, Ravoc.
Estou apenas me aventurando em C++, como eu faria esse Log para lhe enviar?

[Ravoc]

Boa tarde, Ravoc.
Estou apenas me aventurando em C++, como eu faria esse Log para lhe enviar?

Poste algum print onde o erro é executado, pois é estranho com a solução acima era pra ter anulado os erros, caso houvesse erro ele jogava um valor padrão para testes

[embreve]

Poste algum print onde o erro é executado, pois é estranho com a solução acima era pra ter anulado os erros, caso houvesse erro ele jogava um valor padrão para testes

Boa tarde, Ravoc.
Meu código não apresenta erro algum. Apenas não copia o espaço e o restante da string.

Abraço.