C++, Função strcpy_s

Boa noite a todos.
Para os mais experientes , poderia me tirar uma duvida?
Estou copiando do meu arquivo .txt uma String com o nome Diamante I com a função str_cpy_s, mais ela só está copiando o primeiro nome que seria Diamante.
Está ignorando o espaço e o restante da String.

Alguém pra me ajudar a corrigir?
Muito obrigado.

poste um trecho do código onde você está utilizando strcpy_s para que possamos analisar melhor.

Citação:

---

Originally Posted by Ravoc

poste um trecho do código onde você está utilizando strcpy_s para que possamos analisar melhor.

---

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

Citação:

---

Originally Posted by embreve

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

---

NamePatente - procure por isto, nele vai estar declarado qual tipo de variavel ela está (muito provavelmente seja do tipo char, veja qual o tamanho desse array imposto nela: quando falo tamanho é: char NamePatent[50] (50 seria seu tamanho)]

Citação:

---

Originally Posted by Denis Alves

NamePatente - procure por isto, nele vai estar declarado qual tipo de variavel ela está (muito provavelmente seja do tipo char, veja qual o tamanho desse array imposto nela: quando falo tamanho é: char NamePatent[50] (50 seria seu tamanho)]

---

O tamanho está ok! O problema é na função mesmo. Como eu disse: a função não copia o espaço em branco e restante do nome. Fiz o teste usando _ ex: Diamante_I copiou todo o nome. Usando Diamant I com espaço, a função só puxa o primeiro nome o espaço e o I são ignorado. Espero que conseguido explicar.

Tente usar uma função alternativa de cópia, como strncpy_s, para ver se o comportamento muda

const char* tempStr = Section.Rows[i].GetStringPtr(2);size_t length = strlen(tempStr);

strncpy_s(this->PatentStruct[this->_Count].NamePatente, sizeof(this->PatentStruct[this->_Count].NamePatente), tempStr, _TRUNCATE);

Citação:

---

Originally Posted by Ravoc

Tente usar uma função alternativa de cópia, como strncpy_s, para ver se o comportamento muda

const char* tempStr = Section.Rows[i].GetStringPtr(2);size_t length = strlen(tempStr);

strncpy_s(this->PatentStruct[this->_Count].NamePatente, sizeof(this->PatentStruct[this->_Count].NamePatente), tempStr, _TRUNCATE);

---

Obrigado, Ravoc. Irei testar.

Olá... Primeiro que essa função não é do C++, ela é de C e é uma alternativa segura a strcpy. Talvez se vc usar strcpy resolva o seu problema, mas existem outros métodos e funções melhores para fazer isso e evitar problemas de vazamento de memória.

Citação:

---

Originally Posted by Romário

Olá... Primeiro que essa função não é do C++, ela é de C e é uma alternativa segura a strcpy. Talvez se vc usar strcpy resolva o seu problema, mas existem outros métodos e funções melhores para fazer isso e evitar problemas de vazamento de memória.

---

Sim, é do C. Preciso passar três parâmetros para a função, no caso essa aceita apenas duas.
Sabe me dizer quais seriam essas funções melhores para que eu possa corrigir o meu problema? Se possível hoje irei tentar resolver utilizando a ajuda do amigo, Ravoc.
Obrigado pelo seu comentário.

3 Horas 47 minutes:------- Atualizado -------

Citação:

---

Originally Posted by Ravoc

Tente usar uma função alternativa de cópia, como strncpy_s, para ver se o comportamento muda

const char* tempStr = Section.Rows[i].GetStringPtr(2);size_t length = strlen(tempStr);

strncpy_s(this->PatentStruct[this->_Count].NamePatente, sizeof(this->PatentStruct[this->_Count].NamePatente), tempStr, _TRUNCATE);

---

Infelizmente testei aqui, e continua com o mesmo comportamento.

4 minutes:------- Atualizado -------

Citação:

---

Originally Posted by Romário

Olá... Primeiro que essa função não é do C++, ela é de C e é uma alternativa segura a strcpy. Talvez se vc usar strcpy resolva o seu problema, mas existem outros métodos e funções melhores para fazer isso e evitar problemas de vazamento de memória.

---

Utilizei a função strcpy e também não funcionou.

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?

Citação:

---

Originally Posted by Ravoc

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);
}
}

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.

Código:

---

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);
    }
}

---

Citação:

---

Originally Posted by 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.

Código:

---

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

Citação:

---

Originally Posted by 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.

Código:

---

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.

Vish, consegue preparar um log mais detalhado?

Citação:

---

Originally Posted by Ravoc

Vish, consegue preparar um log mais detalhado?

---

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

Citação:

---

Originally Posted by embreve

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

Citação:

---

Originally Posted by Ravoc

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.