Вкусный код

Муад'Диб · **Joined:** Sat Sep 08, 2007 10:06 pm **Posts:** 12456

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

Alex · **Posted:** Wed Feb 24, 2010 2:16 pm

Орел wrote:

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

Угумц. Чтобы избежать фрагментации, так как приложение монолитно, должно бежать непрерывно в течение очень долгого времени, а виртуальная память и "полновесные" процессы не задействуются.
Иногда таки используют динамическую аллокацию, но при этом создают несколько куч. Каждая такая куча содержит энное количество блоков фиксированного (но своего для каждой кучи размера).

Domnitch · **Posted:** Fri Feb 26, 2010 7:08 am

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

Code:

if( myVar == true)
    return true;
else if( myVar == false )
    return false;
else
    return !true && !false;

Pan Novichek · **Posted:** Fri Feb 26, 2010 7:14 am

Напоминает почему-то анекдот:
- Манана, ты мальчика родила?
- Нет...
-А кого же?

Но вообще-то над таким кодом надо потрудиться. С ходу такой не создашь.

kopf · **Joined:** Wed Sep 19, 2007 4:41 pm **Posts:** 21834

Самый цимес был, когда индюк конвертировал булеву переменную в строку, мерял ее длину, и по длине (4 или 5) решал, что вернуть, TRUE или FALSE.

marine · **Joined:** Sat Sep 08, 2007 11:13 am **Posts:** 4618

kopf wrote:

Самый цимес был, когда индюк конвертировал булеву переменную в строку, мерял ее длину, и по длине (4 или 5) решал, что вернуть, TRUE или FALSE.

Тут бы еще индусские комментарии к коду привести. Комментируют они часто и многословно. Жалею что сниппеты не сохранял, но иногда по полу каталась вся наша компания. Баш нервно курит в сторонке.

хмельник · **Posted:** Sat Feb 27, 2010 7:19 am

Лежу я в подвале Главспирта зимой
Вскормлённый завскладом орёл молодой... :roll:

Domnitch · **Posted:** Sat Feb 27, 2010 7:35 am

Вот код, написанный китайской программисткой, творившей софт у наших партнеров в Дании.
Ознакомившись с этим шедевром, наш директорат решил прекратить партнерство. Демонстрирую только малую часть, сохранив орфографию:

Code:

U8 GetNode(U8 MAC[6])
{ U8 IPaddr;
  U8 i;
for ( i=0; i<2; i++){
    if (MAC[0]==ARPtable[i].MAC[0]){
      if (MAC[1]==ARPtable[i].MAC[1]){
        if (MAC[2]==ARPtable[i].MAC[2]){
          if (MAC[3]==ARPtable[i].MAC[3]){
            if (MAC[4]==ARPtable[i].MAC[4]){
              if (MAC[5]==ARPtable[i].MAC[5]){
                 IPaddr=ARPtable[i].IP;
              }
          }
        }
      }
     }
    }
   }
  return IPaddr;
}

alpolo · **Posted:** Sat Feb 27, 2010 9:13 am

(заинтересованно) Домнич, и че этот код должен делать или не делать?

Domnitch · **Posted:** Sat Feb 27, 2010 9:49 am

alpolo wrote:

(заинтересованно) Домнич, и че этот код должен делать или не делать?

Этот код должен по 6-байтному коду МАС, уникальному для каждой сетевой карты Ethernet, выдавать IP-адрес. Отыскивая MAC в т.н. ARP-таблице (ARP - address resolution protocol).
Однако здесь мы наблюдаем (по степени тяжести)
1) выдачу одного байта вместо 4-байтного адреса
2) выдачу неопределенного/неприсвоенного значения, если МАС-адрес в таблице не найден
3) искусственно зафиксированный размер таблицы = 2
4) крайне неэффективное сравнение - особенно с учетом того, что эта хня вызывается для каждого пролетевшего в сети пакета
5) варварскую запись

JOURFIXE

Вкусный код

Who is online