Как упростить сравнение 4 чисел с помощью if?

Question

[Ruslan Malkhozov]

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

start:
scanf("%d %d %d %d",&a ,&b, &c, &d);
if(a!=b&&a!=c);
    else{
        printf("Not good, repeat\n");
        goto start;
    }
if(a!=d&&b!=d);
    else{
        printf("Not good, repeat\n");
        goto start;
    }
if(b!=c&&c!=d);
    else{
        printf("Not good, repeat\n");
        goto start;
    }

Comments

[Владислав Лысков]

Или?

Answer 1

[Andrey Tsvetkov]

Складываешь чиселки в мапу, сет или что там есть в Си, далее если длина не равна числу входных чиселок, значит есть совпадения.

Comments

[BorLaze]

В чистом С ничего такого нет

Answer 2

[Сергей delphinpro]

$Maccив = ПолучитьМассивЧисел()
ОтсортироватьМассив($Массив)
$Врем = 0
Цикл $Число Из $Массив
  Если $Число == $Врем То 
    Принт("Есть совпадение")
    ОстановитьЦикл
    ВыйтиИзПодпрограммы
  Иначе
    $Врем = $Число

Comments

[Владимир Олохтонов]

Это решение хорошо своей обобщаемостью на произвольный размер массива :)

[Сергей delphinpro]

Владимир Олохтонов, Именно. Сейчас четыре числа, завтра пять. Нужно быть готовым.

[GavriKos]

Пользователь ввел 0 1 2 3 - и сломал ваш алгоритм.
Идея хорошая, реализация с ошибкой )
Сортировка кстати не нужна.

[Сергей delphinpro]

GavriKos, и правда =)
сортировка нужна, иначе введенная последовательность 2-3-5-2 не сработает

[GavriKos]

Сергей delphinpro, тоже верно )

Answer 3

[Владимир Т]

if(a == b || a == c || a == d || b == c || b == d || c == d) {
    printf("Not good, repeat\n");
}
else {
// start hete
// ...
}

Comments

[Griboks]

можно даже с while

[Ruslan Malkhozov]

А что если нужно оставить три функции if, и при этом скомпоновать мой код. Есть идеи?

[Владимир Т]

Ruslan Malkhozov, ничего не понял. if - это не функция, это statement.
Можно и с тремя if, но зачем?

if(a != b && a != c && a != d) {
  if( b != c && b != d){
 if( c != d) {
    // start hete
    // ...
    
    goto exit;
}}}
printf("Not good, repeat\n");
exit:

[Евгений Шатунов]

Ruslan Malkhozov , а для чего надо оставить все три if? Какая у них цель на самом деле?

[h8nor]

Евгений Шатунов, никакой цели нет. В задаче перечислены все условия, при которых каждое последующие число не должно равняться одному из предыдущих.

Answer 4

[Василий Мельников]

Чем проще написать тем лучше. Если значений только четыре и не предполагается это количество менять. То просто написать условие. Но лучше эту проверку вынести в правильно названую функцию, чтобы читающий код понимал что происходит.