Наименование журнала (JournalName)

[kalex_a]

Здравствуйте, у меня такой вопрос. Создаю новое наименование журанла (в LedgerJournalName), например "Наименование 1", в LedgerJournalTable создаю журнал с этим именем "Наименование 1", так вот при разноске журнала в зависимости от его наименования, мне необходимо вызывать разнае классы, как правильнее проверить наименование журнала. Мне кажется что если так:

if (ledgerJournalTable.JournalName == "Наименование 1")
    {
            // действия
    }

то как то это не правильно. Или я ошибаюсь?

[glibs]

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

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

[lev]

На сколько я знаю, в параметрах склада (Управление запасами \ Настройки \ Параметры), на вкладке Журналы есть группа полей Код журнала по умолчанию. Там заполняются названия журналов по умолчанию.
Так вот. Можно в параметрах добавить поле с вашим новым названием журнала. Потом при вызове разноски сверять с параметрами Название журнала который разносить, и в зависимости от того с чем совпадает вызывать необходимый класс.

[gl00mie]

Цитата:

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

Создаю новое наименование журанла (в LedgerJournalName), так вот при разноске журнала в зависимости от его наименования, мне необходимо вызывать разнае классы

Мне кажется, идеологически неправильно завязываться на названия журналов в данной постановке задачи - у них другое предназначение. Кроме того, классы разноски журналов ГК создаются еще до того, как станет известно, какой именно журнал (с каким кодом названия) надо разносить; тот или иной класс-наследник LedgerJournalCheckPost создается в зависимости от типа журнала, т.е. вам нужно:

• создать дополнительное значение в enum'е LedgerJournalType (выберите значение где-нить в районе 200, чтобы снизить вероятность пересечения с локализаторскими и прочими доработками);
• создать наследника класса LedgerJournalCheckPost, реализовав в нем нужные дополнительные действия, скажем, за счет перегрузки метода postTrans();
• модифицировать метод LedgerJournalCheckPost::construct(), добавив в switch ваше значение LedgerJournalType и создание экземпляра вашего класса разноски;
• в названиях журналов (LedgerJournalName) указывать ваше значение типа журнала и затем использовать соотв. коды при создании журналов, которые вам нужно разносить с помощью вашего класса разноски.

[kalex_a]

Цитата:

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

• создать дополнительное значение в enum'е LedgerJournalType (выберите значение где-нить в районе 200, чтобы снизить вероятность пересечения с локализаторскими и прочими доработками);
• создать наследника класса LedgerJournalCheckPost, реализовав в нем нужные дополнительные действия, скажем, за счет перегрузки метода postTrans();
• модифицировать метод LedgerJournalCheckPost::construct(), добавив в switch ваше значение LedgerJournalType и создание экземпляра вашего класса разноски;
• в названиях журналов (LedgerJournalName) указывать ваше значение типа журнала и затем использовать соотв. коды при создании журналов, которые вам нужно разносить с помощью вашего класса разноски.

Это я сделал, дело в том что 2 наименования относятся к 1-му LedgerJournalType

[gl00mie]

Цитата:

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

в параметрах склада (Управление запасами \ Настройки \ Параметры), на вкладке Журналы есть группа полей Код журнала по умолчанию. Там заполняются названия журналов по умолчанию.
Так вот. Можно в параметрах добавить поле с вашим новым названием журнала. Потом при вызове разноски сверять с параметрами Название журнала который разносить, и в зависимости от того с чем совпадает вызывать необходимый класс.

Мне кажется, этот вариант не будет корректно работать в общем случае, поскольку в тех же УЗ настройки кодов журналов по умолчанию и указанные в них коды журналов относятся как n:m, n ≥ m, т.е. если указать для разных настроек одинаковые коды журналов, то по коду журнала уже нельзя будет однозначно сказать, к какой именно настройке он относится.

Цитата:

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

Это я сделал, дело в том что 2 наименования относятся к 1-му LedgerJournalType

"А вы так не делайте" (с) анек Введите дополнительный тип журнала, укажите его во втором названии журнала и job'ом пропишите его в неразнесенные журналы со вторым кодом названия. Вообще, названия журналов - это ведь просто строковые данные, которые пользователи руками вводят, на них нельзя в коде закладываться. Можно закладываться на значения enum'ов, либо на (не)совпадение данных в одной таблице и в другой, либо на отношения между значениями (больше, меньше), но никак не на строковые всякие данные, которые вводятся пользователями. Ведь у вас так семантика того, что делает код, теряется. Вот удалят названия журналов, коды которых вы в код прописали, - и все, уже непонятно, почему вы какое-то ветвление в коде делаете на основе несуществующих кодов журналов. Или захотят сделать другие названия журналов, чтобы для них было такое же поведение, - что, полезете в код дополнительные строковые литералы прописывать?..

[kalex_a]

2 glibs: Не хотелось бы делать лишних примочек в LedgerJournalTable ((

2 lev: А при чем тут журнал запасов? Или вы имеете ввиду сделать то же самое в Журнале ГК??

[lev]

Цитата:

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

2 glibs: Не хотелось бы делать лишних примочек в LedgerJournalTable ((

2 lev: А при чем тут журнал запасов? Или вы имеете ввиду сделать то же самое в Журнале ГК??

Ой Дико извиняюсь! Что то меня заклинило на складском журнале тогда беру свои слова обратно

[glibs]

Цитата:

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

...
Не хотелось бы делать лишних примочек в LedgerJournalTable ((
...

Нужно работать над собой, искоренять такие неправильные хотелки.

[kalex_a]

Цитата:

Введите дополнительный тип журнала, укажите его во втором названии журнала и job'ом пропишите его в неразнесенные журналы со вторым кодом названия.

Да но, при открытии фомы LedgerJournalTable выводятся журналы только одного типа, а как тогда быть с двумя??

Цитата:

Вообще, названия журналов - это ведь просто строковые данные, которые пользователи руками вводят, на них нельзя в коде закладываться. Можно закладываться на значения enum'ов, либо на (не)совпадение данных в одной таблице и в другой, либо на отношения между значениями (больше, меньше), но никак не на строковые всякие данные, которые вводятся пользователями. Ведь у вас так семантика того, что делает код, теряется. Вот удалят названия журналов, коды которых вы в код прописали, - и все, уже непонятно, почему вы какое-то ветвление в коде делаете на основе несуществующих кодов журналов. Или захотят сделать другие названия журналов, чтобы для них было такое же поведение, - что, полезете в код дополнительные строковые литералы прописывать?..

Согласен, это я и имел ввиду в первом посте,

Цитата:

то как то это не правильно

[gl00mie]

Цитата:

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

Да но, при открытии фомы LedgerJournalTable выводятся журналы только одного типа, а как тогда быть с двумя??

Сделайте отдельный пункт меню для второго типа. Как вариант, пользователи, если им сильно приспичит видеть в одной форме шапки журналов разных типов, могут после открытия формы поменять фильтры в запросе, используемом для отбора шапок журналов. Это, конечно, в том случае, если у вас такая возможность в шапках пока не отключена Да, еще при введении своего типа журналов надо не забыть поправить метод класса LedgerJournalStatic.menuItemStrLines(), чтобы использовалась нужная форма для строк журналов, а также добавить ваш новый тип журналов в метод формы LedgerJournalTable.setButtonVisible(), чтобы кнопка открытия строк была вообще видна на форме журналов.