IMHO, главная ошибка в этом коде - попытка сделать всё сразу и применение велосипедов для каждой мелкой операции. Вместо того, чтобы разбить задачу на блоки, отладить каждый из них, и собрать в готовое решение. В результате получается каша из кода, которую сложно и воспринимать, и отлаживать.
Как вариант. Возможно, не самый эффективный, но куда более понятный и поддающийся наглядной отладке даже глазами:
#include <stdio.h>
#include <ctype.h>
#include <string.h>
/* ------------------------------------------------------------------------- */
static inline int is_vowel( char c )
{
static char vowels[] = "AaEeIiOoUuYy";
return strchr( vowels, c ) != NULL;
}
/* ------------------------------------------------------------------------- */
static char *compress( char *str )
{
char *s = str;
while( *s ) {
/* пропускаем пробелы: */
while( *s && isspace( *s ) ) s++;
if( !*s ) break;
if( is_vowel( *s ) ) {
/*
* 1. ищем конец слова
* 2. пропускаем концевые пробелы
* 3. переносим остаток строки
*/
char *word_end = s;
while( *word_end && !isspace( *word_end ) ) word_end++;
while( *word_end && isspace( *word_end ) ) word_end++;
memmove( s, word_end, strlen( word_end )+1 );
}
else {
/* не гласная - пропускаем слово: */
while( *s && !isspace( *s ) ) s++;
}
}
return str;
}
/* ------------------------------------------------------------------------- */
int main()
{
char str[] = "banana apple potato onion deadbeef";
printf( "Before : '%s'\n", str );
printf( "After : '%s'\n", compress( str ) );
return !*str;
}
/* ------------------------------------------------------------------------- */
if(i == 0 || str[i-1] == ' '){
. 2) Посмотрите наstrchr()
,memmove()
etc, велосипеды не нужны. 3)str[i++];
- этот код что делает? 4) Научитесь пользоваться отладчикомif(i == 0 || str[i-1] == ' ')
?