#1 Pointless reassignement of lastMinerRecord

Открыто
3 лет назад открыта TooManySugar · комментариев: 0
TooManySugar прокомментировал 3 лет назад

https://gogs.veloe.link/Veloe/EthemineTelegramBotCore/src/master/EthermineBotTelegramCore/UpdateData.cs#L88-L90

lastMinerRecord already assigned at line 42. Assignment here assigns the same value as was before.

Otherwise if emit Botdb.SaveChangesAsync() after Botdb.miners.Add(newMinerRecord) (line 46) this assignment would always lead to equality with newMinerRecord.

https://gogs.veloe.link/Veloe/EthemineTelegramBotCore/src/master/EthermineBotTelegramCore/UpdateData.cs#L88-L90 lastMinerRecord already assigned at line 42. Assignment here assigns the same value as was before. Otherwise if emit `Botdb.SaveChangesAsync()` after `Botdb.miners.Add(newMinerRecord)` (line 46) this assignment would always lead to equality with newMinerRecord.
Войдите, чтобы присоединиться к обсуждению.
Нет меток
Нет этапа
Нет ответственного
1 участников
Загрузка...
Отмена
Сохранить
Пока нет содержимого.