Python can tools c source generator

[Thorben (Gast)]

Hallo,

für ein CAN Projekt möchte ich gerne den C Source Generator aus dem 
Python Modul "cantools" https://github.com/cantools/cantools nehmen. 
Soweit ist der generierte Code anhand der DBC Datei verständlich nur zu 
einem Teil hätte ich ein paar Fragen.

In meinen Fragen beziehe ich mich auf folgende Datei
https://github.com/cantools/cantools/blob/master/tests/files/c_source/vehicle.h

Hier ein Auszug des Codes für meine Fragen:

/**
 * The MIT License (MIT)
 *
 * Copyright (c) 2018-2019 Erik Moqvist
 *
 * Permission is hereby granted, free of charge, to any person
 * obtaining a copy of this software and associated documentation
 * files (the "Software"), to deal in the Software without
 * restriction, including without limitation the rights to use, copy,
 * modify, merge, publish, distribute, sublicense, and/or sell copies
 * of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be
 * included in all copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
 * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
 * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
 * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
 * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
 * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
 * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 * SOFTWARE.
 */

/**

/**
     * Range: 0..255 (0..255 -)
     * Scale: 1
     * Offset: 0
     */
    uint8_t accuracy_ins_vel_ne;


/**
 * Encode given signal by applying scaling and offset.
 *
 * @param[in] value Signal to encode.
 *
 * @return Encoded signal.
 */
uint8_t vehicle_rt_sb_ins_vel_ned_1_accuracy_ins_vel_ne_encode(double value);

/**
 * Decode given signal by applying scaling and offset.
 *
 * @param[in] value Signal to decode.
 *
 * @return Decoded signal.
 */
double vehicle_rt_sb_ins_vel_ned_1_accuracy_ins_vel_ne_decode(uint8_t value);

Die Type ist uint8_t für das Signal definiert, mit einem Wertebereich 
von 0..255, Scale 1 und Offset 0.
Wozu sind nun die Funktionen encode und decode vorhanden, insbesondere 
von und zur Type double? Für mich müsste es von uint8_t zu uint8_t 
gewandelt werden bzw. sind die Funktionen ohne Sinn. Wo ist hier mein 
Denkfehler?

Beispiel:

uint8_t vehicle_rt_sb_ins_vel_ned_1_accuracy_ins_vel_ne_decode(uint8_t value);

[c oder d (Gast)]

uint8t warumnichtdouble(double wert1, int sig, char name, time_t);


;)

[Thorben (Gast)]

Ich verstehe deine Antwort nicht

[c oder d (Gast)]

globale variablen in Funktion
?

Ohne die Funktion zu sehen könnte man vielleicht auf c+- tippen;) WM

[MaWin (Gast)]

Thorben schrieb:
> uint8_t vehicle_rt_sb_ins_vel_ned_1_accuracy_ins_vel_ne_encode(double
> value);

Bei diesen Funktionsnamen bekommt man ja Hirnkrebs.

[Thorben (Gast)]

>Bei diesen Funktionsnamen bekommt man ja Hirnkrebs.

Hab schon schlimmere Message- und Signalnamen gesehen. So ist halt die 
Message und das Signal im DBC definiert.

[Eine Vermutung (Gast)]

Eine recht verbreitete Strategie - eine allgemeine Library arbeitet mit 
dem universellsten Datentyp. Und der Generator passt die reinkommenden 
und ausgehenden Daten an die Library an.

Macht halt weniger Arbeit, als den gesamten Programmcode optimal 
generieren.

[Rolf M. (rmagnus)]

Thorben schrieb:
> Die Type ist uint8_t für das Signal definiert, mit einem Wertebereich
> von 0..255, Scale 1 und Offset 0.
> Wozu sind nun die Funktionen encode und decode vorhanden, insbesondere
> von und zur Type double?

Die werden vermutlich einfach standardmäßig für jedes Signal generiert.

> Für mich müsste es von uint8_t zu uint8_t gewandelt werden bzw. sind die
> Funktionen ohne Sinn. Wo ist hier mein Denkfehler?

Wäre der Faktor oder der Offset z.B. 0,5, dann müsste man double nehmen. 
Die Software berücksichtigt einfach den Spezialfall "Faktor=1, Offset=0, 
Größe <= 8 Bit" nicht und nimmt für alles double, weil man diesen Fall 
damit auch abdecken kann. Dementsprechend wird dafür dann auch eine 
Umrechnungsfunktion generiert.