Задача про foobar на JS. Как найти правильный алгоритм?

Question

[Armando]

Есть интересная задача про FooBar.
Суть её заключается в том, что дан массив чисел, допустим от 1 до 100.
Нужно сделать так, что бы числа, делящиеся на 3 без остатка принимали значения Foo, а числа, делящиеся на 5 принимали значение Bar, а числа делящиеся на 3 и на 5 принимали значение Foobar.
И всё вроде бы ничего, и моих скудных знаний хватило, что бы решить данную задачу. Однако, чувство, что я написал 100 срок кода, вместо 5 заставило меня обратиться к вам.
Друзья, подскажите, как можно было решить данную задачу покороче. Спасибо!
Собственно мой код,

var arr = [];
			console.log(arr);
			function createArr(){
				for (var i=1; i<101; i++)
					arr.push(i);
			}
			createArr();
			console.log(arr);
			function divBy3By5(a){
				var ar = [];
				for (var i=0; i<a.length; i++){
					if (a[i]%5 == 0 && a[i]%3 == 0){
						ar.push(a[i]='foobar');
					}
				}
				return ar;
			}
			function divBy3(a){
				var ar = [];
				for (var i=0; i<a.length; i++){
					if (a[i]%3 === 0 && a[i]%5 != 0){
						ar.push(a[i]='foo');		
					}
				}
				return ar;
			}
			function divBy5(a){
				var ar = [];
				for (var i=0; i<a.length; i++){
					if (a[i]%5 === 0 && a[i]%3 != 0){
						ar.push(a[i]='bar');
					}
				}
				return ar;
			}
			divBy3(arr);
			divBy5(arr);
			divBy3By5(arr);
			console.log(arr);

Comments

[dollar]

Да, можно и в 2-3 строчки (короткие я имею в виду, а не тупо убрать переносы).
А у вас много отладки, она и занимает основное место. Вы же еще зачем-то создаете три отдельных массива, а этого нет в условии задачи.

[Armando]

dollar, уточните, пожалуйста, как можно обойтись без использования вспомогательных массивов? Учитываются ли они, если они используются только локально?

[dollar]

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

Answer 1

[Leo Developer]

const fillArray = count => 
  (new Array(count)).fill(1).map((item, i) => i + 1)

const fooBar = array =>
  array.map(number => 
    !(number % 3) && !(number % 5) 
    ? "FooBar"
    : !(number % 3) ? "Foo"
    : !(number % 5) ? "Bar" : number
  )

fooBar(fillArray(100))

Comments

[Armando]

На выходе получил (47) ["Foo", "Bar", "Foo", "Foo", "Bar", "Foo", "FooBar", "Foo", "Bar", "Foo", "Foo", "Bar", "Foo", "FooBar", "Foo", "Bar", "Foo", "Foo", "Bar", "Foo", "FooBar", "Foo", "Bar", "Foo", "Foo", "Bar", "Foo", "FooBar", "Foo", "Bar", "Foo", "Foo", "Bar", "Foo", "FooBar", "Foo", "Bar", "Foo", "Foo", "Bar", "Foo", "FooBar", "Foo", "Bar", "Foo", "Foo", "Bar"]
А остальные элементы массива вывести не получилось?
Должно быть (100) [1, 2, "foo", 4, "bar", "foo", 7, 8, "foo", "bar", 11, "foo", 13, 14, "foobar", 16, 17, "foo", 19, "bar", "foo", 22, 23, "foo", "bar", 26, "foo", 28, 29, "foobar", 31, 32, "foo", 34, "bar", "foo", 37, 38, "foo", "bar", 41, "foo", 43, 44, "foobar", 46, 47, "foo", 49, "bar", "foo", 52, 53, "foo", "bar", 56, "foo", 58, 59, "foobar", 61, 62, "foo", 64, "bar", "foo", 67, 68, "foo", "bar", 71, "foo", 73, 74, "foobar", 76, 77, "foo", 79, "bar", "foo", 82, 83, "foo", "bar", 86, "foo", 88, 89, "foobar", 91, 92, "foo", 94, "bar", "foo", 97, 98, "foo", "bar"]

[Leo Developer]

arman1231, Поправил

Answer 2

[zendor]

Решение

const arr = [...Array(100)].map((_, i) => i + 1);
const res = arr.map(n => `${n % 3 ? '' : 'Foo'}${n % 5 ? '' : 'Bar'}` || n);

Comments

[dollar]

Можете объяснить, почему [...Array(100)].map работает, а Array(100).map - нет?
Это же просто распаковка массива, а потом запаковка снова в массив.

[alex]

dollar,

Метод map вызывает переданную функцию callback один раз для каждого элемента, в порядке их появления и конструирует новый массив из результатов её вызова. Функция callback вызывается только для индексов массива, имеющих присвоенные значения, включая undefined. Она не вызывается для пропущенных элементов массива (то есть для индексов, которые никогда не были заданы, которые были удалены или которым никогда не было присвоено значение. © MDN

[Armando]

Выглядит божественно))

Answer 3

[Рональд Макдональд]

Да нормальное решение. Есть изящнее и короче, но это с опытом приходит.

Comments

[Armando]

Согласен, но мне даже кажется, что это даже относится к стандартам JS, так как они постоянно, что то вводят новое, а я последний раз смотрел на этот синтаксис году в 2012.
Я конечно ничего против не имею, и понимаю, что всё это только совершенствует язык, но когда копаешь глубже, понимаешь, что JS это костыль на костыле...

[Рональд Макдональд]

arman1231, воистину, там уже фрактальный костыль, костылящий сам себя. Вообще же изначально JS был идеален и в нем было 0 команд. А потом в него начали добавлять костыли...

Answer 4

[Andreas Shopen]

Твоё решение вполне годное, но я тут сделал по компактнее, хотя, конечно, Leo Developer написал лучше всего)

const arr = []

fillArray = array => {

    for (let i = 1; i < 101; i++) {

        if (!(i % 3) & !(i % 5)) {
            array.push('foobar')
        } else if (!(i % 3)) {
            array.push('foo')
        } else if (!(i % 5)) {
            array.push('bar')
        } else {
            array.push(i)
        }

    }
    return array
}

console.log(fillArray(arr))

Answer 5

[SheikhRamil]

let arrayOfNums = [];        

    for (let i = 1; i <= 100; i++) {
        arrayOfNums[i] = i;
        if (i % 3 == 0) {
            // alert(i + " Foo");
            arrayOfNums[i] = "Foo";
        }
        if (i % 5 == 0) {
            // alert(i + " bar");
            arrayOfNums[i] = "bar";
        }
        if (i % 3 == 0 && i % 5 == 0) {
            // alert(i + " Foobar")
            arrayOfNums[i] = "Foobar";
        }
    }

        console.log(arrayOfNums);

Считаю это - самый простой и понятный и примитивный, компактный код, который даже новичок 3 урока поймет(хотя я сам еще новичко кстати)

Comments

[Codaconda]

Код не оптимизирован и с багами.
1) При каждой итерации он проверяет все условия.
2) Первое число, которое подходит под все условия - 15. Т.е. в твоём коде сначала в массив с индексом i записывается значение"i", перезаписывается на "Foo", следующий if перезаписывает это значение на "bar" и только после этого доходит до последнего условия и записывает туда "Foobar".
3) Итерация начинается с 1. Логика твой программы делает массив, в котором 101 элемент и элемент с нулевым индексом всегда пустой.

Answer 6

[Nightmare2002]

let arr = []

for (let i = 1; i < 101; i++) {
  arr.push(i)
}


function fooBar(arr) {
  let foo = []
  let bar = []
  let foobar = []

  for (let a = 1 ; a < 101; a++) {
    if (a % 3 == 0) {
      foo.push(a)
    }
  }

  for (let a = 1 ; a < 101; a++) {
    if (a % 5 == 0) {
      bar.push(a)
    }
  }

  for (let a = 1 ; a < 101; a++) {
    if (a % 3 == 0 || a % 5 == 0) {
      foobar.push(a)
    }
  }

  alert(`Foo(value /3) = ${foo}`)
  alert(`Bar(value /5) = ${bar}`)
  alert(`FooBar(value /3 and /5) = ${foobar}`)
}

fooBar(arr)

Я смог решить таким образом