Stringgrid vs. LoadFromCSVFile

[Bart]

Well, they apparently are. Stringgrids retain the line ending in the cell's string. They just don't display it.

With support I meant, it actuay shows up, otherwise one could also state TEdit supports mutiline.

Furthermore, think a bit further. If multiline were not supported, don't you think people will try to read multiline csv files anyway and complain?

Possibly...

If we woud consider doing that, then we cannot use TStringlist internally anymore, and have to design a completey different algorithm.

Doing what? Multiline support? Seems that already works.

OK, currently we load the CSV data into a TStringlist and then process each line, assuming a new line also means a new row in the context of a StringGrid.
If fields are allowed to be mutiline, this algorithm wil not do anymore.

2. Have you looked at csvdocument and its dependencies?

No I have not.
I will take a look sometime.
Mind you, if I understand the wiki correcty CsvDocument is not in LCL.
StringGrid is, and it cannot depend on external component that isn't part of LCL.

Bart

[BigChimp]

Mind you, if I understand the wiki correcty CsvDocument is not in LCL.
StringGrid is, and it cannot depend on external component that isn't part of LCL.

Perhaps you misunderstood my earlier suggestion on that:

1. Have you read what I proposed? How is that "scattered around the place"?
1.1 Use csvdocument for all csv handling in Lazarus now
1.2 Try to get csvdocument in FPC and use that when it is stable (e.g. for dataset export to csv, future perhaps import from csv)
Surely better than the current situation (custom CSV code in the actual procedure)?
2. Have you looked at csvdocument and its dependencies?

Finally, if you don't want to use csvdocument... then you'd just be trying to rewrite it to deal with all exceptions/weird formats (RFC4180, Excel) etc. I don't see any reason to do that.

As previously discussed, FPC sdf format may be shoehorned into supporting csv format. If that is the case, I'd suggest that csvdocument be in FPC and we can rewrite that, to make it seamless to the user.

[BigChimp]

In case somebody needs this: code to load a CSV file into a stringgrid using CSVDocument:
http://wiki.lazarus.freepascal.org/CsvDocument#Loading_a_csv_document_into_a_StringGrid

Improvement suggestions welcome as always...

[madepabloh]

Thanks BigChimp for this nice solution.

I tried to use it, but it does not compile, because return three errors related to "MaxColCount" parameter. This is the message returned by Lazarus:

unit1.pas(125,31) Error: identifier idents no member "MaxColCount"

They are the unique errors returned.
Do you know what is the problem and how to solve it?

Thanks!

[BigChimp]

As the wiki says:

Note: this requires the development version of csvdocument (see above)

You probably didn't get the development version.

Try coping over the development version https://lazarus-ccr.svn.sourceforge.net/svnroot/lazarus-ccr/components/csvdocument/csvdocument.pas over your existing csvdcoument.pas and try again.
(Or do it the proper way and download the entire svn version as detailed in the wiki article)

[madepabloh]

Thanks BigChimp,

My fault. I downloaded both version, but only installed one... I removed the normal and installed the development version.

Now the error is different:

unit1.pas(60,3) Note: Local variable "PrevRow" not used
unit1.pas(24,15) Error: Forward declaration not solved "TForm1.LoadGridFromCSVFile(TStringGrid,AnsiString,Char="44",Boolean="TRUE",Boolean="TRUE");"
unit1.pas(8,36) Hint: Unit "math" not used in Unit1
unit1.pas(137) Fatal: There were 1 errors compiling module, stopping

The critical error now is in line 24, relared to the declaration of the procedure. I copied everything from the wiki...

EDITED:

Ok, i reply to myself. The solution is to include "TForm1." in the procedure declaration at the implementation. Solving the mistake of the example included in the wiki.
[SOLVED]

[madepabloh]

OK, now it compiles, but it does not do what it is expected.

It has a rare visual behaviour. It does show the result into the stringgrid, except when move and click the moise on the cells. Moreover, it moves all the content of the row into one cell of the stringgrid.

I attach the test form to see if you could detect where is the problem.

Thanks!