alois/OpenFrontIO
1
0
Fork 0
mirror of https://github.com/openfrontio/OpenFrontIO.git synced 2026-08-06 10:54:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #163 from PilkeySEK/handledeaddefender-guard

Browse Source 
Add guard clause instead of nesting the if statement in handleDeadDefender()
This commit is contained in:
evanpelle
2025-03-07 08:50:26 -08:00
committed by GitHub
parent 8462c04e65 e979e1184c
commit db3a5e9d6a
1 changed files with 25 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/core/execution/AttackExecution.ts
+25 -25
View File
@@ -283,32 +283,32 @@ export class AttackExecution implements Execution {
} }
private handleDeadDefender() { private handleDeadDefender() {
if (this.target.isPlayer() && this.target.numTilesOwned() < 100) { if (!(this.target.isPlayer() && this.target.numTilesOwned() < 100)) return;
const gold = this.target.gold();
this.mg.displayMessage(
`Conquered ${this.target.displayName()} received ${renderNumber(
gold,
)} gold`,
MessageType.SUCCESS,
this._owner.id(),
);
this.target.removeGold(gold);
this._owner.addGold(gold);
for (let i = 0; i < 10; i++) { const gold = this.target.gold();
for (const tile of this.target.tiles()) { this.mg.displayMessage(
const borders = this.mg `Conquered ${this.target.displayName()} received ${renderNumber(
.neighbors(tile) gold,
.some((t) => this.mg.owner(t) == this._owner); )} gold`,
if (borders) { MessageType.SUCCESS,
this._owner.conquer(tile); this._owner.id(),
} else { );
for (const neighbor of this.mg.neighbors(tile)) { this.target.removeGold(gold);
const no = this.mg.owner(neighbor); this._owner.addGold(gold);
if (no.isPlayer() && no != this.target) {
this.mg.player(no.id()).conquer(tile); for (let i = 0; i < 10; i++) {
break; for (const tile of this.target.tiles()) {
} const borders = this.mg
.neighbors(tile)
.some((t) => this.mg.owner(t) == this._owner);
if (borders) {
this._owner.conquer(tile);
} else {
for (const neighbor of this.mg.neighbors(tile)) {
const no = this.mg.owner(neighbor);
if (no.isPlayer() && no != this.target) {
this.mg.player(no.id()).conquer(tile);
break;
} }
} }
} }