[🐣질문] (겜스)밑에 테두리 구현 코드인데 리뷰해주실분 있나요

[익명(121.128)]

function heal_range_display_script()

{

var s = surface_create(460,360);

var camera_x=camera_get_view_x(view_camera[0])

var camera_y=camera_get_view_y(view_camera[0])

if(surface_exists(s))

{

surface_set_target(s);

draw_clear_alpha(c_black,0);

with healrange_object

if bbox_right>=camera_x && bbox_left<=camera_x+460

&& bbox_bottom>=camera_y && bbox_top<=camera_y+360 // 카메라 안에 있는 놈들만 그리게

draw_sprite_ext(range_outline_sprite,image_index,x-camera_x,y-camera_y,2,2,0,c_blue,1) // 테두리+내용물 그림

gpu_set_blendmode(bm_subtract)

with healrange_object

if bbox_right>=camera_x && bbox_left<=camera_x+460

&& bbox_bottom>=camera_y && bbox_top<=camera_y+360

draw_sprite_ext(range_outline_sprite,image_index+3,x-camera_x,y-camera_y,2,2,0,c_blue,1) // 내용물(+겹치는 테두리) 제거

gpu_set_blendmode(bm_normal)

surface_reset_target();

draw_surface(s,camera_x,camera_y);

surface_free(s);

}

var s = surface_create(460,360); // 다른 종류의 힐타워에 대해 같은 코드 한번 더

if(surface_exists(s))

{

surface_set_target(s);

draw_clear_alpha(c_black,0);

with bobble_range_object

if bbox_right>=camera_x && bbox_left<=camera_x+460

&& bbox_bottom>=camera_y && bbox_top<=camera_y+360

draw_sprite_ext(range_outline_sprite,image_index,x-camera_x,y-camera_y,2,2,0,c_yellow,1)

gpu_set_blendmode(bm_subtract)

with bobble_range_object

if bbox_right>=camera_x && bbox_left<=camera_x+460

&& bbox_bottom>=camera_y && bbox_top<=camera_y+360

draw_sprite_ext(range_outline_sprite,image_index+3,x-camera_x,y-camera_y,2,2,0,c_yellow,1)

gpu_set_blendmode(bm_normal)

surface_reset_target();

draw_surface(s,camera_x,camera_y);

surface_free(s);

}

}

range_outline_sprite는 이렇게 생겼습니다

저거만 드로우하면 초당 프레임이 100에서 40까지 떨어져서 이게 이렇게 무거운 코드인가 싶은데

혹시 제가 잘못하고 있는 부분이나 더 깔끔하게 표현할수 있으면 조언 부탁드립니다

해당 스크립트는 블럭 설치시 미리보기 오브젝트 하나에서만 실행됩니다

[ㅇㅇ 1(125.132)]

일단 저 코드가 매프레임 호출된다라는 전제하에 댓글 달아볼께요 아니면 무시부탁 일단 서피스라는게 렌더타겟 텍스쳐 비슷한거같은데 매프레임 객체를 만들었다 없앴다 할 필요 있나요? 왠지 만들어 둔거 clear만해서 울궈먹을수 있을거같은? 그리고 서피스를 객체를 남겨두는김에 테두리가 애니메이션이 되지 않는다면 애초에 매프레임 다시 그릴필요 있을까 싶습니다 camera_x,camera_y가 움직이거나 새 타워가 생겻을때만 서피스 내용을 갱신해주고 평소에는 그냥 출력만 해보는건 어떨까요.

[익명(121.128)]

드로우 코드라 매프레임 그리는게 맞습니다. camera_x는 캐릭터 중심으로 계속 바뀌는거라 그렇게 할거면 그냥 맵 전체를 대상으로 그려야될거같은데 2560*1440 사이즈라 그거대로 걱정되네요 그리고 서피스를 계속 남겨두는것도 신경쓰이고요 직접 해봐야 알겠지만 피드백 감사합니다!

[익명(125.139)]

var s = surface_create(460,360); << draw 에 박으면 성능 하락 지림 create에서 생성해두고 draw에선 생성된 서퍼스 호출만 해서 지우고 그리고 지우고 그리고 하는게 나음

[익명(121.128)]

헉 그렇군요 생성이벤트에 넣어서 한번 해볼게요

[익명(121.128)]

헐 맙소사 이게 문제였던것같아요 사실 댓글 보면서도 '아무리 그래도 이게 60프레임이나 깎아먹을 실순가?' 했는데 저것만 생성이벤트로 빼서 다시 했더니 바로 100프레임 그대로 방어되네요 정말 감사합니다!!!

[매튜돌(matthewstone218)]

서피스는 생성 코드에 surf = -1; 이렇게 선언한 다음에, 드로우 코드 맨 처음에 if(surface_exists(surf)){surface_create(460,360)} 이런식으로 존재하는지 확인하고, 없으면 생성해야합니다. 굳이 저렇게 매 프레임마다 있는지 확인하는 이유는, 전체화면 전환이나 컴퓨터 내부 사정으로 vram이(서피스가) 날아갈 수 있기 때문입니다. 그래서 만약 서피스가 계속 저장되어야 하는 경우(중간에 날라가면 안되는 경우) 매 프레임마다 버퍼에 저장해줘야합니다./

[매튜돌(matthewstone218)]

추가적으로, surface_set_target은 그자체로 성능을 꽤 먹기 때문에 최대한 적게 호출하는게 좋습니다.

[매튜돌(matthewstone218)]

윗 댓에서 다 설명해서 이미 하셨을 것 같긴 한데, 당연히 서피스 프리도 필요 없어용

[ㅇㅇ 2(121.128)]

서피스 프리가 필요없단건 무슨말인지 설명해주실수 있나요 다썼을땐 항상 surface_free 해줘야하는걸로 알고있는데.. 일단 서피스 생성을 생성이벤트로 옮겨서 서피스 프리도 삭제이벤트로 옮겼긴 합니다

[매튜돌(matthewstone218)]

서피스를 매 프레임마다 프리시켜줄 이유가 없다는 뜻이었어요. 삭제 혹은 클린 이벤트로 옮기셨으면 문제 없습니다. 다만 게임 재시작 등의 작업은 삭제 이벤트를 트리거하지 않으니, clean 이벤트로 surface_free 코드를 옮기는걸 추천드립니다.

[WWS(clip1446)]

나 같은 경우는 create에서 서피스에 쓸 변수를 선언하고 (생성해도 상관없음) 드로우에서 서피스가 없을 경우에 생성하는 방식으로 짜는데 혹시나 실수로 다른 코드에서 free하거나 창 관련 함수로 윈도우를 바꾸면 서피스가 없어질 수도 있기 때문에 그렇게 하는 게 안전함 surface_set_target, draw_surface, gpu_set_blendmode 같은 함수는 gpu 버텍스 버퍼 배치를 나눠버려서 성능에 영향을 줄 수 있으니 최소한으로 호출하는 게 좋음

[WWS(clip1446)]

겜메단 화이팅~

[ㅇㅇ 2(121.128)]

전체적으로 무겁다고 들어서 지금까지 사용을 자제해왔는데 이번에 써보니 요주의 코드 몇개만 조심하면 생각보다 어마어마하게 무겁진 않네요 참고하겠습니다 겜메단 파이어~