Lotto step4 #23
Lotto step4 #23
Conversation
| <tr> | ||
| <th>5</th> | ||
| <td>[13, 14, 16, 38, 42, 45]</td> | ||
| <th>{{@index_1}}</th> |
There was a problem hiding this comment.
{{@index}}는 알겠는데, {{@index_1}} 은 어떻게 출력되는 건가요??
There was a problem hiding this comment.
@nokchax
http://mozmonkey.com/2014/03/ember-getting-the-index-in-each-loops/
handlebars에서 기본으로 지원하는 기능인듯합니다! 저도 처음 보네요... 헬퍼 안만들어도 됐을 걸 그랬네요...ㅠㅠ
| static { | ||
| webConfiguration = new WebConfiguration(); | ||
| lottoMatchService = new LottoMatchService(); | ||
| } |
There was a problem hiding this comment.
별건 아닌데 이부분 그냥 처음에 선언할 때
private static WebConfiguration webConfiguration = new WebConfiguration();
private static LottoMatchService lottoMatchService = new LottoMatchService();이렇게 안하시고 나중에 따로 초기화 하신 이유가 있나요??
| post("/buyLotto", (request, response) -> { | ||
|
|
||
| final Money money = Won.from(Integer.parseInt(request.queryParamOrDefault("inputMoney", "0"))); | ||
| final String[] manualNumbers = request.queryParams("manualNumber").split("\r\n"); |
There was a problem hiding this comment.
매직 문자열 상수로 빼보아요! 저도 안빼긴 했던거 같기도 하네요...ㅎㅎ 😅
| public class WebConfiguration { | ||
|
|
||
| static { | ||
| port(8080); | ||
| } | ||
| } |
There was a problem hiding this comment.
웹 관련 설정 파일을 빼니까 한곳에서 관리할 수 있어서 더 좋겠네요 👍
| Map<String, Object> model = new HashMap<>(); | ||
| WinningLottoDto winningLottoDto = lottoResultService.getWinningLottoByResults(money, rankResults); | ||
|
|
||
| List<WebWinningLottoDto> webWinningLottoDtos = new ArrayList<>(); |
There was a problem hiding this comment.
final 을 여기서도 사용해도 될 것 같은데 제외하신 이유가 있나요? 👀
| <th> | ||
| {{matchCount}} 개 일치{{#if bonus}},보너스 볼 일치{{/if}} ({{money}}원) - {{count}}개 | ||
| </th> |
There was a problem hiding this comment.
아.. 전 보너스볼일치 텍스트넣는걸 깜빡했네요 저도 if 사용해서 변경해봐야겠네요
| <tr> | ||
| <th>5</th> | ||
| <td>[13, 14, 16, 38, 42, 45]</td> | ||
| <th>{{@index_1}}</th> |
There was a problem hiding this comment.
@nokchax
http://mozmonkey.com/2014/03/ember-getting-the-index-in-each-loops/
handlebars에서 기본으로 지원하는 기능인듯합니다! 저도 처음 보네요... 헬퍼 안만들어도 됐을 걸 그랬네요...ㅠㅠ
| public class WebConfiguration { | ||
|
|
||
| static { | ||
| port(8080); | ||
| } | ||
| } |
| <th> | ||
| {{matchCount}} 개 일치{{#if bonus}},보너스 볼 일치{{/if}} ({{money}}원) - {{count}}개 | ||
| </th> |
There was a problem hiding this comment.
아.. 전 보너스볼일치 텍스트넣는걸 깜빡했네요 저도 if 사용해서 변경해봐야겠네요
| private long count; | ||
|
|
||
| public WebWinningLottoDto(Rank rank, long count){ | ||
| this.bonus = (rank.name().equalsIgnoreCase("SECOND")); |
There was a problem hiding this comment.
요기는, rank.matchType.isMatchBous를 가져오면 좀더 유연한 대응이 될 수 있을것 같아요!
| public class WebConfiguration { | ||
|
|
||
| static { | ||
| port(8080); | ||
| } | ||
| } |
| final WinningLotto winningLotto = new WinningLotto(winningLottos, bonusNumber); | ||
| final RankResults rankResults = winningLotto.getRankResults(lottos); | ||
|
|
||
| Map<String, Object> model = new HashMap<>(); |
There was a problem hiding this comment.
model을 사용하는위치(하단)에서 선언해도 좋을것 같아요!
handlebar.js? 이거 커스텀헬퍼 등록하다가 js 로 잘 안돼길래 시간을 너무 많이 잡아먹었네요.. ㅜ
수정사항이 많긴하지만, 시간이 늦은 관계로 일단 제출 먼저 하겠습니다.ㅜ
잘부탁드립니다.