Кто может проверить грамотность кода?

Question

[Quintis]

Ссылка на само задание :https://learn.freecodecamp.org/javascript-algorith...

function fearNotLetter(str) {
let arr = [];
let arr1 = [];

for (let i = 0; i < str.length ; i++){
arr.push(str.charCodeAt([i]));
}

let indx = arr.length-1;
  console.log(arr);
for (let j =arr[0] ; j <= arr[indx] ;j++ ){
arr1.push(j);
}
  console.log(arr1);

var findElement = arr1.filter(function(element) {
    return arr.includes(element) == false;
});
 console.log(findElement);

 console.log(String.fromCharCode(findElement));
if (findElement == false) {
  return undefined
} else{
 return String.fromCharCode

Кто может сказать как улучшить мой код или может показать аналогичную функцию .Спасибо

Answer 1

[0xD34F]

как улучшить мой код

Подобный код не улучшают, а переписывают.

function fearNotLetter(str) {
  const missing = Array
    .from(str, n => n.charCodeAt(0))
    .find((n, i, a) => n !== a[0] + i);

  return missing && String.fromCharCode(missing - 1);
}

Comments

[MasterMike]

Лол, самый краткий и конструктивный вариант от самого куратора тега и не отметили решением )

upd. Хотя немного странно, что не через стрелки записана сама функция.

[Absolute138]

Извините, не могли бы объяснить эту строчку:
missing = [...str]
Внутри массива - это чё за точки?!

[0xD34F]

Это spread.

Правда, его тут на самом деле не надо, ответ обновил.

[Absolute138]

0xD34F, благодарю, крутая и супер-полезная вещь. На хабре классно разжовано: https://habr.com/ru/post/348718/
Буду практиковаться обязательно.
Еще раз спасибо Вам.

Answer 2

[Петр Мухуров]

function fearNotLetter(str) {
  const letters = str.split("");
  
  const letter = letters.reduce((acc, letter, index) => {
    if (acc !== undefined) return acc;

    const prevLetterCode = (letters[index - 1] || letter).charCodeAt();

    if (letter.charCodeAt() - prevLetterCode  > 1) return String.fromCharCode(prevLetterCode  + 1)

    return undefined;
  }, undefined);

  return letter;
}

fearNotLetter("abce");
//d

Answer 3

[Алексей]

Ну раз уж тут все скидывают свои варианты

function fearNotLetter(str) {
  var missingLetter;
  str.split('').map(v => v.charCodeAt(0)).forEach((v,i,arr) => {
    if (!missingLetter && arr[++i]) {
      if ((++v) !== arr[i]) {
        missingLetter =  String.fromCharCode(v);
      };
    };
  });
  return missingLetter;
}

console.log(fearNotLetter("bcdf"));

Либо можно через создание заведомо правильного ряда

function fearNotLetter(str) {
	var missingLetter;
	Array(str.length).fill(str.charCodeAt(0)).map((v,i) => v + i ).forEach(v => {
		if (!missingLetter && !str.includes(String.fromCharCode(v))) {
			missingLetter = String.fromCharCode(v);
		};
	});
  return missingLetter;
}
console.log(fearNotLetter("stvwx"));

Answer 4

[h88p]

var fearNotLetter = str => {
	str = str.split('');
	if (str.length < 2) return undefined;
	let res = [];
	for (var i = 0; i < str.length-1; i++){
		if (str[i].charCodeAt() > str[i+1].charCodeAt()) {
			return ['Error'];
		} else
		if (str[i].charCodeAt()+1 !== str[i+1].charCodeAt()) {
			res.push(String.fromCharCode(str[i].charCodeAt()+1));
			str.splice(i+1, 0, String.fromCharCode(str[i].charCodeAt()+1));
			i--;
		}
	}
	return res.length > 0 ? res.join(', ') : undefined;
}

Answer 5

[Кирилл]

const fearNotLetter = str => {
  // guard expression на случай пустой строки
  if (str === "") {
    return undefined;
  }
  // на всякий случай приводим строку к нижнему регистру
  const normalizedString = str.toLowerCase();
  // берём код первого символа, ниже в цикле это значение будет обновляться.
  let charCode = normalizedString[0].codePointAt(0);
  // перебираем строку посимвольно,
  // если код текущего символа соответствует коду из переменной charCode,
  // то увеличиваем charCode на единицу
  // если код символа не соответствует ожидаемому коду из переменной charCode,
  // значит мы нашли пропавший символ и вернём его.
  for (let char of normalizedString) {
    if (charCode === char.codePointAt(0)) {
      charCode = charCode + 1;
    } else {
      return String.fromCodePoint(charCode);
    }
  }
};

Comments

[Quintis]

Спасибо за разяснение)