syncfusionexamples / xlsio-examples Goto Github PK

consider this GH example

1. opens the sample InputTemplate.xlsx in 3 folders higher [i.e. the program source directory] which is bad practice because would be wrong on deploying to PROD [like good chefs your never reveal your sauces!]. Instead I propose you mark that xls to be copied to target folder on build, so then available in target to both dev and prod
2. uses legacy explicit type, i.e. does not reflect current "var" guideline of using implied type
3. legacy use of .Close() does not accord with best practice to wrap disposable objects within a using block [or statement for method scope], which will protect against any exceptions
4. I would prefer file-scoped namespace instead of legacy block-scope
5. this GH project does not contain the necessary .gitignore file, so any unwitting PR would contain all bin/obj/debug detritus

I will submit a PR (items 1-3) for consideration by SF, but the above issue occurs elsewhere and needs fixing throughout.

When I tested this demo, when excel file contains Chinese characters , I found that Chinese characters were not recognized

Hi there, when I was tring to use CalculatedValue function with round formula, I got some trouble.
The cell value Round(3944.5,0) in CalculatedValue function get 3944, I think the correct answer is 3945.
I use Syncfusion.XlsLO.Net.Core - 20.4.0.42 thanks