Что не так с моим C кодом?

Question

[Андрей Таранов]

Одна американская фирма ищет себе тимлида - разработчика встраиваемых систем. Я подошел по опыту, навыкам, но со мной отказались дальше беседовать после того как я прислал по их просьбе кусок кода С для микроконтроллера. содержимое main файла тут: ideone.com/V7dhJy Код рабочий. Специально не стал его откарамеливать, послал так как было сразу после отладки.
Скажите, что в моем коде так ужасно?

Comments

[Roman K]

Тимлида, говоришь? =)

[Adamos]

"Специально не стал его откарамеливать" - из каких соображений? Тимлид, образно говоря, должен отвечать не только за выезд на шашлыки, но и за вывоз мусора после них.
Вы же, прислав небрежный кусок кода, однозначно продемонстрировали, что доводить работу до состояния "под ключ" вам не свойственно (вариант - лениво). Какой же вы, к черту, тимлид?

Answer 1

[LeEnot]

Сразу говорю, я в Си и контроллерах особо не смыслю и Вас, как специалиста, не оцениваю. Все нижесказанное лишь мои предположения, ничего личного.
Возможно за мусор в коде и отсутствие именованных констант:

puTX.bytes[0]=(!IN_1<<0)+
												(!IN_2<<1)+
												(!IN_3<<2)+
												(!IN_4<<3)+
												(!IN_5<<4)+
												(!IN_6<<5)+
												(!IN_7<<6)+
												(!IN_8<<7);

Сравните, например, с куском кода от STM из вашего же примера:

InitGPIO(GPIOB,GPIO_PIN_7,GPIO_MODE_INPUT); //1
	InitGPIO(GPIOB,GPIO_PIN_6,GPIO_MODE_INPUT);
	InitGPIO(GPIOB,GPIO_PIN_5,GPIO_MODE_INPUT);
	InitGPIO(GPIOB,GPIO_PIN_4,GPIO_MODE_INPUT);

Ну или такое:

if (reset >= 2)
					{
						reset = 0;
						timeout = MAX_TIMEOUT;
						//PCKG_SwitchOffAllLeds(); // выключить все лампы
					}
					else
					{
						reset++;
					}

это код джуниора, никак не тимлида, который должен код ревью делать за джунами...

Answer 2

[asd111]

Комментарии в коде на русском .
Представьте что они на китайском и всё станет понятно.
Остальное несущественно если код рабочий.