디시인사이드 갤러리

마이너 갤러리 이슈박스, 최근방문 갤러리

갤러리 본문 영역

[🐣질문] 너네는 코드 리뷰 누구한테 부탁하냐. 나 초보개발자라 도움이 필요하다.

ㅆㅇㅆ(124.216) 2024.05.20 03:46:19
조회 538 추천 0 댓글 20
														

viewimage.php?id=2abcdd23dad63db0&no=24b0d769e1d32ca73ce983fa11d0283107ab5cf91d66c5d1ce22a5f13c0960fddbb0c89c26d79cfc97b263d118fe8995501631ba3893bd150506f915aa015a7bd0087545



이렇게 싱글톤으로 잘 되던거

viewimage.php?id=2abcdd23dad63db0&no=24b0d769e1d32ca73ce983fa11d0283107ab5cf91d66c5d1ce22a5f13c0960fddbb0c89c26d79cfc97b263bc75fc809829e8d50332fa6693dae6212279b9b39bb08b4d




3dbec374abc236a14e81d2b628f1726f1dbe


이렇게 동일하게 짰는데, 일단 아이템 이미지를 받아오질 못하는건 일단 내가 코드 구조때문에 일부러 뺀거


근데 이거는 문제 원인 파악 못한게 아니라. 사실 어디에 이미지 받느냐 코드 수정 문제라 .이거는 뭐 수정하면 되는 부분이라 별거 아님


어쨌건


거의 다 완성됐음. 리팩토링만 계속했는데 초보 개발자인 내 수준에 맞게 코드 짰지만 잘 짰는지를 모르겠다.


후에 문제가 있을지 없을지도 모르겠고


근데 게임 개발 연차가 1년이 안되니까


내 코드가 맞는지 안맞는지 도저히 가늠이 안된다.


이럴때는 너네는 코드 리뷰 하거나 하냐. 어떻게 하냐


대략 내 코드 구조는 이럼



3db4c568f5dc3f8650bbd58b36807d653aad




1.게터 세터는 메서드로 가져와서 가져오게함. 모든 필드는 private이고 외부에서 가져와야할 필드만 게터 세터 메서드 씀


3db4c574abc236a14e81d2b628f177695c2393


2. 직접 참조 안하고, 다른 매니저에서 참조해야할 게터와 세터를 인터페이스로 해서 가져옴


3db4c575abc236a14e81d2b628f176643c2e3a

3db4c572abc236a14e81d2b628f1756b9257d9


3.인벤토리 시스템 팩토리에서 매니저들 초기화와 더불어서 의존성을 주입함.


여기에서 필드 아이템, 인벤토리 아이템 생성시 의존성 주입도 담당함


인벤토리 플로우는 다음과 같음


0. 인벤토리 시스템 팩토리에서 의존성 주입

1. 슬롯 그리드에서 슬롯 생성

2. 슬롯 그리드 매니저에서 슬롯 그리드에서 정해둔 수량만큼 받음

3. 인벤토리에 저장되는 아이템 매니저-슬롯 매니저-컬러 매니저가 있음.

아이템 매니저: 인벤토리에 저장된 아이템 데이터 사용 및 아이템의 x값 y값을 리스트로 저장해둠

슬롯 매니저:아이템이 점유 됐을때, 슬롯 색깔 변경 및, x,y값 리스트를 슬롯에 맞게 체크함

컬러 매니저-아이템이 점유됐을때 아이템 점유수에 따라 슬롯 색상 변경

4.저장된 아이템 체크->리스트에 저장된 아이템의 위치를 슬롯 매니저에서 체크->슬롯 위치 체크 되면 컬러매니저에서 그리드 컬러 변경-> 인벤토리 그리드 매니저에서 최종적으로 전체 데이터 다시 값 체크


이런식의 구조를 하고 있는데


내가 코드를 잘 짜고 있는거 맞냐.

GPT한테 물으려고 하니까 GPT는 내 코드 전부 저장을 못해서 답변을 제대로 못해줌.

코드를 너무 나누니까 얘가 없는 기능을 자꾸 추가 시킬려고 함.

이렇게 분리 시켜서 관리해야하는거 맞는지도 모르겠고.


그냥 시리얼 라이즈 필드로 인스펙터 상에서 추가할 것을 괜히 귀찮게 짜고 있는지도 모르겠고


지금 대부분 코드를 이렇게 해서 짜다보니 생산성이 많이 낮은 편인데, 이렇게 짜는 방식이 맞는지도 모르겠다.


현업인 애들은 어째 짜냐


이렇게 구조를 짰을때 뭔가 문제가 될만한게 있을까? 싱글톤을 안쓰고 팩토리 패턴을 이용해서 이렇게 구현을 했는데


이게 맞는 방법인지를 전혀 모르겠음..


의존성 주입<<이거 갤에서 키워드만 얻고 내 나름대로 진짜 생각하면서 만들어봤는데 이게 진짜 맞는 방법인지 몰겠다


내 프로그래밍 버릇들이 협업에 문제가 될 만한게 있냐?


일단 최대한 패턴 코딩 익히면서 구현하고 있는데, 진짜 맞게 하고 있는건지 내 자신이 긴가 민가하다.


아무래도 선배가 없어서 독학 때리고 있어서 그런가


image_translator_chrome_icon.png


자동등록방지

추천 비추천

0

고정닉 0

0

댓글 영역

전체 댓글 0
등록순정렬 기준선택
본문 보기
자동등록방지

하단 갤러리 리스트 영역

왼쪽 컨텐츠 영역

갤러리 리스트 영역

갤러리 리스트
번호 말머리 제목 글쓴이 작성일 조회 추천
2868 설문 힘들게 성공한 만큼 절대 논란 안 만들 것 같은 스타는? 운영자 24/06/10 - -
2870 이슈 [디시人터뷰] 웃는 모습이 예쁜 누나, 아나운서 김나정 운영자 24/06/11 - -
161634 공지 취.업관련 이야기 금지 [9] 파베르Dev갤로그로 이동합니다. 24.06.05 557 6
159490 공지 인디 게임 개발 갤러리 규칙 - 질문 전 필독!! [25] 에뚜갤로그로 이동합니다. 24.05.10 963 3
52873 공지 게임제작 초심자를 위한 글 모음 [9] Goranic갤로그로 이동합니다. 20.07.02 64681 52
123789 공지 뉴비 개임개발 하려는데 뭐부터 해야하나요 복사본 [14] 흉황갤로그로 이동합니다. 23.04.11 12512 22
162119 💬 쉐이더 작업을 효율적으로 잘하는 방법이 있긴할까 [7] ㅆㅇㅆ(124.216) 03:59 32 0
162118 👮검거 아직도 완강 못함 ㄷㄷ [4] 고도전도사갤로그로 이동합니다. 03:33 53 0
162116 💬 덕분에 유니띠에 집중하기로 했어요 ㄳㄳ [7] 불타는키보드갤로그로 이동합니다. 02:38 104 0
162113 💬 님들 이거보세요 [2] 검은고양0I갤로그로 이동합니다. 02:13 94 0
162112 💬 Pencil2D랑 비슷한데 더 안정적인 2D 애니메이션 툴 업슴?? [10] 김메가갤로그로 이동합니다. 01:35 139 2
162111 📜일지 최적화 진짜 신경 안쓰고 만들어왔긴 한데 [4] ㅇㅇ(121.128) 01:12 150 0
162109 💬 탑다운 2.5d용 md캐릭터 에셋 만들어 주면 무조건 다 살텐데 ㅇㅇ(39.114) 00:39 66 0
162107 📜일지 오늘 지피티 겁나 털고 왔다[개초보주의] [1] ㅇㅇ(220.87) 00:25 128 0
162106 💬 어디서 주워들었는데 인싱 깊었던 말 [2] SR4갤로그로 이동합니다. 00:11 159 1
162103 💬 뉴비라 감이 안잡혀서 그런데 슬더스 같은 게임도 제작 기간 오래 걸림? [19] 유나fan갤로그로 이동합니다. 06.11 245 0
162102 💬 용기 하나로 아직까지 모델링 중 [3] 이세상에서빵빠레갤로그로 이동합니다. 06.11 193 1
162101 💬 미소녀엉덩이나오는 올라가기류나 만들어라. [4] ㅇㅇ(61.97) 06.11 158 0
162100 🐣질문 c# 이벤트 vs 유니티 이벤트 [4] ㅇㅇ(112.187) 06.11 181 0
162099 🐣질문 무작정 기획서 쓰려하니 어렵네 [1] ㅇㅇ갤로그로 이동합니다. 06.11 96 0
162098 🐣질문 게임기획책중에 타이난이 쓴거 이거 좋음? ㅇㅇ갤로그로 이동합니다. 06.11 80 0
162097 🐣질문 유니티 좆병신같네.. 씬 안 바뀌는 버그 머임? ㅅㅂ [7] ㅇㅇ(218.232) 06.11 202 0
162096 🐣질문 키우기 연령대가 40대라고 오해하는 분들이 있네 [5] ㅇㅇ(218.232) 06.11 263 2
162094 💬 엔터 더 건전 다 좋은데, 왜 보스 잡을때 한대도 안맞아야 [8] ㅆㅇㅆ(124.216) 06.11 266 1
162093 💬 스팀 넥스트 페스트 떠오르는 출시 예정 게임이거 [1] ㅇㅇ갤로그로 이동합니다. 06.11 132 0
162092 💬 흥한 게임에서 사용된 에셋을 쓰는 것은 어떰 [3] ㅇㅇ(124.53) 06.11 193 0
162091 💬 스팀에서 게임업데이트 하고 국가별 언어적용하기 [5] ㅇㅇ(117.53) 06.11 303 12
162090 💬 실베가 뭔지 몰랐는데 [9] MAYONEZU갤로그로 이동합니다. 06.11 395 0
162089 🐣질문 언리얼 좆고수만 [4] 미월이갤로그로 이동합니다. 06.11 241 0
162087 💬 슬더스가 빠르게 베낀 게임 [17] dawnDev갤로그로 이동합니다. 06.11 523 3
162085 💬 유니티랑 구성 똑같은 애니메이션 패키지가 언리얼에선 28만원이네... [6] ㅇㅇ(14.55) 06.11 128 0
162084 💬 에셋스토어에 2d 캐릭터 몬스터 패키지 만들어서 팔아보려는데 [5] ㅇㅇ(106.252) 06.11 121 0
162083 💬 영광입니다....! [4] Lighter갤로그로 이동합니다. 06.11 262 3
162082 💬 달리면서 고양이 줘패는 게임 업데이트.gif [17] ㅇㅇ(210.95) 06.11 384 1
162081 📜일지 좋은 UI를 찾기위한 나의 여정 (진행중) [5] MAYONEZU갤로그로 이동합니다. 06.11 375 6
162080 💬 중국어로 출시하면 판매에 도움을 많이 줄까요? [1] ㅇㅇ(61.101) 06.11 178 0
162079 💬 또 실베 끌려갔네 [6] 매튜돌갤로그로 이동합니다. 06.11 365 2
162078 💬 심심해서 찍어봤는데 어떰? [6] ㅇㅇ(223.222) 06.11 230 3
162077 🐣질문 인디 게임 중에 또 재미있는 거 없나요? [2] ㅇㅇ(183.101) 06.11 179 1
162075 🐣질문 steamworks에 통계 추가하고있는데 [9] ㅇㅇ(220.83) 06.11 207 0
162074 📢홍보 비주얼노벨 <봄을 부르는 설녀 - 스핀오프> 펀딩 오픈했습니다! [2] JinYong갤로그로 이동합니다. 06.11 152 1
162072 💬 2D 게임 스테이지 하나 모작하는데 1달이면 짧은 시간임? [2] ㅇㅇ(210.222) 06.11 105 0
162071 💬 스팀 출시해본 사람 질문 좀. 리뷰 해줄테니 게임키 보내달라는데. [6] ㅇㅇ(121.168) 06.11 260 0
162069 💬 밑에 질문글이랑 진짜 어울리는 학과가 있는데 ㅇㅇ(124.28) 06.11 179 0
162068 🐣질문 유니티엔진에서 SetActive 함수 쓸때 주의사항 뭐있음? [1] ㅇㅇ(119.199) 06.11 166 0
162067 💬 현실적인 답변 부탁드립니다) 게임 개발을 하고 싶음 전기전자공,,, [29] ㅇㅇ(39.124) 06.11 482 0
162066 💬 내가 20년 이전 심리적 접근한 게임 디자인론을 잘 안믿는 이유 [7] ㅆㅇㅆ(124.216) 06.11 361 7
162065 💬 Assertion failed on expression: 'm_LockC [4] ㅇㅇ(114.204) 06.11 71 0
162064 📢홍보 우당탕탕 보안관 게임 출시했어 (피드백 가능할까요) [6] 홍염갤로그로 이동합니다. 06.11 554 7
162063 📜일지 "크리스탈 연대기" 개발일지 - 보스 및 신맵 - [8] Noah갤로그로 이동합니다. 06.11 275 10
162062 💬 개발에 수학 필요한가요 떡밥 결정체가 쉐이더잖아 [5] ㅇㅇ(124.28) 06.11 316 0
162061 💬 맥스랑 블랜더가 아트의 전문성을 퇴화시켰다 [2] ㅇㅇ(211.203) 06.11 229 0
162060 💬 이 글에 나온 사이트 어디임? [2] ㅇㅇ(210.121) 06.11 226 0
162059 💬 쉐이더는 코딩이 아니라 아트인데 프로그래머만 만질수있음 [8] 마치쨩갤로그로 이동합니다. 06.11 414 1
162058 💬 유니티 유료 에셋 깃허브에서 public 레포로 작업하면 ㅈ됨? [3] ㅇㅇ(180.81) 06.11 159 0
162057 💬 근데 아트 에셋으로 만들고 성공한 겜있냐 [7] 유나fan갤로그로 이동합니다. 06.11 334 0
갤러리 내부 검색
제목+내용게시물 정렬 옵션

오른쪽 컨텐츠 영역

실시간 베스트

1/8

뉴스

디시미디어

디시이슈

1/2