[C] copia stringa e caratteri estranei

[deleted_129]

Il programma qui sotto dovrebbe prendere da input una stringa...copiare la stringa in un'altro array(questo lo fa con una funzione che usa i puntatori) e stampare l'array nuovo....
CODICE:

#include <stdio.h>
void cambia(char* primo,char* secondo){
     while(*primo){
             *secondo = *primo;
             primo++;
             secondo++;
     }
     *secondo='\0';
}
int main(){
char stringa1[50],stringa2[50];
int x;
for(x=0;stringa1[x-1]!= '\n';x++){
         scanf("%c",&stringa1[x]);
}
stringa1[x+1]='\0';
cambia(stringa1,stringa2);
for(x=0; stringa2[x]!='\0';x++){
         printf("%c",stringa2[x]);
}
scanf("%d");
return 0;
}

il programma funziona ...solo che in alcuni casi nell'output sotto la stringa mi appare qualche simbolo strano...non capisco il perche'....

[c0der]

È assolutamente terribile questo programma (terribile come svolto). Comincia per dire a studiare la scanf, per esempio da qui: http://www.cplusplus.com/reference/cstdio/scanf/ (esempio al fondo della pagina).

[deleted_129]

ho risolto grazie ....lo so che e' terribile ma l'esercizio era questo...usare quella funzione e usare i puntatori....

[deleted_129]

cosa dovrei studiare esattamente? non ho usato nello scanf %s perche' altrimenti non leggeva gli spazi....

[c0der]

Quel ciclo for con la scanf dentro è totalmente senza senso. Prova a metterci una printf dentro per vedere quante volte viene chiamato...

[deleted_129]

viene richiamato tante volte quanti i caratteri immessi..e' giusto, cosa c'e' che non va?

[c0der]

viene richiamato tante volte quanti i caratteri immessi..e' giusto,

Ma neanche... aggiungi questa riga subito dopo la tua scanf o subito prima e poi vedi.
printf("%d\n", x);

Tu inserisci tutta la stringa in una volta... scusa adesso riguardo.

[c0der]

Non ho mai visto fare una cosa del genere, comunque non va bene.
Se inserisci una stringa superiore a 50 caratteri cosa succede?

P.S.
Tu volevi tentare di fare una getline... cioè di prendere una intera riga con gli spazi.
Ma l'esercizio non mi pare punti a questo, semmai alla funzione cambia, sostituisci il for con:
scanf("%50s", stringa1);

[deleted_129]

l'esercizio era riuscire a farlo...ecco xk gli array sono di 50 elementi...potevano anche essere di 500..ma non cambiava....cmq ho aggiunto il printf nel ciclo e mi da' il giusto numero di caratteri...

[deleted_129]

#include <stdio.h>
/*programma che copia e stampa un array...*/
void cambia(char* primo,char* secondo){
     while(*primo){
             *secondo = *primo;
             primo++;
             secondo++;
     }
     *secondo='\0';
     return;
}
int main(){
char stringa1[50],stringa2[50];
int x;
for(x=0;stringa1[x-1]!= '\n';x++){
         scanf("%c",&stringa1[x]);
}
stringa1[x]='\0';
cambia(stringa1,stringa2);
for(x=0; stringa2[x]!='\0';x++){
         printf("%c",stringa2[x]);
}
scanf("%d");
return 0;
}

cosi' e' giusto...funziona...magari e' un po' pasticciato ma io non avrei nemmeno usato i puntatori xD e avrei usato strcpy ahah xD pero' va bene dai