Skip to content

Commit 992744d

Browse files
committed
main
1 parent 11c031b commit 992744d

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

readme.md

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929

3030
### Введение
3131

32-
В первую очередь командой преследуется цель оперативного решения проблем связанных с бизнес процессом. По этому весь код должен быть написан согласно [PSR-4](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-4-autoloader-examples.md) и [Чистый код](https://github.com/3xter/clean-code-php). Ели нет уверенности, что вы пишете читабельный код посоветуйтесь с соседом рядом. Не обязательно это должен быть кто-то из команды. Придерживаемся правила: "Если два человека поняли написанное, третий с большой вероятностью тоже поймет".
32+
В первую очередь командой преследуется цель оперативного решения проблем связанных с бизнес процессом. По этому весь код должен быть написан согласно [PSR-4](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-4-autoloader-examples.md) и [Чистый код](https://github.com/3xter/clean-code-php). Еcли нет уверенности, что вы пишете читабельный код посоветуйтесь с соседом рядом. Не обязательно это должен быть кто-то из команды. Придерживаемся правила: "Если два человека поняли написанное, третий с большой вероятностью тоже поймет".
3333

3434
Первые проблемы начинаются с неявно описанных тасков и плохо спроектированной базы. Если при прочтении таска у вас возникли вопросы в первую очередь пишем их в таск комментарием, обращаемся к ПМ-у. Часто бывают ситуации, когда уточнение задачи вы обсуждаете напрямую с постановщиком. Если такое случается берем за правило оповестить ПМ-а про изменения/уточнения. Просим постановщика подправить сам таск или даже просто в таск скинуть переписку в которой вы решили непонятные моменты. В таске обязательно должны быть зафиксированы все уточнения и/или изменения задачи. Это может привести к переэстимейту таска.
3535

@@ -286,7 +286,7 @@ final class BrandRepository
286286
foreach ($query->executeQuery()->iterateAssociative() as $brand) {
287287

288288
yield new BrandObject(
289-
$Uuid::fromString(brand['guid']),
289+
Uuid::fromString(brand['guid']),
290290
$brand['name'],
291291
);
292292
}
@@ -339,7 +339,7 @@ final class BrandCreator
339339
if ($this->brandRepository->isExistByName($brandCreatorObject->brandName)) {
340340
throw new BrandCreatorException("Brand exist");
341341
}
342-
$brand = new BrandEntity();
342+
$brand = new BrandEntity($brandCreatorObject->brandName);
343343

344344
$this->entityManager->persist($brand);
345345

@@ -351,7 +351,6 @@ final class BrandCreator
351351
}
352352
```
353353

354-
355354
### Коллекции
356355

357356
Советуюем использовать реализацию [https://github.com/ramsey/collection](https://github.com/ramsey/collection) так как они типизированы. По возможности не использовать массивы.

0 commit comments

Comments
 (0)