dotnetdbf icon indicating copy to clipboard operation
dotnetdbf copied to clipboard

Configurable exception throwing on parsing error

Open jbezak opened this issue 5 years ago • 3 comments

We have a lot of cases where a high numeric value is stored in DBF which overflows the DBF limits. This is then stored as series of * and the library fails to read the record. In these case we would like to continue reading the DBF and display default value instead. This is also what DBF viewing tools are doing.

I made this configurable in NextRecord() method.

jbezak avatar Feb 21 '20 17:02 jbezak

I'm thinking about this.

Leaning towards accepting it. I just have to be careful, because it's a binary breaking change.

I'm weighing between uping the version number, or changing it to an overload. I have a general preference for optional params.

jbtule avatar Mar 02 '20 16:03 jbtule

I have no problem making it an optional param instead of overload.

Let me know your preference and I can get to this within a day or two.

jbezak avatar Mar 02 '20 17:03 jbezak

@jbtule I came across this pull request today after long time and changed it to optional param

jbezak avatar Oct 06 '20 20:10 jbezak