Complex classes like BlameableTrait 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 BlameableTrait, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
44 | trait BlameableTrait |
||
45 | { |
||
46 | use ConfirmationTrait, |
||
47 | SelfBlameableTrait; |
||
48 | |||
49 | private $blameableLocalRules = []; |
||
50 | private $blameableLocalBehaviors = []; |
||
51 | |||
52 | /** |
||
53 | * @var boolean|string|array Specify the attribute(s) name of content(s). If |
||
54 | * there is only one content attribute, you can assign its name. Or there |
||
55 | * is multiple attributes associated with contents, you can assign their |
||
56 | * names in array. If you don't want to use this feature, please assign |
||
57 | * false. |
||
58 | * For example: |
||
59 | * ```php |
||
60 | * public $contentAttribute = 'comment'; // only one field named as 'comment'. |
||
61 | * ``` |
||
62 | * or |
||
63 | * ```php |
||
64 | * public $contentAttribute = ['year', 'month', 'day']; // multiple fields. |
||
65 | * ``` |
||
66 | * or |
||
67 | * ```php |
||
68 | * public $contentAttribute = false; // no need of this feature. |
||
69 | * ``` |
||
70 | * If you don't need this feature, you should add rules corresponding with |
||
71 | * `content` in `rules()` method of your user model by yourself. |
||
72 | */ |
||
73 | public $contentAttribute = 'content'; |
||
74 | |||
75 | /** |
||
76 | * @var array built-in validator name or validatation method name and |
||
77 | * additional parameters. |
||
78 | */ |
||
79 | public $contentAttributeRule = ['string', 'max' => 255]; |
||
80 | |||
81 | /** |
||
82 | * @var boolean|string Specify the field which stores the type of content. |
||
83 | */ |
||
84 | public $contentTypeAttribute = false; |
||
85 | |||
86 | /** |
||
87 | * @var boolean|array Specify the logic type of content, not data type. If |
||
88 | * your content doesn't need this feature. please specify false. If the |
||
89 | * $contentAttribute is specified to false, this attribute will be skipped. |
||
90 | * ```php |
||
91 | * public $contentTypes = [ |
||
92 | * 'public', |
||
93 | * 'private', |
||
94 | * 'friend', |
||
95 | * ]; |
||
96 | * ``` |
||
97 | */ |
||
98 | public $contentTypes = false; |
||
99 | |||
100 | /** |
||
101 | * @var boolean|string This attribute speicfy the name of description |
||
102 | * attribute. If this attribute is assigned to false, this feature will be |
||
103 | * skipped. |
||
104 | */ |
||
105 | public $descriptionAttribute = false; |
||
106 | |||
107 | /** |
||
108 | * @var string |
||
109 | */ |
||
110 | public $initDescription = ''; |
||
111 | |||
112 | /** |
||
113 | * @var string the attribute that will receive current user ID value. This |
||
114 | * attribute must be assigned. |
||
115 | */ |
||
116 | public $createdByAttribute = "user_guid"; |
||
117 | |||
118 | /** |
||
119 | * @var string the attribute that will receive current user ID value. |
||
120 | * Set this property to false if you do not want to record the updater ID. |
||
121 | */ |
||
122 | public $updatedByAttribute = "user_guid"; |
||
123 | |||
124 | /** |
||
125 | * @var boolean Add combinated unique rule if assigned to true. |
||
126 | */ |
||
127 | public $idCreatorCombinatedUnique = true; |
||
128 | |||
129 | /** |
||
130 | * @var boolean|string The name of user class which own the current entity. |
||
131 | * If this attribute is assigned to false, this feature will be skipped, and |
||
132 | * when you use create() method of UserTrait, it will be assigned with |
||
133 | * current user class. |
||
134 | */ |
||
135 | public $userClass; |
||
136 | public static $cacheKeyBlameableRules = 'blameable_rules'; |
||
137 | public static $cacheTagBlameableRules = 'tag_blameable_rules'; |
||
138 | public static $cacheKeyBlameableBehaviors = 'blameable_behaviors'; |
||
139 | public static $cacheTagBlameableBehaviors = 'tag_blameable_behaviors'; |
||
140 | |||
141 | /** |
||
142 | * @inheritdoc |
||
143 | */ |
||
144 | 37 | public function rules() |
|
145 | { |
||
146 | 37 | return $this->getBlameableRules(); |
|
147 | } |
||
148 | |||
149 | /** |
||
150 | * @inheritdoc |
||
151 | */ |
||
152 | 37 | public function behaviors() |
|
153 | { |
||
154 | 37 | return $this->getBlameableBehaviors(); |
|
155 | } |
||
156 | |||
157 | /** |
||
158 | * Get total of contents which owned by their owner. |
||
159 | * @return integer |
||
160 | */ |
||
161 | 2 | public function countOfOwner() |
|
162 | { |
||
163 | 2 | $createdByAttribute = $this->createdByAttribute; |
|
164 | 2 | return static::find()->where([$createdByAttribute => $this->$createdByAttribute])->count(); |
|
165 | } |
||
166 | |||
167 | /** |
||
168 | * Get content. |
||
169 | * @return mixed |
||
170 | */ |
||
171 | public function getContent() |
||
172 | { |
||
173 | $contentAttribute = $this->contentAttribute; |
||
174 | if ($contentAttribute === false) { |
||
175 | return null; |
||
176 | } |
||
177 | if (is_array($contentAttribute)) { |
||
178 | $content = []; |
||
179 | foreach ($contentAttribute as $key => $value) { |
||
180 | $content[$key] = $this->$value; |
||
181 | } |
||
182 | return $content; |
||
183 | } |
||
184 | return $this->$contentAttribute; |
||
185 | } |
||
186 | |||
187 | /** |
||
188 | * Set content. |
||
189 | * @param mixed $content |
||
190 | */ |
||
191 | public function setContent($content) |
||
192 | { |
||
193 | $contentAttribute = $this->contentAttribute; |
||
194 | if ($contentAttribute === false) { |
||
195 | return; |
||
196 | } |
||
197 | if (is_array($contentAttribute)) { |
||
198 | foreach ($contentAttribute as $key => $value) { |
||
199 | $this->$value = $content[$key]; |
||
200 | } |
||
201 | return; |
||
202 | } |
||
203 | $this->$contentAttribute = $content; |
||
204 | } |
||
205 | |||
206 | /** |
||
207 | * Determines whether content could be edited. Your should implement this |
||
208 | * method by yourself. |
||
209 | * @return boolean |
||
210 | * @throws \yii\base\NotSupportedException |
||
211 | */ |
||
212 | public function getContentCanBeEdited() |
||
213 | { |
||
214 | if ($this->contentAttribute === false) { |
||
215 | return false; |
||
216 | } |
||
217 | throw new \yii\base\NotSupportedException("This method is not implemented."); |
||
218 | } |
||
219 | |||
220 | /** |
||
221 | * Check it has been ever edited. |
||
222 | * @return boolean Whether this content has ever been edited. |
||
223 | */ |
||
224 | public function hasEverEdited() |
||
225 | { |
||
226 | $createdAtAttribute = $this->createdByAttribute; |
||
227 | $updatedAtAttribute = $this->updatedByAttribute; |
||
228 | if (!$createdAtAttribute || !$updatedAtAttribute) { |
||
229 | return false; |
||
230 | } |
||
231 | return $this->$createdAtAttribute === $this->$updatedAtAttribute; |
||
232 | } |
||
233 | |||
234 | /** |
||
235 | * Get blameable rules cache key. |
||
236 | * @return string cache key. |
||
237 | */ |
||
238 | 37 | public function getBlameableRulesCacheKey() |
|
239 | { |
||
240 | 37 | return static::className() . $this->cachePrefix . static::$cacheKeyBlameableRules; |
|
|
|||
241 | } |
||
242 | |||
243 | /** |
||
244 | * Get blameable rules cache tag. |
||
245 | * @return string cache tag |
||
246 | */ |
||
247 | 13 | public function getBlameableRulesCacheTag() |
|
248 | { |
||
249 | 13 | return static::className() . $this->cachePrefix . static::$cacheTagBlameableRules; |
|
250 | } |
||
251 | |||
252 | /** |
||
253 | * Get the rules associated with content to be blamed. |
||
254 | * @return array rules. |
||
255 | */ |
||
256 | 37 | public function getBlameableRules() |
|
257 | { |
||
258 | 37 | $cache = $this->getCache(); |
|
259 | 37 | if ($cache) { |
|
260 | 37 | $this->blameableLocalRules = $cache->get($this->getBlameableRulesCacheKey()); |
|
261 | 37 | } |
|
262 | // 若当前规则不为空,且是数组,则认为是规则数组,直接返回。 |
||
263 | 37 | if (!empty($this->blameableLocalRules) && is_array($this->blameableLocalRules)) { |
|
264 | 30 | return $this->blameableLocalRules; |
|
265 | } |
||
266 | |||
267 | // 父类规则与确认规则合并。 |
||
268 | 13 | if ($cache) { |
|
269 | 13 | TagDependency::invalidate($cache, [$this->getEntityRulesCacheTag()]); |
|
270 | 13 | } |
|
271 | 13 | $rules = array_merge( |
|
272 | 13 | parent::rules(), |
|
273 | 13 | $this->getConfirmationRules(), |
|
274 | 13 | $this->getBlameableAttributeRules(), |
|
275 | 13 | $this->getDescriptionRules(), |
|
276 | 13 | $this->getContentRules(), |
|
277 | 13 | $this->getSelfBlameableRules() |
|
278 | 13 | ); |
|
279 | 13 | $this->setBlameableRules($rules); |
|
280 | 13 | return $this->blameableLocalRules; |
|
281 | } |
||
282 | |||
283 | /** |
||
284 | * Get the rules associated with `createdByAttribute`, `updatedByAttribute` |
||
285 | * and `idAttribute`-`createdByAttribute` combination unique. |
||
286 | * @return array rules. |
||
287 | */ |
||
288 | 13 | public function getBlameableAttributeRules() |
|
289 | { |
||
290 | 13 | $rules = []; |
|
291 | // 创建者和上次修改者由 BlameableBehavior 负责,因此标记为安全。 |
||
292 | 13 | if (!is_string($this->createdByAttribute) || empty($this->createdByAttribute)) { |
|
293 | throw new \yii\base\NotSupportedException('You must assign the creator.'); |
||
294 | } |
||
295 | 13 | $rules[] = [ |
|
296 | 13 | [$this->createdByAttribute], |
|
297 | 13 | 'safe', |
|
298 | ]; |
||
299 | |||
300 | 13 | if (is_string($this->updatedByAttribute) && !empty($this->updatedByAttribute)) { |
|
301 | 3 | $rules[] = [ |
|
302 | 3 | [$this->updatedByAttribute], |
|
303 | 3 | 'safe', |
|
304 | ]; |
||
305 | 3 | } |
|
306 | |||
307 | 13 | if ($this->idCreatorCombinatedUnique && is_string($this->idAttribute)) { |
|
308 | 12 | $rules [] = [ |
|
309 | 12 | [$this->idAttribute, |
|
310 | 12 | $this->createdByAttribute], |
|
311 | 12 | 'unique', |
|
312 | 12 | 'targetAttribute' => [$this->idAttribute, |
|
313 | 12 | $this->createdByAttribute], |
|
314 | ]; |
||
315 | 12 | } |
|
316 | 13 | return $rules; |
|
317 | } |
||
318 | |||
319 | /** |
||
320 | * Get the rules associated with `description` attribute. |
||
321 | * @return array rules. |
||
322 | */ |
||
323 | 13 | public function getDescriptionRules() |
|
324 | { |
||
325 | 13 | $rules = []; |
|
326 | 13 | if (is_string($this->descriptionAttribute) && !empty($this->descriptionAttribute)) { |
|
327 | 1 | $rules[] = [ |
|
328 | 1 | [$this->descriptionAttribute], |
|
329 | 'string' |
||
330 | 1 | ]; |
|
331 | 1 | $rules[] = [ |
|
332 | 1 | [$this->descriptionAttribute], |
|
333 | 1 | 'default', |
|
334 | 1 | 'value' => $this->initDescription, |
|
335 | ]; |
||
336 | 1 | } |
|
337 | 13 | return $rules; |
|
338 | } |
||
339 | |||
340 | /** |
||
341 | * Get the rules associated with `content` and `contentType` attributes. |
||
342 | * @return array rules. |
||
343 | */ |
||
344 | 13 | public function getContentRules() |
|
345 | { |
||
346 | 13 | if (!$this->contentAttribute) { |
|
347 | 2 | return []; |
|
348 | } |
||
349 | 11 | $rules = []; |
|
350 | 11 | $rules[] = [$this->contentAttribute, 'required']; |
|
351 | 11 | if ($this->contentAttributeRule) { |
|
352 | 11 | if (is_string($this->contentAttributeRule)) { |
|
353 | $this->contentAttributeRule = [$this->contentAttributeRule]; |
||
354 | } |
||
355 | 11 | if (is_array($this->contentAttributeRule)) { |
|
356 | 11 | $rules[] = array_merge([$this->contentAttribute], $this->contentAttributeRule); |
|
357 | 11 | } |
|
358 | 11 | } |
|
359 | |||
360 | 11 | if (!$this->contentTypeAttribute) { |
|
361 | 9 | return $rules; |
|
362 | } |
||
363 | |||
364 | 2 | if (is_array($this->contentTypes) && !empty($this->contentTypes)) { |
|
365 | 2 | $rules[] = [[ |
|
366 | 2 | $this->contentTypeAttribute], |
|
367 | 2 | 'required']; |
|
368 | 2 | $rules[] = [[ |
|
369 | 2 | $this->contentTypeAttribute], |
|
370 | 2 | 'in', |
|
371 | 2 | 'range' => array_keys($this->contentTypes)]; |
|
372 | 2 | } |
|
373 | 2 | return $rules; |
|
374 | } |
||
375 | |||
376 | /** |
||
377 | * Set blameable rules. |
||
378 | * @param array $rules |
||
379 | */ |
||
380 | 13 | protected function setBlameableRules($rules = []) |
|
381 | { |
||
382 | 13 | $this->blameableLocalRules = $rules; |
|
383 | 13 | $cache = $this->getCache(); |
|
384 | 13 | if ($cache) { |
|
385 | 13 | $tagDependency = new \yii\caching\TagDependency(['tags' => [$this->getBlameableRulesCacheTag()]]); |
|
386 | 13 | $cache->set($this->getBlameableRulesCacheKey(), $rules, 0, $tagDependency); |
|
387 | 13 | } |
|
388 | 13 | } |
|
389 | |||
390 | /** |
||
391 | * Get blameable behaviors cache key. |
||
392 | * @return string cache key. |
||
393 | */ |
||
394 | 37 | public function getBlameableBehaviorsCacheKey() |
|
398 | |||
399 | /** |
||
400 | * Get blameable behaviors cache tag. |
||
401 | * @return string cache tag. |
||
402 | */ |
||
403 | 13 | public function getBlameableBehaviorsCacheTag() |
|
407 | |||
408 | /** |
||
409 | * Get blameable behaviors. If current behaviors array is empty, the init |
||
410 | * array will be given. |
||
411 | * @return array |
||
412 | */ |
||
413 | 37 | public function getBlameableBehaviors() |
|
414 | { |
||
415 | 37 | $cache = $this->getCache(); |
|
416 | 37 | if ($cache) { |
|
417 | 37 | $this->blameableLocalBehaviors = $cache->get($this->getBlameableBehaviorsCacheKey()); |
|
418 | 37 | } |
|
419 | 37 | if (empty($this->blameableLocalBehaviors) || !is_array($this->blameableLocalBehaviors)) { |
|
420 | 13 | if ($cache) { |
|
421 | 13 | TagDependency::invalidate($cache, [$this->getEntityBehaviorsCacheTag()]); |
|
422 | 13 | } |
|
423 | 13 | $behaviors = parent::behaviors(); |
|
424 | 13 | $behaviors['blameable'] = [ |
|
425 | 13 | 'class' => BlameableBehavior::className(), |
|
426 | 13 | 'createdByAttribute' => $this->createdByAttribute, |
|
427 | 13 | 'updatedByAttribute' => $this->updatedByAttribute, |
|
428 | 13 | 'value' => [$this, |
|
429 | 13 | 'onGetCurrentUserGuid'], |
|
430 | ]; |
||
431 | 13 | $this->setBlameableBehaviors($behaviors); |
|
432 | 13 | } |
|
433 | 37 | return $this->blameableLocalBehaviors; |
|
434 | } |
||
435 | |||
436 | /** |
||
437 | * Set blameable behaviors. |
||
438 | * @param array $behaviors |
||
439 | */ |
||
440 | 13 | protected function setBlameableBehaviors($behaviors = []) |
|
441 | { |
||
442 | 13 | $this->blameableLocalBehaviors = $behaviors; |
|
443 | 13 | $cache = $this->getCache(); |
|
444 | 13 | if ($cache) { |
|
445 | 13 | $tagDependencyConfig = ['tags' => [$this->getBlameableBehaviorsCacheTag()]]; |
|
446 | 13 | $tagDependency = new \yii\caching\TagDependency($tagDependencyConfig); |
|
447 | 13 | $cache->set($this->getBlameableBehaviorsCacheKey(), $behaviors, 0, $tagDependency); |
|
448 | 13 | } |
|
449 | 13 | } |
|
450 | |||
451 | /** |
||
452 | * Set description. |
||
453 | * @return string description. |
||
454 | */ |
||
455 | public function getDescription() |
||
460 | |||
461 | /** |
||
462 | * Get description. |
||
463 | * @param string $desc description. |
||
464 | * @return string|null description if enabled, or null if disabled. |
||
465 | */ |
||
466 | public function setDescription($desc) |
||
471 | |||
472 | /** |
||
473 | * Get blame who owned this blameable model. |
||
474 | * NOTICE! This method will not check whether `$userClass` exists. You should |
||
475 | * specify it in `init()` method. |
||
476 | * @return \vistart\Models\queries\BaseUserQuery user. |
||
477 | */ |
||
478 | 1 | public function getUser() |
|
484 | |||
485 | /** |
||
486 | * Get updater who updated this blameable model recently. |
||
487 | * NOTICE! This method will not check whether `$userClass` exists. You should |
||
488 | * specify it in `init()` method. |
||
489 | * @return \vistart\Models\queries\BaseUserQuery user. |
||
490 | */ |
||
491 | 1 | public function getUpdater() |
|
492 | { |
||
493 | 1 | if (!is_string($this->updatedByAttribute)) { |
|
494 | 1 | return null; |
|
495 | } |
||
496 | $userClass = $this->userClass; |
||
497 | $model = $userClass::buildNoInitModel(); |
||
498 | return $this->hasOne($userClass::className(), [$model->guidAttribute => $this->updatedByAttribute]); |
||
499 | } |
||
500 | |||
501 | /** |
||
502 | * This event is triggered before the model update. |
||
503 | * This method is ONLY used for being triggered by event. DO NOT call, |
||
504 | * override or modify it directly, unless you know the consequences. |
||
505 | * @param \yii\base\Event $event |
||
506 | */ |
||
507 | 11 | public function onContentChanged($event) |
|
508 | { |
||
509 | 11 | $sender = $event->sender; |
|
510 | 11 | $sender->resetConfirmation(); |
|
511 | 11 | } |
|
512 | |||
513 | /** |
||
514 | * Return the current user's GUID if current model doesn't specify the owner |
||
515 | * yet, or return the owner's GUID if current model has been specified. |
||
516 | * This method is ONLY used for being triggered by event. DO NOT call, |
||
517 | * override or modify it directly, unless you know the consequences. |
||
518 | * @param \yii\base\Event $event |
||
519 | * @return string the GUID of current user or the owner. |
||
520 | */ |
||
521 | 37 | public function onGetCurrentUserGuid($event) |
|
522 | { |
||
523 | 37 | $sender = $event->sender; |
|
524 | 37 | if (isset($sender->attributes[$sender->createdByAttribute])) { |
|
525 | 37 | return $sender->attributes[$sender->createdByAttribute]; |
|
526 | } |
||
527 | $identity = \Yii::$app->user->identity; |
||
528 | if ($identity) { |
||
529 | $igAttribute = $identity->guidAttribute; |
||
530 | return $identity->$igAttribute; |
||
531 | } |
||
532 | } |
||
533 | |||
534 | /** |
||
535 | * Initialize type of content. the first of element[index is 0] of |
||
536 | * $contentTypes will be used. |
||
537 | * @param \yii\base\Event $event |
||
538 | */ |
||
539 | 36 | public function onInitContentType($event) |
|
540 | { |
||
541 | 36 | $sender = $event->sender; |
|
542 | 36 | if (!isset($sender->contentTypeAttribute) || !is_string($sender->contentTypeAttribute)) { |
|
543 | 29 | return; |
|
544 | } |
||
545 | 7 | $contentTypeAttribute = $sender->contentTypeAttribute; |
|
546 | 7 | if (!isset($sender->$contentTypeAttribute) && |
|
547 | 7 | !empty($sender->contentTypes) && |
|
548 | 7 | is_array($sender->contentTypes)) { |
|
549 | 7 | $sender->$contentTypeAttribute = $sender->contentTypes[0]; |
|
550 | 7 | } |
|
551 | 7 | } |
|
552 | |||
553 | /** |
||
554 | * Initialize description property with $initDescription. |
||
555 | * @param \yii\base\Event $event |
||
556 | */ |
||
557 | 37 | public function onInitDescription($event) |
|
558 | { |
||
559 | 37 | $sender = $event->sender; |
|
560 | 37 | if (!isset($sender->descriptionAttribute) || !is_string($sender->descriptionAttribute)) { |
|
561 | 31 | return; |
|
562 | } |
||
563 | 6 | $descriptionAttribute = $sender->descriptionAttribute; |
|
564 | 6 | if (empty($sender->$descriptionAttribute)) { |
|
565 | 6 | $sender->$descriptionAttribute = $sender->initDescription; |
|
566 | 6 | } |
|
567 | 6 | } |
|
568 | |||
569 | /** |
||
570 | * Attach events associated with blameable model. |
||
571 | */ |
||
572 | 37 | public function initBlameableEvents() |
|
587 | |||
588 | /** |
||
589 | * @inheritdoc |
||
590 | */ |
||
591 | 11 | public function enabledFields() |
|
617 | |||
618 | /** |
||
619 | * Find all follows by specified identity. If `$identity` is null, the logged-in |
||
620 | * identity will be taken. |
||
621 | * @param string|integer $pageSize If it is 'all`, then will find all follows, |
||
622 | * the `$currentPage` parameter will be skipped. If it is integer, it will be |
||
623 | * regarded as sum of models in one page. |
||
624 | * @param integer $currentPage The current page number, begun with 0. |
||
625 | * @param $userClass $identity |
||
626 | * @return static[] If no follows, null will be given, or return follow array. |
||
627 | */ |
||
628 | public static function findAllByIdentityInBatch($pageSize = 'all', $currentPage = 0, $identity = null) |
||
629 | { |
||
630 | if ($pageSize === 'all') { |
||
631 | return static::findByIdentity($identity)->all(); |
||
632 | } |
||
633 | return static::findByIdentity($identity)->page($pageSize, $currentPage)->all(); |
||
634 | } |
||
635 | |||
636 | /** |
||
637 | * Find one follow by specified identity. If `$identity` is null, the logged-in |
||
638 | * identity will be taken. If $identity doesn't has the follower, null will |
||
639 | * be given. |
||
640 | * @param integer $id user id. |
||
641 | * @param boolean $throwException |
||
642 | * @param $userClass $identity |
||
643 | * @return static |
||
644 | * @throws NotFoundHttpException |
||
645 | */ |
||
646 | public static function findOneById($id, $throwException = true, $identity = null) |
||
658 | |||
659 | /** |
||
660 | * Get total of follows of specified identity. |
||
661 | * @param $userClass $identity |
||
662 | * @return integer total. |
||
663 | */ |
||
664 | public static function countByIdentity($identity = null) |
||
668 | |||
669 | /** |
||
670 | * Get pagination, used for building contents page by page. |
||
671 | * @param integer $limit |
||
672 | * @param $userClass $identity |
||
673 | * @return Pagination |
||
674 | */ |
||
675 | public static function getPagination($limit = 10, $identity = null) |
||
684 | } |
||
685 |
In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code:
Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: