Приветствую любящих программирование :) !
Есть такое понятие, как умение использовать принципы ООП. Так вот хочется:
расширить свое представление о том, что такое плохо в общем случае и почему оно плохо;
натренироваться на практике распознавать нехорошие ситуации и понимать, в чем именно проблема, к каким сложностям она ведет и как ее устранить.
Чтобы было понятнее, какого характера знания-умения хочется получить, приведу несколько примеров :) :
Что видим в коде? Какие сложности ждут? Где локализуется проблема? Как устранить? Название методы интерфейса IInt работают группами, группы почти не пересекаются
а) при использовании интерфейса некоторые методы все время мешаются, не нужны даже гипотетически;
б) при чтении непонятно, какая часть методов будет задействована в данном куске кода само объявление интерфейса и все классы, использующие его выделить в IInt группы методов, работающих вместе, и разнести эти группы в разные интерфейсы сопряжение нескольких сущностей в одном интерфейсе вместо четкого определения конкретной функциональностиprivate-метод класса ClassParent использует абстрактный метод, реализованный в классе ClassChildBase private-метод воспринимается как вспомогательный метод с четко фиксированным поведением, а на самом деле любой может изменить это поведение через наследников общая структура ClassParent, контракты ClassParent и ClassChildBase может быть, в данной ситуации так и надо? возможность изменить поведение родителя через его наследников
Еще примеры Что видим в коде? Какие сложности ждут? Где локализуется проблема? Как устранить? Название класс-родитель и его наследники, в наследниках много похожих/одинаковых реализаций методов а) сложнее выделить специфику каждого из наследников, она "замутнена" лишним кодом; б) дублирование кода со всеми вытекающими конкретные методы в классах-наследниках вынести "одинаковость" в родителя и оставить в нем абстрактный ключик, ведущий в наследников и отвечающий за непохожесть неудачное распределение функциональности между наследником и родителемCEntityBase реализует IEntity, имеет наследников CEntity1 и CEntity2, не имеет собственных полей и методов CEntityBase как-то дополняет/конкретизирует/расширяет функциональность IEntity? когда нужно использовать интерфейс, а когда - базовый класс? контракты IEntity и CEntityBase оставить или IEntity, или CEntityBase в зависимости от контекста размножение сущностей без необходимостиметоды класса получают на вход объект; иногда класс сам проводит подготовительные-проверяющие действия для объекта, а иногда считает, что ему на вход поступает уже правильный объект а) поведение не определено точно и неизвестно, чего ожидать - класс сложно использовать; б) проверка-подготовка то проводится несколько раз, то не проводится ни одного; в) размазывание ответственности сам контракт класса и все контексты, в которых он использовался повешаться или четко распределить ответственность и привести весь код в соответствие этому распределению невнятный контракт, нечеткое определение зоны ответственности
Надеюсь, более-менее понятно описала, что хочется :) . Причем нужно
не прочитать готовый анализ или вообще готовое решение (как в
известной книге четырех ), а пропустить все через себя,
пройти (почти) весь путь анализа самостоятельно .
Пока знаю ровно два способа такие навыки развить:
разглядывание существующего кода и попытки сделать что-то с ним (если коряво спроектировано, сразу же почувствуешь, где препятствия ; если все хорошо, то потребуется больше усилий, чтобы понять, засчет чего так получается);
беседа с опытными и хорошими товарищами (садится рядом, обращает внимание на какую-то часть кода, предлагает подумать о сложностях сейчас, при разрастании и при изменении требований, о способах устранения, может помогать наводящими вопросами, дополняет, ...).
Буду безумно счастлива, если удастся найти нечто, дающее более целенаправленный и быстрый результат, чем разглядывание существующего кода, и поменьше нагружающее других людей.
Может быть, есть какие-то книги-тренажеры (
"вот, смотрите, код... какая проблема в нем? подумайте... теперь можете подсмотреть в мой ответ... а как решить проблему? ну-ну, давайте, догадывайтесь... да, именно так! а как еще эта проблема проявляется? можете другие примеры кода придумать?" )?
Или какие-то сайты, на которых собраны плохие примеры, их анализ и способы исправления?
