Грамотно ли написан код?

Question

[Lolipop999]

Есть задание: Составьте массив дней недели. С помощью цикла for выведите все дни недели, а выходные дни выведите жирным. Я выполнил вот так. Это говнокод на убогих костылях или можно написать грамотнее?

const week = ['Понедельник', 'Вторник', 'Среда', 'Четверг', 'Пятница', 'Суббота', 'Воскресенье'];

for (i = 0; i < week.length; i++) {
  document.write(week.slice(0,-2) + ',' + week.slice(-2).join().bold());
  break;
}

Comments

[VoidVolker]

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

function weekDays(locale, weekday = 'long') { // 'narrow', 'short', 'long'
    locale = locale || 'default'
    let i = 5 // Понедельник
    let days = []
    while(i < 12) {
        days.push(new Date(1970, 0, i++).toLocaleString(locale, { weekday: weekday }))
    }
    return days
}

weekDays()
[ 'понедельник',  'вторник',  'среда',  'четверг',  'пятница',  'суббота',  'воскресенье']
weekDays('jpn')
[ '月曜日', '火曜日', '水曜日', '木曜日', '金曜日', '土曜日', '日曜日' ]

[Василий Банников]

VoidVolker, интересно, а есть ли такие календари (современные страны с календарями), в которых не 7 дней в неделе?)

[VoidVolker]

Василий Банников Не знаю - не приходилось сталкиваться.

Answer 1

[StiflerProger]

В вашем коде от цикла нет смысла, если без цикла просто

document.write(week.slice(0,-2).join(',') + ',' + week.slice(-2).join(',').bold());

то код хороший
в цикле надо перебирать по одному дню и выводить

const week = ['Понедельник', 'Вторник', 'Среда', 'Четверг', 'Пятница', 'Суббота', 'Воскресенье'];

for (let i = 0; i < week.length; i++) {
  if (week[i] === 'Суббота' || week[i] === 'Воскресенье') {
    document.write(week[i].bold());
  } else {
    document.write(week[i]);
  }
}

чтобы не сравнивать выходные по названию, можно вместо строк использовать объекты

const week = [{title: 'Понедельник', weekend: false}, ... {title: 'Воскресенье', weekend: true}];

for (let i = 0; i < week.length; i++) {
  document.write(week[i].weekend ? week[i].title.bold() : week[i].title);
}

Comments

[Lolipop999]

а как вписать join, чтобы через запятую выводило?

const week = ['Понедельник', 'Вторник', 'Среда', 'Четверг', 'Пятница', 'Суббота', 'Воскресенье'];

for (let i = 0; i < week.length; i++) {
  if (week[i] === 'Суббота' || week[i] === 'Воскресенье') {
    document.write(week[i].bold());
  } else {
    document.write(week[i]);
  }
}

[StiflerProger]

Lolipop999,

const week = ['Понедельник', 'Вторник', 'Среда', 'Четверг', 'Пятница', 'Суббота', 'Воскресенье'];

for (let i = 0; i < week.length; i++) {
  if (week[i] === 'Суббота' || week[i] === 'Воскресенье') {
    document.write(week[i].bold());
  } else {
    document.write(week[i]);
  }
  if (i < week.length - 1) document.write(", ");
}

[StiflerProger]

ну и week я бы на weekdays заменил, чтобы понятней было

Answer 2

[Александр]

Можно так.

const week = ['Понедельник', 'Вторник', 'Среда', 'Четверг', 'Пятница', 'Суббота', 'Воскресенье'];
week.forEach((day, index) => {
  const element = document.createElement('p');
  if (index < 5) {
    element.textContent = day;
  } else {
    const wrapper = document.createElement('strong');
    wrapper.textContent = day;
    element.append(wrapper);
  }
  document.body.append(element);
});