А индексированная таблица не спасает?

УстановитьМодельХранения - это безусловно тонкий и нужный инструмент.
только глобального толку думаю мало будет. подавляющее число 1с-ников не работают на таком уровне (вплоть все до мелочей...).

steban

Степ, поддержу здесь Степанова в том плане, что не получится сделать единую коллекцию и не будет единого интерфейса.
Ибо все коллекции того же STL (а их вполне можно взять за основу) - разные.

Даже итераторы для листа нельзя сравнивать на <>, только на == и !=, а для вектора, к примеру, можно.
И это правильно.

К примеру, как будет работать предлагаемая коллекция, у которой ТипКлюча == число, а МодельХранения == список?
Если она не будет работать, то теряет смысл общий интерфейс.
Если она будет работать, но с затратами N - то это уже совсем плохо.

Таким образом, думаю, надо просто определиться, какие коллекции нам нужны.
Мне лично не хватает одной - единственной коллекции, о которой я вчера уже упомянул.

Это вектор, заодно индексированный строкой ("ИменованныйВектор").
Можно будет использовать вместо структуры и СЗ, в том числе во множестве мест в ИТ.
Прост в реализации, может вырождаться в просто "Вектор", если задавать пустые ключи, к примеру.
Прост в применении, ибо жестко не требует наличия итераторов, достаточно индексов.
Для реализации в icpp уже есть все, что нужно (CIStringMap/std::vector (или std::deque)).
И т.д.

Именованный вектор также можно использовать как hash_set / hash_map.
В принципе, структура - это по-сути и есть hash_map.

Конечно, есть задачи, где требуется вставка и удаление в начало (deque это решает), или в середину.
Над списками и прочим можно еще подумать.
Но еще интереснее подумать над алгоритмами, которые можно будет к этим контейнерам применить.

Блин, лучше сделайте в 1С++ закладки и панель инструментов, не зависимые от офисов и прочей хрени
А структура сейчас достаточно хорошо работает, её хватает .

trdm писал(а) 04. Июля 2008 :: 08:44:

Для некоторых задач недостаточно хорошо

trdm писал(а) 04. Июля 2008 :: 08:44:

Дык это очень просто.
Я перестал заморачиваться и начал юзать ТП с плоской границей.
Только, кажется, я не до конца еще в CVS это залил.
Значит, будет.

А вот с коллекциями - мне не нравится то, что делал Deb.
Все, что можно было переделать, уже переделали.
Мне даже не нравится Делегат, который когда-то вполне нравился.

Все потому, что в свое время Deb шарашил не заботясь об интерфейсе.
А надо было слушать Палыча, и только потом брать в руки перфоратор.

kms писал(а) 04. Июля 2008 :: 09:26:


Могу назвать много причин невозможности (ну лично для меня) такого перехода... но боюсь не по теме... вполне поддерживаю trdm в необходимости создания панели и закладок... вот реализовал же Chieftain панель управления, однако на дельфи... правда тема как то затерялась.
Кстати Миша, я там счас в ТП одну хотелку накидаю может она тебе интересной покажется.
А плоская граница в ТП есть в текущей сборке... и намано работает огромное спасибо   Правда там тоже есть пара "но" ... но я в другую тему пошел  

_{Пы.Сы. Арта оказался проворнее  
Пы.Сы.2 Прошу прощения Chieftain конечно же... ошибся в нике}

trdm писал(а) 04. Июля 2008 :: 08:44:

офф.
Дима, может это подойдет http://www.1cpp.ru/forum/YaBB.pl?num=1215168467

artbear писал(а) 04. Июля 2008 :: 10:37:

Ну как - например, как большинство коллекций, используемых в ТП.
Типа колонок или полей поставщика ODBC.

artbear писал(а) 07. Июля 2008 :: 04:57:

Зарегистрировал (icpp).

Описание соответствует коллекции "ИндексыКартинок":
http://www.1cpp.ru/docum/icpp/html/TableField.html#id52

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

Например, выгрузку в СЗ я сделал универсальной для всех коллекций.
Можно поддержать и дополнительные классы, ИТ например.
Но хотелось бы, чтобы это было на этот раз универсально, через внешние интерфейсы, типа ResultLoader'а - т.е. надо продумать.

Цитата:

Ну, это пока было в планах.
Давайте тоже сделаем.

Может тогда действительно сделать прокси-объекты для загрузки и выгрузки
- строки ТЗ
- колонки ТЗ
- строки ИТ
- колонки ИТ
- СЗ

При этом можно сразу спроектировать интерфейс так, чтобы заодно поддержать и коллекции, индексированные строкой.
Ну, к примеру, для строк ТЗ и ИТ ключом может являться имя колонки, для СЗ - представление.
Для колонок ТЗ и ИТ - хз, может быть, значение другой выбранной колонки?

artbear писал(а) 09. Июля 2008 :: 05:12:

Точнее так,
ИТЗ::ВыгрузитьСтроку(КудаВыгружаем, [Индекс = ""], [Колонки = ""], [НомерСтроки = ""])
для большей вариативности.
Внутренний интерфейс при этом не меняется.

ADirks писал(а) 09. Июля 2008 :: 05:58:

Ага
Ну, типа по аналогии - насколько возможно:

ЗаполнитьСтроку([чНомерСтроки = 0], Источник, [чНомерСтрокиИсточника = 0], [чПоИменамКолонок = 0])
 

ВыгрузитьСтроку(чНомерСтроки = 0, Приемник, [Колонки = ""])
 

[quote author=artbear link=1215152763/15#26 date=1215597543]
И кто реализует?
ЗЫ вопросов по интерфейсу ICollectionLoader нет?[/quote]
Все равно. :)
Но раньше, чем сделаю ассоциативные коллекции, я точно за это не возьмусь.

Если моё мнение как-то учитывается, то пусть коллекции нумеруются с 1.

[quote author=artbear link=1215152763/30#30 date=1215686367]
Михаил, вопрос по Вектору и коллекциям
1 - нумерация в открытых для пользователя коллекциях всегда идет с 0 ?
Не будет ли это неудобно в 1С, где привыкли к нумерации с 1.
Например, в ИТЗ и колонки, и строки нумеруются с 1.
[/quote]
Ну, в ТП так сделано.
В v8 так сделано.

Я за нумерацию с 0 - потом  легче будет. :)

[quote][quote]
2 - ты выгрузку в СЗ проверял?
у меня Вектор.Количество() = 2, а СЗ.РазмерСписка() = 1 после Вектор.Выгрузить(СЗ) :([/quote]
По выгрузке в СЗ все ок, я ошибся :(
[/quote]
Пустые теряются - СЗ так устроенО.

kms писал(а) 10. Июля 2008 :: 10:55:

Да, тут с одной стороны 1С 7.7, где всё нумеруется с единицы, а с другой - все остальные "нормальные" системы, включая 8-ку.
Я пошёл по пути наименьшего сопротивления  Если нумерация будет начинаться с нуля - пусть начинается, вам по-любому виднее. 

Реализован метод ИТЗ::ВыгрузитьСтроку(чНомерСтроки = 0, Приемник, [Колонки = ""]) в 3.0
http://www.1cpp.ru/bugs/show_bug.cgi?id=3978

В наличии тесты + дока

PS пока только для класса Вектор или его наследников, ну или для любых коллекций, которые доступны из 1С.

artbear писал(а) 10. Июля 2008 :: 13:27:

Далее по методу Процедура Вставить(Индекс, Значение)
Сейчас - если Индекс больше/равен количества элементов, он вставляется в конец вектора, независимо от переданного индекса, а не в позицию с указанным индексом.

Соответственно пользователь может быть неверно проинформирован, т.к. он может считать, что он успешно вставил элемент в позицию Индекс
Т.е. опять неоднозначность.

Это поведение метода CSimpleVector::Insert

Варианты исправления:
1. Сделать как в стандартной библиотеке шаблонов std - сделать метод функцией и возвращать номер позиции, куда был вставлен элемент. При этом количество элементов увеличивается так, чтобы новый элемент поместился в указанной позиции.
2. Или явно запретить вставку элементов, в позиции с индексом больше, чем количество текущих элементов.
В этом случае вектор будет расти только при явном изменении размера через Вектор::ИзменитьРазмер или Вектор::Добавить.

Мне лично нравится п.2, как более стабильный и четкий.

Жду ответа.

1. По параметризации выгрузки пустых - не, не будем делать.
Последние дни меня склонили к мысли, что такие параметры должны принадлежать прокси-объектам - загрузчикам результатов.
А интерфейс выгрузки должен быть простым, четким и односторонним (т.е. учитывать только интересы источника).

2. По параметрам по умолчанию - согласен.
Их там сейчас просто нет, это есть в планах.

3. По Вставке за границей.
Увеличивать размер - точно не будем.
Насчет исключения - ну можно, наверное... немного еще подумаю над этим.

P.S.
[quote author=artbear link=1215152763/30#36 date=1215696467]
это я тесты на Вектор пишу :) - специально опечатался, и сразу и нарвался :(
[/quote]
Поддерживаю тебя в этом.
Я положительно отношусь к TDD, хотя считаю, что по значимости логика намного важнее.
TDD снижают требования по применению логики, и в этом его сила и опасность.

[quote author=kms link=1215152763/30#40 date=1215726402]
Интерфейс добавил свой - это на пути к универсальным интерфейсам обмена данными.
Вообще, идея меня вдохновляет - например, скоро мы получим возможность задания значений ключей индекса в ИТ вообще из любой доступной коллекции
(из вектора, сз, строки или колонки ТЗ и т.д.).

Не правда ли, чУдно? :)

P.S.
Как вы с Лехой думаете - не стоит пока не поздно переделать чуть порядок параметров на:
[tt]
ВыгрузитьСтроку(Приемник, чНомерСтроки, [Колонки = ""], флВыгружатьИмена)
[/tt]
Все же приемник всегда указывается, в отличие от номера строки и колонок.[/quote]
Да по всем пунктам.
По порядку параметров я и сам хотел предложить.
Если ты еще не исправил, сейчас исправлю тесты и затем код.

[quote author=kms link=1215152763/30#40 date=1215726402]
Доработал немного выгрузку.
Теперь можно получать выгрузку имен колонок, данных строки или всего вместе.

Интерфейс добавил свой - это на пути к универсальным интерфейсам обмена данными.
Вообще, идея меня вдохновляет - например, скоро мы получим возможность задания значений ключей индекса в ИТ вообще из любой доступной коллекции
(из вектора, сз, строки или колонки ТЗ и т.д.).
[/quote]
Тестами дополнительно проверил, все отлично.

[quote author=kms link=1215152763/30#40 date=1215726402]
Доработал немного выгрузку.
Теперь можно получать выгрузку имен колонок, данных строки или всего вместе.

Интерфейс добавил свой - это на пути к универсальным интерфейсам обмена данными.
Вообще, идея меня вдохновляет - например, скоро мы получим возможность задания значений ключей индекса в ИТ вообще из любой доступной коллекции
(из вектора, сз, строки или колонки ТЗ и т.д.).
[/quote]
А зачем ты убрал фичу выгрузки строки в КОП-наследник от Вектора/Коллекции ?
Я специально так сделал для полной аналогии с ИТЗ::Выгрузить()

ИМХО удобная и полезная фича.

kms писал(а) 11. Июля 2008 :: 07:05:

Я добавил шаблонную функцию ::CValue2Type<Type>(const CValue&), посмотри ее код.
Она сначала пытается через dynamic_cast привести контекст с типу Type, если неудача и переданный контекст это КОП, то ищем Type в наследниках.
Аналогично сделано в ИТЗ для выгрузки в ТЗ, СЗ и ИТЗ.
А ты заменил вызов этой функции просто на dynamic_cast

kms писал(а) 11. Июля 2008 :: 08:55:

Ага, поправлю.
Кстати, давно хотел насчет safe-cast поговорить.
Опиши, плиз, ситуацию, при которой может быть подобное падение, типа назови объекты и т.д.
Я что-то пропустил упоминание об этом.

Вопрос по реализации выгрузки строки ИТЗ в коллекцию.
Коллекция у нас представлена в каком виде?
- одна строка и куча колонок - строка таблицы,
или куча строк и одна колонка - колонка таблицы ?
Мне просто не очень понятна твоя реализация выгрузки строки ИТЗ в коллекцию.

Михаил, сделай, плиз, код на базе этого интерфейса для выгрузки из ИТЗ в ТЗ или СЗ.
Я сейчас делаю выгрузку строки ИТЗ в СЗ.

artbear писал(а) 10. Июля 2008 :: 13:43:

ОК, согласен.

Будем бросать исключение за вставкой за границей.
Вставка на границе - штатная ситуация.

[quote author=kms link=1215152763/30#40 date=1215726402]
Теперь можно получать выгрузку имен колонок, данных строки или всего вместе.

Интерфейс добавил свой - это на пути к универсальным интерфейсам обмена данными.
Вообще, идея меня вдохновляет - например, скоро мы получим возможность задания значений ключей индекса в ИТ вообще из любой доступной коллекции
(из вектора, сз, строки или колонки ТЗ и т.д.).

[tt]
ВыгрузитьСтроку(Приемник, чНомерСтроки, [Колонки = ""], флВыгружатьИмена)
[/tt]
[/quote]
Добавил к ВыгрузитьСтроку фичу выгрузки в СписокЗначений.

Т.е. теперь ВыгрузитьСтроку понимает как Вектор, так и СписокЗначений.

ЗЫ реализовал универсальный класс CVLResultLoader, служающий для выгрузки данных в СписокЗначений, на базе универсального интерфейса IGeneralResultLoader

PPS Логика ВыгрузитьСтроку подтверждена многочисленными юнит-тестами.

JohnyDeath писал(а) 14. Июля 2008 :: 07:16:

Будет, конечно.
Щас за чашкой чая Степан на Луа напишет.

Смысл тут, Жень, - в желании сделать стандартные коллекции, интегрированные в единое пространство.
Чтобы 1sqlite, например, мог выгружать в любую коллекцию, которая написана по определенным правилам.
Ну и т.д.

Добавил к ВыгрузитьСтроку фичу выгрузки в ТаблицаЗначений.

Т.е. теперь ВыгрузитьСтроку понимает как Вектор, так и СписокЗначений с ТаблицаЗначений.

ЗЫ реализовал универсальный класс CVTResultLoader, служающий для выгрузки данных в ТаблицаЗначений, на базе универсального интерфейса IGeneralResultLoader

ЗЗЫ Логика ВыгрузитьСтроку подтверждена многочисленными юнит-тестами.

kms писал(а) 15. Июля 2008 :: 08:09:

Ага, сам уже над этим раздумывал.

kms писал(а) 15. Июля 2008 :: 08:09:

Расскажи, подробнее, как сделать? желательно с кодом.

ЗЫ у меня вообще уже готова и протестирована выгрузка из ИТЗ через универсальный интерфейс для ТЗ, СЗ и коллекций.
Нарисовал еще кучу тестов для ИТЗ::Выгрузить, все проходят
Сейчас буду заливать.

Завел новую тему об универсальных интерфейсах загрузки/выгрузки
http://www.1cpp.ru/forum/YaBB.pl?num=1216110955/0#0
Прошу обсуждать их там, а не в этой ветке.

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

Да, думаю, это нормально.
Единственно, если еще не сделал - неплохо бы, чтобы при выгрузке 2х колонок вторая колонка попадала в представление.

kms писал(а) 15. Июля 2008 :: 16:22:

Да, это понятно.
В какой момент вызывать Release при выгрузке в этот приемник? после окончания выгрузки, т.е. после ldrDone ?

ЗЫ а вообще внутри 1С++ пока обошелся просто shared_ptr

artbear писал(а) 10. Июля 2008 :: 12:25:


В документации ошибка:

Цитата:


Или вы ещё с интерфейсом не определились?

artbear писал(а) 23. Июля 2008 :: 05:25:


Ну тогда ещё реализацию исправьте  . А то неработает.

artbear писал(а) 23. Июля 2008 :: 05:45:

Ну, бывает...
Мы же тогда не приняли решение, просто продекларировали, что это было бы неплохо.

Научил коллекции (Вектор, АссоциативныйВектор) выгружаться в любую другую коллекцию или СписокЗначений, или ТаблицаЗначений.

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

PS тесты подтверждают.

Цитата:

Армянское радио отвечает: да и хрен с ним.
Ну, можно кидать исключение при передаче пустых - но мое мнение - это лишнее.

Залил тесты (Вектор, АссоцВектор, ИТЗ) + код по загрузке в СЗ.

Интересная ситуация - падают тесты для
Вектор::Выгрузить(СЗНаФорме)
АссоцВектор::Выгрузить(СЗНаФорме)
но не падают
ИТЗ::Выгрузить(СЗНаФорме ...)
ИТЗ::ВыгрузитьВСтроку(СЗНаФорме ...)

Очень странно
Ведь реализация приемника одинакова во всех случаях.

ЗЫ также добавил тесты для ТЗ на форме, они проходят нормально.

Михаил, дока на АссоцВектор::Сортировать(Параметр) непонятна
что означает
Сортировать / Sort
---------------------------------------------------------------------
     :Синтаксис:
           Сортировать(Выражение)

     :Параметры:
           - Выражение, тип: Строка
                 Выражение сортировки.
                 Состоит из перечисления "колонок", разделенных запятыми,
                 каждая из которых может иметь один или несколько модификаторов

                 **Колонки:**
                       + "Ключ"
                       + "Значение"

Насчет колонок/ключ/значение не понял
Поясни.

Раз дока непонятна, значит, непонятно, как использовать и тестов не могу составить
Жду.

1. Залей эту инфу в репозитарий, плиз.
Думаю, что не мне одному будет непонятна текущая дока

2. Значит, для простого Вектора можно написать Сортировать("*Значение") и т.д., т.е. можно писать значение, а можно не писать?