fix(weapons): drop unneeded check in useWeapon

[ranisalt]

Pull Request Prelude

• I have followed proper The Forgotten Server code styling.
• I have read and understood the contribution guidelines before making this PR.
• I am aware that this PR may be closed if the above-mentioned criteria are not fulfilled.

Changes Proposed

Drops an unneeded check in WeaponDistance::useWeapon, if bow->getHitChance() is 0 it doesn't matter if it's added to chance or not.

[Copilot]

Pull request overview

This pull request refactors a conditional check in the WeaponDistance::useWeapon method by removing an unnecessary comparison. The change simplifies the code by eliminating the check for bow->getHitChance() != 0, since adding 0 to the chance variable has no effect on the calculation. The refactoring also modernizes the code by using a C++ if-initializer pattern.

• Removes redundant zero-value check for bow->getHitChance()
• Refactors to use if-initializer for more concise pointer null checking

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[ramon-bernardo]

LGTM 😃