#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 учасників
Завантажується...
Скасувати
Зберегти
Тут ще немає жодного вмісту.