[C] Problema con stringhe

[Manugal]

Ciao

Devo scrivere un programma che prenda da un file di testo una serie di stringhe e mi stampi solamente quelle che terminano con "ed".

Il codice che ho scritto è il seguente, mi sembra giusto però non stampa niente.

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

#define MAXCHAR 200

void numerostringhe2(char *s1, char *s2);

int main(void){

   char *s1=(char *)malloc((MAXCHAR+1)*sizeof(char));
   char *s2=" ";
   int i=0;
   char c;

   while((c=getchar())!=EOF)
      s1[i++]=c;

   s1[i]='\0';

   numerostringhe2(s1,s2);
   return 0;
}

void numerostringhe2(char *s1, char *s2){

   int i=1;
   char *tmp;

   tmp=strtok(s1,s2);
   if(tmp[strlen(tmp)]=='d' && tmp[strlen(tmp)-1]=='e')
      printf("%s", *tmp);
   while((tmp=strtok(NULL,s2))!=NULL)
      if(tmp[strlen(tmp)]=='d' && tmp[strlen(tmp)-1]=='e')
         printf("%s", *tmp);
}

Chi mi può aiutare? Grazie.

[Fenomeno85]

domanda scema è ... ma il file ando sta? non mi sembra proprio che te lavori sui file. Poi ti sei innamorato di strtok?

~§~ Sempre E Solo Lei ~§~

[Manugal]

Il file lo do in input io tramite la redirezione.

Uso strtok in modo da isolare le varie sottostringhe e controllarle separatamente. Altrimenti come posso fare?

[Fenomeno85]

allora tra che si potrebbe verificare un bufferoverflow come ridere ... a parte questo insignificante problema ... puoi anche usare un for.

~§~ Sempre E Solo Lei ~§~

[Manugal]

Scusa ma così non mi aiuti molto. Il for al posto di cosa? In qualunque caso uso strtok per prendere le sottostringhe

[andbin]

Ciao, ti riscrivo (corretto) il ciclo nella funzione numerostringhe2():

tmp = strtok (s1, s2);

while (tmp != NULL)
{
    if (tmp[strlen(tmp)-2]=='e' && tmp[strlen(tmp)-1]=='d')
        printf ("%s", tmp);

    tmp = strtok (NULL, s2);
}

[Manugal]

Ok grazie proverò

[Manugal]

Purtroppo ancora non stampa niente

Ho creato un file di testo con scritto just married (quindi dovrebbe stampare married), però quando faccio partire il programma con questo testo in input non stampa niente.

[andbin]

Originariamente inviato da Manugal

Purtroppo ancora non stampa niente

Riposta il codice così come è nella tua ultima versione che verifichiamo.

[Manugal]

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

#define MAXCHAR 200

void numerostringhe2(char *s1, char *s2);

int main(void){

   char *s1=(char *)malloc((MAXCHAR+1)*sizeof(char));
   char *s2=" ";
   int i=0;
   char c;

   while((c=getchar())!=EOF)
      s1[i++]=c;

   s1[i]='\0';

   numerostringhe2(s1,s2);
   return 0;
}

void numerostringhe2(char *s1, char *s2){

   char *tmp;

   tmp=strtok(s1,s2);
   while(tmp!=NULL){
   if((tmp[strlen(tmp)-1]=='d') && (tmp[strlen(tmp)-2]=='e'))
      printf("%s", *tmp);
   tmp=strtok(NULL,s2);
   }
}

Grazie sempre molto disponibili

[DanieleC88]

Originariamente inviato da andbin

Ciao, ti riscrivo (corretto) il ciclo nella funzione numerostringhe2():

tmp = strtok (s1, s2);

while (tmp != NULL)
{
    if (tmp[strlen(tmp)-2]=='e' && tmp[strlen(tmp)-1]=='d')
        printf ("%s", tmp);

    tmp = strtok (NULL, s2);
}

Bene, ma... non era molto piu' semplice questa?

  if (strcmp((tmp + strlen(tmp) - 2), "ed") == 0) {
    printf ("%s", tmp);
  }

(oppure quest'altra, che fa lo stesso lavoro:

  if (strcmp(&tmp[strlen(tmp) - 2], "ed") == 0) {
    printf ("%s", tmp);
  }

)

[andbin]

L'errore è qui:

printf("%s", *tmp);

non devi dereferenziare il puntatore (nel codice che avevo postato io non c'era infatti il *). Quindi:

printf("%s", tmp);

[DanieleC88]

Originariamente inviato da andbin

L'errore è qui:

printf("%s", *tmp);

non devi dereferenziare il puntatore (nel codice che avevo postato io non c'era infatti il *). Quindi:

printf("%s", tmp);

Non solo. Il programma non toglie il '\n' alla fine delle linee, quindi legge male il carattere finale.

[DanieleC88]

Infatti. Il codice funzionante:

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

#define MAXCHAR 200

void numerostringhe2(char *, char *);

int main(void)
{
	char *s1 = (char *) malloc((MAXCHAR+1) * sizeof(char));
	int i = 0;
	char c = '\0';

	while ((c = getchar()) != EOF) {
		if (c != '\n') {
			s1[i++] = c;
		}
		else {
			s1[i++] = ' ';
		}
	}
	s1[i] = '\0';

	numerostringhe2(s1, " ");
	return 0;
}

void numerostringhe2(char *s1, char *s2)
{
	char *tmp;

	tmp = strtok(s1, s2);
	while (tmp) {
		if (strcmp(&tmp[strlen(tmp) - 2], "ed") == 0) {
			printf("%s\n", tmp);
		}

		tmp = strtok(NULL, s2);
	}
}

[Manugal]

Grazie ora infatti funziona .

In effetti sarebbe stato più semplice come aveva detto DanieleC88, ma non mi sono proprio ricordato di strcmp

[andbin]

Originariamente inviato da DanieleC88

Bene, ma... non era molto piu' semplice questa?

  if (strcmp((tmp + strlen(tmp) - 2), "ed") == 0) {
    printf ("%s", tmp);
  }

(oppure quest'altra, che fa lo stesso lavoro:

  if (strcmp(&tmp[strlen(tmp) - 2], "ed") == 0) {
    printf ("%s", tmp);
  }

)

Certamente!!! Ma Manugal aveva iniziato nel suo post mettendo il test sui singoli caratteri (non so perché ma probabilmente voleva fare così) ed io ho semplicemente mantenuto lo stesso concetto.

[andbin]

Originariamente inviato da DanieleC88

Infatti. Il codice funzionante:

Basta semplicemente specificare alla strtok di usare come separatore anche il '\n' oltre allo spazio. Quindi:

numerostringhe2(s1, " \n");

Senza fare test vari sui caratteri!

[DanieleC88]

Originariamente inviato da andbin

Certamente!!! Ma Manugal aveva iniziato nel suo post mettendo il test sui singoli caratteri (non so perché ma probabilmente voleva fare così) ed io ho semplicemente mantenuto lo stesso concetto.

Ah, ok.

Originariamente inviato da andbin

Basta semplicemente specificare alla strtok di usare come separatore anche il '\n' oltre allo spazio.

Argh! Non c'avevo pensato!

ciao