abhishekbharthi / yahoo-finance-managed

Automatically exported from code.google.com/p/yahoo-finance-managed
0 stars 0 forks source link

Error on QuotesDownload - class ImportExport.cs #57

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
for symbol like Cisco (CSCO), the result is null, in fact the result is 20 
items, but you are looking for 19, I'll attach a screenshot.

Regards /// Angel

Original issue reported on code.google.com by i...@advancedcomputing.ch on 5 Apr 2012 at 9:22

Attachments:

GoogleCodeExporter commented 9 years ago
The problem is the company name : "Cisco Systems, In", and you split on this 
"," ...

Original comment by i...@advancedcomputing.ch on 5 Apr 2012 at 9:29

Attachments:

GoogleCodeExporter commented 9 years ago
Possible Fix:
test the item[2] for numeric, if not, add 1 to the index.

Original comment by i...@advancedcomputing.ch on 5 Apr 2012 at 9:48

GoogleCodeExporter commented 9 years ago

Original comment by i...@advancedcomputing.ch on 5 Apr 2012 at 9:50

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks!
It's time for a scanner (goes char per char through the text and ignore special 
signs inside quotation marks) instead of a splitter.

Original comment by Maas...@gmail.com on 5 Apr 2012 at 9:53

GoogleCodeExporter commented 9 years ago
True ...
small fix on that line for forex:
if (i == 2 && !IsNumeric(rowItems[i]) && rowItems[i] != "N/A") k++;

Original comment by i...@advancedcomputing.ch on 5 Apr 2012 at 12:50

GoogleCodeExporter commented 9 years ago

Original comment by Maas...@gmail.com on 30 Apr 2012 at 12:50