<!-- canonical: https://0xsimao.com/findings/fuji-finance-useless-else-statement -->

# Useless else statement

Low/Info · Three Sigma · Lending aggregator · 6th May, 2023

Finding 3S-FUJI-N17 of the Fuji Finance security review.

- Report: /reports/fuji-finance
- Source: https://cdn.sanity.io/files/qoqld077/staging/32181a28eac3175d15fb8924d249bb0d91ca350c.pdf

---

### Description

If a call reverts in the first if statement, the following else is not required.

### Recommendation

Replace

```solidity
if (balance < amount) {
    revert ConnextRouter__xReceive_notReceivedAssetBalance();
} else {
    _tempTokenToCheck = Snapshot(asset, balance - amount);
}
```

with

```solidity
if (balance < amount) {
    revert ConnextRouter__xReceive_notReceivedAssetBalance();
}
_tempTokenToCheck = Snapshot(asset, balance - amount);
```

### Status

Addressed here: [Fujicracy/fuji-v2#625](https://github.com/Fujicracy/fuji-v2/pull/625) and [Fujicracy/fuji-v2#622](https://github.com/Fujicracy/fuji-v2/pull/622)