Task 152

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

Task 152

Olesya

Задача: Почини код. Доступ должны получить только пользователи с ролью admin.
Вроде как всё написала, ответ выдаёт нужный, но одно из check-task не принимает, а именно пишет ошибка в "4. checkAccess должна возвращать false, если роль пользователя не является admin."

При этом у меня в helper:

export const checkAccess = (user, password) => {
  return user.password === password;
}
export const grantAccess = (name) => {
  console.log(`Congratulations, ${name}. Access granted.`)
}
export const denyAccess = (name) => {
  console.log(`Sorry, ${name}. Access denied.`)
}

В functions :

import { checkAccess, grantAccess, denyAccess } from './helper.js';
export const auth = (user, password) => {
  if (checkAccess(user, password) === true) {
    if (user.role === 'admin') {
      return(grantAccess(user.name));
    }
    else {
      return(denyAccess(user.name));
    }
  }
}

При запуске выдаёт:

Authenticating John!
Sorry, John. Access denied.
undefined
Authenticating Sally!
Congratulations, Sally. Access granted.
undefined

Подскажите, что не так? 🙁
И не могу понять, почему в конце "undefined", ведь у меня всего два "console.log()"?

Coderslang_Master

Olesya третий консоль лог, скорее всего из solution.js, который я не вижу в первом посте.

По логике не вижу смысла проверять пароль если юзер админ.

Т.е. если админ - пропускаем сразу. Если нет, проверяем пароль и дальше действуем по результатам этой проверки.

salton

начните с того, что внимательно прочтите еще раз п.4 : "4. checkAccess должна возвращать false, если роль пользователя не является admin."

export const checkAccess = (user, password) => {
  return user.password === password;
}

я не вижу тут проверки на админа...
ну и потом уже нужно проводить работы над другими ошибками

Olesya

salton
Эта часть была в первоначальном задаче, и на мой взгляд логично её оставить без изменений, ведь прежде чем проверять роль, сперва нужно проверить, а правильно ли в принципе введены данные. А уже потом в блоке функций, если пароль правильный, то идет условие if (user.role === 'admin')

salton

Возможно и логичнее. Но есть условие задачи - "checkAccess должна возвращать false, если роль пользователя не является admin." И сколько бы нам не хотелось, но проверка задачи не пройдет, пока четкое условие не будет выполнено! У вас же функция возвращает булево только в зависимости от пароля.

Olesya

Coderslang_Master
Переставила куски кода, чтоб первой шла проверка на админа. Но п. 4 задачи всё равно пишет ошибку:

import { checkAccess, grantAccess, denyAccess } from './helper.js';

export const auth = (user, password) => {
    if (user.role === 'admin') {
      return(grantAccess(user.name));
    }
    else {
      if (checkAccess(user, password) === true) {
        return(denyAccess(user.name));
      }
    }
}

при проверке задачи выдаёт:

**_True_** 1. auth должна вызывать checkAccess с аргументами user и password
_**True**_ 2. checkAccess должна возвращать true, если пароли совпадают, а роль пользователя - admin.
_**True**_ 3. checkAccess должна вернуть false, если пароли не совпадают
**_False_** 4. checkAccess должна возвращать false, если роль пользователя не является admin.
_**True**_ 5. auth должна вызыватьgrantAccess с именем пользователя, если checkAccess вернул true
_**True**_ 6. auth должен вызывать denyAccess с именем пользователя, если checkAccess вернул false

Coderslang_Master

Olesya так, я прошлым ответом похоже тебя запутал. Пароль нужно проверять обязательно, даже если роль админ. Об этом явно говорим в условии:

Доступ должны получить только пользователи с ролью admin

Плюс у админов нужно обязательно проверить пароль.

Проблема в том, что проверку на админа нужно сделать внутри checkAccess, а ты ее делаешь в auth.
Так и получается, что это требование не выполняется:

False 4. checkAccess должна возвращать false, если роль пользователя не является admin.

Код внутри auth должен выглядеть так:

export const auth = (user, password) => {
  if (checkAccess(user, password)) {
    grantAccess(user.name);
  } else {
    denyAccess(user.name);
  }
}

salton

Где код checkAccess ???

Olesya

salton
Он уже был прописан авторами задачи в отдельном файле:

export const checkAccess = (user, password) => {
  return user.password === password;
}

salton

Отлично, вы видете, что функция checkAccess возвращает булево при проверке роли пользователя?
Я по крайней мере - нет, только проверку пароля.

Alexey


import { checkAccess, grantAccess, denyAccess } from './helper.js'; 
export const auth = (user, password) => {
  if (checkAccess(user, password)) {
    if (user.role = 'admin') 
    grantAccess(user.name);
  } else {
    denyAccess(user.name);
  }
}

но не выдает false

Alexey

salton

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

Maxim

Подскажите где я сделал промашку в логике
Все проверки пройдены кроме пункта

checkAccess должна возвращать true, если пароли совпадают, а роль пользователя - admin.

function.js


export const auth = (user, password) => {
  if (checkAccess(user, password)) {
    if(user.role === 'admin'){
      grantAccess(user.name);
      return true;}
      }   else {
      denyAccess(user.name);
      }
  }

helper.js

export const checkAccess = (user, password) => {
  if (user !== 'admin'){
    return false} {
  if (user.password === password && user.role === 'admin')
    return true;
  }
}

export const grantAccess = (name) => {
  console.log(`Congratulations, ${name}. Access granted.`)
}

export const denyAccess = (name) => {
  console.log(`Sorry, ${name}. Access denied.`)
}
}

salton

В одном случае вы сравниваете роль, а во втором?

Maxim

salton Спасибо)