는 : 소나타 관리, 단일 관리 클래스는 내가 추가 방법 몇 가지 필드를 추가하고 있다면, 우리의 품질 게이트 그것은을 것을, 나에게 말하고이 경우프레임 워크 표준 구문에서 문자열을 추출하는 것이 좋습니다. 예를 들어
$formMapper->add('test', null, ['label' => 'testlabel']);
$formMapper->add('test1', null, ['label' => 'testlabel1']);
$formMapper->add('test2', null, ['label' => 'testlabel2']);
처럼 항상 반복 구문 'label'문자열을 2 번 이상 사용하여 냄새를 맡으십시오. 나는
const KEY_LABEL = 'label'
$formMapper->add('test2', null, [self::KEY_LABEL => 'testlabel2']);
처럼 상수의 전체 많은 많은 수업을하는 좋은 또는 나쁜 관행은 이제
인가 ... 그 문자열 상수를 추가해야합니다 ???
나는, 언젠가는 레이블 키가 변경됩니다 경우 큰 도움이 .. 프레임 워크에 의해 제공되는 무슨, 내가이 물건 중 하나의 방법을 변경할 필요가 알아낼 수 없습니다 ...
상수를 사용할 이유가 없습니다. 코드 스니퍼가 항상 올바른 것은 아닙니다. – Andrew
내가 사용하는 번들이 작동하는 방식을 변경하지 않을 것을 제안합니다. 바르게 지적하면 레이블이 변경 될 수 있으며 레이블을 변경하기 위해 전체 코드 기반을 거치지 않아도됩니다. 0이 아니라면 효과는 0에 가까워집니다. @Andrew가 말했듯이, 코드 스니퍼는 그들이 확인하고있는 코드를 실제로 이해하지 못하고,이 경우에 그들은 바보가됩니다. – tchap
@tchap 외에도 PSR2 만 사용하면 충분하다고 생각합니다. /usr/local/bin/phpcs --standard = PSR2. – staskrak