diff --git a/easyexcel-core/src/main/java/com/alibaba/excel/analysis/csv/CsvExcelReadExecutor.java b/easyexcel-core/src/main/java/com/alibaba/excel/analysis/csv/CsvExcelReadExecutor.java index 5a901b5b..814f5b52 100644 --- a/easyexcel-core/src/main/java/com/alibaba/excel/analysis/csv/CsvExcelReadExecutor.java +++ b/easyexcel-core/src/main/java/com/alibaba/excel/analysis/csv/CsvExcelReadExecutor.java @@ -4,11 +4,7 @@ import java.io.IOException; import java.io.InputStream; import java.io.InputStreamReader; import java.nio.file.Files; -import java.util.ArrayList; -import java.util.Iterator; -import java.util.LinkedHashMap; -import java.util.List; -import java.util.Map; +import java.util.*; import com.alibaba.excel.analysis.ExcelReadExecutor; import com.alibaba.excel.context.csv.CsvReadContext; @@ -54,7 +50,7 @@ public class CsvExcelReadExecutor implements ExcelReadExecutor { @Override public List sheetList() { - return sheetList; + return (this.sheetList != null ? Collections.unmodifiableList(this.sheetList) : Collections.emptyList()); } @Override diff --git a/easyexcel-core/src/main/java/com/alibaba/excel/analysis/v07/XlsxSaxAnalyser.java b/easyexcel-core/src/main/java/com/alibaba/excel/analysis/v07/XlsxSaxAnalyser.java index beda8503..66e813ca 100644 --- a/easyexcel-core/src/main/java/com/alibaba/excel/analysis/v07/XlsxSaxAnalyser.java +++ b/easyexcel-core/src/main/java/com/alibaba/excel/analysis/v07/XlsxSaxAnalyser.java @@ -3,13 +3,7 @@ package com.alibaba.excel.analysis.v07; import java.io.File; import java.io.IOException; import java.io.InputStream; -import java.util.ArrayList; -import java.util.HashMap; -import java.util.Iterator; -import java.util.List; -import java.util.Map; -import java.util.Optional; -import java.util.UUID; +import java.util.*; import javax.xml.parsers.ParserConfigurationException; import javax.xml.parsers.SAXParser; @@ -211,7 +205,7 @@ public class XlsxSaxAnalyser implements ExcelReadExecutor { @Override public List sheetList() { - return sheetList; + return (this.sheetList != null ? Collections.unmodifiableList(this.sheetList) : Collections.emptyList()); } private void parseXmlSource(InputStream inputStream, ContentHandler handler) { diff --git a/easyexcel-core/src/main/java/com/alibaba/excel/metadata/format/DataFormatter.java b/easyexcel-core/src/main/java/com/alibaba/excel/metadata/format/DataFormatter.java index 1bd591e3..bf45bde5 100644 --- a/easyexcel-core/src/main/java/com/alibaba/excel/metadata/format/DataFormatter.java +++ b/easyexcel-core/src/main/java/com/alibaba/excel/metadata/format/DataFormatter.java @@ -853,7 +853,7 @@ public class DataFormatter { * non-cachable wrapper around a * {@link CellFormatResult} */ - private final class CellFormatResultWrapper extends Format { + private static final class CellFormatResultWrapper extends Format { private final CellFormatResult result; private CellFormatResultWrapper(CellFormatResult result) {