sunghun7511 / gotop Goto Github PK

문제점

core/process.go 파일의 GetCPUUsage 함수와 GetMemUsage 함수의 내용이 대부분 겹치는 것 같아요.

해결 방법

파싱할 파일 이름, index를 제외하면 내용이 중복되니 이 두 값을 인자로 받는 Private 함수를 만들면 될 것 같아요!

현재 cpu 총 사용량 - 이전 cpu 총 사용량 을 이용하여 계산하는데, 처음 켜지는 경우에는 이전 cpu 총 사용량이 0이여서 아주 큰 값이 나오게됨.
처음에는 max(percent, 100) 과 같은 방식도 생각했는데, 대부분의 프로세스들이 100으로 나와서 여전히 이상한것 같아서 다른 해결 방식이 필요해 보임

어떤 내용인가요?

기본적으로 CPU 전체 사용량을 보여주고, 특정 키를 눌러 개별 CPU의 사용량을 각각 보여주도록 해주세요

각 widget에서 처음 terminal 크기 기준으로 데이터를 저장하고 있어서 생긴 이슈입니다. resize 이벤트가 발생 했을 때, 조치가 필요해 보입니다.

추가적으로, 가능하다면 크기를 main에서 각 위젯으로 넘겨주던가 해서 widget의 위치나 크기 상관없이 그려질 수 있으면 좋을 것 같아요. 지금은 [cpuWidget width] == [terminal width] && [memoryWidget width] == [terminal width] / 2 를 가정하고 있는데, 이게 위치가 바뀌어서 widget width와 terminal width의 비율이 달라지더라도 잘 작동할 수 있도록 하면 좋을 것 같아요.

린트와 빌드가 되는지 체크를 하고, 테스트는 굳이 의미가 없어서 안해도 될 것 같아요.