📝 код-ревью 08: разобрал fixme-комменты из задачи — типы, маппинг доменов в конфиг

Author: esemi

code-review-practice/solutions/08_email_validator.md

@@ -31,6 +31,14 @@
 
 8. `if "@" not in email` избыточен: следующая проверка `len(parts) != 2` уже это ловит.
 
+9. **Нет аннотаций типов** (все три функции). `email: str -> bool`,
+   `emails: list[str] -> list[str]` — контракт валидатора должен читаться из сигнатуры.
+
+10. **Маппинг доменов захардкожен** (`normalize_email`, строка 32). `if domain == "googlemail.com": domain = "gmail.com"` —
+    одиночный `if` для алиаса домена. Таких алиасов много; их место — в словаре-константе
+    (`DOMAIN_ALIASES = {...}`) или конфиге, а не в ветвлении. Тогда добавление нового алиаса
+    не трогает код.
+
 **Итого:** главное — поймать рассинхрон «валидируем сырое, нормализуем изменённое» и
 порядок операций, плюс набор питон-идиом (`is None`, `== True`). Валидатор email руками —
 сам по себе повод для комментария на ревью.