# Findings

|                  |                                                 |              |                                            |
| ---------------- | ----------------------------------------------- | ------------ | ------------------------------------------ |
| **Type**         | **Severity**                                    | **Location** | **Status**                                 |
| Gas Optimization | <mark style="color:green;">Informational</mark> | \*           | <mark style="color:red;">Unresolved</mark> |

**Description**

The public isExcluded, totalFees , reflect and reflectionFromToken functions should be declared as external.

**Recommendation**

These functions are only called from outside of the contract, consider using the external attribute instead of public to save

|                  |                                                 |              |                                            |
| ---------------- | ----------------------------------------------- | ------------ | ------------------------------------------ |
| **Type**         | **Severity**                                    | **Location** | **Status**                                 |
| Gas Optimization | <mark style="color:green;">Informational</mark> | \*           | <mark style="color:red;">Unresolved</mark> |

**Description**

State variables that are never changed should be constant.

**Recommendation**&#x20;

The state variables “\_name”, “\_symbol” and “\_decimals” are never changed. Consider changing their attribute to constant to save on gas.

|               |                                                 |              |                                            |
| ------------- | ----------------------------------------------- | ------------ | ------------------------------------------ |
| **Type**      | **Severity**                                    | **Location** | **Status**                                 |
| Best Practice | <mark style="color:green;">Informational</mark> | \*           | <mark style="color:red;">Unresolved</mark> |

**Description**

\_tTotal value has too many digits. Literals with too many digits are difficult to read and review.

**Recommendation**

Our recommendation is to represent the number by a power of 10.

|               |                                                 |              |                                            |
| ------------- | ----------------------------------------------- | ------------ | ------------------------------------------ |
| **Type**      | **Severity**                                    | **Location** | **Status**                                 |
| Best Practice | <mark style="color:green;">Informational</mark> | \*           | <mark style="color:red;">Unresolved</mark> |

**Description**

There are many places in the code that interact with taxFee variable, but since it is set to zero it has no effect. Therefore, the user pays extra gas costs for no reason.

**Recommendation**

Consider deleting the code that has no effect.