Можно ли улучшить код?

Question

[towncitybr]

если да, то как
или нормально?

https://jsfiddle.net/kwdg1e8r/

Comments

[towncitybr]

Ankhena , подскажите , пожалуйста, как сделать картинки квадратными?

например, добавил контейнер
сейчас размеры картинок 150 на 187.5

но в макете они квадратные
тоесть ширина картинки равняется её высоте

https://jsfiddle.net/prqfdeh0/

[Ankhena]

towncitybr,

но в макете они квадратные

Ну так и вставляйте их сразу квадратными. Это проще всего.

Альтернативы: сделать квадратным блок и использовать object-fit.
Квадратный блок делается хаком с паддингом (гуглится) или с помощью aspect-ratio, если устраивает его поддержка.

Answer 1

[Дмитрий]

Вполне нормально!

var nums = 123123;
var arr = `${nums}`.split('');
var result1 = 0;
var result2 = 0;

arr.forEach((curr, index) => {
  (index <= 2) ? result1 += Number(curr) : result2 += Number(curr);
})

console.log((result1 == result2) ? 'сумма первых цифр равняется остальным трем' : 'нет, не равняется')

Comments

[WbICHA]

index == 0 || index <= 2

Какова цель первого сравнения? Ещё и нестрого.

[towncitybr]

WbICHA, суть задачи просто сравнить сумму первых трех чисел с остальными тремя

[WbICHA]

towncitybr, и? index == 0 для чего?

[Дмитрий]

WbICHA towncitybr Действительно! index == 0 - Лишнее. Только зачем там строгое сравнение... И без того понятно что там будет число... Но если нра. то почему бы и нет!

Answer 2

[Сергей delphinpro]

Array.prototype.amount = function() {
    return this.reduce((a, i) => a + Number(i), 0);
}

let nums = 123123;
let arr = `${nums}`.split('');

const sum1 = arr.slice(0, 3).amount();
const sum2 = arr.slice(3, 6).amount();

if (sum1 == sum2) {
    console.log('сумма первых цифр равняется остальным трём');
} else {
    console.log('нет, не равняется');
}