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