M3-154 마이페이지 걸음수 Ui만들기

[tkdals802]

작업 내용*

• 마이페이지 걸음수, 픽셀수 표시창 구현
• 마이페이지 현재 걸음수가 목표 걸음수의 몇%인지 보여주는 그래프 구현

고민한 내용*

• 그래프의 전체 길이 = 화면 폭(width)-20
• walking controller에서 현재 걸음수 받아오기
• pixel service로 db에서 현재 픽셀수, 누적 픽셀수 받아오기

리뷰 요구사항

• 함수, 변수 이름

스크린샷

|

[qjvk2880]

api url "/count" 맞나요?? api 명세서 상에는 "/api/pixels/count"로 되어있습니다!

[koomin1227]

pixel 들의 카운드를 담는 모델인데 UserPixelCount 는 어떨까요?

[koomin1227]

단순히 공간을 띄우시려고 쓴거 같은데 Container 대신 SIzedBox 도 고려 해볼 수 있을 것 같아요!

[koomin1227]

마찬가지로 getUserPixelCount 가 어떨까요?

[koomin1227]

이건 왜 고정값으로 바뀌었나요?
컨트롤러에서 값을 가져오는 것으로 다시 수정하셔야 마이페이지에서 내 정보가 나올거 같네요~

[koomin1227]

위랑 똑같습니다

[koomin1227]

보니까 비슷한 형식이 반복되는 것 같습니다.

Container(
               width: 120,
               height: 130,
               decoration: BoxDecoration(
                 color: Colors.white,
                 borderRadius: BorderRadius.circular(10),
               ),
               child: Column(
                 mainAxisAlignment: MainAxisAlignment.center,
                 children: [
                   Text('현재 픽셀수'),
                   Image.asset(
                     'assets/currentTile.png',
                     width: 80,
                     height: 80,
                   ),
                   Obx(
                     () => Text(
                       myPageController.getCurrentUserPixel().toString(),
                     ),
                   ),
                 ],
               ),
             ),

이부분을 공통 위젯으로 분리하고 생성할때 다른 이미지와 값을 받게하면 코드 중복을 개선할 수 있을 것 같아요

[koomin1227]

StepWindow 라는 이름을 잘 모르겠습니다.

현재 픽셀수, 누적 픽셀수, 현재 걸음수 에 대한 정보를 나타내는 이름이 더 좋지 않을까요?

지금은 Step 만 강조되어있는 것 같아요

[koomin1227]

리뷰가 잘 반영된것 같습니다~