[C] Errori e dubbi vari

[414N]

scusate... non ho letto i codici
ma qui si parla di programmi che prendono in input una stringa di char 01 e ne restituiscono il corrispettivo ascii???

Se non ho male interpretato pure io, direi di sì.
Prima si era anche discussa la riga

Codice: Seleziona tutto

if(c=getchar()&&(c='0'||c='1'))

dalla quale deduco che i caratteri vengano letti da stdin uno alla volta.

[Blizzard]

se ho capito bene ed è come detto sopra beccatevi sto codice scritto con i piedi

BIN2ASCII secondo me

Codice: Seleziona tutto

/* bin2ascii by Giovanni "Blizzard" Santostefano*/

#include <stdio.h>
#include <stdlib.h>
 
 /*convert binary string to ascii string*/
char* ConvertBinToAscii(char *bstr)
{
	int count=0;
	char *astr=NULL;
	char *p=NULL;
	int i=0;
	
	count=(int)(strlen(bstr)/8);
	astr=malloc(count+1);
	memset(astr,0,count+1);
	
	p=astr;
	for(i=0; i<strlen(bstr); ++i) p[i/8] = p[i/8] | ( (bstr[i]-'0') << (7-(i%8)));
	
	return astr;
}

int main()
{
	char *t=ConvertBinToAscii("011000010110001001100011");
	printf("ascii: %s\n",t);
	free(t);
	
	return 0;
}

lo facciamo diventare un contest??

[anycolouryoulike]

Me lo spieghi questo?

Codice: Seleziona tutto

p[i/8] = p[i/8] | ( (bstr[i]-'0') << (7-(i%8)));

[Blizzard]

i cicla le cifre binarie.
dato che la stringa è fatta di char (8 bit)
p[i/8] ti dice il byte che stai calcolando in quel momento
(i%8) ti dice il bit che stai calcolando relativamente a quel byte
quindi piazzo quel bit con un or logico nella giusta posizione shiftandolo di (7-(i%8)) a sinistra.
Ovviamente dato che tu passi un carattere '0' o '1' basta che a questi caratteri sottrai il carattere '0' e avrai
'0'-'0'=0
'1'-'0'=1

molto uovo di colombo... scritta strana ma semplicissima

ciao
Gio

[Mario Vanoni]

Scusate, reinventare la ruota?

Nella thread Convertire ASCII
avevo citato un link con un programma in C

bin2dec di Mike Golvach

bin2dec.c

a me funziona ed e` semplice.

gcc -o bin2dec bin2dec.c

PS Nel frattempo Mike ha pubblicato le versioni perl e bash.

[Blizzard]

ma no Mario!
penso che l'autore [edit]autore del post (anycoloryoulike)[/edit] lo abbia fatto per esercitarsi e a seguire era così per giocare un po col codice :P

bambini dentro

[anycolouryoulike]

ma no Mario!
penso che l'autore [edit]autore del post (anycoloryoulike)[/edit] lo abbia fatto per esercitarsi e a seguire era così per giocare un po col codice :P

bambini dentro

Si infatti.
E poi quell'articolo sono riuscito a trovarlo solo ora, non è nella pagina che hai linkato...
E comunque la versione di Blizzard è migliore, perché non effettua la conversione da binario a decimale.

Ma perché hai usato quel cast con l'strlen e anche un puntatore apparentemente inutile (astr è uguale a p)?
Come mai li hai subito inizializzati i puntatori e le variabili?

[Blizzard]

ciao

Ma perché hai usato quel cast con l'strlen

si in quel caso il cast è inutile sorry

e anche un puntatore apparentemente inutile (astr è uguale a p)?

p si muove, invece astr punta sempre al primo char così posso restituirlo correttamente.
Comunque quello che dici è vero e p è effettivametne inutile.
Cambia così

[dato che non mi ERO capito bene ho cambiato la return che era sbagliata]

Codice: Seleziona tutto

char* ConvertBinToAscii(char *bstr)
{
   int count=0;
   char *astr=NULL;
   int i=0;
   
   count=(int)(strlen(bstr)/8);
   astr=malloc(count+1);
   memset(astr,0,count+1);
   
   for(i=0; i<strlen(bstr); ++i) astr[i/8] |= ( (bstr[i]-'0') << (7-(i%8)));
   
   return astr;
}

non l'ho provato ma forse funge.

Come mai li hai subito inizializzati i puntatori e le variabili?

per non rischiare di perdere i capelli in ore e ore di debugging
A parte gli scherzi è sempre meglio inizializzare tutto dopo la dichiarazione.
Ti ho scritto un piccolo esempio per farti venire la voglia di inizializzazione

Codice: Seleziona tutto

#include <stdio.h>

void cattivo()
{
	int a;
	
	if(a==5)
	{
		printf("smuahuahauahua\n");
	}
	else
	{
		printf("hello!\n");
		a=5;
	}
}

int main()
{
	printf("chiamo cattivo la prima volta\n");
	cattivo();
	printf("chiamo cattivo la seconda volta\n");
	cattivo();
	
	return 0;
}

vai col liscio :P

ciao
Gio

[anycolouryoulike]

Grazie a tutti!
I miei dubbi si sono temporaneamente esauriti.

PS: comunque astr e p non vanno da nessuna parte, è i che si muove

[Blizzard]

la prima volta facevo scorrere p poi l'ho ottimizzato maledetto me che mi distraggo! e mi è rimasta la convinzione che p si muoveva nonostante poi avevo fatto il giochetto p[i/8]

sorry ma ste cose le faccio in media

[edit]ho modificato la return del codice sopra[/edit]