Taking baby-developer steps

2022.02.04. gnl refactoring - 완료! 본문

Logs/학습 log

2022.02.04. gnl refactoring - 완료!

Surin Lee 2022. 2. 4. 20:38

오늘 목표

  • gnl refactoring
    • gnl 함수 25줄 이하로 줄이기
    • get_a_line 함수 25줄 이하로 줄이기+할 수 있다면 최대한 뒤에 덕지덕지 붙은 if문 없애기

오늘은 백신 3차를 맞고 와선지 계속 피곤한 느낌이 있어서 잠깐 무리하지 않는 선에서 코드를 보도록하자.


오늘 한 일

  • gnl 함수 25줄 이하로 줄이기

이 부분중에서 한줄만 줄이면 함가 25줄 미만이 되는데 잘 안되서, 윗 부분에서 가독성을 좀 포기하고, if()~else()문에서 else()문에 들어가는 곳을 문제가 되지 않게 꺼내면 'else' / '{' / '}' 이렇게 3줄을 줄일 수 있어서 그렇게 먼저 수정했다가, 아무래도 else{}문으로 꺼내는게 가독성도 좋을거 같애서 나중에 다시 else{}문을 꺼냈다.

이렇게 else문을 없애는 쪽으로 수정했다가
가독성을 위해 다시꺼냈다

결국에 초반에 목표했던 gnl 함수 끝부분에 덕지덕지 붙은 if문 중 한개를 다른 부분함수 내에서 예외처리해 널값을 반환하도록 수정함으로써 25줄 이하에 가독성 있는 코드로 완성하였다.

 

  • get_a_line 함수 25줄 이하로 줄이기+할 수 있다면 최대한 뒤에 덕지덕지 붙은 if문 없애기

코드를 전체적으로 검토하던 중, 부분함수에서 필요없는 메모리를 해제하고 해당 포인터가 널값을 가리키게 하는 부분을 추가하면서, get_a_line 함수에서 필요 없어진 부분이 보였다. 

  • gnl refactoring 완료!

 

Comments