[🐣질문] 업데이트문에서 foreach쓰는거 비싸??

[사슴발레리나(dearvalerie)]

유물 슬롯을 만들고 유물 소지 여부를 유물 슬롯 배열 쭉 훑으면서

원하는 유물 번호가 있을 경우 true, 아니면 false를 리턴하는 함수를 만들었는데

이걸 어떻게 할까 생각하다가 foreach로 했단말이야

슬롯이 최대 10개밖에 안되고 중간에 찾으면 나머지 안찾고 리턴되니까

괜찮을까? 싶었는데 이걸 어떤 함수에서 딱 한번만 호출하는게 아니라

플레이어 업데이트문에서도 계속 호출하고 있어서(그것도 여러번) 모바일 성능 이슈가 생길까 걱정됨..

예를 들어 커피(1번) 소유 중이면 이동속도가 100퍼 이하로 제한되지 않고

개구리 모자(4번) 소유 중이면 대시 횟수가 하나 더 늘어나거든

이걸 업데이트에서

if (1번찾는 함수) {

이동속도 1 미만이면 1로

}

else {

이동속도 1 미만이면 초당 0.15씩 회복

}

이런식으로 만들었는데 그럼 이동속도 여부를 떠나 1번 찾는 함수가 계속 호출되잖아

그 함수는 10 크기의 배열을 훑으면서 1이란 요소가 들어있는지 여부를 확인해주는 거고

이런게 여러곳에서 한꺼번에 업데이트문으로 계속 호출되면... 문제가 있을까?

차라리 if문 순서를 바꿔서

이동속도를 먼저 체크한다음 변화를 줘야 할 때만 유물을 찾아주고 참거짓 여부에 따라 다른 액션을 주는게 맞는건가 싶음.

대충 유물 30개라고 치면 그중에 이런식으로 동작하는 애들 절반만이라고 쳐도 15개인데

얘네들을 다 바꿔줘야 하면 바꿔주고 아니면 머리아파서 냅두려구

[익명(118.40)]

딕셔나리를 써는 방법도 있기는 한데 그냥 이벤트 형식으로 탈착 이벤트가 생길 때 값을 수정하고 되돌리는 형식으로 하면 되지 않냐?

[사슴발레리나(dearvalerie)]

헐..? 이벤트 처음들어보는데 검색해서 함 공부해보겠음 키워드 제공 감사. 그런식으로 하면 이동속도 상한, 하한 등등 변수가 더 늘어날 거 같은데 그래도 코딩 편의성보다 성능 챙기는게 우선이겠지.. 다시한번 땡큐

[사슴발레리나(dearvalerie)]

딕셔너리는 방금 뇌내코딩으로 시뮬 돌려보니까 어차피 슬롯 몇번에 어떤 유물 번호가 있는지 확인하려면 현재 하는 방식이랑 크게 다를 거 없어서 일단 보류했음 ㅎ

[익명(211.51)]

10개뿐이면 큰 영향은 없을거같긴한데. 나라면 유물 갯수만큼 bool 배열 만들어서 해당 값이 true면 효과 발동 하게하는 if문을 만들거같아. 예를들면 if(유물[1]== true){ 이동속도 1미만이면 1로 } 이렇게만..? 그러면 조금 더 적게 검색하니까 조금 더 성능이 나아질지도..?

[사슴발레리나(dearvalerie)]

나도 처음엔 그렇게 했었는데 유물 얻는 순서별로 차곡차곡 쌓이게 하고 싶어서.. 이렇게 바꿨음. 안그럼 유물슬롯이랑 유물획득여부를 따로 관리해야 하는데 본인 빡통이라 나중에 힘들어질거같더라

[익명(211.51)]

이러면 유물마다 넘버링해주면 좋을듯..

[사슴발레리나(dearvalerie)]

역시 어렵네... 일단 하던거 중단하고 다른 사람들은 어떻게 하는지 검색좀 하다올게 고마워

[익명(211.51)]

그럼 배열 2개 쓰는건 어때? 획득순서 그릴때는 얻은 순서대로 만들어진 배열 쓰고 기능 호출할땐 위의 방식으로 하고.. 그럼 훨씬 깔끔해질거같은데

[익명(211.51)]

유물 7번 획득 > 유물 배열의 7번쨰 값을 true , 획득 배열의 0번쨰 값을 7로 저장 해서

[사슴발레리나(dearvalerie)]

헐 한 칸에 두개 이상 들어가는 배열도 있어???????????? 아니면 유물 소지 여부 배열이랑 유물 획득 시기(슬롯) 배열을 따로따로 두란 말인가? ㅇㅋ 이렇게 해볼게 어떤 방법이든 지금 하고 있는 방식은 너무 찝찝한듯... 그렇게 해야겠다

[익명(211.51)]

후자를 말한거야

[사슴발레리나(dearvalerie)]

[익명(14.32)]

10개 미만이면 그냥 둬도 괜찮을 것 같아

[사슴발레리나(dearvalerie)]

일단 지금까지 만든거 빌드한다음 내 똥폰으로 함 확인해볼게 ㄳㄳ 나중에 개발일지같은거 올리면 실험 결과 공유하겠음... 고수들한텐 너무 자명한 결과일지 모르겠지만

[익명(14.32)]

10개면 이정도까지 갈 필요는 없겠지만 그 10개 안에서도 가장 자주 참조되는 놈들이 앞에 있는게 성능에 도움이 될 것 같네

[사슴발레리나(dearvalerie)]

똥폰 테스트 결과: 60프레임 유지가 안될때가 있음. 아마 유물 체크하는 함수가 업데이트문 안에서도 여러차례 호출되고 있음 + 적들 스크립트 + 기타 다른 연산 전부 포함해서 이렇게 된 거겠지만 고수님들 말씀대로 고치는게 맞을듯. 감사합니다

[익명(180.231)]

??? : 오빠 오빠 오빠 나 비싸

[사슴발레리나(dearvalerie)]

[해당 댓글은 삭제되었습니다.]

해당 댓글은 삭제되었습니다.

[사슴발레리나(dearvalerie)]

ㅇㅋ 위 댓글 + 님 댓글 합쳐서 그럼 Awake()에서 유물장착스탯 결정한다음 그걸로 계속 쓰다가 유물 획득하는 시점에서 유물장착스탯만 다시 불러오는 함수로 해보겠음.. 감사합니다

[사슴발레리나(dearvalerie)]

헐 hash set 뭐임 이거 처음보는건데 공부해보겠습니다

[사슴발레리나(dearvalerie)]

[익명(61.85)]

요즘 컴퓨터는 뒤지게 빨라서 매 프레임에 100개 포문 돌리는걸 10개 돌려도 렉 안걸림

[사슴발레리나(dearvalerie)]

컴터는 걱정없는데 모바일이 저번에 테스트해보니까 생각보다 성능 안좋더라고. 게다가 모바일 특성상 유저들 기기 성능 차이도 심할건데 "이거 제폰에서 안돌아감 똥겜임 ㅅㄱ" << 이런 리뷰 받으면 가슴찢어질거같음

[익명(61.85)]

정말 안좋은 방법이고 버그나 핵에 취약해지는 방법이지만, 나같으면 이동속도 감소 디버프가 걸릴 떄에만 if문 달아서 체크할거같음 매 프레임마다 체크할 필요가 없다고 판단되면 코루틴으로 연산횟수를 확 줄여서 사용하거나 아니면 특정 동작에만 값이 변하게 하는 쪽으로

[사슴발레리나(dearvalerie)]

아 그러네 그냥 이동속도가 기본 이속보다 낮을때만 체크하면 평상시엔 안돌아가니까 괜찮겠네 ㄳㄳ if문 구조도 좀 더 싸게 바꿔보겠음

[익명(220.83)]

아이템 착용 해제 하는 함수 안에 딕셔너리 검사하는 친구 만들고, 게임 시작하면 딕셔너리 읽어와서 적용시키면 쌀듯

[사슴발레리나(dearvalerie)]

윗댓보고 딕셔너리 찍먹해봤는데 아직 어떻게 쓰는건지 잘 모르겠음.. ㅈㅅ합니다

[태보미(taebomi)]

스테이지 상에서 유물 보유여부가 매프레임 변경되는게 아니기때문에 보유여부를 계속 체크할게 아니라 유물을 스테이지에서 획득한 경우 + 게임 시작할 때 유물 보유여부를 체크하고, 각 유물이 건드리는 스텟을 변수화 한 다음에 유물 보유여부에 따라 그 스텟만 수정해주는 식으로 할 듯? 예를 들면 이동관련 함수에서는 최소 이동속도 및 이동관련 변수들 두고 매프레임 이동속도 관련해서 체크하는 함수에서 현재 이동속도가 최소 이동속도 아래면 최소이동속도로 대입, 이후 현재 이동속도가 보정받을 이동속도 아래면 보정받을 속도 더해주기, 게임 시작시나 유물 획득 시에만 1번 유물 획득하면 이 이동속도 관련 변수값을 수정해주는 방식으로.

[사슴발레리나(dearvalerie)]

대고수 ㅇㅈ합니다 이게 젤 쉬울듯 ㄳㄳ