непонятное поведение поля enum NoYes

[timaluhs]

Есть таблица со всеми льготами
EmplID, NumOfbenefits, и разные поля в которых есть дополнительные характеристики

Нужно созадть табличку в которой в одной строке будут все льготы для сотрудника.
Создала табличку типа EmplID, Benefit1, характеристики для Benefit1, Benefit2, характеристики для Benefit2 ну и так до 27...
поля Benefit1 типа Enum, EnumType NoYes
В классе запускается цикл. Внутри цикла для каждого benefits

select BenefitTbl where BenefitTbl .emplId == MyTbl.EmplId
         && BenefitTbl .BenefitTypeId == '01';
//если есть такая льгота то 
        if (BenefitTbl != NULL)
        {
            MyTbl.Benefit1= NoYes::Yes;
                 
        }
        else
        {
            MyTbl.Benefit1= NoYes::No;
        }

и это повторяется для всех benefits только со сменой номера .

Вопрос 1: Почему-то в enum поля не всегда правильно записывается значение. С дебагером захожу внутрь цикла - т.е. должно быть Yes -1, a в БД - 0
Есть какая-то хитрость?
Вопрос 2: Можно ли это как-то оптимизировать. Муторно очень всё копировать
Спасибо

[Zabr]

1) после записи значения не забываете делать MyTbl.update() ?
2) вместо "if (BenefitTbl != NULL)" лучше писать "if (BenefitTbl)" или "if (BenefitTbl.recid)"

[samolalex]

попробуйте убрать

if (BenefitTbl != NULL)

и оставить

if (BenefitTbl)

[lev]

Цитата:

Сообщение от samolalex

попробуйте убрать

if (BenefitTbl != NULL)

и оставить

if (BenefitTbl)

а ещё лучше:

if (BenefitTbl.RecId != 0)

[pitersky]

Цитата:

Сообщение от timaluhs

Нужно созадть табличку в которой в одной строке будут все льготы для сотрудника.
Создала табличку типа EmplID, Benefit1, характеристики для Benefit1, Benefit2, характеристики для Benefit2 ну и так до 27...
поля Benefit1 типа Enum, EnumType NoYes

Ужасное решение. Просто ужасное. НЕ делайте так никогда.
Надо было создать табличку с полями EmplId/BenefitType/BenefitDescription. В BenefitType записать все возможные типы льгот (создать соответствующий Enum или использовать таблицу льгот, если она есть), в BenefitDescription - то, что относится по типу к EmplId. А дальше на EmplTable создаёте закладку, на которой отображаете льготы для конкретного человека.
Если надо всё-таки в одну строку выводить (именно выводить, а не хранить!!!) - нарисуйте соответствующую формочку. Но я не вижу в этом особого смысла.

[Krasher]

Я бы ещё бы проверил, что присутствует select forupdate MyTbl..., присутствуют операторы открытия/закрытия транзакции ttsbegin/ttscommit..., ну и разумеется MyTbl.update();

[Ivanhoe]

Цитата:

Сообщение от timaluhs

Вопрос 2: Можно ли это как-то оптимизировать. Муторно очень всё копировать
Спасибо

Оптимизация простая - сделать по-другому. Поясните что и зачем вы делаете, тогда, возможно, получите дельный совет

[timaluhs]

Цитата:

Сообщение от pitersky

Ужасное решение. Просто ужасное. НЕ делайте так никогда.
Надо было создать табличку с полями EmplId/BenefitType/BenefitDescription. В BenefitType записать все возможные типы льгот (создать соответствующий Enum или использовать таблицу льгот, если она есть), в BenefitDescription - то, что относится по типу к EmplId. А дальше на EmplTable создаёте закладку, на которой отображаете льготы для конкретного человека.
Если надо всё-таки в одну строку выводить (именно выводить, а не хранить!!!) - нарисуйте соответствующую формочку. Но я не вижу в этом особого смысла.

Всё это уже есть. просто наши HR хотят всё видеть сразу. Поэтому создаётся такая никому не нужная таблица в классе, который будет по ночам обноблять в batch.
поэтому кажется forupdate несколько не в тему
в начале есть
delete_from MyTbl;

и в конце каждой итерации MyTbl.insert();

Попробую заменить
if(BenefitTbl != NULL) на if (BenefitTbl.RecId != 0)
Хотя с дебагером заходит в if , но значение почему-то не меняет

[timaluhs]

Цитата:

Сообщение от Ivanhoe

Оптимизация простая - сделать по-другому. Поясните что и зачем вы делаете, тогда, возможно, получите дельный совет

писала выше. HR заказали форму в которой будут показаны все льготы всех сотрудников включая характеристики некоторых льгот или галочку (есть/нет)
их не устраивает лазить по каждому отдельному сотруднику, подозреваю что это будет некая отчётность для поставщиков (машина, сотовый, комп и т.д.)

посему решено создать подтаблицу и её показывать в форме

[Krasher]

Вот собственно и первое предложение по оптимизации. Вместо delete_from и потом нового её заполнения, почему не использовать .update() только того, что изменилось?

[pitersky]

Цитата:

Сообщение от timaluhs

Всё это уже есть. просто наши HR хотят всё видеть сразу.

Мне кажется, что это не тот случай, когда надо под HR прогибаться (извините за слэнг). Полученная таблица крайне ненаглядна и неудобна в работе. Могу предположить, что утром это копипастят в Excel и дальше работают автофильтрами, отбирая нужное. Если это так, то может проще нарисовать им нужную выгрузку, а не плодить лишние таблицы?

Цитата:

Сообщение от Krasher

Вот собственно и первое предложение по оптимизации. Вместо delete_from и потом нового её заполнения, почему не использовать .update() только того, что изменилось

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

[timaluhs]

Цитата:

Сообщение от pitersky

Если это так, то может проще нарисовать им нужную выгрузку, а не плодить лишние таблицы?

а как это сделать?

[timaluhs]

Цитата:

Сообщение от pitersky

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

подозреваю что HR не заботит проблема с RecID
а искать то что надо обновить кажется ещё непригляднее

[pitersky]

Цитата:

Сообщение от timaluhs

а как это сделать?

А что тут сложного? Делаете цикл по EmplTable (это строка выгрузки), внутри него делаете вывод в Excel данных по каждому типу льгот (делая запросы к таблице льгот по типу и EmplId)

[S.Kuskov]

Цитата:

Сообщение от timaluhs

а как это сделать?

Точно также как вы делали вставку в таблицу, только вместо инициализации полей таблицы найденными значениями, отправляйте эти значения в excel или лучше во временную таблицу ADORecordset, а уже её заполненную одним махом в excel (так будет быстрее )
Есть ещё конечно вариант с OLAP и RS, но это не обязательно
Программное добавление столбцов в отчёт

И на будущее, если необходимо сделать в таблице 27 столбцов одного типа, отличающихся только индексом, то используйте расширенный тип данных основанный на масиве полей (стандартный пример - тип Dimension)

[timaluhs]

Не уверена что excel устроит. Я спрошу у HR. Там ещё должна быть ещё фильтрация по типу льгот, которую они хотят делать по уровню доступа(не знаю как по русски) кто что может видеть. т.е. поставщики машин - только машины и всё что с ними связано, сотовая связь - отдельно, но и т.п. а кто-то может всё.

Цитата:

Сообщение от S.Kuskov

И на будущее, если необходимо сделать в таблице 27 столбцов одного типа, отличающихся только индексом, то используйте расширенный тип данных основанный на масиве полей (стандартный пример - тип Dimension)

На самом деле однотипных полей не 27 , штук 15. И для каждого 2 характеристики. То есть создавать типа 3х мерный массив на 15 полей? пока не очень понятно, попробую разобраться

[S.Kuskov]

Цитата:

Сообщение от timaluhs

Не уверена что excel устроит. Я спрошу у HR. Там ещё должна быть ещё фильтрация по типу льгот, которую они хотят делать по уровню доступа(не знаю как по русски) кто что может видеть. т.е. поставщики машин - только машины и всё что с ними связано, сотовая связь - отдельно, но и т.п. а кто-то может всё.

Вы имеете в виду RLS (record level security)? Права доступа на уровне записей.
Пусть отчёт в Excel формируют сами пользователи под своими правами. Тогда в Excel попадут только те данные, к которым у них есть доступ.
Скрытие поля

Цитата:

Сообщение от timaluhs

На самом деле однотипных полей не 27 , штук 15. И для каждого 2 характеристики. То есть создавать типа 3х мерный массив на 15 полей? пока не очень понятно, попробую разобраться

Для хранения характеристик можно сделать ещё два масива по 15 элементов, под каждую характеристику свой масив.

[timaluhs]

а что делать с enum?
С дебагером захожу внутрь цикла - вижу как значение меняется с No на Yes.
захожу в БД - 0, хотя есть поля-enum которые принимают правильное значение
поменяла с if(MyTbl != NULL) на if(MyTbl.RecID != 0) та же петрушка

[S.Kuskov]

Цитата:

Сообщение от timaluhs

В классе запускается цикл. Внутри цикла для каждого benefits
...
и это повторяется для всех benefits только со сменой номера

Судя по всему при написании подобного кода использовалась технология Copy/Past .

Цитата:

Сообщение от timaluhs

С дебагером захожу внутрь цикла - вижу как значение меняется с No на Yes.

Вполне могли где-нибудь забыть исправить наименование поля.
Поствьте точку останова не на инструкции присваивания, а непосредственно на строке с MyTbl.insert() и посмотрите на значения полей именно в тот момент.

[lev]

Цитата:

Сообщение от timaluhs

а что делать с enum?
С дебагером захожу внутрь цикла - вижу как значение меняется с No на Yes.
захожу в БД - 0, хотя есть поля-enum которые принимают правильное значение
поменяла с if(MyTbl != NULL) на if(MyTbl.RecID != 0) та же петрушка

так если if отрабатывает, то проблема то не в нем
можете выложить весь код, который заполняет табличку? так будет проще найти ошибку.

и кстати, посмотрите метод insert() на MyTable возможно там есть какое то переопределение Вашего Enum.