есть метод AddPort и возвращает Tport
Тоже ведь обычное дело?
На каждый сервер в списке добавит по tports и по одному tport с Inservise=false
нет, не обычное ))) нарушается принцип инкапсуляции
получается, что ты рулишь внутренностями сервера/портами "снаружи".. по сути, устанавливаешь поля класса вне самого класса - это ппц как неправильно с точки зрения контроля
Тут смотреть в контексте надо. Кстати, присваивание False полю это инициация? Тогда ее можно внедрить в конструктор объекта TPort. Вернее даже не нужно это делать, так как итак проинициируется этим значением. AddCards неговорящее название функции. Вернее неправильно говорящие. Я бы все-таки сделал просто Cards (необязательно как свойство, можно оставить функцию) ну или GetCards
for i := 0 to Servers.Count - 1 do with Servers.Data[i].AddPort do KeyData['inservice'] := False;
Обсуждают сегодня