artbear писал(а) 22. Июля 2008 :: 08:45:

Ну, родил типа наконец интерфейсы и реализацию для векторов.
Получилось, на мой взгляд, нормально.

Если поглядеть на код Вектор::Загрузить() и Вектор::Выгрузить(), то становится довольно забавно.
Когда-нибудь две телефонные трубки соединять доводилось, чтобы дружбаны могли между собой побазарить?
Вот так, коллекции ничего не нужно, кроме реализации двух симметричных интерфейсов.
То же самое можно сделать для остальных контейнеров и прокси-объектов.

По интерфейсам: в принципе, можно использовать.
Я постарался дать развернутое описание, и здесь важно в реализации его поддержать аккуратно.
В том числе - корректно обработать критичные и некритичные ошибки и 0, возвращаемый Init в nRowCount.

Насчет nRowCount у меня остались сомнения, нужен ли он вообще.
Да, теоретически его можно где-то применить для оптимизации получения данных.
Но случай такой так и не представился, так что я серьезно задумываюсь, стоит ли здесь усложнять интерфейс.

Еще описание Загрузить в доку забыл занести.

kms писал(а) 29. Июля 2008 :: 08:06:

Так ты прочти выше мою фразу.
Я предлагаю юзать Done только при успешном завершении выгрузки!
Done уже пригодился для выгрузки в ТЗ на форме.
Еще может пригодиться для какого-нибудь контейнера, который может каким-то образом оптимизировать выгрузку, например, кешируя данные в ldrAddRow и отправляя их окончательно только в Done - т.е. скорость за счет памяти и т.д.

т.е. Done - нужная вещь

artbear писал(а) 29. Июля 2008 :: 08:39:

Знаешь, я подумал и согласился.
На самом деле, независимо от результата операции, Done нужно вызывать, чтобы закрыть какие-то сетевые подключения, освободить какие-либо блокировки и т.д.

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

P.S.
Да, и Done - теперь типа деструктора, исключений не бросает и ошибок не возвращает.
Соответственно, инициатор коды ошибок Done проверять не обязан.

kms писал(а) 29. Июля 2008 :: 09:16:

"Но теперь Done - обязателен в том числе и при неуспешном завершении." - так сделаем?
Тогда нужно исправлять реализации универсальной выгрузки для ИТЗ - Выгрузить и ВыгрузитьСтроку + коллекции (Вектор/АссоцВектор).
Ты их исправлял?

Исправил остальное - ldrDone сделан обязательным.

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