1

E tendo um problema na minha função, estou recebendo o endereço dela e estou a percorrendo, aonde for maiúsculo eu converto para minusculo e aonde tiver ponto, exclamação, interrogação, dois pontos e ponto e virgula eu troco por um espaço.

Mas de acordo com o valgrind o problema esta na linha 82 e 155 do código, ou seja, na função diminuieeretira(); e na main aonde estou recebendo o retorno da função diminuieretira();. Não estou entendendo qual o problema para dar segmentation fault.

Segue abaixo o codigo:

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

int contagemCaracteres(char* p) //Declaracao da funcao que conta quantos caracteres existem no texto
{
    int i = 0;

    while(p[i] != '\n')
    {
        i++;
    }
    return i;

}

void contagemPalavras(char* p , int r) //Declaracao de funcao que conta o numero de palavras existentes no texto
{
    int palavras = 0, parada = 0, i;


    for(i = 0; i < r ; i++)
    {
        if(p[i] >=  48 && p[i] <= 57 || p[i] >= 65 && p[i] <= 90 || p[i] >= 97 && p[i] <= 122)
        {
            palavras++;
            while(p[i] != 32)
            {
                parada++;
                i++;
            }
        }

    }

    printf("%d\n", palavras);
}


int  contagemFrases(char* p, int r)
{   
    int frases = 0, parada = 0, i;

    for(i = 0; i < r; i = parada + 1)
    {

        if(p[i] == 46 || p[i] == 33 || p[i] ==  63)
        {
            frases++;
            parada = i;
        }
        else if(p[i] == 32)
            parada = i;

        else if (p[i] >= 48 && p[i] <= 57 || p[i] >= 65 && p[i] <= 90 || p[i] >= 97 && p[i] <= 122)
        {
            if(i == 0)
                parada = 0;
            else
                parada = i;
        }
    }

    return frases;
}

void mediaCaracteres(int c, int f)
{
    float media = 0;

    media = (float)(c/f);

    printf("%.1f\n", media);
}

char* diminuieretira(char** p, int r)
{
    int i;

    for(i = 0; i < r; i++)
    {
        if(isupper(*p[i]))
            tolower(*p[i]);

        else if((*p[i]) == 46 || (*p[i]) == 44 || (*p[i]) == 59 || (*p[i]) == 58 || (*p[i]) == 33 || (*p[i]) == 63);
            (*p[i]) == 32;
    }

    printf("%s", (*p));
    return (*p);
}

void retiraespaco(char **p1, int r)
{
    int i , j , e = 0, counter = 1;
    char* paragrafo2 = NULL;

    for(i = 0; j < r; i++)
    {
        for(j = 0; j < r; j++)
        {
            if((*p1[j]) != 32)
            {
                paragrafo2 = (char*) realloc(paragrafo2, counter * sizeof(char));
                paragrafo2[i] = (*p1[j]);
                counter++;
                e = 0;
                break;
            }

            if((*p1[j]) == 32)
                e++;

            if(e == 1)
            {
                paragrafo2[i] = 32;
                break;
            }
        }

    }

    printf("%s\n", paragrafo2);
}

int main (void)
{
    int counter = 0, i = 0, recebe1, recebe2, caso;
    char* paragrafomod = NULL; 
    char* paragrafo = NULL;
    size_t tamalocado = 0;


    getline(&paragrafo, &tamalocado, stdin);
    do{
        scanf("%d", &caso);

        switch(caso)
        {
            case 1: //Primeira etapa
                recebe1 = contagemCaracteres(paragrafo);
                printf("%d\n", recebe1);

                contagemPalavras(paragrafo, recebe1);

                recebe2 = contagemFrases(paragrafo, recebe1);
                printf("%d\n", recebe2);

                mediaCaracteres(recebe1, recebe2);

            break;

            case 2:

                paragrafomod = diminuieretira(&paragrafo, recebe1); //Segudna etapa
                retiraespaco(&paragrafomod, recebe1);

            break;

            case 3: //Terceira etapa
            break;
        }   

    }while(caso != 0);


    free(paragrafo);

    return 0;
}
3
  • Uma dica, sempre use titulos intuitivos ao problema e organize por "tópicos", assim fica mais legivel.
    – Syzoth
    Commented 10/06/2015 às 20:04
  • media = (float)(c/f); faz primeiramente a divisao inteira c/f e sepois converte o resultado pata float. Acho que devia ser media = (float)c/f; para fazer a divisao com numeros de virgula flutuante.
    – pmg
    Commented 10/06/2015 às 21:17
  • tolower(*p[i]); nao altera valor de nenhuma variavel. Precisa de fazer *p[i] = tolower(*p[i]);
    – pmg
    Commented 10/06/2015 às 21:19

1 Resposta 1

0
char* diminuieretira(char** p, int r)
{
    int i;

    for(i = 0; i < r; i++)
    {
        if(isupper(*p[i]))
            tolower(*p[i]);

        else if((*p[i]) == 46 || (*p[i]) == 44 || (*p[i]) == 59 || (*p[i]) == 58 || (*p[i]) == 33 || (*p[i]) == 63);
            (*p[i]) == 32;
    }

    printf("%s", (*p));
    return (*p);
}

Neste pedaco de codigo, a sub-expressao *p[i] nao faz o que pretendes. O que pretendes 'e (*p)[i].

Mas, para esta funcao, nao precisas de complicar e passar um valor de tipo char **. Funciona bem com um simples char *.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .