Tastenkollision mit Linienverfolgung kombinieren

[markusj]

Zur Fehlersuche könntest du dir Mal T1 und T2 via serieller Schnittstelle ausgeben lassen, wenn du in den Stop-Fall gerätst. Ich bin geneigt auf Schwankungen der Spannungsversorgung zu tippen, evtl. gepaart mit einem grenzwertigen Kalibrierfaktor für die Tasterauswertung.

Davon abgesehen solltest du dich demnächst Mal mit dem Thema Zustandsautomaten beschäftigen - Das wird nämlich erforderlich wenn das Verhalten noch komplexer werden soll.
Außerdem kannst du das Problem der Verlangsamung des Linienfolgers beheben in dem du statt einer Warteschleife (sleep) einfach mehrere Iterationen der Hauptschleife überspringst und nur bei jeder n-ten Iteration die Taster abfragst. Eine noch schönere Lösung wäre natürlich, wenn du Linienfolger und Tasterabfrage zeitgesteuert durchführen würdest, unter Verwendung einer festen Zeitbasis (Timer!)

mfG
Markus

[radbruch]

Hallo

Vielleicht bringt das eine Besserung:

	T1= PollSwitch();			/ erst den Tastenstatus einlesen...
	MotorSpeed(SpeedLeft,SpeedRight);	/ ... und dann die Antriebe starten		
	
	while (1)
	{
		T2=T1;				/ alten Tastenstatus merken für Vergleich mit dem neuen Status
		T1= PollSwitch();		/ Tasten nur einmal pro Zyklus einlesen
		
		if( ((T1 > 0) && (T1 == T2)) || (Stop == true) ) / klammern, wenn man die Rangfolge nicht auswendig kennt :)

Gruß

mic

[Valen]

Wie Markusj schon angegeben hat. PollSwitch kann falsche werten ausgeben. Eben andere wenn die Motoren angeschaltet sind. Erst testen ob es die richtige Werten gibt, oder kalibriert werden muss.

Wieso muss die Funktion wait ein Float wert bekommen? Und wie viel Sekunden Dauer verwendest du als Parameter für Endloss-schleife-verzögerung. Vielleicht entsteht hier mehr Verzögerung als gewünscht. Float-rechnerei kostet sehr viel Prozessorkraft (und Speicherplatz) für ein einfache Mikrocontroller als ein Atmega8. Du könntest es viel einfacher für Asuro machen wenn du es zu ein Msleep Funktion umbaut das nur mit integer werten arbeitet. Und ein ganz-Zahl von Millisekunden wartet.

void Msleep (int dauer)
{
  int z;
  for (z = 0; z < dauer; z++)
  Sleep (72); // Warte ein Millisekunde ( mit Originale Asuro Lib!!)
 }

Dazu wundere ich mich auch über dein Letzte Else-zweig:

...
else
{
BackLED(OFF,OFF);
StatusLED(YELLOW);
SpeedLeft= 120;
SpeedRight= 130;
}
...

Wie arbeitet es wenn du SpeedLeft und SpeedRight nicht auf 120 und 130 einstellt (aber unverändert last) wenn die Linie im mitten der Sensor befindet (absoluten diff-offset weniger als 4). Dein Änderung auf diese niedriger Motorspeed werten macht das steuern vielleicht schlimmer.