PHPコードスニファの三項演算子でエラー

PHPコードスニファと三項演算子に問題があります。私は、オペレータPHPコードスニファの三項演算子でエラー

<rule ref="Squiz.WhiteSpace.OperatorSpacing"/>

とif文今、私はショートのエラーを持った後と前にスペースをチェックするためのルールを追加しました：

return ($this->get('router')->getContext()->getHttpPort() == 80) 
    ? '//'.$this->get('router')->getContext()->getHost() 
    : '//'.$this->get('router')->getContext()->getHost().':'.$this->get('router')->getContext()->getHttpPort();

誰もが知っている：

37 | ERROR | [x] Expected 1 space before "?"; newline found 
38 | ERROR | [x] Expected 1 space before ":"; newline found

は私のコードは次のように見えます問題がありますか？私はルールセットファイル全体を貼り付けることができますが、OperatorSpacingルールを削除した後はすべてOKです。

挨拶

出典

2017-05-31 jager91

よく働いています

<rule ref="Squiz.WhiteSpace.OperatorSpacing"> <properties> <property name="ignoreNewlines" value="true"/> </properties> </rule>

を追加し、エラーは、システムがあなたの事業者のそれぞれの前にスペースを期待していた、一目瞭然ようだが、改行を発見しました。それらを1つのスペースで置き換えてください。 – roberto06

この目的の機能の詳細はこちらhttps://github.com/squizlabs/PHP_CodeSniffer/issues/348 – roberto06

Squizにはあなたが従わなければならない標準があります – RiggsFolly

[OK]をみんな、助けに感謝しかし、私は解決策は、@ roberto06、リンクM8に感謝を発見！

した後、それは完璧:)

出典

2017-05-31 08:07:08 jager91

あなたの回答には、受け入れ可能とマークできますか？だから私たちは皆、この質問が解決されたことを知っている、ありがとう –

@TomášVotruba完了:) – jager91

それだけですべてが同じ行にする必要がありますと言います。

ところで、あなたのケースでより良い何かが次のようになります。

$value = '//'.$this->get('router')->getContext()->getHost(); 

return $value . ($this->get('router')->getContext()->getHttpPort() !== 80) ? (':'.$this->get('router')->getContext()->getHttpPort()) : '';

出典

2017-05-31 07:58:41 Asakuura

PHPコードスニファの三項演算子でエラー

答えて

関連する問題