어추구니 없는 실수..

오랜만에 WiMAX 구현한 부분에서 ARQ 관련 보안할 부분이 있어서
오래전에 코딩했던 소스를 다시 살펴보았는데 정말로 내가 봐더 어처구니가 없는 실수가 많이 있었다..

첫번째, 상황에 맞는 변수를 사용
바보같이 BSN은 2Byte 즉 unsigned short인데, BSN을 비교하는 함수내에서
unsigned char tempBSN..... 을 사용해서..허걱..이때 졸았었나?
왜 이런 실수를 했는지 반성반성..

두번째, #define 사용시 주의
또 어처구니 없이 Block Size 를 16으로 해놓고 배열의 개수를 32개로 해서 총 16*32=512 까지만 Data를 저장하게 해논 실수..

세번째, 쓸데없는 루틴들
이건 진짜 바보같은 실수인데..for문을 두번돌면서 비슷한 루틴을 진행했다.
이런건 한번에 돌면서 모든 루틴을 다 해결할 수 있는건데 왜 이렇게 했는지 앞으로 조심하자~

네번째, 1의 오류
이부분도 조심해야 하는 부분인데 특히 for문을 돌때 많이 발생한다.
for(i=0;i<XXXXX;i++) 에서 XXXXX부분에서 내가 생각했던 값보다 1을 작게 넣어서 실수 -_-;;

앞으로 코딩할때 한번 더 생각해보고~ 루틴도 줄이면서 필요한 변수만 사용하면서 해보자
좀 더 Professional 하게~