Программа, чтобы найти, сколько слов в тексте не содержит определенный символ

Question

Следующая программа работает без печати чего-либо на экране, возможно, потому что цикл проходит через нулевой символ. Я не могу понять, почему и как это происходит, и как это исправить.

//program to find how many word in the text doesn't contain p char

#include<stdio.h>
#include<stdbool.h>
#define space ' '

void find_word(char s[]) {
  bool wordfound = false;

  int i, j = 0, word = 0;
  i = 0;

  while (s[i]) { //s[i]!='\0'  does not

    if (s[i] != 'p' && s[i + 1] != space) { //for the first word 
      wordfound = true;
      word++;
    }

    wordfound = false;

    if (s[i] == space && s[i + 1] != space) { //for more than one word in the text 
      for (j = i + 1; s[j] != space; j++)
        if (s[j] != 'p' && s[j + 1] != space)
          wordfound = true;
    }
    if (wordfound) {
      word++;
    }
    wordfound = false;
    i = j;
    i++;
  } //end while loop

  printf("Number of words not contain p character%d\n\n", word);

}

int main(void) {
  char s[] = {"pppp zzzz ppp ssss dfg sfsfdsf"};
  find_word(s);
  return 0;
}

Answer 1

Есть несколько проблем с этим кодом, но главная из них состоит в том, что внутри цикла вы присваиваете j i , что вызывает бесконечный цикл как while ( s [i]) условие никогда не выполняется. Почему бы вам не попытаться сделать это просто, например:

//program to find how many word in the text doesn't contain p char

#include<stdio.h>
#include<stdbool.h>
#define space ' '

void find_word(char s[]) {
  bool is_in = false;
  short words_count = 0, i = 0; 

  while (s[i]) {
      if (s[i] == 'p') { // if this letter is a 'p', mark the word
          is_in = true;
      }

      if (s[i] == space) { // if it's end of word
          if (!is_in) { // check if 'p' is present and increase the count
              words_count++;
          }
          is_in = false;
      }

      i++;
  }

  if (!is_in) { // check if the last word has 'p'
      words_count++;
  }

  printf("no. of words without p is %d\n", words_count);

}

int main(void) {
  char s[] = {"pppp zzzz ppp ssss dfg sfsfdsf"};
  find_word(s);
  return 0;
}

Answer 2

Похоже, что ваше условие завершения for-loop установлено неудовлетворительным с учетом вашего ввода.

if (s[i] == space && s[i + 1] != space) { //for more than one word in the text 
  for (j = i + 1; s[j] != space; j++)
    if (s[j] != 'p' && s[j + 1] != space)
      wordfound = true;
}

Здесь вы проверяете начальный пробел во входной строке. Если вы найдете его, то увеличиваете проверку индекса, пока не достигнете другого места. Что если в вашей строке нет пробела?

Вместо этого попробуйте установить второе условие для пустых и пробелов для завершения цикла:

if (s[i] == space && s[i + 1] != space) { //for more than one word in the text 
  for (j = i + 1; s[j] != '\0' && [j] != space; j++)
    if (s[j] != 'p' && s[j + 1] != space)
      wordfound = true;
}

И тогда вы установите:

    wordfound = false;
    i = j;
    i++;
  } //end while loop

Это продолжит переустанавливать ваш цикл, я не совсем понимаю ваши аргументы в пользу этого, но это будет запускать ваш цикл бесконечно.

Если вы сделаете эти изменения, ваш код завершится:

#include<stdio.h>
#include<stdbool.h>
#define space ' '

void find_word(char s[]) {
  bool wordfound = false;

  int i, j = 0, word = 0;
  i = 0;

  while (s[i]) { //s[i]!='\0'  does not

    if (s[i] != 'p' && s[i + 1] != space) { //for the first word 
      wordfound = true;
      word++;
    }

    wordfound = false;

    if (s[i] == space && s[i + 1] != space) { //for more than one word in the text 
      for (j = i + 1; s[j] && s[j] != space; j++)
        if (s[j] != 'p' && s[j + 1] != space)
          wordfound = true;
    }
    if (wordfound) {
      word++;
    }
    wordfound = false;
    i++;
  } //end while loop

  printf("Number of words not contain p character%d\n\n", word);

}

int main(void) {
  char s[] = {"pppp zzzz ppp ssss dfg sfsfdsf"};
  find_word(s);
  return 0;
}

Выход:

Number of words not contain p character24