jExcelApi - повторное использование формата даты в ячейке не работает - PullRequest
4 голосов
/ 22 сентября 2009

Я создал оболочку для классов jExcelApi, чтобы легко экспортировать списки объектов в Excel. Для минимизации создания объектов форматы ячеек создаются как статические поля и повторно используются при последовательных вызовах экспорта. Но у меня проблемы с форматом даты - первый вызов работает хорошо, но во всех последовательных экспортах ячейки даты имеют числовой формат вместо формата даты. Если я создаю новый объект для формата даты вместо использования статического поля, все в порядке. Есть ли какая-либо причина, по которой использование одного и того же формата для разных листов или книг не удается?
Вот код с упрощенной обработкой исключений, другие типы данных пропущены и, возможно, некоторые операции импорта отсутствуют:

ExcelCellGenerator.java:

import jxl.write.WritableCell;

public interface ExcelCellGenerator<T> {
 WritableCell getCell(int col, int row, T arg);
}

ExcelCellGeneratorFactory.java:

import jxl.write.DateFormat;
import jxl.write.DateTime;
import jxl.write.Label;
import jxl.write.NumberFormat;
import jxl.write.NumberFormats;
import jxl.write.WritableCell;
import jxl.write.WritableCellFormat;
import ExcelExporter.DateTimeExtractor;

final class ExcelCellGeneratorFactory {
 private ExcelCellGeneratorFactory() {}

 private static final WritableCellFormat DATE_FORMAT = new WritableCellFormat ( new DateFormat ("dd MMM yyyy hh:mm:ss")); // reusing this field fails

 static public <T> ExcelCellGenerator<T> createDateCellGenerator(final DateTimeExtractor<T> extractor) {
  return new ExcelCellGenerator<T>() {
   public WritableCell getCell(int col, int row, T arg) {
    return new DateTime(col, row, extractor.extract(arg), DATE_FORMAT);
                // if there is new WritableCellFormat(new DateFormat(...)) instead of DATE_FORMAT, works fine
   }

  };
 }
}

ExcelExporter.java:

import jxl.Workbook;
import jxl.write.DateFormat;
import jxl.write.DateTime;
import jxl.write.Label;
import jxl.write.NumberFormat;
import jxl.write.WritableCellFormat;
import jxl.write.WritableSheet;
import jxl.write.WritableWorkbook;

public class ExcelExporter<T> {
 // describe a column in Excel sheet
 private static class ColumnDescription<T> {
  public ColumnDescription() {}

        // column title
        private String title;
        // a way to generate a value given an object to export
        private ExcelCellGenerator<T> generator;
 }

    // all columns for current sheet
 private List<ColumnDescription<T>> columnDescList = new ArrayList<ColumnDescription<T>>();

    // export given list to Excel (after configuring exporter using addColumn function
    // in row number rowStart starting with column colStart there will be column titles
    // and below, in each row, extracted values from each rowList element
 public byte[] exportList(int rowStart, int colStart, List<? extends T> rowList) {
  final ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
  WritableWorkbook workbook;
  try {
   workbook = Workbook.createWorkbook(outputStream);
  } catch (IOException e) {
   e.printStackTrace();
  }
  final WritableSheet sheet = workbook.createSheet("Arkusz1", 0);

  int currRow = rowStart;
  try {
   int currCol = colStart;
   for (ColumnDescription<T> columnDesc : columnDescList) {
    final Label label = new Label(currCol, currRow, columnDesc.title);
    sheet.addCell(label);
    currCol++;
   }
   currRow++;

   for (T object : rowList) {
    currCol = colStart;
    for (ColumnDescription<T> columnDesc : columnDescList) {
     sheet.addCell(columnDesc.generator.getCell(currCol, currRow, object));
     currCol++;
    }
    currRow++;
   }

   workbook.write();
  } catch (Exception e) {
   e.printStackTrace();
  } finally {
   try {
    workbook.close();
   } catch (Exception e) {
    e.printStackTrace();
   }
  }

  return outputStream.toByteArray();
 }

    // configure a Date column
    public ExcelExporter<T> addColumn(String title, DateTimeExtractor<T> extractor) {
  final ColumnDescription<T> desc = new ColumnDescription<T>();
  desc.title = title;
  desc.generator = ExcelCellGeneratorFactory.createDateCellGenerator(extractor);
  columnDescList.add(desc);
  return this;
 }

    // and test that shows the problem
    public static void main(String []args) {
        final ExcelExporter<Date> exporter = new ExcelExporter<Date>();
        exporter.addColumn("Data", new DateTimeExtractor<Date>() {
            public Date extract(Date date) {
                return date;
        }});

        // this file looks OK
        FileOutputStream ostream = new FileOutputStream("C:\\tmp\\test1.xls");
        try {
            ostream.write(exporter.exportList(0, 0, Collections.singletonList(new Date())));
        } finally {
            ostream.close();
        }

        // but in this file date is shown in cell with numeric format
        final ExcelExporter<Date> exporter2 = new ExcelExporter<Date>();
        exporter2.addColumn("Data", new DateTimeExtractor<Date>() {
            public Date extract(Date date) {
                return date;
        }});

        ostream = new FileOutputStream("C:\\tmp\\test2.xls");
        try {
            ostream.write(exporter2.exportList(0, 0, Collections.singletonList(new Date())));
        } finally {
            ostream.close();
        }
    }
}

Ответы [ 4 ]

5 голосов
/ 28 сентября 2009

Ответ Телконтара был полезен, поскольку указывал, что это функция, а не ошибка, но его было недостаточно, чтобы не давать ссылки на FAQ или документ. Поэтому я провел небольшое исследование и обнаружил FAQ , в котором говорится:

также важно, чтобы вы не объявляли форматы своих ячеек как статические . При добавлении формата ячейки на лист ему присваивается внутренний индексный номер.

Таким образом, ответ таков: форматы не могут быть повторно использованы на разных листах, потому что они не предназначены для повторного использования таким образом.

2 голосов
/ 15 февраля 2011

Это на самом деле хуже, чем это. Шрифты и форматы неявно полагаются на «рабочую книгу». Вопрос о том, какая рабочая тетрадь освещает проблему. По-видимому, их необходимо переназначить после создания последующих рабочих книг.

    final WritableWorkbook workbook = Workbook.createWorkbook(response
            .getOutputStream());

    // We have to assign this every time we create a new workbook.
    bodyText = new WritableCellFormat(WritableWorkbook.ARIAL_10_PT);
    ...

API должен быть изменен так, чтобы конструкторы требовали в качестве аргумента рабочую книгу, к которой они относятся, или конструкторы должны быть частными, а шрифты и форматы должны быть получены из рабочей книги.

    WritableCellFormat bodyText = new WritableCellFormat(workbook,
            WritableWorkbook.ARIAL_10_PT);

или

    WritableCellFormat bodyText = workbook.getCellFormat(
            WritableWorkbook.ARIAL_10_PT);
2 голосов
/ 22 сентября 2009

В формате jxl объекты нельзя повторно использовать в нескольких книгах. я не знаю почему.

1 голос
/ 25 сентября 2009

вы можете попробовать SmartXLS , формат ячейки можно повторно использовать где угодно.

...