[💬] 코드에 상수가 너무 많음

[익명(112.144)]

뭔가 문제가 있는걱같은데,, 원래 이렇게 상수가 많아도 되나 싶음.

지금 짜고있는건 능력치를 추가하는건데, 기존 능력치의 일정한 계산식에 따라 이뤄짐.

예를들면 "리더쉽" 이라는 능력치를 추가함

계산식은 다음과 같음: 외향성요인(0: 0, 1: 10, 2: 40) + 성실성요인(0: 0, 1: 10, 2: 20) + 나이요인((나이-20)*3) + 랜덤값(0~20)

외향성과 성실성은 0(낮음), 1(보통), 2(높음)으로 기존에 구분해놨기 때문에,

만약 외향성이 1이면 10을, 2면 40을 리더쉽이라는 능력치에 추가하라는 의미임

그리고 나이는 말그대로 나이-20의 *3만큼 추가하라는 얘기 (25살이면 25-20인 5 * 3 해서 15를 추가)

랜덤값은 뭐 랜덤함수 쓰고...

이러다보니 내가 위에 파란색 처리한부분이 다 상수가 됨.

리더쉽같은 능력치가 10개정도 있고 앞으로 더 추가할건데 계속 상수떡칠되는게 맞는가 싶어. 원래 이렇게하는거 맞나...? 유린이라 잘 모르겠음..

소스코드는 점점 길어지는데 나중에 밸런스패치라도 하려면 힘들거같기도 하고... 주석처리 잘 해놓음 되려나?ㅠ

[정공개발자(firetruck4545)]

define이라도 쓰는게 어때

[정공개발자(firetruck4545)]

readonly나

[익명(112.144)]

너도 상수같은거 쓸때 다 그런걸로 저장해서 써?

[정공개발자(firetruck4545)]

무조건 참조가 딱 한번일 것이라 생각되는건 그냥 바로 숫자로 쓰고, 나머지는 상수 전용 static 클래스에 몰아서 써놓음. 사실 좋은 방법인지는 모름 그냥 귀찮아서 그렇게함

[익명(112.144)]

저것들 참조가 딱 하나일것들이긴 함... 각자 계산식이 천차만별이니 ㅜ 그래도 고마움 잘자 - dc App

[AnonymousD(kenshin122)]

상수가 많은건 전혀 문제 없음. 상수도 결국 게임에 쓰이는 데이터일 뿐임. 그게 코드에 존재할지, 외부파일로 빼낼지는 선택의 문제임. 옛날 게임 데이터들은 전부 롬에 상수처럼 박혀있었음. 밖으로 빼내는 이유가 크게 두가지가 있음. 첫째, 기획자한테 맡기기 용이하다. 둘째, 외부파일은 수정해도 코드를 다시 빌드할 필요가 없음. 나도 저런 상수는 Excel에서 Const_Table이라는 시트에 별도로 다 모아놓고 쓰고있어.

[익명(112.144)]

const가 뭔지 지금에서야 알앗네... 간단하게 말하면 (수정불가능한) 변수화 하는거네 나도 어디 그냥 전역변수로 막 선언해놓을까 생각도 해봣는데,, 저기로 치면 10이니 40이니 -20 이런걸 당최 어떤변수명으로 일일이 저장할지 까마득해서 안햇는데,, 너는 하나하나 다 이름지어주면서 저장한거야? 이름 대충지어주면 또 그것대로 유지보수하기 힘들거아냐ㅠ

[AnonymousD(kenshin122)]

이걸 빼낼지 말지는 전적으로 너의 판단기준에 달려있음. 예를들어, 마리오 게임을 만드는데, 중력속도가 초당 100픽셀이래. 게임개발과정에서 변할일이 거의 없음. 그럼 이건 코드에 그냥 박아버림. 위의 예시에서 "외향성이 1이면 10을, 2면 40을 리더쉽이라는 능력치에 추가" 이 경우 이 능력치가 기획자가 게임밸런스 도중 수정이 많이 가해진다. 그럼 이거 외부로 빼서 기획자 줘버리면 되는거임. 이름은 대충 ONE_LEADERSHIPPOINT_PER_EXTROVERT = 10, TWO_LEADERSHIPPOINT_PER_EXTROVERT = 40 빼낸 상수종류가 많을때 구별이 어렵지 않게 이름은 풀네임으로 지음

[AnonymousD(kenshin122)]

저 상수를 너만 수정할거다 하면 위에 정공개발자 말대로 define문같은걸로 상수 이름으로 정의해두는것도 매우 좋은 방법이다. 숫자 10, 20으로 박아놓으면 너가 적었다 하더라도, 나중에 와서 너조차 저 상수의 의미를 까먹을 가능성이 높아.

[익명(112.144)]

이름 기니까 정신나갈거같음 ..ㅋㅋㅋ define문 찾아보니까 비교적 간단하게 쓸수잇을거같네 하 그래도 너무 귀찮음 1인개발이기도 하고 게임규모도 막 큰것도 아니니 그냥써도될거같음.. 그래도 이번에 진짜많이배웟다 두번이상 써야되는거는 다 디파인해놔야할듯ㅋㅋ 고맙다 - dc App

[익명(221.155)]

어차피 엑셀로 뺄 거면 이름 짓는 거 고민 안 하고 인덱스, 변수1, 변수2, 변수3, 변수4...,설명 이런식으로 만들어도 됨. 설명에는 '외향성 1일 때 변수1만큼, 2일 때 40만큼 리더쉽 추가. 나이 + 변수3 * 변수4만큼 리더쉽 추가.' 뭐 대충 이런식으로 써놓고

[AnonymousD(kenshin122)]

우리기획자가 이렇게하드라. 근데 그렇게하면 코드에서 var1 var2 이런식으로 표기가되버려서 내가 그 변수가뭔지알려면 엑셀로 들어가야돼. 그러기싫어서 이렇게한거임. 개인플젝에선 코더인 내가 곧 기획이라서리. 근데 실제 플젝에선 이친구 말대로 하고있음

[익명(221.155)]

그게 싫은 거면 뭐 코드에 주석 달아두는 방법도 있고... 상수마다 이름 달아주려면 진짜 밑도 끝도 없어서 난 이게 편하다고 보긴 함 ㅋㅋ

[AnonymousD(kenshin122)]