Hilfe beim optimieren meiner BMA020 Klasse

Ok hab noch ein paar sachen geändert, wie zum Beispiel unötiges auslesen 
von Registern, und noch ein bisschen kommentiert.

Fällt sonst wehm noch was auf?
Und könnte jemand die Klasse vllt mal auf einem anderen AVR als nem 
m328p ausprobieren? Hab leider keinen anderen da.

Wenn sonst alles in Ordnung ist werde ichs die Tage zur Codesammlung 
packen.

Gruss Robin

Was stört dich daran? Die funktion vordert nen array und das muss bevor 
man was einträgt inizialisiert werden.

Ich würde es nicht als Klasse implementieren. Klassen bringen einen 
gewissen Overhead mit sich (sagt man, hab' ich nicht überprüft), den man 
bei Mikrocontrollern wegen dem bisschen Speicher vermeiden möchte.

Ja der Overhead kann vorkommen hab ich jetzt bei der Klasse aber nicht 
bemerckt.
Und wenn doch was ist fügt man (bei eclipse) "${COMMAND} --cref -s -Os 
${OUTPUT_FLAG}${OUTPUT_PREFIX}${OUTPUT} ${INPUTS} -lm ${FLAGS}" mit ans 
avr-gcc im c++ linker menu das sollte auch die größe reduzieren. Aber 
man sollte erst ohne probieren.

Das Beispiel hat zum Beispiel als hex datei 11,6KB. Ich find das geht 
oder?

Erst mal danke,
und da es zum lernen war/ist dachte ich mir lass ich es schon noch 
übersichtlich das ich es auch in 2,3 Jahren noch versteh.
Ob es hier jetzt sinnvoll war als Klasse zu machen lässt sich auch 
bestimmt drüber streiten denk ich mal, aber zum lernen und im handling 
find ichs ganz gut ;)

Achja so weit ich weis kann man nur einen BMA020 in der schaltung haben 
der hat nämlich ne feste adresse.

Hab noch nen fehler entdeckt "shadow_disable" muss bit 3 und nicht 2
gesetzt werden. Und hab noch nen bisschen drann rumgebastelt, z.B. die
tmp Variabeln existieren werden nicht mehr benötigt und schon wieder
500byte weniger.

Ferner hab ich noch die i2c_communicate in ne eigene Datei gepackt um
die Funktion im selben Projeckt in verschieden libs haben kann ohne sie
doppelt und dreifach einzubinden.

@Stefan: Hab mal geguckt wie groß der unterschied zwischen Klasse und
nicht Klasse ist er beträgt einen knappen KB.

Wenn euch oder mir bis Morgen nix aufällt pack ichs zur Codesammlung 
wurde jetzt schon so oft gedownloaded und das obwohl noch Fehler drinn 
waren xD

@Peter II: Jo danke is mir nicht aufgefallen Compiler hat nix gesagt und 
ich bins von PHP und JavaScript gewohnt^^ Änder ich nacher noch

@Peter II: Vielen dank für die Tipps vor allem der hinweis mit den 
Arrays auf einmal spar ich 1KB ein (@Stefan: kein unterschied mehr ohne 
Klasse).

Das &reg funktioniert auch was mich freut. Und die typecast hab ich auch 
weg gemacht sonst noch hilfereiche Tipps für Anfänger?

Ja.
SChau dein Header File noch mal durch.

Welche der #define muss jemand kennen, der deine Klasse verwendet?
Welche dieser #define sind nur für die Klasse selber interessant und 
müssen daher von niemandem ausserhalb der Klasse gewusst werden?

Alle #define, die in letztere Kategorie gehören, gehören NICHT ins 
Header File sondern ins CPP selber.

Das Headerfile ist in erster Linie eine Beschreibung für denjenigen, der 
die Klasse benutzen will. Dafür, wie das ganze intern funktioniert, soll 
er sich nach Möglichkeit nicht interessieren. Das geht in C++ nicht 
immer besonders gut, weil es keine saubere Trennung zwischen externer 
und interner Beschreibung auf File Ebene gibt (zumindest nicht ohne 
Aufwand, sprich zb PIMPL Idiom), trotzdem solltest du versuchen, es so 
gut es geht umzusetzen. Ein #define welches die Registernummer des 
Low-Bytes der Y-Achse beschreibt, ist für jemanden, der die Klasse 
benutzen möchte, völlig uninteressant. Daher hat das im Header File 
nichts verloren. Er kann daran sowieso nichts ändern und er gewinnt 
daraus keine Information.

Ist ein Makro
#define acc_10Bit(LSB,MSB)  (((int8_t)MSB << 2) | LSB >> 6)
wirklich für jemanden der ein BMA20 Objekt benutzt so interessant, dass 
es im Header File sein muss?


Deine Klasse hat mir ein wenig zu viele statische Funktionalität! Das 
ist nicht wirklich der Sinn der Objektorientierung, dass man Klassen als 
eine Art Namensraum benutzt. In erster Linie möchte man Objekte haben, 
die eine Funktionalität kapseln. Und von diesen Objekten will man viele 
haben können! Soll es nur 1 derartiges Objekt geben können, dann macht 
man ein Singleton, wenn man sich selbst nicht über den Weg traut, das 
einzuhalten. Das hat den Vorteil, dass es dann auch einen Konstruktor 
gibt, der das Objekt zuverlässig als Ganzes in einen Defaultzustand 
bringt.

Wenn du einfach nur alle Variablen, die du bisher hattest und alle 
Funktionen die du bisher hattest, als static in eine neue Klasse 
einfügst, dann kannst du Objektorientierung auch bleiben lassen. Denn 
das ist kein.

So habs jetzt in nen namespace gepackt und die defines sortiert.
Siehts jetzt besser aus?

Nette idee aber da ich mich mit mehreren I2C Bus nicht auskenn wüsst ich 
net wie. xD

Ich glaub da muss ich erstmal noch ne ganze Menge lernen bis ich das 
hinbekomm^^

Kannst mir ja nen paar tuts posten wenn du lust und zeit hast, dann 
versuch ich mich bestimmt mal drann ;)