New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify, replace method to using equals #86

Merged
merged 2 commits into from Dec 21, 2015

Conversation

Projects
None yet
2 participants
@mwkang
Copy link
Contributor

mwkang commented Dec 2, 2015

String을 비교할 때 ==을 사용하던 것을 equals를 호출하도록 변경하였습니다.
for문을 foreach문을 사용하도록 변경하였습니다.
konlpy는 JDK 1.7 이상 설치를 요구하고 있기 때문에 new ArrayList<String>()new ArrayList<>()으로 변경하였습니다.
list.toArray(new String[0])의 경우 list에 element가 존재할 경우 array의 크기가 리스트보다 작아서 array를 항상 다시 생성합니다. 이를 list.toArray(new String[list.size()]) array를 list 크기만큼 만들어서 전달하도록 변경하였습니다.
for 문을 돌면서 list.add(morphemes[j]) list에 element를 추가하던 구문을 Collections.addAll(list, morphemes)로 변경하고 for 문을 제거하였습니다.

@e9t

This comment has been minimized.

Copy link
Member

e9t commented Dec 5, 2015

안녕하세요. 고수님께서 살펴주시니 코드가 훨씬 깔끔해지네요 ^^
Merge하기 전에 한 가지 부탁드리고 싶은 부분이 있는데, auto import 사용해서 import java.util.*;을 필요한 util만 import하도록 바꿔주실 수 있나요? 현재로서도 별 다른 문제가 되지는 않겠지만, 패키지 전체를 import하지 않고 필요한 부분만 import하는 코드 스타일을 유지하는 것이 좋을 것 같습니다.

@mwkang

This comment has been minimized.

Copy link
Contributor

mwkang commented Dec 12, 2015

import 변경하였습니다~^^
고수님이라고 해주시니 정말 고수가 된 것 같네요 ㅎㅎ 😅
하지만 현실은 고수가 아니라서 부끄럽네요 😣
기여가 아닌 것 같은 기여.. Java 쪽만 끄적거려서 죄송할 따름입니다.

@e9t e9t merged commit a199f6b into konlpy:master Dec 21, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 56.985%
Details

e9t added a commit that referenced this pull request Dec 21, 2015

@e9t

This comment has been minimized.

Copy link
Member

e9t commented Dec 21, 2015

죄송하긴요. 중요한 부분들 많이 수정해주셔서 감사드릴 뿐입니다.
그리고 드디어 머지 했습니다! 기여 다시 한 번 감사합니다 ^^

@e9t

This comment has been minimized.

Copy link
Member

e9t commented Dec 21, 2015

그런데 음 로컬에서는 문제 없었는데 travis에서 build가 fail하네요! 뭐가 문젠지 한 번 들여다봐야겠어요.

@mwkang

This comment has been minimized.

Copy link
Contributor

mwkang commented Dec 22, 2015

@e9t 머지 감사합니다~^^
class 파일로 직접 컴파일도 해주셨네요~ 감사합니다.
travis 빌드 실패한 것은 해결이 된건가요?

@e9t

This comment has been minimized.

Copy link
Member

e9t commented Dec 22, 2015

네 결국 뚜렷한 원인은 발견하지 못했는데 일단 모든 클래스 파일들을 다시 컴파일했더니 해결되었습니다 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment