return 이후의 unset() 호출은 실행되지 않음 정보
return 이후의 unset() 호출은 실행되지 않음본문
shop/cart.php
// 테마에 cart.php 있으면 include
if(defined('G5_THEME_SHOP_PATH')) {
$theme_cart_file = G5_THEME_SHOP_PATH.'/cart.php';
if(is_file($theme_cart_file)) {
include_once($theme_cart_file);
return;
unset($theme_cart_file);
}
}전역변수 정리를 위한 코드라고 생각되는데 return 문 이후에 위치한 unset()은 실행되지 않기 때문에 의미 없는 코드로 보입니다. 해당 코드는 제거하거나, 필요하다면 if문 밖으로 옮기는 것이 적절해 보입니다.
또한, cart.php 외에도 테마 파일을 불러오는 부분에서 유사한 패턴이 반복되고 있습니다.
같은 로직이 여러 곳에서 반복되고 있으므로, 테마 파일을 조건에 따라 불러오는 기능을 별도의 함수로 분리해 재사용하면 전역 스코프에 영향을 주지 않으면서, 향후 유지보수도 더욱 용이해질 것으로 보입니다.
추천
0
0
관련링크
댓글 1개
안녕하세요. SIR 입니다.
알려주셔서 감사합니다.
해당 내용을 수정하겠습니다.
90레벨 이상 댓글을 남길 수 있습니다.