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 외에도 테마 파일을 불러오는 부분에서 유사한 패턴이 반복되고 있습니다.
같은 로직이 여러 곳에서 반복되고 있으므로,
테마 파일을 조건에 따라 불러오는 기능을 별도의 함수로 분리해 재사용하면
전역 스코프에 영향을 주지 않으면서, 향후 유지보수도 더욱 용이해질 것으로 보입니다.
shop/cart.php
전역변수 정리를 위한 코드라고 생각되는데
return 문 이후에 위치한 unset()은 실행되지 않기 때문에 의미 없는 코드로 보입니다.
해당 코드는 제거하거나, 필요하다면 if문 밖으로 옮기는 것이 적절해 보입니다.
또한, cart.php 외에도 테마 파일을 불러오는 부분에서 유사한 패턴이 반복되고 있습니다.
같은 로직이 여러 곳에서 반복되고 있으므로,
테마 파일을 조건에 따라 불러오는 기능을 별도의 함수로 분리해 재사용하면
전역 스코프에 영향을 주지 않으면서, 향후 유지보수도 더욱 용이해질 것으로 보입니다.