Ответ
Да, ревьювил код напарников. Это важная часть процесса разработки, которая помогает выявить баги, улучшить качество кода и обменяться знаниями.
На что обращаю внимание:
- Читаемость и стиль кода (Kotlin/Java guidelines)
- Оптимизация (избегание лишних операций, memory leaks)
- Архитектурные принципы (SOLID, Clean Architecture)
- Тестируемость (мокинг, изолированность логики)
Пример:
// Было:
fun loadData() {
// смешение логики и UI
}
// Стало:
fun loadData() = viewModelScope.launch {
repository.fetchData().collect { data ->
_uiState.value = UiState.Success(data)
}
} Всегда стараюсь давать конструктивную обратную связь.
Ответ 18+ 🔞
Ёпта, ну вот сидишь, значит, ревьювишь код этих... напарников. А что делать-то, важная же часть работы, блядь. Иначе потом вылезет какая-нибудь манда с ушами, и всем будет хиросима и нигерсраки. Так что это не просто придирки, а реально помогает баги выловить, код улучшить и мозги друг другу пошевелить, понимаешь?
Ну и на что я, блядь, смотрю, как орёл:
- Читаемость и стиль. Чтобы код был не как хуй в пальто — непонятно, зачем и почему. Слежу за гайдлайнами, чтобы всё по-человечески.
- Оптимизация. Чтобы ничего лишнего не делал, память не жрал как не в себя. Искаю места, где можно впендюрить поаккуратнее.
- Архитектура. SOLID там, Clean Architecture. Чтобы не было одной большой функции, где всё в кучу свалено — пиздопроебибна получается, а не код.
- Тестируемость. Чтобы логику можно было нормально замокать и потестить. Если всё завязано на сто зависимостей — это доверия ебать ноль, что оно работать будет.
Вот, смотри, пример из жизни, прямо как вчера было:
// Было: пиздец, смешали всё в одну кучу, логику с UI.
fun loadData() {
// тут и сетевой запрос, и обновление TextView, и ещё бог знает что
}
// Стало: нормальный, человеческий подход.
fun loadData() = viewModelScope.launch {
repository.fetchData().collect { data ->
_uiState.value = UiState.Success(data) // чистое состояние, красота!
}
}Главное в этом деле — не быть пидарасом шерстяным, который просто так придирается. Всегда стараюсь объяснить, почему так лучше, дать нормальную, конструктивную обратную связь. Чтобы человек не охуел от комментария, а реально понял и в следующий раз написал красивше. А иначе какой смысл, да похуй тогда вообще?