Что лучший инструмент должен обнаружить утечки памяти в [закрытом] Delphi

Как все отметили, команда игроков не является списком игроков. Эта ошибка совершается многими людьми во всем мире, возможно, на разных уровнях знаний. Часто проблема тонкая, а иногда и очень грубая, как в этом случае. Такие проекты плохи, потому что они нарушают Принцип замещения Лискова. В Интернете есть много хороших статей, объясняющих эту концепцию, например http://en.wikipedia.org/wiki/Liskov_substitution_principle

Итак, есть два правила, которые должны быть сохранены в отношениях между родителями и детьми между классами:

• a Ребенок не должен иметь никакой характеристики, которая меньше, чем то, что полностью определяет родителя.
• Родитель должен не требовать характерный в дополнение к тому, что полностью определяет ребенка.

Другими словами, родитель является необходимым определением для ребенка, а ребенок является достаточным определением родителя.

Вот способ продумать одно решение и применить вышеприведенный принцип, который должен помочь избежать такой ошибки. Следует проверить гипотезу, проверяя, действительны ли для всех классов родительский класс для производного класса как структурно, так и семантически.

• Является ли футбольная команда списком футболистов? (Все ли свойства списка относятся к команде в том же значении) Является ли команда коллекцией однородных объектов? Да, команда - это сборник игроков. Является ли порядок включения игроков в описание состояния команды и обеспечивает ли команда сохранение последовательности, если она явно не изменена? Нет, и Нет. Ожидается, что игроки будут включены / сброшены на основе их последовательной позиции в команде? Нет

Как вы видите, только первая характеристика списка применима к команде. Следовательно, команда не является списком. Список будет деталью реализации того, как вы управляете своей командой, поэтому ее следует использовать только для хранения объектов игрока и манипулировать с помощью методов класса Team.

На этом этапе я хотел бы отметить что класс Team должен, на мой взгляд, даже не реализовываться с использованием List; в большинстве случаев он должен быть реализован с использованием структуры данных Set (например, HashSet).