importfile regexp thousands separator returns NaN when negative

3 views (last 30 days)
I used "Import Data" to generate a function importfile for importing a csv file. Unfortunately, the function returns NaN for every number less than or equal to -1,000.
When I step through the generated code, I discover that the error happens in the following code:
% Detected commas in non-thousand locations.
invalidThousandsSeparator = false;
if numbers.contains(',')
thousandsRegExp = '^\d+?(\,\d{3})*\.{0,1}\d*$';
if isempty(regexp(numbers, thousandsRegExp, 'once'))
numbers = NaN;
invalidThousandsSeparator = true;
end
end
Let's say that:
numbers = "-1,167.67627";
Then:
regexp(numbers, thousandsRegExp, 'once')
ans =
[]
Hence, the code returns a NaN when it should be giving me a number of -1167.67627.
Compare this to the result when the number is positive:
numbers = "1,167.67627";
regexp(numbers, thousandsRegExp, 'once')
ans =
1
Obviously, the error comes from the following code, but I have no idea how to fix this 'matched regular expression':
thousandsRegExp = '^\d+?(\,\d{3})*\.{0,1}\d*$';

Accepted Answer

madhan ravi
madhan ravi on 17 Jul 2019
str2double(regexprep(numbers,',',''))
  2 Comments
Craig Cowled
Craig Cowled on 17 Jul 2019
Thank you, madhan ravi. Something very similar is in the next section of code:
% Convert numeric text to numbers.
if ~invalidThousandsSeparator
numbers = textscan(char(strrep(numbers, ',', '')), '%f');
numericData(row, col) = numbers{1};
raw{row, col} = numbers{1};
end
It seems that I just need to remove the section of code that is causing the problem.
madhan ravi
madhan ravi on 17 Jul 2019
Attach your data file and illustrate what you want the desired result to be.

Sign in to comment.

More Answers (0)

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!