Php Inspections - 'isset(...)' constructs can be merged / 여러 `isset(...)` 문을 하나로 병합합니다.

Php Inspections ​(EA Extended)​

저는 일정 수준의 코드 퀄리티를 유지하기 위해 Php Inspections ​(EA Extended)​ 정적 분석기를 사용하고 있습니다.

특히 위 정적 분서기는 Phpstorm의 플러그인으로 적용되고 있기 때문에 사용하는 게 매우 쉽고 용이합니다.

Phpstorm 에 설치하면 좋은 플러그인은 아래 게시글을 보면 확인할 수 있습니다.

(https://sparkweb.co.kr/forum/Phpstorm-%EC%B6%94%EC%B2%9C-%ED%94%8C%EB%9F%AC%EA%B7%B8%EC%9D%B8)

 


 

'isset(...)' constructs can be merged

이번에 소개할 것은 위 정적 분석기에서 알려주는 검사 중 하나인 `'isset(...)' constructs can be merged` 를 설명하고 있습니다.

`'isset(...)' constructs can be merged` 은 여러 `isset(...)` 문을 하나로 병합할 수 있는 경우 알려줍니다.

저는 이 검사를 통해 처음으로 `issset(...)` 가 병합되어 사용될 수 있다는 사실을 알게 되었습니다.

 

Php Inspections ​(EA Extended)​ 에서 설명하고 있는 예시

/* before */
$value = isset($first) && isset($second);
    
/* after */
$value = isset($first, $second);

 

내가 경험한 실제 코드

/** before */
if (isset($apiToken) && isset($id) && isset($isAdmin)) {
    ````
}

/** after */
if (isset($apiToken, $id, $isAdmin))
 ````
}

 

참고

https://github.com/kalessil/phpinspectionsea/blob/master/docs/control-flow.md#isset-constructs-can-be-merged

  • share