6

Если встречается согласная буква, то пропускать одну позицию в строке, если встречается гласная - то две. В итоге должно получится слово hello. Проблема в том, что значение в pos после первого раза не меняется, как это поправить?

var str = "hieeelalaooo"; //строка
var vowels = "aeiouy"; //гласные
var pos=0, res;
var res = str.charAt(0);
for (var i = 0; i <= str.length; i++) {
  for (var j = 0; j <= vowels.length; j++) {
    if (pos == i) continue;//если есть позиция букву которой мы взяли то пропускаем 
    if (str[i] != vowels[j]) {//если согласная то 
      pos = i + 2;//позиция увеличивается на два
    } else {
      pos = i + 3;//если гласная то на три
    }
  }
  res += str[pos];//получаем букву
  console.log(res)
}

7
  • а как pos может быть одновременно равен и i и undefined?
    – Grundy
    30 июн 2017 в 11:24
  • ну вот, теперь pos - всегда undefined, и соответственно str[pos] так же undefined
    – Grundy
    30 июн 2017 в 11:26
  • почему в условиях <= str.length?
    – Grundy
    30 июн 2017 в 11:28
  • инициализируйте pos. Вообще начните скрипт с "use strict"; сразу такие моменты отпадут 30 июн 2017 в 11:29
  • 1
    попробуй прокомментировать каждую строку внутреннего цикла - он непонятно что делает
    – Grundy
    30 июн 2017 в 11:32

5 ответов 5

3

Наличие символа в массиве гласных можно делать с помощью indexOf, т.е. по факту вам не нужны 2 цикла с тремя счетчиками.

var str = "hieeelalaooo"; //строка
var vowels = "aeiouy"; //гласные
var pos, res = "";
var i = 0;
while (i < str.length) {
  res += str[i];
  if(vowels.indexOf(str[i]) > -1) {
    i += 3;
  } else {
    i += 2;
  }
}
console.log(res);

6
  • спасибо ,однако не подскажите как поправить мой код, а именно почему в переменной pos ничего не меняется
    – J.Joe
    30 июн 2017 в 11:27
  • @J.Joe, потому что ты всегда делаешь continue если pos==undefined, и нигде не заполняешь pos
    – Grundy
    30 июн 2017 в 11:27
  • @Grundy почему же? ` if (str[i] != vowels[j]) { pos = i + 2; }`
    – J.Joe
    30 июн 2017 в 11:28
  • 1
    @J.Joe, до твоего последнего изменения кода вопроса - ты не доходил до этой строки, потому что перед ней было условие: if(pos==undefined) continue;
    – Grundy
    30 июн 2017 в 11:29
  • @Grundy поскажите, как именно можно мою писанину переписать чирбы сработало,потому что я не понимю пчему pos меняется только в первый раз
    – J.Joe
    30 июн 2017 в 11:30
2

Ошибка в логике: вместо того, чтобы определить гласная буква или согласная, на каждую букву pos меняется столько раз, сколько букв в массиве vowels

Кроме этого:

  • проблемы со счетчиками (i <= str.length;, j <= vowels.length;) - в случае когда счетчик равен str.length, str[i] будет равен undefined, так как индексация идет с нуля.

  • строка res меняется после изменения pos, но до проверок, что pos не выходит за границы строки. Поэтому в конце результата выводится много undefined

  • if (pos == i) continue; - бесполезное условие, которое всегда false

  • непонятно как связан счетчик i с переменной pos.


Если убрать недочеты, то может получиться так:

// Если встречается согласная буква, то пропускать одну позицию в строке, если встречается гласная - то две.
var str = "hieeelalaooo"; //строка
var vowels = "aeiouy"; //гласные
var res = '';
for (var pos = 0; pos < str.length;) {
  res += str[pos];
  var isVowels = false;
  for (var j = 0; j < vowels.length; j++) { // проверяем тип буквы
    if (str[pos] == vowels[j]) { // если гласная
      isVowels = true; // выставляем флаг
      break; // выходим 
    }
  }
  if (isVowels) {
    pos += 3;
  } else {
    pos += 2;
  }
  console.log(res)
}

вместо внутреннего цикла, можно воспользоваться методом indexOf как в ответе @br3t

либо перевести строку в объект, с ключами соответствующими гласным буквам и использовать его:

// Если встречается согласная буква, то пропускать одну позицию в строке, если встречается гласная - то две.
var str = "hieeelalaooo"; //строка
var vowels = "aeiouy".split('').reduce((acc, el) => acc[el] = true && acc, {}); //гласные
var res = '';
for (var pos = 0; pos < str.length;) {
  res += str[pos];
  if (vowels[str[pos]]) {
    pos += 3;
  } else {
    pos += 2;
  }
  console.log(res)
}

0

var str = "hieeelalaooo"; //строка
var vowels = "aeiouy"; //гласные
var vowels = new Set(vowels.split(''));
var res = '';
for (var i = 0; i < str.length; i++) {
  if (str[i] === ' ')
    res += str[i++];
  res += str[i];
  i = vowels.has(str[i]) ? i + 2 : i + 1;
}
console.log(res);

7
  • что это за код? Он явно делает не то, что описано в вопросе. Кроме того, у массивов нет метода has
    – Grundy
    30 июн 2017 в 12:07
  • Хотя если убрать лишний if и заменить has на includes то получится еще одна вариация уже данных ответов
    – Grundy
    30 июн 2017 в 12:09
  • Ну с Set, это хотя бы не кидает синтаксические ошибки :) split('') можешь убрать, он не нужен
    – Grundy
    30 июн 2017 в 12:10
  • ну проблему то решает) ну и без второго вложенного цикла
    – J.Joe
    30 июн 2017 в 12:11
  • а в чем смысл if (str[i] === ' ') res += str[i++];? У тебя в строке нет пробелов
    – Grundy
    30 июн 2017 в 12:11
0
var str = "hieeelalaooo";
var vowels = 'aeiouy';
var word = '';
for (var i = 0; i < str.length; i++) {
  if (str[i] == ' ') {
      word += ' ';
  } else if(vowels.indexOf(str[i]) > -1 ) {
      word += str[i];
      i += 2;
  } else {
      word += str[i];
      i++;
  }
}
console.log(word);
-1

=> что-то типа того, осмысленней и по-современней, чем предлагали выше:

{
const str = "hieeelalaooo"; //строка
const glass = "aeiouy"; //гласные
const res = [];

str.split('').forEach(e=>
  glass.split('').includes(e) && !res.includes(e) ? res.push(e):''
)
console.log('гласные не повторяющиеся -> '+res.join(''));
}

///////еще короче:

{
const str = "hieeelalaooo"; //строка
const glass = "aeiouy"; //гласные

str.split('').reduce( (sum,e) =>glass.split('').includes(e) && !sum.includes(e) && sum.push(e)&&sum||sum, []
)
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.