디시인사이드 갤러리

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

갤러리 본문 영역

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

ㅆㅇㅆ(124.216) 2024.05.20 03:46:19
조회 533 추천 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 - -
2865 AD 호요버스 신작 <젠레스 존 제로> 7월 4일 오픈! 운영자 24/06/05 - -
161634 공지 취.업관련 이야기 금지 [9] 파베르Dev갤로그로 이동합니다. 24.06.05 528 6
159490 공지 인디 게임 개발 갤러리 규칙 - 질문 전 필독!! [25] 에뚜갤로그로 이동합니다. 24.05.10 959 3
52873 공지 게임제작 초심자를 위한 글 모음 [9] Goranic갤로그로 이동합니다. 20.07.02 64661 52
123789 공지 뉴비 개임개발 하려는데 뭐부터 해야하나요 복사본 [14] 흉황갤로그로 이동합니다. 23.04.11 12493 22
162063 📜일지 "크리스탈 연대기" 개발일지 - 보스 및 신맵 - Noah갤로그로 이동합니다. 13:40 14 0
162062 💬 개발에 수학 필요한가요 떡밥 결정체가 쉐이더잖아 [4] ㅇㅇ(124.28) 13:10 124 0
162061 💬 맥스랑 블랜더가 아트의 전문성을 퇴화시켰다 [1] ㅇㅇ(211.203) 13:06 74 0
162060 💬 이 글에 나온 사이트 어디임? [2] ㅇㅇ(210.121) 12:27 116 0
162059 💬 쉐이더는 코딩이 아니라 아트인데 프로그래머만 만질수있음 [7] 마치쨩갤로그로 이동합니다. 12:22 200 1
162058 💬 유니티 유료 에셋 깃허브에서 public 레포로 작업하면 ㅈ됨? [2] ㅇㅇ(180.81) 12:09 91 0
162057 💬 근데 아트 에셋으로 만들고 성공한 겜있냐 [5] 유나fan갤로그로 이동합니다. 11:54 184 0
162056 🐣질문 님들 유니티3d랑 3d 모바일이랑 개발할 때 큰차이는 없음? [3] ㅇㅇ(210.204) 11:19 119 0
162055 💬 인디는 인원이 적다보니 잘팔렸다의 기준이 많이 다른듯해요 [17] ㅇㅇ(125.132) 11:13 275 0
162054 💬 뱀서라이크 기획하다 알게 된 사실 [6] 태율 (222.98) 10:43 244 0
162053 💬 근데 개발 진짜 쫌 어렵다 [8] ㅇㅇ(118.42) 09:54 433 0
162052 💬 여기서 구인 어캐하는거임? [3] ㅇㅇ(121.129) 09:36 184 0
162051 💬 Time To Live 스팀 넥페에서 보러오세요 [2] 강화인간갤로그로 이동합니다. 08:03 148 1
162050 💬 유니티로 클릭형 공포게임 처음 만들어봤습니다! [1] ㅇㅇ(211.170) 07:38 117 1
162048 🐣질문 님들 나이가 다 어떻게 되나요? [14] ㅇㅇ(125.137) 06:23 329 0
162047 💬 이렇게 카메라를 스킬쓸때마다 움직이려면 어떻게 해야하나요? [4] ㅇㅇ갤로그로 이동합니다. 06:17 114 0
162046 💬 구글 플레이 개발자 주소지 비상주 오피스로한 갤럼 있어? ㅠ [1] ㅇㅇ(58.225) 04:51 179 0
162044 📜일지 UI 거의 다 만든 것 같구나 [13] dryrain갤로그로 이동합니다. 02:45 717 18
162043 💬 GPT가 발전해서 쉐이더도 짜줬음 좋겠다. [9] ㅆㅇㅆ(124.216) 02:07 263 0
162042 💬 보통 아트나 음악 작업할 사람은 어디서 구해? [6] ㅇㅇ(221.160) 01:55 220 0
162041 💬 개억까 당하는중 ㅇㅇ(61.98) 01:54 119 0
162040 💬 인갤 응원영상 만들어왔습니다 [4] 왕크롱이갤로그로 이동합니다. 01:29 243 8
162039 🐣질문 배경에 실존 장소 넣는 거에 대해서 어떻게 생각함? [4] ㅇㅇ(39.114) 01:23 148 0
162038 🐣질문 언리얼 VT는 언제 쓰는건가요? [2] ㅇㅇ(121.124) 00:58 89 0
162037 📢홍보 덱빌딩 로그라이크랑 전략 게임 섞어봤다(피드백이 필요해) [21] 호라갤로그로 이동합니다. 00:10 551 9
162036 📜일지 요즘 작업하는 것 [11] jipoiz갤로그로 이동합니다. 06.10 628 13
162035 💬 취미로 게임하나 만들라고 유니티 받았다 ㅇㅇ갤로그로 이동합니다. 06.10 129 0
162034 💬 아, 쉐이더 만들어서 실험하는게 제일 짜증나 내뜻대로 안될때 [3] ㅆㅇㅆ(124.216) 06.10 173 0
162033 💬 포케로그 << 생태계 교란종 [3] ㅇㅇ(175.213) 06.10 272 0
162032 💬 1학기 팀프로젝트 끝....!(+광기의 원화) 허당갤로그로 이동합니다. 06.10 237 4
162031 📚정보 스팀에 게임파일 올리는 방법 [18] ㅇㅇ(117.53) 06.10 600 24
162030 💬 돈이 목적이라 어쩔수 없이 미소녀틱한 요소가 들어가는 게임 만들고 있는데 [6] ㅇㅇ(220.65) 06.10 294 0
162028 🐣질문 게임 프로젝트 팀원 모집할때 다들 어떻게 하시나요? [10] 검은고양0I갤로그로 이동합니다. 06.10 317 0
162027 🐣질문 컷씬 궁금한거 있음 [3] ㅇㅇ(59.18) 06.10 173 0
162026 💬 벡터의 연산과 활용 [7] vertex갤로그로 이동합니다. 06.10 550 17
162025 💬 유니티 안드 빌드 권한 도와주실분~~~ [2] ㅇㅇ(220.86) 06.10 112 0
162024 🐣질문 기획이 흔한가요? [14] ㅇㅇ(183.101) 06.10 484 0
162023 💬 익스텐션 에셋 옮겨서 다른 계정에서 써도됨? [2] ㅇㅇ(218.146) 06.10 87 0
162022 📜일지 Moe Moe Island 일지 119 [14] 개발하는똥개갤로그로 이동합니다. 06.10 367 10
162021 💬 도박이 도파민 폭탄이긴한듯 [7] ㅇㅇ(182.227) 06.10 379 0
162020 💬 언리얼 액터 에디터 이동시 컴포넌트 재생성 되는거 확 와닿네요 [3] ㅇㅇ(125.132) 06.10 120 0
162019 🐣질문 유니티로 기획할때 제일 어려운건 뭘까요? [10] 앞섬갤로그로 이동합니다. 06.10 351 0
162018 💬 게임아트 겁나 잘하는 사람 없나연~ [23] ㅇㅇ(121.124) 06.10 533 0
162017 🐣질문 유니티 투사체 반전질문;;; [7] ㅇㅇ(121.145) 06.10 188 0
162016 💬 맵, 보스는 3D인데 캐릭터는 2D인 게임 많이 이상할까? [14] 개발용용갤로그로 이동합니다. 06.10 358 0
162015 💬 텀블벅 후원 걸어놓을러면 보통 어느정도 궤도여야 하냐? [8] ㅇㅇ(1.218) 06.10 417 0
162014 💬 dom기반 웹게인 엔진 니즈 파악 [5] ㅇㅇ(125.128) 06.10 173 0
162013 💬 파란 버튼을 만들고 싶었습니다... [2] ㅇㅇ(125.132) 06.10 297 0
162012 💬 어제 게임쇼에나온 덱빌딩 배틀로얄 파베르Dev갤로그로 이동합니다. 06.10 277 2
162011 💬 Phaser로 만들었는데 김치랜덤디펜스 한판하실분~ [5] 인생전망(101.235) 06.10 252 2
갤러리 내부 검색
제목+내용게시물 정렬 옵션

오른쪽 컨텐츠 영역

실시간 베스트

1/8

뉴스

디시미디어

디시이슈

1/2