Ok, jetzt kann man es beurteilen und es sieht so grottig aus, wie erwartet.
Ich habe nach erstem Drübersehen einen Verdacht:
Dieses "float microsalt" sieht verdächtig aus. Der Rückgabewert von micros() ist nicht float, sondern unsigned long. Bei allen diesen Berechnungen treten auf jeden Fall Fehler auf, da float nicht so viele gültige Stellen hat.
Zitat von Mxt
Ist grottig ein Kompliment? Kann mich aber nicht erschüttern, da dies mein erstes Projekt mit dem Arduino ist.
Das mit dem "unsigned long" kann ich ja ändern, ist aber vermutlich nicht die Ursache, weil genau diese UP's auch von dem Programmteil verwendet werden, mit dem die Motore schnell laufen, den ich aber durch das Abspecken weg gelassen habe.
vG
fredyxx
Den Motoren ists auch völlig egal, wie der Code aussieht. Die laufen auch mit hässlichem. In deinem Falle nur halt langsam.
Nur wenn sich das jemand ansehen soll, ist das Aussehen nicht egal. Schon die Verwendung von Quote-Tags statt Code-Tags zeigt, dass das Aussehen egal ist.
ich glaube, mxt will dir damit den versteckten Tipp geben, dass du doch bitte deinen Code in Code-Tags posten möchtest...
ändere doch mal bitte dein float in uint32_t und teste es, und dann korrigiere und formatiere deinen Code anschließend um, und dann berichte, ob es mit uint32_t schneller läuft oder nicht
Ich sehe beim drüberschauen zumindest keine irgendwie blockierenden Konstrukte. Besonders effizient ist das goto Herumgehüpfe auf dem kleinen 8-Bit Prozessor sicher nicht. Kann er vielleicht nicht schneller ?
Ansonsten hätte ich die Zeilen
oderCode:Schrittdauer = int(...
im Verdacht.Code:if (dir == 0 && (micros() - ...
Float-Operationen auf 8-Bittern sind schnarchlangsam.
was heißt das?
- - - Aktualisiert - - -
Hallo,
>>>>>>>>>>>>>> eine große ENTSCHULDIGUNG, dass ich euch belässtigt habe <<<<<<<<<<<<<<<<<<<<<
Die Ursache ist total primitiv!
Ich habe in den Aufrufen von Mx_drive dauernd statt der Geschwindigkeit die Anzahl der Umdrehungen verändert und die Geschwindigkeit stand eben auf klein.
M1_drive(22, 24, 26, 28, M1_n, M1_v, M1_dir)
Ich denke, ich brauche mal eine Pause!!!!!
Trotzdem noch die Fragen:
was bedeutet uint32_t und
was ist damit gemeint: "Quote-Tags statt Code-Tags" ?
vG
fredyxx
Siehst du, Computer machen immer genau das, was man sagt. Das ist das Problem beim Programmieren.
Zu uint32_t:
Typen wie int, long usw. sind auf unterschiedlichen Prozessoren unterschiedlich groß, haben also unterschiedlich viele Bits, z.B. int bei einem 8 Bit-Arduino 16 Bits, bei einem 32-Bit Arduino 32 Bit.
Darum gibt es in C und C++ Datentypen, die garantierte Größen haben.
http://en.cppreference.com/w/cpp/types/integer
Damit sind Programme besser auf andere Rechner übertragbar.
Tags:
richtig wäre aberDu zitierst deine Code
Dann bleiben auch die Einrückungen stehen.Code:auf "Erweitert" klicken und das # Symbol nehmen
- - - Aktualisiert - - -
Ach ja, weiterhin macht eine Zeile wie
nicht das, was du glaubst.Code:else if (dir == 1 && (micros() - M2_microsalt) > Schrittdauer) {
Erstmal wird micros() - M2_microsalt berechnet. micros() liefert unsigned long, das wird in float umgewandelt und dann M2_microsalt abgezogen.
Dann muss Schrittdauer von unsigned int nach float umgewandelt werden, damit es verglichen werden kann.
Die ganze Zeile dauert wahrscheinlich etliche Mikrosekunden, weil der 8-Bit Prozessor die 32 Bit Werte nur in Häppchen verarbeiten kann.
Das musste ich auch schon von ca. 46 Jahren erfahren.
Danke für den Tipp.
Wenn ich das richtig verstehe, sollte ich also M2_microsalt und Schrittdauer als unsigned long definieren?? Geht das dann schneller?
vG
fredyxx
Berechtigungen
Zitieren
Lesezeichen