#java #excel #for-loop #arraylist #apache-poi
#java #excel #цикл for #arraylist #apache-poi
Вопрос:
У меня проблемы с моим кодом.
Это работает не так, как я хотел, чтобы это работало. Извините, если я так плохо объясняюсь. 🙁
(Я использую Apache POI версии 4.1.1)
Моя проблема: у меня есть таблица Excel с несколькими записями. Каждая запись содержит, среди прочего, дату отправки в столбце 1, имя клиента в столбце 2, идентификатор поддона в столбце 15 и статус доставки в столбце 16. В довершение ко всему, для столбцов есть заголовок.
Excel содержит несколько клиентов, и некоторые из них по нескольку раз, единственное отличие — идентификаторы поддонов. Мой класс объектов customer содержит, среди прочего, список массивов, который может содержать несколько разных идентификаторов поддонов.
Я хочу перечислить каждого клиента, который отправляется сегодня, только один раз, но со всеми его идентификаторами поддонов.
Мой класс customer выглядит следующим образом:
import java.util.ArrayList;
import java.util.Date;
public class Customer {
private String customername;
private ArrayList<String> pallet = new ArrayList<>();
private String date;
private String status;
public Customer(){
}
public String getCustomerName() {
return customername;
}
public void setCustomerName(String customername) {
this.customername = customername;
}
public String getFromPallet(int i) {
return pallet.get(i);
}
public ArrayList<String> getPallet() {
return pallet;
}
public ArrayList<String> get(int j) {
return pallet;
}
public void setPallet(ArrayList<String> pallet) {
this.pallet = pallet;
}
public void addPallet(String string) {
this.pallet.add(string);
}
public String getDate() {
return date;
}
public void setDate(String date) {
this.date = date;
}
public String getStatus() {
return status;
}
public void setStatus(String status) {
this.status = status;
}
public String toString(){
return String.format("%1s - %2s - %3s", customername, date, status);
}
}
мой основной класс выглядит следующим образом:
import java.io.File;
import java.io.FileInputStream;
import java.io.IOException;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.Date;
import java.util.List;
import org.apache.poi.xssf.usermodel.XSSFRow;
import org.apache.poi.xssf.usermodel.XSSFSheet;
import org.apache.poi.xssf.usermodel.XSSFWorkbook;
public class NewTrueSimpleExcelReader {
SimpleDateFormat SDF = new SimpleDateFormat("dd.MM.yyyy");
Date today = new Date();
Date BBB;
int lastfound = 0;
public List<Customer> readCustomersFromExcelFile(String excelFilePath) throws IOException, ParseException {
List<Customer> listCustomers = new ArrayList<>();
FileInputStream inputStream = new FileInputStream(new File(excelFilePath));
XSSFWorkbook wb = new XSSFWorkbook(inputStream);
XSSFSheet sheet = wb.getSheetAt(0);
XSSFRow firstrow = sheet.getRow(1);
int fr = firstrow.getRowNum();
XSSFRow row;
while(fr < sheet.getLastRowNum()){
row = sheet.getRow(fr);
if(!row.getCell(0).getStringCellValue().equals("")){
String AAA = row.getCell(0).getStringCellValue().trim();
BBB = SDF.parse(AAA);
if((SDF.format(BBB)).equals(SDF.format(today))){
Customer aCustomer = new Customer();
System.out.println("Customer: " row.getCell(1).getStringCellValue().trim());
if(listCustomers.size() > 0 amp;amp; row.getCell(16) == null){
for(int i=0; i<listCustomers.size(); i ){
System.out.println("i is: " i);
String string1 = listCustomers.get(i).getCustomerName();
String string2 = row.getCell(1).getStringCellValue();
if(string1.equals(string2) amp;amp; row.getRowNum() != lastfound){
lastfound = row.getRowNum();
System.out.println("Dupl found");
for(int j=0; j<listCustomers.get(i).get(j).size(); j ){
if(!row.getCell(15).getStringCellValue().trim().equals(listCustomers.get(i).getFromPallet(j))){
listCustomers.get(i).addPallet(row.getCell(15).getStringCellValue());
}
}
} else{
if(row.getRowNum() != lastfound){
System.out.println("second : " row.getCell(0).getStringCellValue().trim());
lastfound = row.getRowNum();
aCustomer.setDate(row.getCell(0).getStringCellValue().trim());
aCustomer.setCustomerName(row.getCell(1).getStringCellValue());
aCustomer.setStatus("unused");
if(row.getCell(15) != null){ //amp;amp; !string1.equals(string2)
aCustomer.addPallet(row.getCell(15).getStringCellValue());
}
listCustomers.add(aCustomer);
}
}
}
} else{
if(row.getCell(16) == null){
lastfound = row.getRowNum();
System.out.println("first: " row.getCell(0).getStringCellValue().trim());
aCustomer.setDate(row.getCell(0).getStringCellValue().trim());
aCustomer.setCustomerName(row.getCell(1).getStringCellValue());
aCustomer.setStatus("unused");
if(row.getCell(15) != null){
aCustomer.addPallet(row.getCell(15).getStringCellValue());
}
listCustomers.add(aCustomer);
}
}
}
}
fr ;
}
System.out.println(listCustomers.size());
wb.close();
inputStream.close();
return listCustomers;
}
}
Некоторые клиенты перечислены несколько раз с одним и тем же именем на одну и ту же дату, но с разными идентификаторами поддонов, поэтому мой код должен просто добавить разные идентификаторы к уже существующим клиентам в списке массивов, содержащем объекты. Но это не работает. 🙁 Он просто добавляет их как новые объекты в список!
И я не понимаю, почему. 🙁
Я знаю, что, возможно, я мог бы использовать итераторы или что, вероятно, есть лучшие способы для достижения того, чего я хочу, но мои знания о них просто недостаточно хороши.
Я надеюсь, что вы, ребята, смогли понять мою проблему и можете мне помочь. 🙂
Искренне ваш, Shathos
Обновить:
Благодаря помощи Hades мне удалось найти решение моей проблемы! Я заменил оба списка массивов хэш-картами и скорректировал класс customer. Огоооо! У меня все работает! Большое тебе спасибо, Аид! Вот моя рабочая функция:
public HashMap<String, Customer> readCustomersFromExcelFile(String excelFilePath) throws IOException, ParseException {
HashMap<String, Customer> map = new HashMap<String, Customer>();
FileInputStream inputStream = new FileInputStream(new File(excelFilePath));
XSSFWorkbook wb = new XSSFWorkbook(inputStream);
XSSFSheet sheet = wb.getSheetAt(0);
int fr = sheet.getRow(1).getRowNum();
XSSFRow row;
while(fr < sheet.getLastRowNum()){
row = sheet.getRow(fr);
if(!row.getCell(0).getStringCellValue().equals("")){
String AAA = row.getCell(0).getStringCellValue().trim();
BBB = SDF.parse(AAA);
if((SDF.format(BBB)).equals(SDF.format(today))){
Customer aCustomer = new Customer();
System.out.println("Customer: " row.getCell(1).getStringCellValue().trim());
if(row.getCell(15) == null || row.getCell(15).getCellType() == CellType.BLANK){
INPUTSTRING = "not found";
} else {
INPUTSTRING = row.getCell(15).getStringCellValue();
}
if(map.containsKey(row.getCell(1).getStringCellValue()) amp;amp; (row.getCell(16) == null || row.getCell(16).getCellType() == CellType.BLANK)) { //Check the map for new customer
System.out.println("Input is: " INPUTSTRING);
if(map.get(row.getCell(1).getStringCellValue()).getPallet().containsKey(INPUTSTRING)){
System.out.println("Duplicate found! -> " INPUTSTRING);
if((INPUTSTRING.equals("not found"))){
System.out.println("Empty duplicate found!");
}
} else {
System.out.println("No Duplicate found! -> " INPUTSTRING);
map.get(row.getCell(1).getStringCellValue()).getPallet().put(INPUTSTRING, INPUTSTRING); //If already in map add Pallet to the Object
}
}else if(row.getCell(16) == null || row.getCell(16).getCellType() == CellType.BLANK){
aCustomer = new Customer(); //populate this with the current values
aCustomer.setCustomerName(row.getCell(1).getStringCellValue());
aCustomer.setDate(row.getCell(0).getStringCellValue());
aCustomer.setStatus("unverladen");
if(row.getCell(15) == null || row.getCell(15).getCellType() == CellType.BLANK){
System.out.println("EMPTY FOUND: " row.getCell(1).getStringCellValue());
aCustomer.addPallet("not found", "not found");
} else {
aCustomer.addPallet(row.getCell(15).getStringCellValue(), row.getCell(15).getStringCellValue());
}
map.put(row.getCell(1).getStringCellValue(), aCustomer); //add to the map when customer encounter for the first time
}
}
}
fr ;
}
wb.close();
inputStream.close();
return map;
}
🙂
Комментарии:
1. будет ли статус одинаковым для одного и того же клиента с другим идентификатором поддона?
2. Привет! Спасибо за ваш ответ 🙂 Если столбец 16 в Excel пуст, статус будет «неиспользуемый». Если столбец 16 не пуст, запись должна быть пропущена, потому что я хочу перечислить клиентов только с сегодняшней датой доставки, а столбец 16 пуст.
Ответ №1:
Создайте карту a ниже
Map<String, Customer> map = new HashMap<String, Customer>();
Ключом будет строка, т. е. Именем клиента и значением будет класс Customer.
Когда ваше условие даты удовлетворяет, добавьте поддон к существующему клиенту, как показано ниже
if(map.containsKey(customerName)) { //Check the map for new customer
map.get(customerName).getPallet.add(palletId); //If already in map add Pallet to the Object
}else {
customers = new Customer(); //populate this with the current values
map.put(customerName, customer); //add to the map when customer encounter for the first time
}
Комментарии:
1. Привет! Спасибо за ваш отзыв! Я заставил ваш код работать. 🙂 Но моя проблема в следующем: как мне запретить клиенту использовать один и тот же идентификатор поддона несколько раз? Если я правильно понимаю ваш код, если клиент тот же, он просто добавляет идентификатор поддона к уже существующему, но не проверяет, есть ли этот идентификатор уже там.
2. Привет, @Hades! Я заставил это работать на меня! Даже при проверке на дубликаты в объекте customer! Ого-го-го! Большое спасибо за вашу помощь. Я думаю, что без этого я бы не нашел решения своей проблемы! Большое спасибо! 🙂 (Я обновил свой пост своей рабочей функцией!)
3. @Shathos Я рад 🙂