Copiar String

[Luis Ferreira 0]

Olá pessoal, boa noite!

Estou com dificuldade em um exercício. Preciso solicitar uma string ao usuário e utilizando notação de ponteiros copiar essa string para outra. SEM UTILIZAR A BIBLIOTECA string.h .

[LazaroBinda 3]

Amigo, posta aí como vc já tentou fazer que eu tento te ajudar....

[Mateus GP 13]

#include <stdio.h>

 

char*

string_copy (const char* source, char* destination)

{

while(*source != '\0')

*destination++ = *source++;

*destination = '\0';

 

return destination;

}

 

int

main (int argc, char** argv)

{

char s[256];

 

string_copy("Teste", s);

puts(s);

return 0;

}

 

[LazaroBinda 3]

Tecnicamente seu código funciona. Porém sua função string_copy tem um retorno sem utilidade.

 

 

 

 

1. char* string_copy (const char* source, char* destination)
2. {
3. while(*source != '\0')
4. *destination++ = *source++;
5. *destination = '\0';
7. return destination;
8. }

 

Toda vez que vc passa um ponteiro como argumento pra alguma função, é como se vc estivesse passando este argumento por referência. Isso significa que qualquer alteração que vc fizer no argumento no escopo da função estaria na realidade alterando a variável passada como argumento. O ideal seria:

void string_copy (const char* source, char* destination)
{
    while(*source)
      *destination++ = *source++;
    *destination = '\0';
}

Somente isso já bastaria pra efetuar a cópia, isso levando em conta que a variável ponteiro passada pra destination já possui memória suficiente alocada pra efetuar a cópia.

 

Se vc quiser colocar tb a declaração da variável em notação de ponteiro, vc pode escrever assim

char *s = malloc(sizeof(char[256]));

No final, sua função main ficaria:

int main (int argc, char** argv)
{
    char *s = malloc(sizeof(char[256]));
    char *d = malloc(sizeof(char[256]));
    
    gets(s);
    string_copy(s, d);
    puts(d);
    
    free(s);
    free(d);

    return 0;
}

[Mateus GP 13]

 

Tecnicamente seu código funciona. Porém sua função string_copy tem um retorno sem utilidade.

 

 

 

Toda vez que você passa um ponteiro como argumento pra alguma função, é como se você estivesse passando este argumento por referência. Isso significa que qualquer alteração que você fizer no argumento no escopo da função estaria na realidade alterando a variável passada como argumento. O ideal seria:

void string_copy (const char* source, char* destination)
{
    while(*source)
      *destination++ = *source++;
    *destination = '\0';
}

Somente isso já bastaria pra efetuar a cópia, isso levando em conta que a variável ponteiro passada pra destination já possui memória suficiente alocada pra efetuar a cópia.

 

Se você quiser colocar tb a declaração da variável em notação de ponteiro, você pode escrever assim

char *s = malloc(sizeof(char[256]));

No final, sua função main ficaria:

int main (int argc, char** argv)
{
    char *s = malloc(sizeof(char[256]));
    char *d = malloc(sizeof(char[256]));
    
    gets(s);
    string_copy(s, d);
    puts(d);
    
    free(s);
    free(d);

    return 0;
}

 

Neste contexto, sim, entretanto, esta função faz parte de um programa maior que escrevi.

[_Isis_ 202]

Não use gets.

[LazaroBinda 3]

Isis, você fala por causa do perigo de se ultrapassar o limite do array???

 

Se for isso, você tem razão. É melhor utilizar fgets