- 1
- 2
- 3
- 4
if (MACaddress == 0)
MACaddress = pAdapterInfo->Address [5] + pAdapterInfo->Address [4] * 256 +
pAdapterInfo->Address [3] * 256 * 256 +
pAdapterInfo->Address [2] * 256 * 256 * 256;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+137
if (MACaddress == 0)
MACaddress = pAdapterInfo->Address [5] + pAdapterInfo->Address [4] * 256 +
pAdapterInfo->Address [3] * 256 * 256 +
pAdapterInfo->Address [2] * 256 * 256 * 256;
daemon_master 12.12.2010 15:59 # 0
Dummy00001 12.12.2010 16:40 # +2
ну хоть бы уже куда комментарий "лопата" добавляли.
TarasB 12.12.2010 17:17 # +1
Dummy00001 12.12.2010 17:29 # +2
это стандартный портабельный способ собирания целого числа из байтов, работающий в независимости от strict alignment и от endianess. "стандртный" способ. я такой код (с мелкими вариациями) вижу почти каждый день.
и меньше писать по сравненю со сдвигами, потому что для сдвигов нужно еще и конвертировать каждый байт из памяти в unsigned int - иначе обрезание происходить будет.
Oleg_quadro 12.12.2010 19:31 # +3
inkanus-gray 12.12.2010 20:17 # 0
Dummy00001 12.12.2010 20:22 # +2
да и цикл то как бы проще не будет. доп переменная понадобится. или две (для множителя/сдвига). народ пытается что бы простое обращение к памяти было как можно проще, и уж точно без flow control. разворачивание циклов оптимизатором тоже не гарантируется.
Oleg_quadro 12.12.2010 20:28 # 0
eth0 12.12.2010 17:09 # +2
inkanus-gray 12.12.2010 17:22 # +8
xaionaro 13.12.2010 13:26 # 0
TarasB 13.12.2010 13:31 # +4
Анонимус 13.12.2010 13:47 # 0
шифтов мы не проходили
xaionaro 13.12.2010 13:56 # 0
Анонимус 13.12.2010 14:05 # 0
xaionaro 13.12.2010 14:53 # 0
TheCalligrapher 14.12.2010 20:30 # 0
Можно поспорить о том, не лучше ль было бы тут использовать сдвиги (возможно, лучше)... Но говнокодом тут и близко и не пахнет. Пахнет профессионализмом очень высокого уровня или по крайней мере зачатками такого профессионализма.
xynta 15.12.2010 00:49 # +1
bugmenot 17.12.2010 19:53 # +1
sanchousf 17.12.2010 18:21 # 0
xaionaro 18.12.2010 00:57 # 0
Говногость 17.12.2010 23:16 # 0