Complex classes like Schema often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Schema, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 13 | class Schema implements \JsonSerializable, \ArrayAccess { |
||
| 14 | /** |
||
| 15 | * Trigger a notice when extraneous properties are encountered during validation. |
||
| 16 | */ |
||
| 17 | const VALIDATE_EXTRA_PROPERTY_NOTICE = 0x1; |
||
| 18 | |||
| 19 | /** |
||
| 20 | * Throw a ValidationException when extraneous properties are encountered during validation. |
||
| 21 | */ |
||
| 22 | const VALIDATE_EXTRA_PROPERTY_EXCEPTION = 0x2; |
||
| 23 | |||
| 24 | /** |
||
| 25 | * @var array All the known types. |
||
| 26 | * |
||
| 27 | * If this is ever given some sort of public access then remove the static. |
||
| 28 | */ |
||
| 29 | private static $types = [ |
||
| 30 | 'array' => ['a'], |
||
| 31 | 'object' => ['o'], |
||
| 32 | 'integer' => ['i', 'int'], |
||
| 33 | 'string' => ['s', 'str'], |
||
| 34 | 'number' => ['f', 'float'], |
||
| 35 | 'boolean' => ['b', 'bool'], |
||
| 36 | 'timestamp' => ['ts'], |
||
| 37 | 'datetime' => ['dt'], |
||
| 38 | 'null' => ['n'] |
||
| 39 | ]; |
||
| 40 | |||
| 41 | private $schema = []; |
||
| 42 | |||
| 43 | /** |
||
| 44 | * @var int A bitwise combination of the various **Schema::FLAG_*** constants. |
||
| 45 | */ |
||
| 46 | private $flags = 0; |
||
| 47 | |||
| 48 | /** |
||
| 49 | * @var array An array of callbacks that will filter data in the schema. |
||
| 50 | */ |
||
| 51 | private $filters = []; |
||
| 52 | |||
| 53 | /** |
||
| 54 | * @var array An array of callbacks that will custom validate the schema. |
||
| 55 | */ |
||
| 56 | private $validators = []; |
||
| 57 | |||
| 58 | /** |
||
| 59 | * @var string|Validation The name of the class or an instance that will be cloned. |
||
| 60 | */ |
||
| 61 | private $validationClass = Validation::class; |
||
| 62 | |||
| 63 | |||
| 64 | /// Methods /// |
||
| 65 | |||
| 66 | /** |
||
| 67 | * Initialize an instance of a new {@link Schema} class. |
||
| 68 | * |
||
| 69 | * @param array $schema The array schema to validate against. |
||
| 70 | */ |
||
| 71 | 168 | public function __construct($schema = []) { |
|
| 74 | |||
| 75 | /** |
||
| 76 | * Grab the schema's current description. |
||
| 77 | * |
||
| 78 | * @return string |
||
| 79 | */ |
||
| 80 | 1 | public function getDescription() { |
|
| 83 | |||
| 84 | /** |
||
| 85 | * Set the description for the schema. |
||
| 86 | * |
||
| 87 | * @param string $description The new description. |
||
| 88 | * @throws \InvalidArgumentException Throws an exception when the provided description is not a string. |
||
| 89 | * @return Schema |
||
| 90 | */ |
||
| 91 | 2 | public function setDescription($description) { |
|
| 100 | |||
| 101 | /** |
||
| 102 | * Get a schema field. |
||
| 103 | * |
||
| 104 | * @param string|array $path The JSON schema path of the field with parts separated by dots. |
||
| 105 | * @param mixed $default The value to return if the field isn't found. |
||
| 106 | * @return mixed Returns the field value or `$default`. |
||
| 107 | */ |
||
| 108 | 4 | public function getField($path, $default = null) { |
|
| 125 | |||
| 126 | /** |
||
| 127 | * Set a schema field. |
||
| 128 | * |
||
| 129 | * @param string|array $path The JSON schema path of the field with parts separated by dots. |
||
| 130 | * @param mixed $value The new value. |
||
| 131 | * @return $this |
||
| 132 | */ |
||
| 133 | 3 | public function setField($path, $value) { |
|
| 156 | |||
| 157 | /** |
||
| 158 | * Get the ID for the schema. |
||
| 159 | * |
||
| 160 | * @return string |
||
| 161 | */ |
||
| 162 | 3 | public function getID() { |
|
| 165 | |||
| 166 | /** |
||
| 167 | * Set the ID for the schema. |
||
| 168 | * |
||
| 169 | * @param string $id The new ID. |
||
| 170 | * @throws \InvalidArgumentException Throws an exception when the provided ID is not a string. |
||
| 171 | * @return Schema |
||
| 172 | */ |
||
| 173 | 4 | public function setID($id) { |
|
| 174 | 4 | if (is_string($id)) { |
|
| 175 | 1 | $this->schema['id'] = $id; |
|
| 176 | 1 | } else { |
|
| 177 | throw new \InvalidArgumentException("The ID is not a valid string.", 500); |
||
| 178 | } |
||
| 179 | |||
| 180 | 1 | return $this; |
|
| 181 | } |
||
| 182 | |||
| 183 | /** |
||
| 184 | * Return the validation flags. |
||
| 185 | * |
||
| 186 | * @return int Returns a bitwise combination of flags. |
||
| 187 | */ |
||
| 188 | 1 | public function getFlags() { |
|
| 191 | |||
| 192 | /** |
||
| 193 | * Set the validation flags. |
||
| 194 | * |
||
| 195 | * @param int $flags One or more of the **Schema::FLAG_*** constants. |
||
| 196 | * @return Schema Returns the current instance for fluent calls. |
||
| 197 | */ |
||
| 198 | 8 | public function setFlags($flags) { |
|
| 206 | |||
| 207 | /** |
||
| 208 | * Whether or not the schema has a flag (or combination of flags). |
||
| 209 | * |
||
| 210 | * @param int $flag One or more of the **Schema::VALIDATE_*** constants. |
||
| 211 | * @return bool Returns **true** if all of the flags are set or **false** otherwise. |
||
| 212 | */ |
||
| 213 | 18 | public function hasFlag($flag) { |
|
| 216 | |||
| 217 | /** |
||
| 218 | * Set a flag. |
||
| 219 | * |
||
| 220 | * @param int $flag One or more of the **Schema::VALIDATE_*** constants. |
||
| 221 | * @param bool $value Either true or false. |
||
| 222 | * @return $this |
||
| 223 | */ |
||
| 224 | 1 | public function setFlag($flag, $value) { |
|
| 232 | |||
| 233 | /** |
||
| 234 | * Merge a schema with this one. |
||
| 235 | * |
||
| 236 | * @param Schema $schema A scheme instance. Its parameters will be merged into the current instance. |
||
| 237 | * @return $this |
||
| 238 | */ |
||
| 239 | 3 | public function merge(Schema $schema) { |
|
| 243 | |||
| 244 | /** |
||
| 245 | * Add another schema to this one. |
||
| 246 | * |
||
| 247 | * Adding schemas together is analogous to array addition. When you add a schema it will only add missing information. |
||
| 248 | * |
||
| 249 | * @param Schema $schema The schema to add. |
||
| 250 | * @param bool $addProperties Whether to add properties that don't exist in this schema. |
||
| 251 | * @return $this |
||
| 252 | */ |
||
| 253 | 3 | public function add(Schema $schema, $addProperties = false) { |
|
| 257 | |||
| 258 | /** |
||
| 259 | * The internal implementation of schema merging. |
||
| 260 | * |
||
| 261 | * @param array &$target The target of the merge. |
||
| 262 | * @param array $source The source of the merge. |
||
| 263 | * @param bool $overwrite Whether or not to replace values. |
||
| 264 | * @param bool $addProperties Whether or not to add object properties to the target. |
||
| 265 | * @return array |
||
| 266 | */ |
||
| 267 | 6 | private function mergeInternal(array &$target, array $source, $overwrite = true, $addProperties = true) { |
|
| 319 | |||
| 320 | // public function overlay(Schema $schema ) |
||
| 321 | |||
| 322 | /** |
||
| 323 | * Returns the internal schema array. |
||
| 324 | * |
||
| 325 | * @return array |
||
| 326 | * @see Schema::jsonSerialize() |
||
| 327 | */ |
||
| 328 | 15 | public function getSchemaArray() { |
|
| 331 | |||
| 332 | /** |
||
| 333 | * Parse a short schema and return the associated schema. |
||
| 334 | * |
||
| 335 | * @param array $arr The schema array. |
||
| 336 | * @param mixed ...$args Constructor arguments for the schema instance. |
||
| 337 | * @return static Returns a new schema. |
||
| 338 | */ |
||
| 339 | 160 | public static function parse(array $arr, ...$args) { |
|
| 344 | |||
| 345 | /** |
||
| 346 | * Parse a schema in short form into a full schema array. |
||
| 347 | * |
||
| 348 | * @param array $arr The array to parse into a schema. |
||
| 349 | * @return array The full schema array. |
||
| 350 | * @throws \InvalidArgumentException Throws an exception when an item in the schema is invalid. |
||
| 351 | */ |
||
| 352 | 160 | protected function parseInternal(array $arr) { |
|
| 384 | |||
| 385 | /** |
||
| 386 | * Parse a schema node. |
||
| 387 | * |
||
| 388 | * @param array $node The node to parse. |
||
| 389 | * @param mixed $value Additional information from the node. |
||
| 390 | * @return array Returns a JSON schema compatible node. |
||
| 391 | */ |
||
| 392 | 154 | private function parseNode($node, $value = null) { |
|
| 442 | |||
| 443 | /** |
||
| 444 | * Parse the schema for an object's properties. |
||
| 445 | * |
||
| 446 | * @param array $arr An object property schema. |
||
| 447 | * @return array Returns a schema array suitable to be placed in the **properties** key of a schema. |
||
| 448 | */ |
||
| 449 | 99 | private function parseProperties(array $arr) { |
|
| 475 | |||
| 476 | /** |
||
| 477 | * Parse a short parameter string into a full array parameter. |
||
| 478 | * |
||
| 479 | * @param string $key The short parameter string to parse. |
||
| 480 | * @param array $value An array of other information that might help resolve ambiguity. |
||
| 481 | * @return array Returns an array in the form `[string name, array param, bool required]`. |
||
| 482 | * @throws \InvalidArgumentException Throws an exception if the short param is not in the correct format. |
||
| 483 | */ |
||
| 484 | 154 | public function parseShortParam($key, $value = []) { |
|
| 542 | |||
| 543 | /** |
||
| 544 | * Add a custom filter to change data before validation. |
||
| 545 | * |
||
| 546 | * @param string $fieldname The name of the field to filter, if any. |
||
| 547 | * |
||
| 548 | * If you are adding a filter to a deeply nested field then separate the path with dots. |
||
| 549 | * @param callable $callback The callback to filter the field. |
||
| 550 | * @return $this |
||
| 551 | */ |
||
| 552 | 1 | public function addFilter($fieldname, callable $callback) { |
|
| 556 | |||
| 557 | /** |
||
| 558 | * Add a custom validator to to validate the schema. |
||
| 559 | * |
||
| 560 | * @param string $fieldname The name of the field to validate, if any. |
||
| 561 | * |
||
| 562 | * If you are adding a validator to a deeply nested field then separate the path with dots. |
||
| 563 | * @param callable $callback The callback to validate with. |
||
| 564 | * @return Schema Returns `$this` for fluent calls. |
||
| 565 | */ |
||
| 566 | 2 | public function addValidator($fieldname, callable $callback) { |
|
| 570 | |||
| 571 | /** |
||
| 572 | * Require one of a given set of fields in the schema. |
||
| 573 | * |
||
| 574 | * @param array $required The field names to require. |
||
| 575 | * @param string $fieldname The name of the field to attach to. |
||
| 576 | * @param int $count The count of required items. |
||
| 577 | * @return Schema Returns `$this` for fluent calls. |
||
| 578 | */ |
||
| 579 | 1 | public function requireOneOf(array $required, $fieldname = '', $count = 1) { |
|
| 628 | |||
| 629 | /** |
||
| 630 | * Validate data against the schema. |
||
| 631 | * |
||
| 632 | * @param mixed $data The data to validate. |
||
| 633 | * @param bool $sparse Whether or not this is a sparse validation. |
||
| 634 | * @return mixed Returns a cleaned version of the data. |
||
| 635 | * @throws ValidationException Throws an exception when the data does not validate against the schema. |
||
| 636 | */ |
||
| 637 | 132 | public function validate($data, $sparse = false) { |
|
| 653 | |||
| 654 | /** |
||
| 655 | * Validate data against the schema and return the result. |
||
| 656 | * |
||
| 657 | * @param mixed $data The data to validate. |
||
| 658 | * @param bool $sparse Whether or not to do a sparse validation. |
||
| 659 | * @return bool Returns true if the data is valid. False otherwise. |
||
| 660 | */ |
||
| 661 | 35 | public function isValid($data, $sparse = false) { |
|
| 669 | |||
| 670 | /** |
||
| 671 | * Validate a field. |
||
| 672 | * |
||
| 673 | * @param mixed $value The value to validate. |
||
| 674 | * @param ValidationField $field A validation object to add errors to. |
||
| 675 | * @param bool $sparse Whether or not this is a sparse validation. |
||
| 676 | * @return mixed|Invalid Returns a clean version of the value with all extra fields stripped out or invalid if the value |
||
| 677 | * is completely invalid. |
||
| 678 | */ |
||
| 679 | 132 | protected function validateField($value, ValidationField $field, $sparse = false) { |
|
| 738 | |||
| 739 | /** |
||
| 740 | * Validate an array. |
||
| 741 | * |
||
| 742 | * @param mixed $value The value to validate. |
||
| 743 | * @param ValidationField $field The validation results to add. |
||
| 744 | * @param bool $sparse Whether or not this is a sparse validation. |
||
| 745 | * @return array|Invalid Returns an array or invalid if validation fails. |
||
| 746 | */ |
||
| 747 | 19 | protected function validateArray($value, ValidationField $field, $sparse = false) { |
|
| 802 | |||
| 803 | /** |
||
| 804 | * Validate a boolean value. |
||
| 805 | * |
||
| 806 | * @param mixed $value The value to validate. |
||
| 807 | * @param ValidationField $field The validation results to add. |
||
| 808 | * @return bool|Invalid Returns the cleaned value or invalid if validation fails. |
||
| 809 | */ |
||
| 810 | 21 | protected function validateBoolean($value, ValidationField $field) { |
|
| 818 | |||
| 819 | /** |
||
| 820 | * Validate a date time. |
||
| 821 | * |
||
| 822 | * @param mixed $value The value to validate. |
||
| 823 | * @param ValidationField $field The validation results to add. |
||
| 824 | * @return \DateTimeInterface|Invalid Returns the cleaned value or **null** if it isn't valid. |
||
| 825 | */ |
||
| 826 | 12 | protected function validateDatetime($value, ValidationField $field) { |
|
| 851 | |||
| 852 | /** |
||
| 853 | * Validate a float. |
||
| 854 | * |
||
| 855 | * @param mixed $value The value to validate. |
||
| 856 | * @param ValidationField $field The validation results to add. |
||
| 857 | * @return float|Invalid Returns a number or **null** if validation fails. |
||
| 858 | */ |
||
| 859 | 8 | protected function validateNumber($value, ValidationField $field) { |
|
| 867 | |||
| 868 | /** |
||
| 869 | * Validate and integer. |
||
| 870 | * |
||
| 871 | * @param mixed $value The value to validate. |
||
| 872 | * @param ValidationField $field The validation results to add. |
||
| 873 | * @return int|Invalid Returns the cleaned value or **null** if validation fails. |
||
| 874 | */ |
||
| 875 | 28 | protected function validateInteger($value, ValidationField $field) { |
|
| 884 | |||
| 885 | /** |
||
| 886 | * Validate an object. |
||
| 887 | * |
||
| 888 | * @param mixed $value The value to validate. |
||
| 889 | * @param ValidationField $field The validation results to add. |
||
| 890 | * @param bool $sparse Whether or not this is a sparse validation. |
||
| 891 | * @return object|Invalid Returns a clean object or **null** if validation fails. |
||
| 892 | */ |
||
| 893 | 87 | protected function validateObject($value, ValidationField $field, $sparse = false) { |
|
| 905 | |||
| 906 | /** |
||
| 907 | * Validate data against the schema and return the result. |
||
| 908 | * |
||
| 909 | * @param array|\ArrayAccess $data The data to validate. |
||
| 910 | * @param ValidationField $field This argument will be filled with the validation result. |
||
| 911 | * @param bool $sparse Whether or not this is a sparse validation. |
||
| 912 | * @return array|Invalid Returns a clean array with only the appropriate properties and the data coerced to proper types. |
||
| 913 | * or invalid if there are no valid properties. |
||
| 914 | */ |
||
| 915 | 86 | protected function validateProperties($data, ValidationField $field, $sparse = false) { |
|
| 980 | |||
| 981 | /** |
||
| 982 | * Validate a string. |
||
| 983 | * |
||
| 984 | * @param mixed $value The value to validate. |
||
| 985 | * @param ValidationField $field The validation results to add. |
||
| 986 | * @return string|Invalid Returns the valid string or **null** if validation fails. |
||
| 987 | */ |
||
| 988 | 56 | protected function validateString($value, ValidationField $field) { |
|
| 1077 | |||
| 1078 | /** |
||
| 1079 | * Validate a unix timestamp. |
||
| 1080 | * |
||
| 1081 | * @param mixed $value The value to validate. |
||
| 1082 | * @param ValidationField $field The field being validated. |
||
| 1083 | * @return int|Invalid Returns a valid timestamp or invalid if the value doesn't validate. |
||
| 1084 | */ |
||
| 1085 | 7 | protected function validateTimestamp($value, ValidationField $field) { |
|
| 1096 | |||
| 1097 | /** |
||
| 1098 | * Validate a null value. |
||
| 1099 | * |
||
| 1100 | * @param mixed $value The value to validate. |
||
| 1101 | * @param ValidationField $field The error collector for the field. |
||
| 1102 | * @return null|Invalid Returns **null** or invalid. |
||
| 1103 | */ |
||
| 1104 | protected function validateNull($value, ValidationField $field) { |
||
| 1111 | |||
| 1112 | /** |
||
| 1113 | * Validate a value against an enum. |
||
| 1114 | * |
||
| 1115 | * @param mixed $value The value to test. |
||
| 1116 | * @param ValidationField $field The validation object for adding errors. |
||
| 1117 | * @return mixed|Invalid Returns the value if it is one of the enumerated values or invalid otherwise. |
||
| 1118 | */ |
||
| 1119 | 130 | protected function validateEnum($value, ValidationField $field) { |
|
| 1138 | |||
| 1139 | /** |
||
| 1140 | * Call all of the filters attached to a field. |
||
| 1141 | * |
||
| 1142 | * @param mixed $value The field value being filtered. |
||
| 1143 | * @param ValidationField $field The validation object. |
||
| 1144 | * @return mixed Returns the filtered value. If there are no filters for the field then the original value is returned. |
||
| 1145 | */ |
||
| 1146 | 132 | protected function callFilters($value, ValidationField $field) { |
|
| 1156 | |||
| 1157 | /** |
||
| 1158 | * Call all of the validators attached to a field. |
||
| 1159 | * |
||
| 1160 | * @param mixed $value The field value being validated. |
||
| 1161 | * @param ValidationField $field The validation object to add errors. |
||
| 1162 | */ |
||
| 1163 | 130 | protected function callValidators($value, ValidationField $field) { |
|
| 1183 | |||
| 1184 | /** |
||
| 1185 | * Specify data which should be serialized to JSON. |
||
| 1186 | * |
||
| 1187 | * This method specifically returns data compatible with the JSON schema format. |
||
| 1188 | * |
||
| 1189 | * @return mixed Returns data which can be serialized by **json_encode()**, which is a value of any type other than a resource. |
||
| 1190 | * @link http://php.net/manual/en/jsonserializable.jsonserialize.php |
||
| 1191 | * @link http://json-schema.org/ |
||
| 1192 | */ |
||
| 1193 | public function jsonSerialize() { |
||
| 1228 | |||
| 1229 | /** |
||
| 1230 | * Look up a type based on its alias. |
||
| 1231 | * |
||
| 1232 | * @param string $alias The type alias or type name to lookup. |
||
| 1233 | * @return mixed |
||
| 1234 | */ |
||
| 1235 | 150 | protected function getType($alias) { |
|
| 1246 | |||
| 1247 | /** |
||
| 1248 | * Get the class that's used to contain validation information. |
||
| 1249 | * |
||
| 1250 | * @return Validation|string Returns the validation class. |
||
| 1251 | */ |
||
| 1252 | 132 | public function getValidationClass() { |
|
| 1255 | |||
| 1256 | /** |
||
| 1257 | * Set the class that's used to contain validation information. |
||
| 1258 | * |
||
| 1259 | * @param Validation|string $class Either the name of a class or a class that will be cloned. |
||
| 1260 | * @return $this |
||
| 1261 | */ |
||
| 1262 | 1 | public function setValidationClass($class) { |
|
| 1270 | |||
| 1271 | /** |
||
| 1272 | * Create a new validation instance. |
||
| 1273 | * |
||
| 1274 | * @return Validation Returns a validation object. |
||
| 1275 | */ |
||
| 1276 | 132 | protected function createValidation() { |
|
| 1286 | |||
| 1287 | /** |
||
| 1288 | * Check whether or not a value is an array or accessible like an array. |
||
| 1289 | * |
||
| 1290 | * @param mixed $value The value to check. |
||
| 1291 | * @return bool Returns **true** if the value can be used like an array or **false** otherwise. |
||
| 1292 | */ |
||
| 1293 | 87 | private function isArray($value) { |
|
| 1296 | |||
| 1297 | /** |
||
| 1298 | * Cast a value to an array. |
||
| 1299 | * |
||
| 1300 | * @param \Traversable $value The value to convert. |
||
| 1301 | * @return array Returns an array. |
||
| 1302 | */ |
||
| 1303 | private function toArray(\Traversable $value) { |
||
| 1309 | |||
| 1310 | /** |
||
| 1311 | * Return a sparse version of this schema. |
||
| 1312 | * |
||
| 1313 | * A sparse schema has no required properties. |
||
| 1314 | * |
||
| 1315 | * @return Schema Returns a new sparse schema. |
||
| 1316 | */ |
||
| 1317 | 2 | public function withSparse() { |
|
| 1321 | |||
| 1322 | /** |
||
| 1323 | * The internal implementation of `Schema::withSparse()`. |
||
| 1324 | * |
||
| 1325 | * @param array|Schema $schema The schema to make sparse. |
||
| 1326 | * @param \SplObjectStorage $schemas Collected sparse schemas that have already been made. |
||
| 1327 | * @return mixed |
||
| 1328 | */ |
||
| 1329 | 2 | private function withSparseInternal($schema, \SplObjectStorage $schemas) { |
|
| 1357 | |||
| 1358 | /** |
||
| 1359 | * Filter a field's value using built in and custom filters. |
||
| 1360 | * |
||
| 1361 | * @param mixed $value The original value of the field. |
||
| 1362 | * @param ValidationField $field The field information for the field. |
||
| 1363 | * @return mixed Returns the filtered field or the original field value if there are no filters. |
||
| 1364 | */ |
||
| 1365 | 132 | private function filterField($value, ValidationField $field) { |
|
| 1389 | |||
| 1390 | /** |
||
| 1391 | * Whether a offset exists. |
||
| 1392 | * |
||
| 1393 | * @param mixed $offset An offset to check for. |
||
| 1394 | * @return boolean true on success or false on failure. |
||
| 1395 | * @link http://php.net/manual/en/arrayaccess.offsetexists.php |
||
| 1396 | */ |
||
| 1397 | 3 | public function offsetExists($offset) { |
|
| 1400 | |||
| 1401 | /** |
||
| 1402 | * Offset to retrieve. |
||
| 1403 | * |
||
| 1404 | * @param mixed $offset The offset to retrieve. |
||
| 1405 | * @return mixed Can return all value types. |
||
| 1406 | * @link http://php.net/manual/en/arrayaccess.offsetget.php |
||
| 1407 | */ |
||
| 1408 | public function offsetGet($offset) { |
||
| 1411 | |||
| 1412 | /** |
||
| 1413 | * Offset to set. |
||
| 1414 | * |
||
| 1415 | * @param mixed $offset The offset to assign the value to. |
||
| 1416 | * @param mixed $value The value to set. |
||
| 1417 | * @link http://php.net/manual/en/arrayaccess.offsetset.php |
||
| 1418 | */ |
||
| 1419 | public function offsetSet($offset, $value) { |
||
| 1422 | |||
| 1423 | /** |
||
| 1424 | * Offset to unset. |
||
| 1425 | * |
||
| 1426 | * @param mixed $offset The offset to unset. |
||
| 1427 | * @link http://php.net/manual/en/arrayaccess.offsetunset.php |
||
| 1428 | */ |
||
| 1429 | public function offsetUnset($offset) { |
||
| 1432 | } |
||
| 1433 |
This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.
To visualize
will produce issues in the first and second line, while this second example
will produce no issues.