BT
сеттеры буллщит, транзакции нет
Size: a a a
2021 March 09
BT
атомарность? забудь
AK
Го обсудим код
```
```
Что можно улучшить?
```
public function change_balance($request, $user1, $user2, $amount) {
$u1 = $this->repo->find($user1);
$u2 = $this->repo->find($user2);
$u1->setAmount($u1->getAmount() - $amount);
$u2->setAmount($u1->getAmount() + $amount);
$u1->save();
$u2->save();
}```
Что можно улучшить?
название метода не по psr
RS
```
```
<?php
public function changeBalance(int $user1,int $user2, $amount) {
$u1 = $this->repo->find($user1);
$u2 = $this->repo->find($user2);
$u1->setAmount($u1->getAmount() - $amount);
$u2->setAmount($u1->getAmount() - $amount);
$u1->save();
$u2->save();
}```
BT
```
```
<?php
public function changeBalance(int $user1,int $user2, $amount) {
$u1 = $this->repo->find($user1);
$u2 = $this->repo->find($user2);
$u1->setAmount($u1->getAmount() - $amount);
$u2->setAmount($u1->getAmount() - $amount);
$u1->save();
$u2->save();
}```
$userId тогда
BT
и $firstUserId
RS
Отлично!
BT
ну ты понял
AK
Го обсудим код
```
```
Что можно улучшить?
```
public function change_balance($request, $user1, $user2, $amount) {
$u1 = $this->repo->find($user1);
$u2 = $this->repo->find($user2);
$u1->setAmount($u1->getAmount() - $amount);
$u2->setAmount($u1->getAmount() + $amount);
$u1->save();
$u2->save();
}```
Что можно улучшить?
вынести изменение баланса в отдельный сервис. использовать там внутри уже объекты user
BT
и транзации, карл
AK
$amount без типа
AK
не понятно шо там вообще
RS
и транзации, карл
+
RS
Бля, так много наговорили. Сча переделаю
D
и это ты 4 строки кинул)
RS
да. я спецом
RS
Короче почти все победили. Там оставалось еще:
```
```
```
$u1 = $this->repo->find($user1);
$u2 = $this->repo->find($user2);```
D
блядь я не успел даже свои 5 копеек вставить
RS
Я сверился со своим собесом
D
как тут отпидарасили код)