Excel IS_AUTO_DETECT_CELL_TYPE Suggestion

[2001 JI Open Discussion]

By: Steve Jenkings - stevejenks

Excel IS_AUTO_DETECT_CELL_TYPE Suggestion

2005-11-02 10:01

I wonder what the Jasper authors think about changing the code in the JRXlsExporter class 'exportText' method as follows..

 

Current code.

 

if (isAutoDetectCellType)

{

try

{

cell.setCellValue(Double.parseDouble(styledText.getText()));

}

catch(NumberFormatException e)

{

cell.setCellValue(JRStringUtil.replaceDosEOL(styledText.getText()));

}

}

 

 

Proposed code.

 

if (isAutoDetectCellType)

{

try

{

cell.setCellValue(Double.parseDouble(styledText.getText().replaceAll(",", "")));

}

catch(NumberFormatException e)

{

cell.setCellValue(JRStringUtil.replaceDosEOL(styledText.getText()));

}

}

 

i.e. by stripping out any commas in the text string before attempting to convert the String to a Double we can successfully convert Strings such as "123,456.00" into numbers.

 

Best,

 

Steve.

 

 

By: Doug Berkland - berkland

RE: Excel IS_AUTO_DETECT_CELL_TYPE Suggestion

2005-11-03 02:16

It would need to be more complex than that so as not to break the locales that use comma as the decimal point and period as the thousands seperator.

[kriegerd]

It can't be done that way as Doug said. There should be a number formatter formatter somewhere that could parse the number properly