[C] problema stampa a video di una stringa

[fenix576]

ho una stringa di nome articolo[1].genere su cui è è salvata la parola CD.
quando faccio
printf(" %s ", articolo[1].genere);
mi viene stampato a video 'R@ e non la parola CD, come posso risolvere???

[cionci]

Come hai allocato articolo[1].genere ?

[fenix576]

Originariamente inviato da cionci

Come hai allocato articolo[1].genere ?

con un comando di una libreria creta da un amico:
initString(articolo[1].genere, "CD")

il codice tratto dalla lib è:
void initString(String *string, String input) {
*string = (String) malloc(strlen(input));
*string = input;
}

[cionci]

Quella funzione è COMPLETAMENTE sbagliata...
Prima di tutto c'è un memory leak, in quanto prima usi malloc e poi vai nuovamente a modificare il puntatore assegnandogli input.
Secondo errore: una stringa non si copia su un'altra con l'operatore uguale, ma bisogna copiare membro a membro o tramite strcpy.
Terzo errore: devi lascia lo spazio per il null character di fine stringa. Quindi per contenere una stringa lunga strlen(input) avrai bisogno di strlen(input) + 1 elementi allocati.

[fenix576]

Originariamente inviato da cionci

Quella funzione è COMPLETAMENTE sbagliata...
Prima di tutto c'è un memory leak, in quanto prima usi malloc e poi vai nuovamente a modificare il puntatore assegnandogli input.
Secondo errore: una stringa non si copia su un'altra con l'operatore uguale, ma bisogna copiare membro a membro o tramite strcpy.
Terzo errore: devi lascia lo spazio per il null character di fine stringa. Quindi per contenere una stringa lunga strlen(input) avrai bisogno di strlen(input) + 1 elementi allocati.

eeeeeeeh ma il mio livello di c è ancora molto basso, riferirò al mio amico di correggere gli errori.

ma per quanto riguarda lo stampare a video il contenuto di articolo[1].supporto ???

[cionci]

Originariamente inviato da fenix576

ma per quanto riguarda lo stampare a video il contenuto di articolo[1].supporto ???

La stampa è corretta, sempre che articolo[1].genere sia uno String genere. Fammi comunque vedere la struttura articolo.
L'errore in teoria è altrove.

[fenix576]

Originariamente inviato da cionci

La stampa è corretta, sempre che articolo[1].genere sia uno String genere. Fammi comunque vedere la struttura articolo.
L'errore in teoria è altrove.

#define strmax 200

typedef char string[strmax]

typedef struct
{
string supporto;
string band;
string album;
string genere;
string anno;
string prezzo;
}archivio;

archivio articolo[200];

[fenix576]

forse credo di aver capito il problema.

io ho definito il tipo string, mentre nella libreria del mio amico è già definito anche li un tipo String con la s maiuscola.
l'allocazione tramite il suo comando usa le stringhe definite String.

è forse li il problema?

il fatto è che se uso la sua definizione di String il mio programma non funziona piu, però se non usavo quel suo initString non riuscivo a salvare "CD" e altre parole all'interno delle stringhe

[cionci]

Allora è ancora tutto più sbagliato

string è già allocata... Non ha alcun senso chiamare la funzione del tuo amico. Anzi fa ancora più casino.
Se il tuo amico aveva definito string nello stesso modo in cui l'hai definita tu allora digli di riguardare tutto perché non funziona.

Per inizializzare un stringa ti basta chiamare:

strcpy(articolo[X].supporto, "CD");

[cionci]

Ok, quello è un altro problema. NON usare la libreria del tuo amico perché è ERRATA.
Come hai definito tu string può andare bene per molti scopi, ma per copiarci un'altra stringa sopra devi usare strcpy.

[fenix576]

Originariamente inviato da cionci

Ok, quello è un altro problema. NON usare la libreria del tuo amico perché è ERRATA.
Come hai definito tu string può andare bene per molti scopi, ma per copiarci un'altra stringa sopra devi usare strcpy.

uando strcpy funziona tutto alla grande, mi terrò alla larga da quella libreria
grazie mille cionci

[fenix576]

*edit*

[fenix576]

devo risolvere un'altro piccolo problemino:

ho aggiunto un'alto campo (presenza) nello struct di sopra
vorrei che ogni volta che viene aggiunto un articolo ci viene salvata sopra la lettera S.

per cui quando viene aggiunto un articolo ho messo anche
strcpy(articolo[i].presenza, "S");

tutto questo per fare in modo che quando uso la funzione per visualizzare la lista degli articoli presenti nell'archivio, vengano visualizzati solo quelli che hanno la S nel campo articolo[i].presenza.

eppure non riesco a far funzionare questa cosa. ho usato un ciclo for

for (i=0; i<z; i++)
{
if (articolo[i].presenza == 'S')
printf("ART %d = %s - %s - %s \n", i, articolo[i].band, articolo[i].album, articolo[i].supporto);
}

dov'è che sbaglio?

[DanieleC88]

Originariamente inviato da fenix576

if (articolo[i].presenza == 'S')

Cosa vorresti fare con questo codice?
Dal momento che usavi strcpy(), suppongo che presenza sia un char*, quindi eventualmente devi confrontarlo con un'altra stringa, usando la funzione strcmp(). Se invece è solo una "flag", fallo diventare un char e allora quel test sarà corretto.

ciao

[fenix576]

un ultimo aiuto con un ciclo che non riesco a far funzionare:

ho 3 variabili:
chose <------ è l'articolo che si vuole cancellare, è un numero preso in input
a =chose+1 <--- articolo successivo a quello che si vuole cancellare
z <---------- numero totale degli articoli presenti.

ho una lista di articoli, quando elimino l'articolo 2, vorrei che il 3 venga salvato nel 2, il 4 nel 3, il 5 nel 4 etc

chose e z sono 2 variabili esterne
articolo[].prezzo e copia sono float e int
il ciclo è:
int a=0;
for (a = chose+1; a< z-1 ; a++) {
strcpy(articolo[a].supporto, articolo[chose].supporto);
strcpy(articolo[a].band, articolo[chose].band);
strcpy(articolo[a].album, articolo[chose].album);
strcpy(articolo[a].anno, articolo[chose].anno);
strcpy(articolo[a].genere, articolo[chose].genere);
articolo[a].prezzo = articolo[chose].prezzo;
articolo[a].copie = articolo[chose].copie;
chose++;
}

dove sbaglio??? il codice attuale mi copia il contenuto chose in tutte le cesse successive fino a z.

[tuccio`]

ops non devi scambiare come non detto, comunque ivnece di fare tutte quelle strcpy puoi usare:

struct archivio tmp;
memcpy(&tmp, &articolo[a]);
memcpy(&articolo[a], &articolo[chose]);
memcpy(&articolo[chose], &tmp);

comunque se devi fare tutti sti scambi meglio una lista concatenata

[cionci]

Inizializzando "a" a l valore di "chose", devi copiare l'articolo "a+1" sull'articolo "a". Ora invece stai copiando l'articolo "chose" sull'articolo "a".

[fenix576]

Originariamente inviato da cionci

Inizializzando "a" a l valore di "chose", devi copiare l'articolo "a+1" sull'articolo "a". Ora invece stai copiando l'articolo "chose" sull'articolo "a".

capito, risolto! grazie mille a tutti