From b0fdfc5f4f23ded83272a4188c69efa79819b72d Mon Sep 17 00:00:00 2001 From: EntireTwix Date: Wed, 7 Jul 2021 14:33:34 -0700 Subject: [PATCH] :racehorse: checking if balance copied is non zero before modify call --- src/bank.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/bank.cpp b/src/bank.cpp index de3923e..d6a29cd 100644 --- a/src/bank.cpp +++ b/src/bank.cpp @@ -233,7 +233,7 @@ BankResponse Bank::DelUser(const std::string &name) noexcept uint32_t bal; if (users.if_contains(name, [this, &bal](const User &u) { bal = u.balance; - })) + }) && bal) { users.modify_if(return_account, [ this, bal ](User & u)) { @@ -250,11 +250,11 @@ BankResponse Bank::DelUser(const std::string &name) noexcept save_flag = true; #endif #endif - return BankResponse{k200OK, "\"User deleted!\""}; + return BankResponse(k200OK, "\"User deleted!\""); } else { - return BankResponse{k404NotFound, "\"User not found\""}; + return BankResponse(k404NotFound, "\"User not found\""); } } void Bank::Save()