From ffbbf81cd3322cccc9873b433f46edb1cfbf9f4d Mon Sep 17 00:00:00 2001
From: Matthias Braun <matze@braunis.de>
Date: Thu, 12 Jan 2006 01:19:58 +0000
Subject: [PATCH] fish should behave like the old fish again

SVN-Revision: 2987
---
 src/badguy/fish.cpp | 65 +++++++++++++++++++++++++++++------------------------
 src/badguy/fish.hpp | 10 ++++++---
 2 files changed, 43 insertions(+), 32 deletions(-)

diff --git a/src/badguy/fish.cpp b/src/badguy/fish.cpp
index 5fca83773..5ac6d39bf 100644
--- a/src/badguy/fish.cpp
+++ b/src/badguy/fish.cpp
@@ -20,14 +20,13 @@
 #include <config.h>
 
 #include "fish.hpp"
+#include "tile.hpp"
 #include "object/tilemap.hpp"
 
-static const float FISH_JUMP_POWER = 700;
-static const float FISH_FALL_BY_Y = 10;
-static const float FISH_WAIT_TIME = 3;
+static const float FISH_JUMP_POWER = 600;
+static const float FISH_WAIT_TIME = 1;
 
 Fish::Fish(const lisp::Lisp& reader)
-    : waiting(false)
 {
   reader.get("x", start_position.x);
   reader.get("y", start_position.y);
@@ -37,7 +36,6 @@ Fish::Fish(const lisp::Lisp& reader)
 }
 
 Fish::Fish(float pos_x, float pos_y)
-    : waiting(false)
 {
   start_position.x = pos_x;
   start_position.y = pos_y;
@@ -69,6 +67,15 @@ Fish::collision_badguy(BadGuy& , const CollisionHit& chit)
   return hit(chit);
 }
 
+void
+Fish::draw(DrawingContext& context)
+{
+  if(waiting.started())
+    return;
+
+  BadGuy::draw(context);
+}
+
 HitResponse
 Fish::hit(const CollisionHit& chit)
 {
@@ -80,25 +87,22 @@ Fish::hit(const CollisionHit& chit)
 }
 
 void
+Fish::collision_tile(uint32_t tile_attributes)
+{
+  if(tile_attributes & Tile::WATER) {
+    start_waiting();
+    movement = Vector(0, 0);
+  }
+}
+
+void
 Fish::active_update(float elapsed_time)
 {
   BadGuy::active_update(elapsed_time);
 
-  // check state and modify accordingly
-  if (!waiting && physic.get_velocity_y() < 0
-               && (get_pos().y - start_position.y) >= FISH_FALL_BY_Y) // we fell far enough
-  {
-  	start_waiting();
-  }
-  else if (waiting)
-  {
-  	waiting_for += elapsed_time;
-  	if (waiting_for >= FISH_WAIT_TIME) // we've been waiting long enough
-  	{
-  	  waiting = false;
-  	  physic.set_velocity_y(FISH_JUMP_POWER);
-  	  physic.enable_gravity(true);
-  	}
+  // waited long enough?
+  if(waiting.check()) {
+    jump();
   }
   
   // set sprite
@@ -107,23 +111,26 @@ Fish::active_update(float elapsed_time)
   // we can't afford flying out of the tilemap, 'cause the engine would remove us.
   if ((get_pos().y - 31.8) < 0) // too high, let us fall
   {
-  	physic.set_velocity_y(0);
-  	physic.enable_gravity(true);
-  }
-  else if (Sector::current() && // spares us from possible segfaults
-          (get_pos().y - 31.8) > (Sector::current()->solids->get_height() * 32)) // too low, wait.
-  {
-  	start_waiting();
+    physic.set_velocity_y(0);
+    physic.enable_gravity(true);
   }
 }
 
 void
 Fish::start_waiting()
 {
-  waiting_for = 0;
-  waiting = true;
+  waiting.start(FISH_WAIT_TIME);
+  set_group(COLGROUP_DISABLED);
   physic.enable_gravity(false);
   physic.set_velocity_y(0);
 }
 
+void
+Fish::jump()
+{
+  physic.set_velocity_y(FISH_JUMP_POWER);
+  physic.enable_gravity(true);
+  set_group(COLGROUP_MOVING);
+}
+
 IMPLEMENT_FACTORY(Fish, "fish")
diff --git a/src/badguy/fish.hpp b/src/badguy/fish.hpp
index af8b1a88b..f7225f87d 100644
--- a/src/badguy/fish.hpp
+++ b/src/badguy/fish.hpp
@@ -26,19 +26,23 @@ class Fish : public BadGuy
 {
 public:
   Fish(const lisp::Lisp& );
-  Fish(float , float );
+  Fish(float x, float y);
+
+  void draw(DrawingContext& context);
 
   HitResponse collision_solid(GameObject& , const CollisionHit& );
   HitResponse collision_badguy(BadGuy& , const CollisionHit& );
+  void collision_tile(uint32_t tile_attributes);
 
   void write(lisp::Writer& );
   void active_update(float);
 
 private:
   HitResponse hit(const CollisionHit& );
-  bool waiting;
-  float waiting_for;
   void start_waiting();
+  void jump();
+  
+  Timer waiting;
 };
 
 #endif
-- 
2.11.0