Как можно сделать код, что-бы он был красивее?

Question

[ostrooovskyy]

Всем приветик, друзья разработчики. Написал вот такой код для расстановки точки в конце и заглавной буквы в начале. Хотел бы получить этот код, только красивее, настолько это возможно.

const endMessage = (text: string) => {
	if (text.includes("
") || /^(?:\[club\d+\|.*])/.test(text)) {
		return text;
	}
	let newText = text;
	if (/(?=[^\d])[\wа-яё]/i.test(utils.array.last(text.split("")))) {
		newText += ".";
	}
	if (/(?=[^\d])[\wа-яё]/.test(text[0])) {
		newText = newText.substring(1);
		newText = text[0].toUpperCase() + newText;
	}
	return newText;
};

Comments

[Модератор]

См.п.3.4 Необходимо привести вопрос в соответствие Регламенту.
Также необходимо дополнить вопрос, привести критерий красивости кода, а также как вы пробовали это сделать и что не получается.

[WbICHA]

const endMessage = (text: string) => {
    if (/
|^\[club\d+\|.*?\]/.test(text)) {
      return text;
    }
    
    let newText = text;
    
    if (/[a-zа-яё_]/i.test(text[0])) {
      newText = `${text[0].toUpperCase()}${text.slice(1)}`;
    }
    
    if (/[a-zа-яё_]/i.test(text[text.length - 1])) {
      newText = `${newText}.`;
    }
    
    return newText;
};

Модератор, неудобные комментарии удалять нехорошо.)

[WbICHA]

Хотя лучше так:

const endMessage = (text: string) => {
    if (/
|^\[club\d+\|.*?\]/.test(text)) {
      return text;
    }

    const firstSymbol = /[a-zа-яё_]/i.test(text[0]) ? text[0].toUpperCase() : text[0];
    const lastSymbol = /[a-zа-яё_]/i.test(text[text.length - 1]) ? '.' : '';

    return `${firstSymbol}${text.slice(1)}${lastSymbol}`
};

const endMessage = (text: string) => {
    if (/
|^\[club\d+\|.*?\]/.test(text)) {
      return text;
    }

    return `${text[0].toUpperCase()}${text.slice(1)}${/[a-zа-яё_]/i.test(text[text.length - 1]) ? '.' : ''}`
};

[ostrooovskyy]

Спасибо за краткое решение!

Answer 1

[Станислав Макаров]

• на функции побейте, с вменяемыми названиями
  
• литералам регулярок можно тоже дать нормальные имена, сейчас не разберёшь что собственно тестится регуляркой - приходится её читать и думать
  
• непонятно зачем тут стрелочная функция вместо обычной, ради чего весь этот синтаксический шум
  
• непонятно зачем это присваивание: newText = newText.substring(1);, чтобы что? Почему не завести отдельную переменную tailText или перенести это во второе выражение?
  
• лично я запутался в использовании text и newText - непонятно почему в последнем условии тестируется text[0], а хвост берём у newText. Выглядит так, что это сделано специально, но не пойму почему. А если НЕ специально - то вот вам пример как вы усложнили код на ровном месте.
  

Чем меньше вы пользуетесь сменой состояния, иными словами, чем больше у вас констант, а не переменных, и чем меньше присваиваний - тем код лучше читается. Я надеюсь вы не ради экономии чего-то там это делаете.

Comments

[WbICHA]

непонятно зачем тут стрелочная функция вместо обычной, ради чего весь этот синтаксический шум

Это же шутка, да?

[Aetae]

WbICHA, ну так ответ - потому что модно-молодёжно. Обычная функция - сразу видно что функция, писать же стрелки где ни попадя - хипстерство. Change my mind.)

[WbICHA]

Aetae, точно так же прекрасно видно... Конечно, если вы не в блокноте пишите. В вскоде со стандартным хайлайтом функции жёлтого цвета, как их можно перепутать с синими переменными?

[ostrooovskyy]

Aetae, интересный комментарий. Но зачем вы это пишете?