java.lang.OutOfMemoryError：Java 堆空间 - 无法修复

Question

我有这个代码，这个错误不断出现。我只有一个 excel ，但似乎没有任何效果，我已经尝试了很多我在网上冲浪时发现的选项，但根据我想做的事情似乎没有任何效果。

我使用不同的案例来简化我的业务逻辑，我不会改变它，所以我不知道如何解决这个问题。

private static final String nombreArchivo = "casoPrueba.xlsx";
private static final String rutaArchivo = "src\\test\\resources\\data\\" + nombreArchivo;




 public static XSSFSheet SacaHojaSegunTipo(String tipo) throws IOException {
        if (workbook == null) {
            try (FileInputStream fis = new FileInputStream(new File(rutaArchivo))) {
                workbook = new XSSFWorkbook(fis);
            }
        }
        XSSFSheet spreadsheet = null;
        switch (tipo) {
            case "Candidatos Minorista":
                spreadsheet = workbook.getSheetAt(1);
                break;
            case "Conversion Candidatos":
                spreadsheet = workbook.getSheetAt(2);
                break;
            case "Cuentas":
                spreadsheet = workbook.getSheetAt(3);
                break;
            case "Detalle Cuenta":
                spreadsheet = workbook.getSheetAt(4);
                break;
            case "Historial de Cuentas":
                spreadsheet = workbook.getSheetAt(5);
                break;
            case "Cuentas Financieras":
                spreadsheet = workbook.getSheetAt(6);
                break;
            case "AR Estado Automático":
                spreadsheet = workbook.getSheetAt(7);
                break;
            case "Oportunidades":
                spreadsheet = workbook.getSheetAt(8);
                break;
            default:
                spreadsheet = workbook.getSheetAt(0);
                break;
        }
        return spreadsheet;
    }

我知道这不是一个有效的方法。希望任何人都可以帮助我。

Answer 1

类似这样的东西（我试图尽可能少地改变你的代码，所以它并不完美）

private static final String nombreArchivo = "casoPrueba.xlsx";
private static final String rutaArchivo = "src\\test\\resources\\data\\" + nombreArchivo;

private static XSSFWorkbook workbook = null;

public static XSSFSheet SacaHojaSegunTipo(String tipo) throws IOException {
    if (workbook == null) {
        try (FileInputStream fis = new FileInputStream(new File(rutaArchivo))) {
            workbook = new XSSFWorkbook(fis);
        }
    }
    XSSFSheet spreadsheet = null;
    switch (tipo) {
    case "Candidatos Minorista":
        spreadsheet  = workbook.getSheetAt(1);          
        break;
    case "Conversion Candidatos":
        spreadsheet  = workbook.getSheetAt(2);          
        break;
    case "Cuentas":
        spreadsheet  = workbook.getSheetAt(3);          
        break;
    case "Detalle Cuenta":
        spreadsheet  = workbook.getSheetAt(4);          
        break;
    case "Historial de Cuentas":
        spreadsheet  = workbook.getSheetAt(5);          
        break;
    case "Navegar Cuentas":
        spreadsheet  = workbook.getSheetAt(6);          
        break;
    case "Validar Número Operación":
        spreadsheet  = workbook.getSheetAt(7);          
        break;
    case "Validar Tipos de Productos":
        spreadsheet  = workbook.getSheetAt(8);          
        break;
    case "Validar Referencia y Cód. Auto.":
        spreadsheet  = workbook.getSheetAt(9);          
        break;
    default:
        spreadsheet = workbook.getSheetAt(0);
    }
    return spreadsheet;
}

Answer 2

首先，顺便说一句：值得注意的是来自
https://poi.apache.org/apidocs/dev/org/apache/poi/xssf/usermodel/XSSFWorkbook.html#XSSFWorkbook-java.io.InputStream-

的以下内容

使用 InputStream 比使用 File 需要更多内存，所以如果 文件可用，那么您应该改为执行类似的操作

   OPCPackage pkg = OPCPackage.open(path);
   XSSFWorkbook wb = new XSSFWorkbook(pkg);
   // work with the wb object
   ......
   pkg.close(); // gracefully closes the underlying zip file

（虽然wb.close() 也会关闭文件和流）。

现在，您的核心问题是您需要在不再需要工作表或工作簿后释放资源，但目前您无法这样做，因为它们隐藏在方法内部。

因此，您需要授予调用者访问权限，以便在完成后关闭它们。这是一个偏好问题，但我个人更喜欢将电子表格封装到它自己的类中——毕竟，电子表格本身就是一个明确定义的对象！因此，这将需要从 static 进行更改，因此类似于：

public class RutaArchivo implements AutoCloseable {
    private static final String nombreArchivo = "casoPrueba.xlsx";
    private static final String rutaArchivo = "src\\test\\resources\\data\\" + nombreArchivo;

    public static final String CANDIDATOS_MINORISTA = "Candidatos Minorista";
    public static final String CONVERSION_CANDIDATOS = "Conversion Candidatos"
    public static final String CUENTAS = "Cuentas";

    private XSSFWorkbook workbook;

    public RutaArchivo() throws InvalidFormatException, IOException {
        workbook = new XSSFWorkbook(new File(rutaArchivo));
    }

    @Override
    public void close() throws Exception {
        if (workbook != null) {
            workbook.close();
            workbook = null;
        }
    }

    public XSSFSheet sacaHojaSegunTipo(String tipo) {
         if (workbook == null) {
             throw new IllegalStateException("It's closed");
         }
         XSSFSheet spreadsheet = workbook.getSheetAt(0);
         if (tipo .equals(CANDIDATOS_MINORISTA)) {
             spreadsheet  = workbook.getSheetAt(1);
         }else if(tipo.equals(CONVERSION_CANDIDATOS)){
             spreadsheet  = workbook.getSheetAt(2);
         }else if(tipo.equals(CUENTAS)){
              spreadsheet  = workbook.getSheetAt(3);
         // etc, etc
         }

         return spreadsheet;
    }
}

有几点需要注意：

• 如果我们想让调用者关闭文件，那么我们应该明确地让他们采取一些行动来打开它，否则它太容易被挂起。在上面的示例中，这在创建对象时是隐含的——就像 FileInputStream 等标准 Java 类型一样。

• 使RutaArchivoAutoCloseable 意味着它可以在try-with-resources 中使用，因此自动关闭-例如：

  try (RutaArchivo rutaArchivo = new RutaArchivo()) {
      XSSFSheet cuentas = rutaArchivo.getSheet(RutaArchivo.CUENTAS);
  }
  

• 为工作表名称使用常量可减少错误（例如，调用方法时不会出现拼写错误）

• 由于这是它自己的类而不是 static 方法，因此在编写单元测试时更容易替换或模拟。

无论如何，一些想法 - 希望他们有所帮助。