Задачка JS

[Schamil74]

Изучаю JS, решаю задачки.

C первым справился, а вот со вторым ступор + вопрос про запись ключей

 

 

 

[Switch74]

var lang = 'ru';
var day = 1;

var arr = {
    'ru': ['пн', 'вт', 'ср'], // Ключи в кавычках
    'en': ['mo', 'tu', 'we'],
};
console.log(arr[lang][day]);

 

[Schamil74]

Это было так просто оказывается)))

[Schamil74]

И новая задачка)))

Уже битый час сижу и не могу сложить два куска кода во единый. Или вообще неправильно мыслю?

Дан массив с числами. Числа могут быть положительными и отрицательными. Найдите сумму положительных элементов массива.

 

Выбирает положительные числа

var num = [4, -1, 2, 5, -8, -5, 9, -4];
for(var i = 0; i < num.length; i++) {
    if (num > 0) {
        var plus = num;
    }
}

 

Сумма положительных чисел

var sum = 0;
for (var j = 0; j < plus.length; j++) {
    sum += plus[j];
}
console.log(sum);

 

 

[andrey7287]

var sum = num.reduce((sum, i)=> i > 0 ? sum + i : sum);

 

[Schamil74]

До методов я еще не дошел

[Svetlana_P]

Так можно https://jsfiddle.net/ubukmekf/2/ ?

Извиняюсь, что влезаю без хорошего знания JS, но любопытно самой)

Edited June 11, 2017 by Svetlana_P

[DivMan]

var num = [4, -1, 2, 5, -8, -5, 9, -4];

function sumNum(arr) {
  var primer = '';
  var sum = 0;

  for(var i = 0; i < arr.length; i++) {
    if(arr[i] > 0) {
      sum = sum + arr[i];
      primer = primer + arr[i] + '+';
    }
  }
  primer = primer.substring(0, primer.length - 1)
	
  return primer + '=' + sum;
}

console.log(sumNum(num))

 

Edited June 11, 2017 by DivMan

[Schamil74]

Нужно было уточнить, сам виноват, что решение должно быть без функции и сложных махинаций, в рамках if-else-for-while.

Но все равно спасибо. Закидали ответами)))

Edited June 11, 2017 by Schamil74

[Николя223]

6 часов назад, Svetlana_P сказал:

Извиняюсь, что влезаю без хорошего знания JS, но любопытно самой)

работать то работает,  но слишком много кода.   если делать через цикл, то можно и одним ограничится

[Svetlana_P]

5 часов назад, Николя223 сказал:

работать то работает,  но слишком много кода.   если делать через цикл, то можно и одним ограничится

Вот так? https://jsfiddle.net/fhuj82tv/2/

[Николя223]

14 часа назад, Svetlana_P сказал:

Вот так?

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

https://jsfiddle.net/fhuj82tv/3/

[andrey7287]

2 часа назад, Николя223 сказал:

я делаю так 

bad pratice, да и зачем ...

[Николя223]

11 минуту назад, andrey7287 сказал:

bad pratice, да и зачем ..

не согласен) фильтры фильтрами, но перебор массивов мне нужен часто. другое дело что я не часто использую нативный JS =)

тем более что в данном случае речь идет о цикле for  ) есть ещё foreach, но вот его то как раз я вообще не использую

[andrey7287]

 

13 минуты назад, Николя223 сказал:

не согласен)

Да пожалуйста. Метод некорректно обрабатывает разряженные массивы и перебирает свойства прототипа. Когда споткнётесь, тогда согласитесь -) 

21 минуту назад, Николя223 сказал:

тем более что в данном случае речь идет о цикле for  ) есть ещё foreach, но вот его то как раз я вообще не использую

Если так нравится императивный подход, используйте for of, 2017 год на дворе. 

[Николя223]

1 час назад, andrey7287 сказал:

разряженные массивы

а как часто ты перебираешь в JS разреженные массивы? )  на стороне сервера это одно, на стороне клиента -  совершенно другое. тут больше вопрос как они в клиент попали 
upd :  если память не изменяет, в JS такие массивы должны представлять собой объект.  проверил в эмуляторе IE 8 даже перебирает) 

1 час назад, andrey7287 сказал:

Когда споткнётесь, тогда согласитесь

скажем так - я много на чем спотыкался и  выносил тот или иной урок)   речь идет о конкретном примере  

for of )  допустим в данном примере эта конструкция работает, а как быть если нужен индекс или же изменить значение массива на какое то другое?   =) 

но спорить особо не вижу смысла) так как не верстальщик )

[andrey7287]

16 часов назад, Николя223 сказал:

upd :  если память не изменяет, в JS такие массивы должны представлять собой объект.  проверил в эмуляторе IE 8 даже перебирает) 

Любой массив объект. Речь шла про то что вы не получите элементы когда они undefined.

var arr = [];
arr[6] = true;

16 часов назад, Николя223 сказал:

а как быть если нужен индекс

Как угодно, кроме for in -))) Он перечислимые свойства объекта отдаёт а не индексы массива (который в свою очередь тоже объект, ога  ) -))) Вот тебе строчка, которая поломает все твои итерации -) 

Array.prototype.test;

То что они в большинстве случаев совпадают и JS прощает подобные действие вовсе не означает что делать так хорошая идея. Почитай на досуге 

https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Statements/for...in 

Edited June 13, 2017 by andrey7287

[Николя223]

21 час назад, andrey7287 сказал:

Любой массив объект. Речь шла про то что вы не получите элементы когда они

эмм не согласен. если массив оформлен именно как объект, то он легко перебирается. попробуй.   

var arr = {};
arr[6] = true;

то что ниже - зависит от ситуаций)) Меня на JS можно подловить)) но на решениях его багов и неоднозначных проблемм - собаку съел)) несмотря на то, что больше практики на других языках) 

21 час назад, andrey7287 сказал:

Вот тебе строчка, которая поломает все твои итерации -) 

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

[Schamil74]

А вы все еще спорите)))

Лучше помогите правильно решить задачку

Нужно последнюю букву 't', сделать заглавной используя строковые функции без циклов. У меня хрень какая-то.

var str = 'lorem ipsum dolor sit amet.';
var letter = str.substr(-2,1).toUpperCase();
var find = str.lastIndexOf('t');
var result = str.replace(str[find], letter);
console.log(result);

[Николя223]

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

result = str.slice(0, find) + letter + str.slice(find + 1)

upd: может регулярками и возможно

upd2: replace по умолчанию заменит первое вхождение.  

[Svetlana_P]

Вот так работает https://jsfiddle.net/Benedetta/foetr995/ Но наверняка можно сделать умнее. Покажи потом, к чему пришел в итоге

[wwt]

вполне себе строковая функция replace()

var str = 'lorem ipsum dolor sit amet.';
console.log( str.replace(/t([^t]*)$/,'T$1'));

 

[Schamil74]

Ничег не понял)))

[DivMan]

это регулярное выражение