Доклад является сбором опыта code review и самых частых косяков, которые я находил за последнее время. Капитанство этой презентации зашкаливает, но я до сих пор не понимаю, почему многие так все ровно пишут.

1. Код
Чебулаев Олег,
Perpetuum Mobile Lab
CocoaHeads 2016

2. Литература

3. Грязный код
• Увеличивает количество ошибок в приложении
• Снижает производительность и скорость
разработки
• Увеличивает стоимость поддержки и дальнейших
изменений

4. Чистый код
«Чистый код прост и
прямолинеен.
Чистый код читается, как
хорошо написанная проза.
Чистый код никогда не
затемняет намерения
проектировщика; он полон
четких абстракций и простых
линий передачи управления»Грэди Буч

5. Главный принцип
именования
Имя должно полно и точно описывать
сущность, представляемую
переменной

6. Передают намерения
программиста
NSObject *dataObject = [self
dataObjectWithRow:indexPath.row];
CustomLabel *customLabel = [CustomLabel
new];
id<SectionDescription> sectionDescription = [self
sectionDescriptionByRow:indexPath.row];
ScrollableLabel *ticker = [ScrollableLabel new];

7. Не дают ложную
информацию
NSArray *usersList = [NSArray new];
NSArray *usersList = [NSDictionary new];

8. Длинна имени
Должна соответствовать размеру его области видимости
// StyleDefines.h
static CGSize const TransportSectionCellIconSize
= {18.0f, 18.0f};
// StyleDefines.h
static CGSize const iconSize = {18.0f, 18.0f};

9. Соблюдайте порядок слов
и окончания
NSArray *dreamsList;
pagesCount или countOfPages
NSArray *dreamList;
countPage;
Прилагательное+Существительное

10. Числа
Никаких чисел в названиях
переменных

11. Имена классов
• Состоят из существительных, прилагательных и их
комбинаций
• Отражают суть намерений
ScrollableLabel *ticker;
CustomCellBackground *backgorund;
Users

12. Одно слово для одной
концепции
Points, Coins, Rewards, Stars
Appointment, record, registration, reservation

13. Часто используемые
антонимы
add/remove
increment/decrement
open/close
begin/end
insert/delete
show/hide
create/destroy
lock/unlock
source/target
first/last
min/max
start/stop
get/put
next/previous
up/down
get/set
old/new

14. Конвенции именования
NYTimes Style Guide
https://github.com/NYTimes/objective-c-style-guide
Raywenderlich Style Guide
https://github.com/raywenderlich/objective-c-style-guide

15. Конвенции везде
• Исходные коды
• Файлы локализации
• Именорование ресурсов

16. Разумные причины создать
метод
• снижение сложности;
• формирование понятной
промежуточной абстракции;
• предотвращение дублирования
кода;
• поддержка наследования;
• сокрытие очередности действий;
• сокрытие операций над
указателями;
• улучшение портируемости;
• упрощение сложных булевых
проверок;
• повышение быстродействия
• изоляция сложности;
• сокрытие деталей реализации;
• ограничение влияния
изменений;
• сокрытие глобальных данных;
• создание центральных точек
управления;
• облегчение повторного
использования кода;
• выполнение специфического
вида рефакторинга.

17. Ментальные барьеры
if (user.sessionToken)
VS
if ([user isAuthenticated])

18. Side эффекты функций
- (BOOL)isValidEmail:(NSString *)email
- andPassword:(NSString *)password
{
if ([email isEqualToString:@""] || [email isEqualToString:@""]) {
return NO;
}
self.session = [PMSession sessionWith:email
password:password];
return YES;
}

19. Псевдокод

20. Комментарии

21. Принципы
GRASP
SOLID
CQS
Law of Demeter
Единый уровень абстракции

22. Литература
Блог Александра Бындю http://blog.byndyu.ru/

23. Вопросы?