The daily Axapta WTF - Страница 2 - Курилка

belugin · 07.03.2007, 09:17

Цитата:

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

Интересно, если Аксапта исполняет две строчки кода за, скажем, 2 микросекунды, то какова вероятность наступления полуночи за этот промежуток времени?

2 E(-6) / 3600 / 24 ~ 1,5 E(-10)

Это если этот код исполняется 1 раз в сутки. А если в полнось что-то пишет в лог большую кучу записей, то вероятность равна времени между соседними вызовами/общее время выполнения повторяющегося куска.

Интересно, а этот код сделан умозрительно или человек сторлкнулся с проблемой?

ziva · 07.03.2007, 12:54

Цитата:

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

Интересно, а этот код сделан умозрительно или человек столкнулся с проблемой?

Интеграция AX c другой системой. Сделано специально - дабы проблем не возникло.

Волчара · 07.03.2007, 14:08

Цитата:

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

Вероятно человек боялся что код запустят в полночь....

Ну если полночь произойдет после whilt при выполнении return, то изврат не поможет.

проще так
return [today(), timenow()];

kashperuk · 07.03.2007, 14:11

Может когда время = полночь минус 1 минута, запускать sleep на 2 минуты?

А уже на второй минуте след. дня начинать то, чего там планировалось

belugin · 07.03.2007, 14:30

Цитата:

Сообщение от Волчара

Ну если полночь произойдет после whilt при выполнении return, то изврат не поможет.

проще так
return [today(), timenow()];

Тут не главное, чтобы дата сопадала с чем-то - а главное, чтобы время дата относилась к тому времени которое обозначено.
Чтобы не было 02.01.07 23:59 вместо 01.01.07 23:59 пусть лучше будет
02.01.07 00:01

glibs · 07.03.2007, 14:44

Да... программистский подход был у автора.

Я бы запомнил дату до и после запроса времени, если бы они отличались, то вывел бы полночь для второй даты, а если нет, то время и вторую дату.

Вообще по БестПрактису код лучше писать наглядно (ну типа он без комментариев д.б. понятен).

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

ziva · 07.03.2007, 15:05

Цитата:

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

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

А на этот случай перед методом имеются комментарии:

// Получение текущей даты и времени с проверкой
// на случай, что между вызовами today() и timenow() наступит новый день

Их я опустил намеренно - чтобы всеобщий разум пошевелил извилинами. Вон EVGL "выдал" совершенно понятный код.

belugin · 07.03.2007, 15:09

лично мне исходный вариант нравится больше всего. По-моему всё понятно.

glibs · 07.03.2007, 15:52

Цитата:

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

...
А на этот случай перед методом имеются комментарии:
...

"...
Вообще по БестПрактису код лучше писать наглядно (ну типа он без комментариев д.б. понятен).
..."

Для тех, кто перед праздником утратил бдительность, обращаю внимание, что я не против комментариев, а наоборот — за их использование. Но одно второму не мешает, и друг друга не исключает.

belugin · 07.03.2007, 16:13

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

DTD · 07.03.2007, 16:24

Цитата:

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

Они, британцы, к этому серьезно относятся. В одном решении, которое адаптировали для англичан, пришлось поменять всех "vendors" на "suppliers", "production order" на "works order", "inventory" на "stock".

Надо было labels сразу использовать, не пришлось бы менять )

DTD · 07.03.2007, 16:27

Мне еще нравится подобная практика использования комментариев:

X++:

info('@NEO258'); //DocuView

EVGL · 07.03.2007, 16:27

Цитата:

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

Надо было labels сразу использовать, не пришлось бы менять )

Обижаете. Естественно, что использовали labels, но сначала сделали их только для языка en-us.

DTD · 07.03.2007, 16:33

Цитата:

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

Обижаете. Естественно, что использовали labels, но сначала сделали их только для языка en-us.

Если речь o vendors, inventory .. то это стандартные labels, в en-gb они соответственно suppliers, stock ..

EVGL · 19.04.2007, 16:47

Из Reports\JmgAbsenceColumn\fetch в 3.0:

X++:

while select sum(seconds) from jmgStampTrans
 group by JobIDAbs
 where jmgStampTrans.emplId    == _jmgEmployee.emplId                    &&
          jmgStampTrans.startDate >= jmgReport_AbsenceColumn.parmDateFrom() &&
          jmgStampTrans.startDate <= jmgReport_AbsenceColumn.parmDateTo()   &&
          jmgStampTrans.stampType == JmgStampTypeEnum::Absence
{
   wrkIndex = str2Int(jmgStampTrans.jobIdAbs);
   if (! wrkIndex)
      wrkIndex = str2Int(jmgStampTrans.jobId);
   if ((wrkIndex) && (arrayIndex[wrkIndex]))
      sumSeconds[arrayIndex[wrkIndex]] += jmgStampTrans.seconds;
}

Обратите внимание на str2Int(jmgStampTrans.jobId). Найдите 2 ошибки

belugin · 28.04.2007, 13:09

X++:

void loadParam(container _param = conNull())
{
    int i;
    if (conLen(_param) > 0)
    {
        param = conNull();
        for (i = 1; i <= conlen(_param); i++)
        {
            param = conIns(param, i, conPeek(_param, i));
        }
    }
}

kashperuk · 28.04.2007, 13:14

Прикольная тема.
И особо мне нравится, как мне приходят уведомления о новых сообщениях в ней на почту.

Название темы там отображается не так "The Daily Axapta WTF"
а так

kashperuk · 28.04.2007, 13:36

Цитата:

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

Из Reports\JmgAbsenceColumn\fetch в 3.0:

X++:

while select sum(seconds) from jmgStampTrans
 group by JobIDAbs
 where jmgStampTrans.emplId    == _jmgEmployee.emplId                    &&
          jmgStampTrans.startDate >= jmgReport_AbsenceColumn.parmDateFrom() &&
          jmgStampTrans.startDate <= jmgReport_AbsenceColumn.parmDateTo()   &&
          jmgStampTrans.stampType == JmgStampTypeEnum::Absence
{
   wrkIndex = str2Int(jmgStampTrans.jobIdAbs);
   if (! wrkIndex)
      wrkIndex = str2Int(jmgStampTrans.jobId);
   if ((wrkIndex) && (arrayIndex[wrkIndex]))
      sumSeconds[arrayIndex[wrkIndex]] += jmgStampTrans.seconds;
}

Обратите внимание на str2Int(jmgStampTrans.jobId). Найдите 2 ошибки

Евгений.
Первая ошибка - простая. Не выбирается JobId
А в чем вторая ошибка-то?

Я же так понимаю ошибка должна быть именно в этой строке?
И это ведь не вызов str2Int без проверки, что это таки число? И не длина строки? И не номерная серия?
Просвети

EVGL · 28.04.2007, 21:01

Вторая ошибка: JobId типа ProdJobId - это строка с обычной номерной серией, у которой вполне может быть буквенный префикс или суффикс, а они преобразуют его в индекс массива.

А в примере Белугина программист, похоже, был уверен, что контейнеры передаются по ссылке. Тяжелый случай.

kashperuk · 29.04.2007, 00:25

Цитата:

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

Вторая ошибка: JobId типа ProdJobId - это строка с обычной номерной серией, у которой вполне может быть буквенный префикс или суффикс, а они преобразуют его в индекс массива.

А в примере Белугина программист, похоже, был уверен, что контейнеры передаются по ссылке. Тяжелый случай.

JmgJobId - EDT для этого поля в 3ке. И на него не настроена номерная серия.

PS В четверке, кстати, у меня тоже так. Только что проверил.