[c] prendere stringa in input da tastiera

[hurricane81]

Ciao a tutti! Ho questo programma che prende in input una frase e la restituisce senza le s; esiste una frase di default e ho provato ad arricchirlo con la possibilità di scegliere la frase da tastiera; però non funziona: quando scelgo di cambiare frase, mi viene visualizzata la riga "inserisci nuova frase", quindi una riga bianca e poi il programma termina. dove sbaglio? (non posso usare scanf al posto di gets perchè devo prendere una frase, non una parola!)

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

char * cancella(char *str, char c)
{
   int i, j=0;
   char *app;
   app=(char*) malloc(strlen(str)+1);
   for (i=0; i<= strlen(str); ++i)
      if (str[i]!=c)
         app[j++]=str[i];
   str=(char*) malloc(strlen(app)+1);
   strcpy(str, app);
   free(app);
   return str;
}

main()
{  
   char x[2], y[100];
   char a[]="s", b[]="n";
   
   char *s="sschissa se ss s sfunzionassssss";
   printf("La frase da esaminare è: %s\n", s);
   printf("Vuoi cambiare la frase? s/n\n");
   scanf("%s", x);
   if (strcmp(x, b)==0)
      printf("%s\n", cancella(s, 's'));
   else if (strcmp(x, a)==0)
   {
      printf("Scrivi la nuova frase:\n");
      gets(y);
      printf("%s\n", cancella(y, 's'));
   }
   else
      printf("Opzione non valida\n");
}

Ho quindi cercato di sostituire gets() con il seguente codice (definendo BUFSIZE=512 e naturalmente passando buffer e non y alla funzione cancella), ma il problema permane esattamente uguale!

char buffer[BUFSIZE];
  		char *p = NULL;
  		p = fgets (buffer, BUFSIZE, stdin);
  		if (!p)
    	{
      	fprintf(stderr, "Impossibile leggere la stringa (errore o EOF inatteso)\n");
      	exit (1);
    	}
  		/* Rimuove il newline finale o termina forzatamente la stringa */
  		p = (char *) memchr (buffer, '\n', BUFSIZE);
  		if (p == NULL)
    	buffer[BUFSIZE - 1] = '\0';
  		else
    	*p = '\0';

[andbin]

Ti posto la "mia" versione.

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

int input_line (char *buf, int bufsize)
{
    char *p;

    if (fgets (buf, bufsize, stdin) == NULL)
        return 0;
    if ((p = strchr (buf, '\n')) != NULL)
        *p = '\0';

    return 1;
}


char *strip_char (char *str, char c)
{
    char *out;
    int   i, j, l = strlen (str);

    if ((out = (char*) malloc (l+1)) == NULL)
        return NULL;

    for (i = j = 0; i < l; i++)
        if (str[i] != c)
            out[j++] = str[i];

    out[j++] = '\0';

    if ((str = (char*) realloc (out, j)) != NULL)
        out = str;

    return out;
}


int main (void)
{
    char  buf[80];
    char *def = "sssprova sseliminaziones caratterisss";
    char *str = NULL;

    printf ("La frase da esaminare e`: \"%s\"\n", def);
    printf ("Vuoi cambiare la frase? (s/n): ");

    if (input_line (buf, sizeof (buf)))
    {
        if (strcmp (buf, "s") == 0)
        {
            printf ("Scrivi la nuova frase: ");

            if (input_line (buf, sizeof (buf)))
                str = buf;
        }
        else if (strcmp (buf, "n") == 0)
            str = def;
        else
            printf ("Opzione non valida\n");
    
        if (str != NULL)
        {
            str = strip_char (str, 's');

            if (str != NULL)
                printf ("Frase elaborata: \"%s\"\n", str);
        }
    }

    return 0;
}

Come vedi, invece di usare un mix di scanf/gets (che sono molto problematiche) ho definito una semplice funzione di input di una riga, molto più comoda.