diff --git a/Coding-style-policy.md b/Coding-style-policy.md index 15aba2f..054ff2c 100644 --- a/Coding-style-policy.md +++ b/Coding-style-policy.md @@ -19,13 +19,44 @@ _Reference_: [`Generic.WhiteSpace.DisallowSpaceIndent`](https://github.com/squiz The concatenation operator should have one space on both sides in order to improve readability. +<details><summary>Example</summary><div><br> - <rule ref="Squiz.Strings.ConcatenationSpacing"> - <properties> - <property name="spacing" value="1"/> - <property name="ignoreNewlines" value="true"/> - </properties> - </rule> +**Bad** + +```PHP +$text = $greeting.' '.$name.'!'; +``` + +**Good** (add spaces) + +```PHP +$text = $greeting . ' ' . $name . '!'; +``` + +</div></details><br> + +You may break long lines into multiple lines using the concatenation operator. That way readability can improve considerable when combining lots of variables. + +<details><summary>Example</summary><div><br> + +**Bad** + +```PHP +$text = $greeting.' '.$name.'!'; +``` + +**Good** (split into multiple lines) + +```PHP +$text = $greeting +. ' ' +. $name +. '!'; +``` + +</div></details><br> + +_Reference_: [`Squiz.Strings.ConcatenationSpacing`](https://github.com/squizlabs/PHP_CodeSniffer/blob/master/src/Standards/Squiz/Sniffs/Strings/ConcatenationSpacingSniff.php) # Maximum line length