Task 1065

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

Task 1065

Immuno

export const hasUpperCaseLetters = (s) => {
    var myArr = s.split('');
    for (let i = 0; i < s.length; i++) {
        return myArr[i] === myArr[i].toUpperCase();
    }
}

В чём ошибка?

Coderslang_Master

Ошибка в том, что ты проверяешь только первую букву и выходишь из цикла.

Immuno

Coderslang_Master Я не могу понять, как мне не выходить из цикла, я попробовал console.log вместо return, всё равно не получается.

Coderslang_Master

console.log не возвращает ничего, поэтому и не получилось.

Тебе нужно добавить if в цикл, но без ветки else.
Один return будет внутри ифа, для ситуаций когда ты найдешь букву в верхнем регистре.

А второй - уже после цикла. Он сработает если ты не найдешь такие буквы.

Immuno

Coderslang_Master

export const hasUpperCaseLetters = (s) => {
    var myArr = s.split('');
    for (let i = 0; i < s.length - 1; i++) {
        if (myArr[i] === myArr[i].toUpperCase()) {
            return true;
        }
    }
return false;
}

Не получается.

Coderslang_Master

Что будет если ты встретишь "не-букву", например точку или зяпятую?

Immuno

Coderslang_Master

Что ему возвращать, если он после первого цикла выходит и не проверяет дальше.

/**
 * Implement a function hasUpperCaseLetters in helper.js that checks if there are any upperCase letters in the string
 * */
import { hasUpperCaseLetters } from './helper.js';
const s = 'Bingo!';
console.log(hasUpperCaseLetters(s));

export const hasUpperCaseLetters = (s) => {
    var myArr = s.split('');
    for (let i = 0; i < s.length; i++) {
        var myA = myArr[i].toString('');
        if (myA.search(/^[a-zA-Z]+$/) === 0) {
            return myArr[i] === myArr[i].toUpperCase() ? true : false;
        } else {
            return false;
        }
    }
}

Coderslang_Master

Выходит, потому что ты написал return.

Для одного из твоих return есть лучшее место.

Найди его.

Immuno

Coderslang_Master А зачем нам проверять всё буквы, если уже есть совпадение, моя программа выводит True, если есть большая буква и false, если все маленькие. Задание в одну строчку и нет явного условия. Как я могу понять, что вы именно хотите. Я написал функцию и она делает всё правильно. Почему я не прав? Нет же определённых условий, только задание написать функцию проверки на буквы в верхнем регистре.

Coderslang_Master

Твой цикл работает ровно одну итерацию. После первого ифа ты в любом случае выходишь. Это ожидаемое поведение?

Immuno

Coderslang_Master Вы понимаете, что у Вас задание написано не понятно, что в итоге функция должна выводить? Она работает так, как это написано в задании. Нашла букву, вышла из цикла, т.к. зачем её работать дальше, если она уже нашла ответ.

Coderslang_Master

Твоя функция работает некорректно.

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

Задание сформулировано максимально корректно.

Функция hasUpperCaseLetters должна проверять, есть ли буквы в верхнем регистре в строке

Это значит - хотя бы одна большая буква, на любой позиции.

Immuno

export const hasUpperCaseLetters = (s) => {
    console.log(s.length);
    for (let i = 0; i < s.length; i++) {
        var myArr = s.split('');
        var myA = myArr[i].toString('');
        if (myA.search(/^[a-zA-Z]+$/) === 0) {
            var a = myArr[i] === myArr[i].toUpperCase();
            if (a === true) {
                break;
            }
        }
    }
    if (a === undefined) {
        a = false;
    }
    return a;
}

Функция работает корректно, что не так, обьясните пожалуйста.

Coderslang_Master

Буква Ü тоже большая. Таких букв много.

Immuno

Coderslang_Master
Использовал это регулярное выражение и наконец сдал это задание)
/^[a-zA-Z\u00C0-\u00ff]+$/

Coderslang_Master

Эту задачу можно решить без регулярных выражений. Но это тоже рабочий вариант.

ILLIA

И все же вопрос 🙂
Я сделал вот так:
export const hasUpperCaseLetters = (s) => {
return s.split('').find(element => element === element.toUpperCase())
}

нашло букву 'B' но задачу не приняло

Для самопроверки я решил найти букву в нижнем регистре:
export const hasUpperCaseLetters = (s) => {
return s.split('').find(element => element === element.toLowerCase())
}
соответственно нашел "і"

Так почему же не принимает?

media_kot

ILLIA функция должна возвращать true или false, а не букву

Coderslang_Master

ILLIA @media_kot правильно говорит, функция должна вернуть булевое значение, а не строку. Попробуй добавить if. Если нашли что-то, то верни true, иначе - false

ILLIA

Функция hasUpperCaseLetters должна проверять, есть ли буквы в верхнем регистре в строке

не могу понять почему:

export const hasUpperCaseLetters = (s) => {
   if ((s.split('').find(element => element == element.toUpperCase())) === 'undefined') {
      return false
   } else {
      return true
   } 
}

Coderslang_Master

ILLIA На каких строках проверял работоспособность функции?

Следующая страница »