[🐣질문] 프로그래밍 잘 하고있나 봐줄 사람있니? 욕도 환영해

[dsa(58.124)]

대학교를 게임공학과 나왔고,(만드는곳 하는곳아님) 다닐때도 프밍이 참 이렇게 재미없고 짜증나는거 였구나,

하면서 대~~~충 다녀서 학점 2.8임 ㅄ

그러고 졸업했는데 졸작할때 쓴다고 유니티 조금 공부해서 쓰다가 그걸로 나도 언제했는지 모르겠는데 취업사이트에 유니티를 올려뒀나봐,

그걸보고 취업하라고 왔고, 했어 좆소기업(생각보다 돈잘범)에 부산권 센텀시티.

6개월차야 이렇게.

유니티 개발자는 나뿐이라 물어볼 사람도 없고, 좆소라 다들 바빠서 초반한달은 과제 던져주면 그냥 그거하고, (예 : c# 계산기 및 sql 데이터베이스짜기 쿼리문, 프로시저 등등 얕게) 지내다 2달째부터 유니티 조금 손보고 나혼자 임무 하달안되어도 조금씩 조금씩 겜 만드는 느낌으로 했어.

재밌더라고 근데?

이게 내가 프로그래밍이 재미가 없었던게 주먹구구식 헬조선 교육방법으로 주입하려니 재미가 없었나봐

내가 찾아보고 그에대해 맞추어 짜보고 재활용하려고 내 머리로 돌려짜보고 배웠던거 활용해보고,

저번에 배운걸로 새로운거 연계시도 해보고,

데브코리아 활동하고, 1~4개월차까지 질문만 하던놈이 이젠 질문글은 한달에 한번 올릴까말까하고 영어로 찾고, 대답만 매일 하더라고? 나도 웃겨 참 ㅋㅋㅋ

근데 다른 탑클래스 (부장님 이사님 존나 미친듯이 잘해)분들꼐 배우기는하는데, 따로 코드를 보고 검증받아 보진 못했어

무섭더라고... 

그래서 다시금 상기시키고 어떤게 내 부족한점인지 인지해서 공부를 좀 더 하려고 물어보려고 왔어.

이건 그냥 회사에서 가끔 사다리타기해서 내기로 간식사기 하는데, 매번 재미없게 네이버사다리타기 쓰길래

유니티로 사다리타기 조금씩 빌떄 맹들고 있어,

그 코드야 다는 아니고, 여튼 이런 느낌으로 나는 짜고있는데, 잘하고있는데 뭐가 부족하다.

또는 욕지껄도 솔직히 내 좆대로 배운 프밍이라 받을 각오가 되어있으니 조금만 시간내서 봐줄 수 있을까?

고마워.

ps. 아무것도 모르는 나이지만 제일 중요하게 생각하고(주관적) 실천하고 있는게,

질문하기전에 5번은 생각해보고, 모르겠으면 일단 유튜브던 구글이던 데브코리아, 스택오버플로, 유니티포럼에 영어로 하나씩 하나씩 우회해서도 다 찾아보고 질문하고, 프로그래밍 시에는 gc가 최대한 일지않게, 계속 생각 하려해. 또 foreach대신 for문 쓰려고 습관들이고 있구, if문 오질나게 거는거보다 switch가 낫대서 스위치도 최대한 쓰려하고,, 오브젝트풀링은 필수에 싱글톤을 이용한 매니저클래스 할당?정도? 씬매니저(씬 이동시 변수 할당하는 녀석) 및 겜매니저는 대부분 사용하는거같아

여기까지가 게임매니저야. 짧은 사다리타기 겜이구 아직 미완성이라서 ㅎㅎㅎ휴ㅠ

여기가 사다리 만드는 부분 래더풀~

여긴 캐릭터 선택창에서 필요한 기능들이야

나중에 게임 플레이시 캐릭터 가진 데이터도 넘겨줘야겠지? (ㅁ캐릭터매니저)

[dsa(58.124)]

대충읽더라도 글 들어와서 읽어준거 고마워

[익명(223.62)]

시리얼라이즈 필드 어딘 붙이고 어딘 안붙이는거 보니까 프로퍼티 모르네. 변수 선언 하나 더블클릭으로 선택해서 Alt Enter누루고 캡슐화 클릭해봐

[dsa(58.124)]

겟셋 프로퍼티를 혼자하다보니 쓸일이 잘 안생기다 요즘 생겨서 써야하는 이유와 이런 뭐랄까... 좋은점 장점 이런걸 아예 모르는데 혹시 설명해줄 수 있어? 해보고 오 이런게 되네 하고 신기하다 하긴했어. 시리얼라이즈 필드는 저거 뭐랄까.. 인스펙터에 등장시키는 용도로 쓰긴 하는데

[익명(223.62)]

https://m.dcinside.com/board/unityengine/35

[익명(223.62)]

주딱 댓글

[dsa(58.124)]

진짜 고마워 시발 눈물이 날정도다 너무 고마워 진짜 쏚쏙들어오네 근데 무조건 캡슐화 할 필요 까지는 없다는거네 밑에 추댓보면?? 추댓에 의하면 저렇게 걍 변수 돌려줄거면 public 쓰고 그 외에 들어와서 다른 데이터로 바꾸어 줄거면 캡슐화해서 쓰라 이소린거야? + ㅇㅇ : 변수로 처리하면 당연히 갱신이 안되니 set 프로퍼티에 그짓거리를 해주근거 marmotru : 와 이건 생각못했음,난 프로퍼티 get이면 get만하고 set이면 set만하라고 배워서 이게 무슨소린지 마지막엔 이해를 못했는데 무슨소리야?

[익명(223.62)]

겟이랑 셋도 함수거든. 그래서 추가적인 기능을 넣을수 있어. 주딱댓글은 추가적인 기능을 넣지 말라 배웠다이고 ㅇㅇ댓글은 유니티가 부모자식 오브젝트 관계이기 때문에 부모가 바뀌면 그자식의 위치값도 다바뀌어야 하고 그 기능이 set에 적용되있다 이얘기임

[데쿠(hhw000)]

그럼 결론적으로 get set프로퍼티도 , 그냥 고대로 변수 전송하고 추가기능 없을땐 지금처럼 public쓰고, 그러라고 만든거니까 그 외 추가 기능이 들어가는 프로퍼티라면 get set을 써서 캡슐화 하는게 맞다 이거지?

[익명(223.62)]

전혀 아님. Public 변수를 쓰지마셈. 해당 글내용이 왜 유니티의 로테이션을 set으로 넘겨야하는지가 요점이었기때문에 저렇게 말한거고 변수는 닥치고 프라이빗에 인스펙터에서 쓸거면 Serial쓰는거고 외부참조 필요하면 public get set 쓰는거고. 해당글에 ㅇㅇ은 외부에서 쓸일 없으면 프로퍼티는 안만들어도 된다 이얘기임

[데쿠(hhw000)]

그러면 앞으로는 무조건 변수는 private로 짜고, 외부에서 사용해야하는 변수다 (예: 게임매니저에 있는 싱글톤 변수) 하는 것들은 private로 캡슐화 시킨 뒤 public으로 get set메소드 만들어서 빼주면 된다. 무조건 private다 이말이네?? 거기에 조금 더 나아가서 인스펙터에서 수정이 필요한 애들이라면 시리얼라이즈 시켜주고?

[익명(58.120)]

ㄴㅇㅇ 맞음

[익명(58.120)]

상속할경우 protected 잊지말고

[데쿠(hhw000)]

하나만 더 물어봐도 될까? 센빠이~ 프로텍티드 상속얘기가 나와서 생각이 났는데, 난 아직 MMORPG류나 이런 몬스터가 다중으로 있는데 그 몬스터들이 하나의 클래스를 상속받아서 사용하는일이 아직은 없었거든, 대부분 모노비헤비어만 쓰거나 드래그 만들려고 이벤트시스템 정도 사용하는? 근데 프로텍티드를 사용하고 상속을 사용하는 경우면 아까 말한 저 예제에다가 게임매니저를 상속받아서 사용하는 애들을 위해 마지막에 프로퍼티를 public으로 말고 protected로 선언해서 만들어라 이거지? 싱글톤으로 참조해서 사용하는거면 걍 퍼블릭이구?

[익명(223.62)]

그리고 목적이랑 장르때문에 어떤 평가를 구체적으로 하긴 어려움.

[익명(223.62)]

코드평가를 따진다면 유지보수위주로 따지는데 사다리타긴 뭐 어찌되도 상관도 없고 작업자도 너한명이고

[dsa(58.124)]

하긴.. 그건 그러네... 그럼 좀 얕게 보는 용도로 써달라 해야겠다.. 난 이런식으로 하는데 나보다 나은 입장에서는 어덯게 하는게 나을까? 라는 느낌의 조언?

[dsa(58.124)]

그래도 최대한 재활용 할 수 있게 함수형태로 다 짜고 그 함수도 다른 여러케이스에서 쓸 수 있게 짜려고 하긴 하는데,,, 많이 부족하지?ㅠㅠ 이 코드가 워낙 빈코드기도 하고, 다른걸 보여주려니 회사내 코드라 좀 그렇기도 하고..

[도적코볼트(w9716www)]

멋있네 열심히 하는게

[dsa(58.124)]

멋있긴 다들 하는건데~ 고마워

[익명(122.211)]

개인 취향이긴 한데 - 몇몇 군데 개행이 좀 이상하다 - GameManager.Instance.some.... 이런식으로 Instance에 괄호 빼는 거 어떰 - selected[playerNumber - 1] 이거 몇줄이나 반복해서 쓰지 말고 로컬 변수로 받아다가 하면 어떰 - GetChild(0).GetChild(0).... 아무래도 이건 좀; - 스위치문 케이스 안에서 뭔가를 하지 말고 뭘 해야할지에 대한 파라미터만 정의해다가 스위치문 끝난 다음에 파라미터들도 실행 ㄱㄱ

[dsa(58.124)]

1. instance는 싱글톤 만들때 여러군대 참고해봤는데 대부분 instance = this; 쓰길래 이거쓰기보단 저거쓰자해서 누가 짜둔거 쓰는데 싱글톤 내부를 안보여줘서 그런가? ()쓰게 되어있더라고?>??? ////////// 2. 그렇다는 소리는 최대한 보기라도 좋게 GameObject sObj = selected[~~]; 해서 지역변수로 선언해둔 뒤 쓰더라도 저걸 쓰게 하라 이소리야?? ///////////// 3. 저거 퍼블릭으로 걍 변수 선언해서 쓰려니 전역변수에 너무많이 걍 너도나도 넣는 느낌인거 같아서 저랬는데 걍 변수선언해서 인스펙터에서 할당하는게 낫겠지??? //////// 4. 스위치문안에서 함수를 만들지말고 함수()만 해두고 바깥에 함수를 만들어서 그걸 실행하게 해라 이소리지?

[dsa(58.124)]

논외로 dc댓글은 엔터 어케쳐..?

[익명(122.211)]

1. 개취이긴 한데 난 () 없이 씀 존재 의미가 없기도 하고 2. 보기가 좋은 것도 있는데 유지보수가 편하지 seleted[playerNumber - 2]로 바꾸고 싶을 때 하나하나 바꾸는 건 좀 그렇잖아? 문자열 치환하는 방법도 있긴 한데 저런 게 많아지면 그냥 로컬 변수 하나 받아두는 게 코드 한줄 더 늘어나는대신 가독성과 유지보수가 좋아지더라 3. 인스펙터 할당 ㄱㄱ 하이어라키 구조 바뀌면 바뀔 때마다 거기맞춰서 GetChild 해놓은 부분 고치는 것보단 인스펙터 할당이 좋다고 봄 4. ㅇㅇ

[익명(223.62)]

쉬프트 엔터

[익명(122.211)]

댓글엔터 ㅋㅋㅋ 쉬프트 엔터임 사실 나도 몰랐는데 이것저것 치다가 알게 됨

[dsa(58.124)]

오호.. 예전이랑 다르게(별거아니지만) 알아들은 나에게 박수를 쳐주고싶다 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ 고마워 진짜 고마워. 2번에서 말한 유지보수 무슨소린지 알거같음ㅋㅋㅋㅋㅋㅋㅋㅋㅋ이거ㄴ랑 다르게 다른프로젝트 할떄 그걸 느꼈거든 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ고마워

[익명(122.211)]

굿굿 성장잼!

[Notch(notthing)]

인갤 상위 0.1프로 - dc App

[dsa(58.124)]

엥... 너무 그러지마.... 인갤성님들 능력자 많다고~ 난 초짜일뿐이야

[익명(118.47)]

playerNum - 1 만 LastPlayerIndex 로 바꿔주면 보기 좋을듯. 코드 깔끔해 보이고 좋네

[데쿠(hhw000)]

너무 과찬인거 아냐? 개허접인데... 고마워정말

[익명(59.3)]

약간만 더 다듬으면 괜찮을거 같다 위에 댓글들 처럼 그리고 주석을 다는 습관을 하는게 좋음 주석에도 여러가지 종류가 있고 사람이란게 완벽하지 않아서 나중에 보면 결국 생각도 많이 해야되고 ... 공통적인 부분은 클래스나 함수 단위로 묶어서 분할시키는것도 좋음 스테이트 패턴 쓰는거는 따로 배운게 아니면 어느정도 코딩감각이 있는거라고 생각한다

[데쿠(hhw000)]

헐..감각이 있다고까지 말할 정도인가... 그냥 흐르듯이 만들고싶은거 만들다 보니 여러 방법이 있고 좋은방법 찾으면 그거 다음에 써먹어보고 했을뿐인데,,, 고맙네... 너무 큰 칭찬인데 과찬이야 너무 잘본거같애...

[익명(180.230)]

귀찮아도 주석 꼼꼼히 해주자.. 나중에 자기가 만든거여도 다시 들여다보면 막막할때가 많으니까

[데쿠(hhw000)]

주석 꼭 기억할게 난 대강봐도 기억하는건 안남겨도 된다 생각하고 알고리즘이나 조금 계산 들어가는 부분만 다는데 앞으로는 더더욱 신경써야겠네 고마워

[익명(lucid74)]

함수 하나에 기능 하나만 담으삼

[익명(lucid74)]

반복되는 개념 있으면 함수화, 추상화하고