해결된 질문
작성
·
251
·
수정됨
0
안녕하세요. 좋은 강의 감사합니다!
강의 내용 중 구현방향에 대해 궁금한 점이 있어 질문 드립니다.
랜덤 넘버를 초기화 하는 배열에서(randomNumbers) 조건에 따라 padding을 지정하는 과정의 코드가 다음과 같이 구현이 되어 있는데요.
padding: EdgeInsets.only(bottom: x.key == 2 ? 0 : 16.0),
만약 x.key를 직접 참조 하는게 아닌 randomNumbers의 length를 가져와서 조건을 거는 것도 유지보수 상 좋은 코드인가요?
padding: EdgeInsets.only(bottom: x.key == randomNumbers.length - 1 ? 0 : 16.0),
답변 1
1
안녕하세요!
Clean Code라는 책에서 주장하는바를 살펴보면 모든 조건문은 알아보기 쉬운 이름의 변수로 선언하고 사용하는게 좋습니다.
예를들면 아래와 같습니다.
final isLastElement = x.key == randomNumbers.length - 1;
final bottomPaddingSize = isLastElement ? 0 : 16.0;
bottom: bottomPaddingSize
저도 동의하는 바입니다.
감사합니다!