Stringhe in C

[vanki]

Salve ragazzi,
il mio codice c deve convertire un array di interi in un'unica stringa. Il risultato ottenuto però non mi sembra affatto corretto!! Dove sbaglio?

Output: Vettore s di t componenti in versione stringa= �#%1010

 int t=4;

 int s[t];

 int dim=128;

 char *s_dest;
 s_dest = (char *) malloc((t+1)*sizeof(char));
 char *s_orig;
 s_orig = (char *) malloc(t*sizeof(char));
   
 for(i=0;i<t;i++){
        int xx=rand()%2;
        s[i]=xx;
        sprintf(s_orig, "%d", s[i]);
        strcat(s_dest, s_orig);
     }

 printf("Vettore s di t componenti in versione stringa= %s \n",s_dest); 

[KrOW]

Ciao . . . Prova a inserire

s_dest[0] = 0;

prima dei ciclo for . . .

[oregon]

Non capisco il perchè dell'esistenza di tutti questi vettori (alcuni statici, alcuni dinamici ...)

s
s_orig
s_dest


???

[c_junior]

devi inizializzare la stringa s_dest perché la strcat cerca il carattere '\0' e se non è inizializzata non lo trova quindi hai quel risultato

[MrX87]

ma a quanto ho capito io, ti serve convertire un vettore di interi in una stringa in modo da stampare l'intero buffer con il %s; esempio:
vettore v = 11010
stringa s = 11010
ovviamente una volta che riempi la stringa devi ricordare di mettere il carattere terminatore...altrimenti non stamperai correttamente!
io ho provato a scrivere 2 righe di codice, vedi un pò se è quello che volevi:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define T 4

int main()
{

int s[T];
int i;

char *s_dest;
s_dest = (char *) malloc((T+1)*sizeof(char));
char *s_orig;
s_orig = (char *) malloc(t*sizeof(char));

for(i=0;i<T;i++){
        int xx=rand()%2;
        s[i]=xx;
        sprintf (s_dest+i,"%d\n", s[i]);
     }
     s_dest[T+1]='\0';
printf("Vettore s di t componenti in versione stringa= %s \n",s_dest);
return 1;
}

PS: invece di usare int t=4 è preferibile usare una define dichiarando una costante fuori dal main!

[oregon]

La questione è ... il vettore numerico serve? O serve soltanto poter generare una stringa di 0 e 1 di una determinata lunghezza ?

In quest'ultimo caso, la maggior parte del codice non serve ... intendo

#define T 5

int main()
{
    int i;
    char s_dest[T] = {0};

    for(i=0; i<T-1; i++)
        s_dest[i] = '0' + (rand()%2);

    printf("Stringa = %s \n", s_dest);

    return 0;
}

[MrX87]

La questione è ... il vettore numerico serve? O serve soltanto poter generare una stringa di 0 e 1 di una determinata lunghezza ?

In quest'ultimo caso, la maggior parte del codice non serve ...

bhè si, oregon ha ragione, io ho modificato poco di quel codice, ma volendo, si potrebbe minimizzare ancora molto portandolo a poche righe:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define t 4

int main()
{

int i;
char *s_dest;
s_dest = (char *) malloc((t+1)*sizeof(char));

for(i=0;i<t;i++)
        sprintf (s_dest+i,"%d\n", rand()%2);

s_dest[t+1]='\0';
printf("Vettore s di t componenti in versione stringa= %s \n",s_dest);
return 1;
}

inoltre non ha neanche molto senso ricorrere alla allocazione dinamica di s_dest in questo caso poichè basta fare:

char s_dest[t+1];

Spero che così vada meglio...se c'è qualcosa che nn va...segnalate!

[vanki]

Ho risolto grazie ai vostri contributi!
Grazie!!!

[oregon]

Originariamente inviato da vanki
Ho risolto grazie ai vostri contributi!
Grazie!!!

Sì ma come?

[vanki]

Il problema consisteva nella mancata inizializzazione della stringa s_dest.
La soluzione suggeritami inizialmente da oregon, ma ancor di più la stessa ripresa con qualche modifica da MrX87 andava benissimo!

Ciao