Clean code

// On ne sait pas ce que sont 'p', 'l' ou 'v'
const p = 150; 
const l = items.map(i => i.price).reduce((a, b) => a + b, 0);

if (l > p) {
  const v = l * 0.1;
  const t = l - v;
  console.log(t);
}

const MINIMUM_ORDER_FOR_DISCOUNT = 150;
const DISCOUNT_RATE_PERCENTAGE = 0.1; // 10%

const orderTotal = cartItems.reduce((total, item) => total + item.price, 0);

if (orderTotal > MINIMUM_ORDER_FOR_DISCOUNT) {
  const discountAmount = orderTotal * DISCOUNT_RATE_PERCENTAGE;
  const finalPriceWithDiscount = orderTotal - discountAmount;
  
  console.log(finalPriceWithDiscount);
}

// Un service qui fait trop de choses
async handleUserSignup(data: any) {
  // 1. Logique de validation métier
  if (data.password.length < 8) throw new Error("Trop court");

  // 2. Persistance (Base de données)
  const user = await this.userRepository.save(data);

  // 3. Logique de communication (Email)
  const message = `Bienvenue ${user.firstName}`;
  await this.emailService.send(user.email, "Bienvenue !", message);

  // 4. Logique de marketing (Tracker)
  await this.analytics.track('user_created', user.id);
}

async createUser(data: CreateUserDto) {
  const user = await this.repo.save(data);
  
  // On émet un événement. C'est tout.
  this.eventEmitter.emit('user.created', user); 
  
  return user;
}

@OnEvent('user.created')
handleUserCreatedEvent(user: User) {
  const welcomeMessage = this.templateService.buildWelcome(user.name);
  this.emailService.send(user.email, "Bienvenue", welcomeMessage);
}

@OnEvent('user.created')
logUserAnalytics(user: User) {
  this.analytics.track('new_registration', { userId: user.id });
}

const UserProfile = () => (
  <div style={{ padding: '10px', border: '1px solid black' }}>Profil Utilisateur</div>
);
const AdminProfile = () => (
  <div style={{ padding: '10px', border: '1px solid black' }}>Profil Admin</div>
);

const Card = ({ children }) => (
  <div className="card-styles">{children}</div>
);

const UserProfile = () => <Card>Profil Utilisateur</Card>;

// Vérifie si l'utilisateur est actif et a plus de 18 ans
if (u.status === 'active' && u.age > 18) { ... }

const isEligibleAdult = user.isActive && user.isAdult;
if (isEligibleAdult) { ... }

class Invoice {
  calculateTotal(items) { /* ... */ }
  generateHTML() { /* ... */ } // ❌ Trop de responsabilités : l'affichage
  saveToDatabase() { /* ... */ } // ❌ Trop de responsabilités : le stockage
}

// Si on veut ajouter "Espagne", on DOIT modifier cette fonction
function getShippingCost(country: string) {
  if (country === 'France') return 5;
  if (country === 'Belgium') return 7;
  if (country === 'USA') return 15;
  // ... C'est le début d'un enfer de maintenance
}

// 1. On définit une règle générale
interface ShippingStrategy {
  calculate(): number;
}

// 2. On crée des extensions pour chaque cas
class FranceShipping implements ShippingStrategy {
  calculate() { return 5; }
}

class USAShipping implements ShippingStrategy {
  calculate() { return 15; }
}

// 3. Le code principal est "fermé" : il ne change jamais, 
// peu importe le nombre de pays ajoutés.
function displayCost(strategy: ShippingStrategy) {
  console.log(strategy.calculate());
}

src/
 └── users/
     ├── dto/              # Validation des entrées
     ├── entities/         # Schéma de base de données
     ├── users.controller.ts
     ├── users.service.ts
     └── users.module.ts