[C] Funzione che conta le parole distinte di una stringa

[rikki.95]

Salve ragazzi, ho un problema con questa funzione che dovrebbe oltre a contare le parole distinte di una stringa anche stamparle. Premetto che sono alle prime armi e non riesco proprio a capire dov'è l'errore.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define DIM 100




int contaParoleDistinte(char[], char*[]);


int main(int argc, char *argv[]) {
	
	char s[DIM], *token[DIM];
	int i;
	
	printf("Inserire una stringa: (max %d caratteri) ", DIM);
	gets(s);
	
	printf("La stringa e\' costituita da %d parole distinte\n", contaParoleDistinte(s, token));
	printf("Le parole distinte individuate sono:\n");
	for(i=0; i<contaParoleDistinte(s, token); i++){
		puts(token[i]);
	}
		
	system("pause");
	return 0;
}


int contaParoleDistinte(char s[], char *token[]){
	char *ptr;
	int cont=0, i, trovato;
	
	ptr=strtok(s, " ");
	token[cont]=malloc((strlen(ptr)+1)*sizeof(char));
	strcpy(token[cont],ptr);
	cont++;
	
	while(ptr!=NULL){
				
		for(i=0; i<cont; i++){
			if(strcmp(token[i], ptr)==0)	{
				trovato=1;
				break;
			}
			else trovato=0;}
		if(trovato=0){
			token[cont]=malloc((strlen(ptr)+1)*sizeof(char));
			strcpy(token[cont],ptr);
			cont++;	}
			
		ptr=strtok(NULL, " ");	
	}
	return cont;	
}

[oregon]

Questa parte la modifichi così

int par = contaParoleDistinte(s, token);
printf("La stringa e\' costituita da %d parole distinte\n", par);
printf("Le parole distinte individuate sono:\n");
for(i=0; i<par; i++)
   puts(token[i]);

e nella funzione correggi la if così

if(trovato==0){

[rikki.95]

non mi ero proprio accorta di aver usato l'operatore di assegnamento invece di quello di uguaglianza...adesso funziona grazie mille

[MItaly]

non mi ero proprio accorta di aver usato l'operatore di assegnamento invece di quello di uguaglianza...adesso funziona grazie mille

È un problema molto diffuso, difatti praticamente tutti i compilatori forniscono un warning in proposito... se stai usando un compilatore di famiglia gcc (MinGW su Windows) usa il flag -Wall, se usi VC++ specifica /W4.

[oregon]

Oppure scrivi

if(0==trovato){

[MItaly]

Anche dette "Yoda conditions"

[rikki.95]

ahahahaha adesso è sicuro che non mi scorderò più di controllare gli if