Раскрывающие намерение имена

Называй по концепции, а не по структуре данных, которая её случайно держит. Имя должно описывать чем эта сущность является в предметной области, а не её представление во время выполнения. Зашивание типа в имя (accountList, accountArray, userMap) привязывает имя к решению, которое часто меняется, — и когда ты заменишь Array на Set, имя начинает врать.

// тип зашит: имя следит за контейнером, а не за концепцией
const accountList: Account[] = await fetchAccounts();
for (const a of accountList) { /* ... */ }

// названо по концепции: читается как предметная область, переживает смену представления
const accounts: Account[] = await fetchAccounts();
for (const account of accounts) { /* ... */ }

accounts стоит во множественном числе, поэтому уже подразумевает коллекцию — суффикс List/Array добавляет ноль информации и одну обузу сопровождения. Называй роль (activeAccounts, accountsToCharge), когда коллекция — это подмножество; это различие — реальная информация о предметной области, в отличие от типа контейнера.

Удаляй слова-шум — они раздувают имя, не уточняя его. Слова вроде data, info, value, object, manager, processor и избыточный пересказ контекста (user.userId, Account.accountName) не несут информации, которую окружающий код уже не давал бы. Если удаление слова не меняет, на какую сущность ссылается имя, — это слово шум.

// шум: каждый суффикс пересказывает контекст, уже бывший в области видимости
interface UserData {
  userId: string;
  userName: string;
  userAccountInfo: AccountInfo;
}
const userDataManager = new UserDataManager();

// сигнал: тип И ЕСТЬ контекст; поля его не повторяют
interface User {
  id: string;
  name: string;
  account: Account;
}
const users = new UserRepository();

user.id однозначно — ты уже знаешь, что это id пользователя, из переменной. UserDataManager — худший нарушитель: Data и Manager оба заглушки для «я не решил, что это делает». Имя, которое не желает зафиксировать ответственность, — это запах дизайна, носящий ярлык.

Заменяй магические числа именованными константами — имя несёт зачем. Голый литерал заставляет каждого читателя реверс-инжинирить его смысл, а продублированный литерал скрывает, какие вхождения — одна и та же концепция. Именованная константа превращает значение в искабельный факт с единственным источником истины и бесплатно документирует намерение.

// магия: что такое 3? почему 86400000? эти две пятёрки — одна и та же 5?
if (attempt > 3) lock();
if (Date.now() - issuedAt > 86400000) expire();

// именовано: каждая константа отвечает «почему именно это число?»
const MAX_RETRIES = 3;
const ONE_DAY_MS = 24 * 60 * 60 * 1000;
if (attempt > MAX_RETRIES) lock();
if (Date.now() - issuedAt > ONE_DAY_MS) expire();

Выигрыш двойной: читатель понимает порог без комментария, а будущее изменение («повторов → 5») — это одна правка в определении, а не охота-и-замена по точкам вызова, которые могли совпасть с несвязанной 3. Именование числа ещё и разделяет два литерала, выглядевших одинаково, но означающих разное, — класс багов, который магические числа активно скрывают.

Булевы — это предикаты; кодируй единицы измерения и валюты в имени. Булево должно читаться на точке вызова как вопрос «да/нет»: добавь префикс is, has, should или can, чтобы if (expired) стало if (isExpired), а параметр flush стал shouldFlush. И любая величина с единицей измерения должна говорить свою единицу в имени — timeoutMs, priceCents, distanceKm, sizeBytes, — потому что именно в единице прячутся продакшен-баги.

// неоднозначно: `timeout` в секундах или мс? `price` в долларах или центах?
function retry(timeout: number, price: number, active: boolean) {
  setTimeout(fn, timeout); // ...а вызывающий передаёт секунды? БАБАХ.
}

// безошибочно: единица и полярность — в имени
function retry(timeoutMs: number, priceCents: number, isActive: boolean) {
  setTimeout(fn, timeoutMs);
}

setTimeout хочет миллисекунды; вызывающий, передающий 30 с мыслью «30 секунд», вносит баг, который не ловит ни один проверяльщик типов, — number есть number. timeoutMs делает контракт невозможным для неверного прочтения, и баг с ошибкой в 1000 раз просто не пишется. Единицы в именах — не педантизм; это разница между таймаутом в 30мс и 30с в проде.