C Exercício de função em C

[Victor Vilardo]

então eu tenho que fazer esse exercicio, mas ela so ta lendo a minha string, ela nao ta imprimindo. o que tem de errado no meu codigo?
lembrando q nao posso utilizar a função strcopy

 

#include <stdio.h>

//Faça um programa que leia uma string de até 50 caracteres e faça uma cópia para outra string. 
//Para isso, o programa deve implementar um procedimento que execute essa operação, 
//recebendo por parâmetro a string de origem e a de destino. Ao final, o programa deve exibir as strings na tela.


//função

void copiarString (char *original, char *copia){
    
    while (*original != '\0'){
        *original = *copia;
        
            original++;
            copia++;
    }
    
    *original = '\0';
}

int main (){
    
    

    char str1 [50];
    char str2 [50];
    printf ("Informe a string: ");
    gets(str1);
    copiarString (str1, str2);
    printf ("%s", str2);
    
    
    
    }

 

[Midori]

@Victor Vilardo  Você inverteu a atribuição, corrija para,

 

*copia = *original;
...
*copia = '\0';

Obs: não é recomendável usar gets, prefira funções mais seguras como fgets para limitar a quantidade de caracteres.

[Victor Vilardo]

@Midori funcionou!!

 

mas em relação a usar o fgets, aqui no devcc++ ele da erro, ele rodou so com o gets

é problema do compilador? 

[Midori]

@Victor Vilardo Tentou assim?

 

fgets(str1,50,stdin);

 

[Victor Vilardo]

1 minuto atrás, Midori disse:

@Victor Vilardo Tentou assim?

 

fgets(str1,50,stdin);

 

aii funcionou de novo, nao sabia que tinha que deixar os parametros dessa forma

desculpa

mas obrigado!!

[Flávio Pedroza]

Não sei se notou, mas sua função copiaString está invertida (você trocou a origem e o destino). O correto seria:

void copiarString (char *original, char *copia){
    
    while (*original != '\0'){
        *copia = *original;
        
            original++;
            copia++;
    }
    
    *copia = '\0';
}