Помогите пожалуйсто исправить программу, Помогите пожалуйсто исправить программу Опции

[Гость_aligatr-kil_*]

Помогите пожалуйсто исправить программу : Удалите из слова все буквы, совпадающие с последней буквой.

Код

#include <iostream.h>
#include <stdio.h>
#include <conio.h>

void main ()
{
  int i;
  int j;
  char str[100];

  cin>>str[i];

  char sym = str[strlen(str)-1];//запомнили последний
  for( int i = 0; i < strlen(str); i++)
    if( str[i] == sym )// находими нужный символ
      for( int j = i; j < strlen(str)-1; j++)
        str[i] = str[i+1];//сдвигаем хвост
  cout << str[i];
  getch();
}

[SiMM]

str[i] = str[i+1];//сдвигаем хвост

[Гость_Ireul_*]

Типичная ошибка для чайника - сдвигая массив ты "перепрыгиваешь" по i одну букву, так как она из i+1 перемещается в i, а в следующей итерации цикла ты уже увеличиваешь i, и когда у тебя встречается пара целевых букв для сдвига - одну ты пропустишь.

Как вариант - --i; каждый раз как выполняешь сдвиг.
Маленькое дополнение - сдвигать гораздо удобнее при помощи вызова strcpy(str+i,str+i+1);

ЗЫ ты вообще щачем удаляешь что-то из целевой строки? Ты же выводишь посимвольно, так и выводи символ если он не совпал и выводи если совпал, а строку вообще трогать незачем.

[Shasta Dam]

Типичная ошибка для чайника - сдвигая массив ты "перепрыгиваешь" по i одну букву, так как она из i+1 перемещается в i, а в следующей итерации цикла ты уже увеличиваешь i, и когда у тебя встречается пара целевых букв для сдвига - одну ты пропустишь.

Как вариант - --i; каждый раз как выполняешь сдвиг.
Маленькое дополнение - сдвигать гораздо удобнее при помощи вызова strcpy(str+i,str+i+1);

ЗЫ ты вообще щачем удаляешь что-то из целевой строки? Ты же выводишь посимвольно, так и выводи символ если он не совпал и выводи если совпал, а строку вообще трогать незачем.

ппц ты слепой!

Код

for( int j = i; j < strlen(str)-1; j++)
str[i] = str[i+1];//сдвигаем хвост

SiMM уже заметил. У тебя цикл по j!

[Amp]

Мои две копейки - в коде разбираться не хочу, но беглый осмотр говорит о том, что gcc вообще не соберет данную программу.

[Гость_Ireul_*]

ппц ты слепой!

Код

for( int j = i; j < strlen(str)-1; j++)
str[i] = str[i+1];//сдвигаем хвост

SiMM уже заметил. У тебя цикл по j!

Даже с циклом по j всё равно сохраняется та ошибка, не говоря уже о том что правильный сдвиг, при котором данные цикл отпадает, был приведён в моём посте.

[SiMM]

Даже с циклом по j всё равно сохраняется та ошибка

Какая - та?
Алгоритм, конечно, не ахти (одного цикла без строковых функций достаточно) и не лишён недостатка (выводится только один, последний символ). Ну и код был нифига не отформатирован.

[Гость_Smacker_*]

Ну, я конечно не уверен, что авторская задумка была именно такая, но если расставить фигурные скобки и исправить индексирование во втором цикле, то в принципе вырисовывается нечто работоспособное:

Код

#include <iostream>
using namespace std;

void main ()
{
    char str[100];
    cin>>str;

    char sym = str[strlen(str)-1];
    for( unsigned int i = 0; i < strlen(str); i++)
    {
        if( str[i] == sym )
        {
            unsigned int j = i;
            for(; j < strlen(str)-1; j++)
            {
                str[j] = str[j+1];
            }
            str[j] = 0;
        }
    }
    cout << str;
}

[Гость_aligatr-kil_*]

спасибо всем большое !!!

Сообщение отредактировал Gobbs - Jan 7 2008, 13:38

[Гость_Snowm@n_*]

Ну, я конечно не уверен, что авторская задумка была именно такая, но если расставить фигурные скобки и исправить индексирование во втором цикле, то в принципе вырисовывается нечто работоспособное:

Код

#include <iostream>
using namespace std;

void main ()
{
    char str[100];
    cin>>str;

    char sym = str[strlen(str)-1];
    for( unsigned int i = 0; i < strlen(str); i++)
    {
        if( str[i] == sym )
        {
            unsigned int j = i;
            for(; j < strlen(str)-1; j++)
            {
                str[j] = str[j+1];
            }
            str[j] = 0;
        }
    }
    cout << str;
}

Не работает, ибо не удаляет вторую из двух подряд идущих букв, которые нужно удалить Либо заменить if на while либо поставить --i после if(...){...}.
p.s. Вариант, претендующий на оптимальность

Код

#include <iostream.h>
#include <string.h>
void main(){
char str[100];
cin>>str;
int l=strlen(str);
char c=str[l-1];
int i=-1;
while (str[++i]!=c);
int j=i;
for (;i<l-1;++i) if (str[i]!=c) str[j++]=str[i];
str[j]=0;
cout<<str<<endl;
}