The daily Axapta WTF - Курилка

EVGL · 14.10.2008, 00:56

Шокированный коллега прислал сегодня сей изысканный образец творчества местного программиста:

X++:

actorForSpoiling = prodTableRun.QtySched / InventTableRun.qtyPerLayer / prodTableRun.MEM_NumOfLanes;
 numofpage = 1;
 if (factorForSpoiling > 198)
  numofpage = 12;
 else if (factorForSpoiling > 180)
  numofpage = 11;
 else if (factorForSpoiling > 162)
  numofpage = 10;
 else if (factorForSpoiling > 144)
  numofpage = 9;
 else if (factorForSpoiling > 126)
  numofpage = 8;
 else if (factorForSpoiling > 108)
  numofpage = 7;
 else if (factorForSpoiling > 90)
  numofpage = 6;
 else if (factorForSpoiling > 72)
  numofpage = 5;
 else if (factorForSpoiling > 54)
  numofpage = 4;
 else if (factorForSpoiling > 36)
  numofpage = 3;
 else if (factorForSpoiling > 18)
  numofpage = 2;

Остается только подчеркнуть: ребята и деачата, Австрии СРОЧНО нужны программисты!

**oip** · 14.10.2008, 10:06

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

PS. А в Вене действительно замечательно. Только недавно оттуда вернулся.

**a33ik** · 23.11.2010, 22:29

Цитата:

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

X++:

actorForSpoiling = prodTableRun.QtySched / InventTableRun.qtyPerLayer / prodTableRun.MEM_NumOfLanes;
 numofpage = 1;
 if (factorForSpoiling > 198)
  numofpage = 12;
 else if (factorForSpoiling > 180)
  numofpage = 11;
 else if (factorForSpoiling > 162)
  numofpage = 10;
 else if (factorForSpoiling > 144)
  numofpage = 9;
 else if (factorForSpoiling > 126)
  numofpage = 8;
 else if (factorForSpoiling > 108)
  numofpage = 7;
 else if (factorForSpoiling > 90)
  numofpage = 6;
 else if (factorForSpoiling > 72)
  numofpage = 5;
 else if (factorForSpoiling > 54)
  numofpage = 4;
 else if (factorForSpoiling > 36)
  numofpage = 3;
 else if (factorForSpoiling > 18)
  numofpage = 2;

Цитата:

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

X++:

numofpage = (factorForSpoiling ? (factorForSpoiling - 1) : factorForSpoiling) div 18 + 1;

У меня нет знаний по аксапте и X++, но (по моему мнению) результаты работы при factorForSpoiling большем чем 215 будут разниться для приведённых участков кода...

EVGL · 23.11.2010, 23:01

Цитата:

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

У меня нет знаний по аксапте и X++, но (по моему мнению) результаты работы при factorForSpoiling большем чем 215 будут разниться для приведённых участков кода...

Автор исходного кода не предполагал factorForSpoiling > 215. Речь идет, насколько я понимаю, о продольной резке полотна. По моим ощущениям больше 100 ножей не ставят.

Roman777 · 28.04.2012, 14:15

Цитата:

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

Шокированный коллега прислал сегодня сей изысканный образец творчества местного программиста:

X++:

actorForSpoiling = prodTableRun.QtySched / InventTableRun.qtyPerLayer / prodTableRun.MEM_NumOfLanes;
 numofpage = 1;
 if (factorForSpoiling > 198)
  numofpage = 12;
 else if (factorForSpoiling > 180)
  numofpage = 11;
 else if (factorForSpoiling > 162)
  numofpage = 10;
 else if (factorForSpoiling > 144)
  numofpage = 9;
 else if (factorForSpoiling > 126)
  numofpage = 8;
 else if (factorForSpoiling > 108)
  numofpage = 7;
 else if (factorForSpoiling > 90)
  numofpage = 6;
 else if (factorForSpoiling > 72)
  numofpage = 5;
 else if (factorForSpoiling > 54)
  numofpage = 4;
 else if (factorForSpoiling > 36)
  numofpage = 3;
 else if (factorForSpoiling > 18)
  numofpage = 2;

Остается только подчеркнуть: ребята и деачата, Австрии СРОЧНО нужны программисты!

Может так:

X++:

    numofpage = trunc(((factorForSpoiling - 1) / 18) + 1);

?

S.Kuskov · 28.04.2012, 14:49

Цитата:

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

Может так:

X++:

    numofpage = trunc(((factorForSpoiling - 1) / 18) + 1);

?

Я вместо trunc задействовал бы div