Упражнения Модуля 1

Content Coupling. TodoService обращается к приватному полю cache через as any. Это патологическая связанность — модуль ковыряется во внутренностях другого.

Исправление:

class TodoService {
  constructor(private userService: UserService) {}

  getUserTodos(userId: string) {
    const user = this.userService.getUser(userId)  // Публичный метод
    if (!user) throw new Error("User not found")
    return this.fetchTodos(userId)
  }
}

Ещё лучше — зависеть не от UserService, а от минимального интерфейса:

interface UserChecker {
  readonly exists: (id: string) => boolean
}

class TodoService {
  constructor(private users: UserChecker) {}

  getUserTodos(userId: string) {
    if (!this.users.exists(userId)) throw new Error("User not found")
    return this.fetchTodos(userId)
  }
}

Теперь — Data Coupling: TodoService зависит только от boolean, который возвращает exists.

Common Coupling. Оба модуля зависят от одного глобального объекта APP_CONFIG. Изменение структуры конфигурации (переименование поля, разделение на секции) затрагивает оба.

К тому же todo-service имеет доступ к jwtSecret, что ему совершенно не нужно (Stamp Coupling поверх Common).

Исправление — каждый модуль получает только то, что ему нужно:

// todo-service.ts
function createTodo(title: string, maxTodos: number) {
  if (countTodos() >= maxTodos) throw new Error("Limit reached")
}

// auth-service.ts
function verifyToken(token: string, secret: string) {
  return jwt.verify(token, secret)
}

С Effect:

const TodoConfig = Effect.Config.number("MAX_TODOS")
const AuthConfig = Effect.Config.string("JWT_SECRET").pipe(Config.map(Redacted.make))

Каждый модуль декларирует свою конфигурацию независимо — Data Coupling.

Control Coupling. Параметр action управляет внутренней логикой функции. Вызывающий код знает, какие действия поддерживаются, и управляет поведением через строковый флаг.

Исправление — отдельная функция для каждого действия:

const completeTodo = (todo: Todo): Todo =>
  ({ ...todo, status: "completed" as const })

const archiveTodo = (todo: Todo): Todo =>
  ({ ...todo, status: "archived" as const })

const reopenTodo = (todo: Todo): Todo =>
  ({ ...todo, status: "active" as const })

Каждая функция — чистая, с единственной ответственностью. Data Coupling: принимает Todo, возвращает Todo.

Coincidental Cohesion. Пять функций, не связанных друг с другом: форматирование дат, хеширование паролей, генерация slug, математика, отправка email. Они оказались вместе случайно.

Рефакторинг — разместить каждую функцию рядом с её доменом:

domain/todo/todo.ts          → generateSlug (если slug — свойство Todo)
domain/auth/password.ts      → hashPassword
domain/shared/date-format.ts → formatDate
domain/shared/math.ts        → clamp (если вообще нужен)
adapters/email/mailer.ts     → sendEmail

Или, если использовать Effect-подход: hashPassword — эффект (Bun.password.hashSync), sendEmail — эффект. Они должны быть адаптерами, а не утилитами.

Functional Cohesion. Все элементы работают с одной сущностью Todo и обслуживают её жизненный цикл: создание, завершение, проверки. Это целевое состояние — рефакторинг не требуется.

Граф зависимостей:

domain/todo.ts → bun:sqlite        ← НАРУШЕНИЕ! Домен зависит от инфраструктуры
adapters/http-server.ts → domain/todo.ts  (допустимо)
adapters/http-server.ts → bun:sqlite      (допустимо для адаптера)

Нарушения:

1. domain/todo.ts импортирует bun:sqlite — домен знает о базе данных.
2. Функция completeTodo принимает Database — бизнес-правило связано с SQLite.
3. SQL-запрос внутри доменной функции.

Исправление:

// domain/todo.ts — НОЛЬ импортов
export interface Todo { readonly id: string; readonly title: string; readonly done: boolean }

export const completeTodo = (todo: Todo): Todo => {
  if (todo.done) throw new Error("Already done")
  return { ...todo, done: true }
}

// ports/todo-repository.ts
import type { Todo } from "../domain/todo"
export interface TodoRepository {
  readonly findById: (id: string) => Promise<Todo | null>
  readonly save: (todo: Todo) => Promise<void>
}

// adapters/sqlite/todo-repo.ts
import type { TodoRepository } from "../../ports/todo-repository"
import { Database } from "bun:sqlite"
// ... реализация

// app/complete-todo.ts
import { completeTodo } from "../domain/todo"
import type { TodoRepository } from "../ports/todo-repository"

export async function handleCompleteTodo(repo: TodoRepository, id: string) {
  const todo = await repo.findById(id)
  if (!todo) throw new Error("Not found")
  const completed = completeTodo(todo)
  await repo.save(completed)
  return completed
}

Новый граф:

domain/todo.ts           → (ничего)           ✅
ports/todo-repository.ts → domain/todo.ts      ✅
app/complete-todo.ts     → domain/todo.ts      ✅
app/complete-todo.ts     → ports/todo-repo     ✅
adapters/sqlite/         → ports/todo-repo     ✅
adapters/sqlite/         → bun:sqlite          ✅ (адаптеру можно)
adapters/http/           → app/complete-todo   ✅

Все стрелки указывают внутрь, к домену. Ни одна стрелка не идёт наружу от домена.

Подсчёт: Базовая сложность = 1. Считаем каждый if, else if, &&, ||, ?.:

• if (req.method === "GET" && req.path === "/todos") → +2
• else if (req.method === "GET" && req.path.startsWith(...)) → +2
• if (!id) → +1
• if (!todo) → +1
• else if (req.method === "POST" && req.path === "/todos") → +2
• if (!req.headers[...]?.includes(...)) → +1
• if (!req.body || typeof req.body !== "object") → +2
• if (!body.title || body.title.length === 0) → +2
• if (body.title.length > 200) → +1
• else if (req.method === "PUT" && ...) → +2
• if (!id) → +1
• else if (req.method === "DELETE" && ...) → +2
• if (!id) → +1

Цикломатическая сложность ≈ 21. Это в два раза выше допустимого порога.

Рефакторинг — разделение по ответственностям:

// Роутер — O(n) по количеству маршрутов, сложность 1 на маршрут
type Handler = (params: RouteParams) => Response

const routes: ReadonlyArray<{ method: string; pattern: RegExp; handler: Handler }> = [
  { method: "GET",    pattern: /^\/todos$/,       handler: handleListTodos },
  { method: "GET",    pattern: /^\/todos\/(.+)$/,  handler: handleGetTodo },
  { method: "POST",   pattern: /^\/todos$/,       handler: handleCreateTodo },
  { method: "PUT",    pattern: /^\/todos\/(.+)$/,  handler: handleCompleteTodo },
  { method: "DELETE", pattern: /^\/todos\/(.+)$/,  handler: handleDeleteTodo },
] as const

// Каждый обработчик — сложность ≤ 3
function handleCreateTodo(params: RouteParams): Response {
  const parsed = parseTodoInput(params.body)    // Schema.decode
  if (Either.isLeft(parsed)) return badRequest(parsed.left)
  return created(createTodo(parsed.right.title))
}

Цикломатическая сложность каждой отдельной функции: ≤ 5. Общая логика разложена на compose-способные части.

Структура папок:

src/
  domain/
    todo/
      todo.ts              — Entity: Todo, TodoId, TodoStatus, Priority
      todo-events.ts       — Events: TodoCreated, TodoCompleted, TodoArchived
      todo-errors.ts       — Errors: TodoNotFound, InvalidTransition, DuplicateTitle
  ports/
    todo-repository.ts     — Driven Port: CRUD для Todo
    notification.ts        — Driven Port: отправка уведомлений
    clock.ts               — Driven Port: текущее время (детерминизм)
  app/
    create-todo.ts         — Use Case: создание задачи
    complete-todo.ts       — Use Case: завершение задачи
    archive-todo.ts        — Use Case: архивация
    delete-todo.ts         — Use Case: удаление
    list-todos.ts          — Use Case: получение списка
    get-todo.ts            — Use Case: получение одной задачи
  adapters/
    sqlite/
      sqlite-client.ts     — Инфраструктура: подключение к SQLite
      todo-repo-sqlite.ts  — Adapter: реализация TodoRepository для SQLite
      migrations.ts        — Миграции схемы БД
    http/
      todo-routes.ts       — Adapter: HTTP-маршруты для Todo
      error-mapper.ts      — Маппинг доменных ошибок → HTTP-кодов
    notification/
      console-notifier.ts  — Adapter: вывод в консоль (dev)
      email-notifier.ts    — Adapter: email (production)
    test/
      todo-repo-memory.ts  — Adapter: in-memory (тесты)
      noop-notifier.ts     — Adapter: no-op (тесты)
  main.ts                  — Точка сборки: Layer composition

Граф зависимостей:

domain/todo/*           → (ничего)
ports/*                 → domain/todo/*
app/*                   → domain/todo/*, ports/*
adapters/sqlite/*       → ports/todo-repository, domain/todo/*
adapters/http/*         → app/*, domain/todo/* (для типов ошибок)
adapters/notification/* → ports/notification
adapters/test/*         → ports/*
main.ts                 → app/*, adapters/*

Все стрелки от внешних слоёв к внутренним. Домен ни от чего не зависит.

Порты:

Driving (входные):

• Use Case интерфейсы в app/ — API для внешнего мира

Driven (выходные):

• TodoRepository — персистентность
• NotificationService — уведомления
• Clock — текущее время (для детерминированных тестов)