Spring/spring-batch
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

BATCH-700:FixedLengthTokenizer didn't support single value ranges when calculating the maximum line length.

Browse Source
This commit is contained in:
lucasward
2008-07-11 04:14:53 +00:00
parent a61b725bc2
commit 402bb81c40
3 changed files with 28 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/AbstractLineTokenizer.java
View File

@@ -65,10 +65,6 @@ public abstract class AbstractLineTokenizer implements LineTokenizer {
* @return the resulting tokens
*/
public FieldSet tokenize(String line) {
// if (line == null || line.length()==0) {
// return new DefaultFieldSet(new String[0]);
// }
if(line == null){
line = "";

19
spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/FixedLengthTokenizer.java
View File

@@ -47,17 +47,30 @@ public class FixedLengthTokenizer extends AbstractLineTokenizer {
calculateMaxRange(ranges);
}
/*
* Calculate the highest value within an array of ranges. The ranges aren't
* necessarily in order. For example: "5-10, 1-4,11-15". Furthermore, there
* isn't always a min and max, such as: "1,4-20, 22"
*/
private void calculateMaxRange(Range[] ranges){
if(ranges == null || ranges.length == 0){
maxRange = 0;
return;
}
maxRange = ranges[0].getMax();
maxRange = ranges[0].getMin();
for(int i = 0; i < ranges.length; i++){
if(ranges[i].getMax() > maxRange){
maxRange = ranges[i].getMax();
int upperBound;
if(ranges[i].hasMaxValue()){
upperBound = ranges[i].getMax();
}
else{
upperBound = ranges[i].getMin();
}
if(upperBound > maxRange){
maxRange = upperBound;
}
}
}