funprog / funbot

MIT License
5 stars 2 forks source link

네임스페이스 추가 #33

Closed jwChung closed 8 years ago

jwChung commented 8 years ago

Closes #26

com.github.funprog.funbot 이란 네임스페이스 만들었습니다.

jwChung commented 8 years ago

@funprog/core 리뷰 부탁드립니다. :)

naxxster commented 8 years ago

스칼라의 경우 파일 구조를 패키지의 구조와 같게 하지 않아도 되는 것 같던데. 파일 구조를 좀 더 간결하게 가져가는 것은 어떨까요?

jwChung commented 8 years ago

@naxxster 폴더구조와 상관없이 com.github.funprog 를 만들수 있다는 말씀이시죠? 제가 모르는 부분이네요. 어떻게 가능한지 알려주실 수 있으신가요? 그리고 이슈를 제기하셨으니 -1 이모지 추가하시는 것은 어떨까요?

jwChung commented 8 years ago

@naxxster 아 1장 읽고 있는데 이 부분에 대한 언급이 있네요. 그런데 폴더구조가 package 이름과 같은 구조가 되는게 어떤 불편함이 있나요?

역으로 저는 폴더 구조가 같은 구조가 아니면 불편한 점이 생길 것 같아요. IDE에서 소스트리 구조를 보고 package 명을 파악할 수 없겠네요. 그리고 새로운 소스 파일을 추가할 때 pacakge 이름을 일일이 적어야 하는 불편함도 있겠네요.

naxxster commented 8 years ago

@jwChung IDE 를 쓰지 않는 경우 폴더 구조가 겹쳐서 있는 부분이 상당히 거슬리더라구요. com.github.funprog 까지의 경우 우리가 작성한 코드에 모두 들어갈 패키지 구조라면 굳이 저 부분까지의 폴더를 만들 필요가 있을까 싶은 마음입니다. 파일 구조를 통한 패키지명 파악에서는 의미가 있긴할텐데, 코드를 오픈하여 다른 곳에서 쓰일 목적이 아닌 상황에서 과연 실용적일까 하는 생각이 드는군요.

naxxster commented 8 years ago

제 주 언어인 C++ 의 경우 네임스페이스, 파일 구조 등이 굉장히 어설프게 사용되고 있는만큼 아직 제 경험이 부족하기 때문에 실제 사용 상의 문제가 적은 구조를 따르는 것이 맞을 것 같습니다. 어느 정도 컨벤션도 정해져 있는 문제인 것 같네요. 스칼라 자체의 일반적인 컨벤션이 있다고 그것이 어느 정도 문제 해결책이 된다면 그에 따르는 게 가장 낫지 않을까 생각됩니다. (책에서는 모든 디렉토리 구조를 만들어주더군요.)

jwChung commented 8 years ago

@naxxster:

IDE 를 쓰지 않는 경우 폴더 구조가 겹쳐서 있는 부분이 상당히 거슬리더라구요. com.github.funprog 까지의 경우 우리가 작성한 코드에 모두 들어갈 패키지 구조라면 굳이 저 부분까지의 폴더를 만들 필요가 있을까 싶은 마음입니다.

공감이 가는 내용입니다. 다른 분 의견도 들어보시죠.

아 그리고 +1, -1 의견 표시해주면 감사하겠습니다.

myeesan commented 8 years ago

음.. 전혀 생각해 본적 없었던 주제입니다. 실제 패키지 이름을 com.github.progfun.funbot 패키지를 funbot 으로 변경해도 실제 문제가 발생할 확률은 거의 없을 것 같습니다.

그래서, 패키지 이름을 필요한 경우 자유롭게 변경 할 수 있거나 절대 구조를 변경하지 않을 것이라는 확신이 있다면, 그 구조를 간결하게 가져가는 것도 나쁘지 않은 것 같습니다. 그러나 자바&스칼라 패키지 구조의 변경으로 인해 발생하는 비용은 상당히 심각합니다. 그리고 funbot의 프로젝트가 진행됨에 따라 유일성을 가져야 하는 이유도 증가하고, 시점이 미뤄지면 치뤄야 하는 비용도 그에 따라 증가합니다.

어서 빨리 merge 해주세요 >_<

naxxster commented 8 years ago

-1 떼고 컨벤션에 따르겠습니다.

myeesan commented 8 years ago

아직 @naxxster 님의 -1이 붙어 있긴 하지만, 직접 코멘트 해 주셨으니 merge 해도 되지 않을까요?

jwChung commented 8 years ago

@myeesan 직접 머지해주시죠 ㅎ

jwChung commented 8 years ago

@myeesan 작업브랜치가 삭제되는 것이 좋지 않을까요?

myeesan commented 8 years ago

@jwChung 아 깜빡했습니다. 감사합니다.