0

Задача - найти одинаковые значения в массиве. Что не так в коде?

void main(void)
{
    int a[5]={1,5,3,4,5},i=0,k;

    for(k=0;k<=4;k++)
    {
        do
        {
            if(a[k]==a[i]){
                printf("%i",a[i]);
            }
            i++;
        }
        while((i-4)==0);
    }
    _getch();
}

2 ответа 2

4
void main()
{ int a[5]={1,2,3,4,5};
int i=0;k=0;

for (i=0; i<5; i++) {

      for(k=5;k>i;k-1){

        if(a[i]==a[k]){Напечатать сообщение или убить всех человеков;}
     }
  }
}

Смысл в том что программа будет сравнивать 1 элемент со всеми в массиве, затем 2ой элемент со всеми и т.д.

8
  • не получается, при выводе похожих элементов выводится весь массив
    – GALIAF95
    6 янв 2012 в 13:24
  • Каким методом выводите? 6 янв 2012 в 13:31
  • printf("%i",a[i])
    – GALIAF95
    6 янв 2012 в 13:32
  • Аааа... когда i=0, a[i]=1, 1=1 он выводит... Ясно, надо чтобы значения i и k не совпадали, то есть если i=0 то k должен перебираться от 1 до 4 и т.п. 6 янв 2012 в 13:35
  • ^^^^^^^^ исправил 6 янв 2012 в 13:38
2

Ошибка в этой строке: while((i-4)==0);. Замени её на while(i<5);, а также инициализируй i нулём во внешнем цикле перед do. Ты хоть пробовал сам ошибку искать? Перед тем, как запускать код, надо прокрутить алгоритм у себя в голове. Сразу наткнёшься на то, что в while будет сравниваться -4 с 0, и цикл прекратится.

Исправленный вариант должен работать, но мне он не очень нравится. Зачем мешать разные циклы? Цикл do ... while выполняется всегда, как минимум, один раз, независимо от условия. Здесь это не нужно, можно обойтись двумя for, как тебе написали выше.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.