Changeset 3dcc27d7d496d4bdf4b9329acec09d11d077da7d

Timestamp:

05/20/07 12:47:08 (2 years ago)

Author:

Christopher Jung <bktheg@web.de>

git-committer:

Christopher Jung <bktheg@web.de> 1179658028 +0200

git-parent:

[5b2bf669f1f869829e390eabfc26aa967ec02fbd]

git-author:

Christopher Jung <bktheg@web.de> 1179658028 +0200

Message:

Naechster Versuch die Exceptions beim Kapern in den Griff zu bekommen

Files:

• src/net/driftingsouls/ds2/server/battles/Battle.java (modified) (3 diffs)
• src/net/driftingsouls/ds2/server/modules/ks/KSKapernAction.java (modified) (2 diffs)

src/net/driftingsouls/ds2/server/battles/Battle.java

@@ -457 +457 @@
-        
-        /**
+         * Prueft, ob das aktuell ausgewaehlte gegnerische Schiff ein gueltiges Ziel ist
+         * (das Schiff also existiert). Bei gueltigen Zielen ist ein aufruf von {@link #getEnemyShip()}
+         * gefahrlos moeglich.
+         * @return <code>true</code>, falls das Ziel gueltig ist
+         */
+        public boolean isValidTarget() {
+                if( this.activeSEnemy >= this.enemyShips.size() ) {
+                        return false;
+                }
+                if( this.activeSEnemy < 0 ) {
+                        return false;
+                }
+                return true;
+        }
+        
+        /**
-         * Liefert den Index des naechsten feindlichen Schiffes nach dem aktuell ausgewaehlten. 
-         * Bevorzugt werden Schiffe gleichen Typs
@@ -462 +478 @@
-         */
-        public int getNewTargetIndex() {
-
-
-
-
-
-
-
+
+
-                        for( int i=0; i < enemyShips.size(); i++ ) {
-                                SQLResultRow aship = enemyShips.get(i);
-
-
-
-
+
+
-                                        return i;
-                                }
-                        }
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-                        }
-                }
@@ -494 +521 @@
-                for( int i=0; i < enemyShips.size(); i++ ) {
-                        SQLResultRow aship = enemyShips.get(i);
+                        if( aship.getInt("id") == enemyShip.getInt("id") ) {
+                                continue;
+                        }
+                        
-                        if( ((aship.getString("docked").length() == 0) || (aship.getString("docked").charAt(0) != 'l')) && (aship.getInt("action") & Battle.BS_DESTROYED) == 0 && (aship.getInt("action") & Battle.BS_SECONDROW) == 0 ) {
-                                return i;

src/net/driftingsouls/ds2/server/modules/ks/KSKapernAction.java

@@ -292 +292 @@
-                        sid.free();
-        
+                        // TODO: Das Entfernen eines Schiffes aus der Liste sollte in Battle 
+                        // durchgefuehrt werden und den Zielindex automatisch anpassen
+                        // (durch das Entfernen von Schiffen kann der Zielindex ungueltig geworden sein)
+                        
-                        // Ein neues Ziel auswaehlen
-                        battle.setEnemyShipIndex(battle.getNewTargetIndex());
@@ -313 +317 @@
-                                return RESULT_HALT;
-                        }
-
+
+
+
+
+
-                        Ships.recalculateShipStatus(enemyShip.getInt("id"));
-