heise online · c't · iX · Technology Review · Telepolis · mobil · Security · Netze · heise open · heise resale · Autos · c't-TV · Jobs · Kiosk
Zum Inhalt
c't

c't Projekte - c't-Bot und c't-Sim - Mailinglisten

c't-Bot und c't-Sim


[Voriger (Datum)] [Nächster (Datum)] [Voriger (Thread)] [Nächster (Thread)]
[Nach Datum][Nach Thread]

Re: [ct-bot] neues Pfadplanungs-Verhalten nach der Map

Absender: Benjamin Benz
Datum: Mi, 02.05.2007 18:16:47
In-reply-to: <000001c78b5d$62de6570$fe78a8c0@mexpnew>
References: <000001c78b5d$62de6570$fe78a8c0@mexpnew>


Hi,

noch ein paar Anmerkungen, die mir entgangen sind:

* Ich denke Funktionen wie value_in_circle() gehören nach map.c/map.h.
Die sind allgemein gültig
* was tut get_mappos_dist() und gehört die nicht auch eher in map.c/map.h?
* Die Routinen update_map_sensor_hole() und update_map_hole() gehört
doch wohl auch eher zum Map-Verhalten und sollte auch dort permanent
verwendet werden ==> bot_scan_onthefly_behaviour()), oder?
* clear_map() gehört die nicht auch eher in map.c/map.h?
* was genau tut bot_set_border_in_map_behaviour() und soltle das nicht
eher in das bot_scan_onthefly_behaviour() mit rein?
* sollte man bot_check_hang_on_behaviour() nicht auch verallgemeinern?
* sollte man get_dist() nicht auch verallgemeinern?
* sollte man map_in_dest nicht auch verallgemeinern?

So, das hört sich erstmal nach viel gemecker an, aber es ist das
Gegenteil. Ich denke, dass da eine ganze Reihe von Funktionen drin ist,
die man unbedingt allen Code-teilen zugänglich machen sollte. Außerdem
würde das die Datei behaviour_map_go_destination.c deutlich
übersichtlicher machen und den Code Wartungsfreundlicher halten.

MfG Benjamin Benz

-- 
Benjamin Benz
Heise Zeitschriften Verlag
Redaktion c't
eMail: bbe@xxxxxxxx
WWW  : http://www.heise.de

Heise Zeitschriften Verlag GmbH & Co. KG
Registergericht: Amtsgericht Hannover HRA 26709

Persönlich haftende Gesellschafterin:
Heise Zeitschriften Verlag Geschäftsführung GmbH
Registergericht: Amtsgericht Hannover, HRB 60405
Geschäftsführer: Ansgar Heise, Steven P. Steinkraus, Dr. Alfons Schräder




Copyright © 2007 Heise Zeitschriften Verlag Kritik, Anregungen bitte an c't-WWW Datenschutzhinweis   Impressum