BMW Ibus project

Projecten die niet passen in bovenstaande onderwerpen
Gebruikers-avatar
Berichten: 2655
Geregistreerd: 06 Aug 2016, 01:03

Re: BMW Ibus project

Berichtdoor Koepel » 05 Jun 2018, 15:41

Hallo luckhermsen,

Dit forum werkt anders. Wanneer je een sketch laat zien, dan proberen we de vinger op de zere plek te leggen zodat je weer verder kunt.

Verbeter me als ik het verkeerd heb, maar dit lijkt aan de hand:
Via die link die je gaf (http://curious.ninja/blog/arduino-bmw-i-bus-interface-intro/) is er een link bij "Resources & Downloads" naar de libraries. Daar zijn verschillende versies van de library te vinden.

Je hebt daar een library gedownload, en die zelf weer online gezet, zonder te vermelden welke versie je hebt gepakt. Je geeft geen sketch en geen voorbeeld hoe je die library gebruikt en verder niets dat wij zouden kunnen uitproberen. Ik heb nog geen documentatie kunnen vinden hoe die library gebruikt moet worden.
Die library is onder andere gebaseerd op code van iemand die voor het eerst een stukje code maakte: https://forum.arduino.cc/index.php?topic=234125.0. De code is typisch code van een beginner. In de code is niet te lezen waarom de code op die manier is gemaakt en wat de bedoeling van de code is. Er zal best over de code nagedacht zijn, maar het inspringen is een rommeltje en daardoor is de code slecht te lezen. Er is geen rekening mee gehouden dat anderen het ook lezen, laat staan dat anderen er een fout in kunnen ontdekken.

Het zou geschikt zijn voor verschillende Arduino boards, maar ik heb niet kunnen vinden of de laatste versie ook op al die Arduino boards getest is. Ik weet ook niet welk board jij gebruikt.

De code is onnodig complex gemaakt, maar dat is het kenmerk van een beginner die vol aan code gaat intikken. Het probleem is echter niet dat de code complex is, maar dat het een rommeltje is.

Dit ziet er beter uit: https://github.com/harryberlin/Arduino-BMW-IBus-SerialInterface-MCP2025, maar toch zie ik daar een fout met het gebruikt van millis(), een zeer slordige "goto", een fout in het vermenigvuldigen van constanten, en onhandig gebruik van de String class die overal geconverteerd wordt naar gewone tekst.
Misschien zijn er nog andere.

Advertisement

Terug naar Overige projecten

Wie is er online?

Gebruikers in dit forum: Geen geregistreerde gebruikers en 10 gasten