The GenMsgCycleTime field of the message is parsed as 0.

[huohuaing]

I have a DBC file where the characteristics such as GenMsgCycleTime and GenMsgStartDelayTime are of floating-point type. This file can be opened normally by Vector CANdb++.

I found that floating-point numbers cannot be parsed by int.TryParse, resulting in a value of 0 being returned.

I have changed the code to use all the parsing operations with double.TryParse.

[Uight]

@huohuaing we had problems with similar issues before Like:
BA_DEF_ SG_ "GenSigMissingSourceValue" INT 0 1e+09;
which wouldnt get parsed aswell but is valid in can db++

you might wanna create a pull request for this. One thing for your code tough:
All int32 values fit into double so no Problem there. But you can create a silent overflow in casting to int.
We allready have a similar discussion in #91 which isnt fully resolved yet.

[huohuaing]

Thank you for your reply. Before converting to an integer, I added a check. If overflow occurs, the program will throw an exception. But there is another way to limit the post-conversion to the int range. I am not sure which method is better.

[Adhara3]

Hi,

I fear we ended up in the usual lack of spec issue.
If 20.000 is a valid value, why would 20.500 not be? I mean, the cast solution is fine but will not give you the right number.
So the question is: are we sure the GenMsgCycleTime should be an integer value?

Bottom line: I will merge the PR to unlock you but technically/philosophically speaking this may lead to other issues.

Regards
A