The daily Axapta WTF

[Cardagant]

Перенесено из темы Классные программисты MS Бразилии создали сей шедевр

Нашёл интересный пример кода.
DAX 2009 (в AX 3.0 то же самое)

Класс ProdRouteType, метод update.

Привожу код метода:

void update()
{
    ProdTable        prodTable;
    ProdRoute        prodRoute_Orig= prodRoute.orig();
    ;
    ttsbegin;

    this.setBackStatus();

    if (prodRoute.OprPriority)
        prodRoute.LinkType = SchedJobLinkType::None;

    prodRoute.doUpdate();

    // Shop Floor Control begin
    if (Global::isConfigurationkeyEnabled(configurationkeynum(Jmgjob)))
    {
        JmgJobTable::synchronizeProdRoute(prodRoute);
    }
    // Shop Floor Control end

    if (0)
    {
        if  (prodRoute.BackorderStatus != prodRoute_Orig.BackorderStatus)
        {
            prodTable= prodRoute.prodTable(true);
            prodTable.status().setBackStatus();
            prodTable.doUpdate();
        }
    }

    ttscommit;
}

Не совсем понимаю условие

if (0)

По сути оно подразумевает, что код не будет выполняться.
Что вы думаете на этот счёт? Зачем так могли сделать?

[db]

Цитата:

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

Перенесено из темы Классные программисты MS Бразилии создали сей шедевр


По сути оно подразумевает, что код не будет выполняться.
Что вы думаете на этот счёт? Зачем так могли сделать?

1. Существует такой "стиль" выключения блоков кода. Ну типа закомментировать. Про уместность лучше промолчать, но возможно в предыдущей жизни автор его где то использовал
2. Автор просто сделал код компилируемым в надежде дописать условие чуть позже и забыл

[user_ax]

Цитата:

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

2. Автор просто сделал код компилируемым в надежде дописать условие чуть позже и забыл

А не легче написать

//TODO

и не забудешь точно

[Cardagant]

Цитата:

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

А не легче написать

//TODO

и не забудешь точно

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

[db]

Цитата:

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

А не легче написать

//TODO

и не забудешь точно

вот throw error() или debug::assert() дают некую надежду что тебе рано или поздно напомнят в принудительном порядке, а TODO, увы, может жить вечно Если кусок действительно не важен для правильного исполнения, то можно и TODO с нормальным комментарием. Если понимаешь что работать без этого куска нормально не будет, но по каким либо причинам надо привести в компилируемый вид (в жизни всякое бывает, иногда реально надо), то надо прервать исполнение если дойдет до недоделанного места.

[ice]

Цитата:

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

вот throw error() или debug::assert() дают некую надежду что тебе рано или поздно напомнят в принудительном порядке, а TODO, увы, может жить вечно Если кусок действительно не важен для правильного исполнения, то можно и TODO с нормальным комментарием. Если понимаешь что работать без этого куска нормально не будет, но по каким либо причинам надо привести в компилируемый вид (в жизни всякое бывает, иногда реально надо), то надо прервать исполнение если дойдет до недоделанного места.

ага, и теперь аксапта просто наводнена этими debug::assert(). надеемся что дойдут руки и доделают

[belugin]

Цитата:

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

ага, и теперь аксапта просто наводнена этими debug::assert(). надеемся что дойдут руки и доделают

Если, вдруг, кто-нибудь на самом деле думает что debug::assert признак чего-то недоделанного, прочитайте, пожалуйста этот пост

[Zabr]

Цитата:

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

2. Автор просто сделал код компилируемым в надежде дописать условие чуть позже и забыл

Некомпилирующийся код не станет компилироваться от того, что его поставят под условие "if (0)". Вероятно, Вы имели в виду не "компилируемым", а "выполняющим правильные действия".

[S.Kuskov]

Цитата:

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

Некомпилирующийся код не станет компилироваться от того, что его поставят под условие "if (0)".

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

[RVS]

Дивный код попался...

    TNow=TimeNow();
    super();
    switch(true)
    {

        case (23*3600 + 55*60 <tnow ):
            infolog.shutDown(true);
            return;
        break;
    }

... из рабочей Аксы.. она там вся почти такая. И ведь - все-таки работает ))