Task 121

Этот сайт лучше всего просматривать в современном браузере с включённым JavaScript.

Task 121

multik8711

Реализуй функцию min, которая принимает три числа и возвращает самое меньшее из них.

реализовываю

export const min = (a, b, c) => {
    if (a < b && c) {
        return a;

    }
    if (b < a && c) {
        return b;
    } else {
        return c
    }


}

и не то

Coderslang_Master

Что вернет твоя функция, если все ее аргументы будут равны друг другу?

multik8711

Возвращает минимальное 1 но тест не сдан. не понимаю вроде все проверки учел

export const min = (a, b, c) => {
if (a < b && c && b <= c) {
return a;

} else if (b < c && a && a >= c) {
    return b
} else if (c < a && b >= a) {
    return c
} else {
    return 0
}

}

Max

multik8711
Совет: когда не работает логическая цепь и глаза замылились, лучше стереть всё и переписать заново.
В чём смысл проверять наличие "а" в условиях? else if (b < c && a && a >=c) вот здесь вторым условием цепи? функция принимает на вход три аргумента, если будет передано только два, то они будут назначены переменным "a" и "b", если передать только один аргумент, то будет назначен только "а" - значит нет смысла проверять его наличие. Кроме того, оператор && вернет false если хоть один из аргументов равен нулю, false, undefined, etc. это значит, что если в функцию передадут меньше трех аргументов, все проверки посыпятся до else{}

Тем не менее, давайте разберём ваш код по шагам, как это сделал бы я сам, если бы запутался:
- Первый if условия (a < b && c && b <= c)
здесь мы предполагаем, что "а" наименьший из переданных аргументов.
если все условия верны, то мы получаем такую картину: "а" меньше "b", которое меньше или равно "с"
! разве не может так случиться, что "b" будет больше "с" и больше "а"
min(1, 200, 15) a=1 b=200 c=15 здесь "а" будет меньшим и ваша функция обработает его неверно
=> лучше первое условие описать так: if (a <= b && a <= c) таким образом, мы напрямую проверим, что
аргумент "а" меньше остальных переданных аргументов.

-Второй if условия (b < c && a && a >= c)
Здесь мы предполагаем, что аргумент b наименьший из переданных.
Если все условия верны, то "b" меньше "с", которое меньше "а"
! Та же ситуация, аргумент "а" может быть меньше "с" и вместе с тем, больше "b"
=> лучше описать условия похожим с первым шагом образом else if (b <= c && b <= a)

-Третeе if условие (c < a && b >= a)
Как и в первых двух, мы получаем двойственную картину, в которой "c" меньше "а" и "b" больше "а"
с=5 а=10 b=8
=> если мы описываем алгоритм как я предложил выше, то дополнительных условий больше не нужны, здесь мы ставим блок else и возвращаем из него аргумент "с"
(Если не понятно, первой проверкой мы предположили, что "а" точно наименьшее, если это не верно, то во второй проверке мы предполагаем, что "b" наименьшее, если и это не верно, то методом исключения остается только аргумент "с")

Почему "<=", а не строгое "<" ? Чтобы предупредить ситуацию когда в функцию будут переданы одинаковые аргументы.

Полный листинг функции:

export const min = (a, b, c) {
    if (a <= b && a <= c) {
      return a;
    } else if ( b <= a && b <= c) {
      return b;
    } else {
      return c;
   }
}

Если хотите потренироваться в написании логических цепочек: опишите эту же функцию но с переменным количеством аргументов - так, чтобы функция не ломалась, если в неё передадут только "а" или только "а" и "b" или ничего.

Antoxa911

multik8711 "return 0" нужно удалить из кода!!

boafeng

Max
Мое решение! Интересно посмотреть как сделал бы ты?

`export const min = (a, b, c) => {

if (b == undefined && c == undefined && a == undefined ) return 'Потрібно ввести числа, максимум три';
if (b == undefined && c == undefined ) return a;
if (c == undefined && b >= a) return a ;
if (c == undefined && a >= b) return b ;
if (b >= a && c >= a) return a;
if (a >= b && c >= b) return b;
return c
}`

multik8711

спасибо

Coderslang_Master

@Max отличный разбор, разложил все по полочкам 💪

baffy

// export const min = (a, b, c) => {
// if (a<b<c){
// return a;}
// if (b<a<c){ return b;} else{
// return c;
// }
// }
я еще писала так, программа работает, но не проходила проверку. Вариант Макса правильный

Ant

export const min = (a, b, c) => {
if ( a & b > c) {
return c }
else {
if ( a > b ) { return b}
else { return a }
}
};
Вот мой вариант, все так же работает, но проверку не проходит.
Думаю что задания стоит более конкретно описывать, иначе будет множественное количество вариантов которые работают, но не проходят проверку

Coderslang_Master

Ant куда ж конкретнее. Приняли три числа, вернули минимальное. Проблема в том, что не все крайние случаи учитывают ученики.

В твоем коде проблема в a & b

salton

Если целью решения задачи является проверка наличия у проверяемого альтернативного мышления или наличие опыта в построении логических цепочек - можно придумать с полдесятка различных решений, суть которых в принципе сведется к сравнению с помощью операнд сравнения (что в принципе сделали все в этой теме). Ваш вариант немного более расширенный - так как имеет проверку на undefined. Что в принципе не требовалось, но если вам так уж хочется проверять - то проверять нужно именно на то, что переменная является именно числом, а не undefined, строкой, объектом итд.
В реальной же жизни решение должно выглядеть следующим образом:

export const min = (a, b, c) => {
  return Math.min(a, b, c);
}

с целью сделать функцию более универсальной можно воспользоваться спредом и принимать/обрабатывать неограниченное количество переменных. Также можно добавить проверку на на число если это потребуется.

D1z2y

Добрый день. Скажите пожалуйста, а почему нельзя использовать такой код?

if(a<b) return a;
if (b<c) return b;
else return c;

Coderslang_Master

D1z2y потому, что нужно сравнить все 3 числа.

Представь, что a=2, b=3, c=1. Твой код проверит, что a < b и вернет a, даже не попробовав посмотреть на c

prosto_ja

что не так?

Coderslang_Master

prosto_ja не учитываешь ситуацию когда аргументы равны друг другу.

prosto_ja

все работает а не то