Ошибка при заполнении Jlist данными из ArrayList - PullRequest
0 голосов
/ 30 ноября 2018

Я создаю программу, которая будет генерировать электронные письма для имен, извлеченных из листа Excel.У меня проблема в том, что когда я пытаюсь перенести свои данные Arraylist в Jlist ListModel, в графическом интерфейсе ничего не появляется.

Я знаю, что публиковать весь код - это излишне, но id скорее переоценивают, чем заставляют людей запрашивать дополнительные детали, чтобы понять, что происходит.проблема строго между классами Main и Window (наверняка).

Это класс person для создания объектов person

public class person {
    private String fName, lName, serialNum, location, isDone;
// default constructor
public person() {
    fName = "default";
    lName = "default";
    serialNum = "0";
    location = "nowhere";
    isDone = "";
}

public person(String serialNum1, String fName1, String lName1, String 
location1, String isDone1) {
    this.fName = fName1;
    this.lName = lName1;
    this.serialNum = serialNum1;
    this.location = location1;
    this.isDone = isDone1;
}

// set first name
public void setFirst(String newFirst) {
    this.fName = newFirst;
}

// get first name
public String getFirst() {
    return fName;
}

////////
// set last name
public void setLast(String newLast) {
    this.lName = newLast;
}

// get last name
public String getLast() {
    return lName;
}

/////////
// set serial number
public void setSerial(String newSerial) {
    this.serialNum = newSerial;
}

// get serial number
public String getSerial() {
    return serialNum;
}

/////////
// set location
public void setLocation(String newLocation) {
    this.location = newLocation;
}

// get location
public String getLocation() {
    return location;
}

// set done
public void setDone(String newDone) {
    this.isDone = newDone;
}

// get done
public String getDone() {
    return isDone;
}

@Override
public String toString() {
    return ("serial: " + serialNum + " |  Name: " + fName + " " + lName + " |  Location: " + location + " |  Completed: "+isDone+ "\n");
}
public String label() {
    return " Name:                    " + fName + " " + lName + "                 done: "+isDone+ "\n";
}


///////////
// script///
///////////
public String printScript() {
    return "Hello " + fName + " " + lName + "\n\n" +

            "script"+

            "Serial #: " + serialNum + "\n\n" +

            "Location: " + location + "\n\n" +

            ;
}
}

Основной класс читает файл excel и извлекает имена /данные из него.Затем он помещает эти данные в Arr a.«a» успешно сохраняет объекты в массиве, но при попытке переместить содержимое «a» в GUI с помощью «w.fill (a);»это ничего не делает.графический интерфейс отображается с пустым jList.

public class Main extends JFrame {

/**
 * Launch the application.
 */
// connect program to desired excel file
private static final String fileName = "C:\\bigList.xlsx";
public static Arr a = new Arr();
public static Window w = new Window();

public static void main(String[] args) throws URISyntaxException {
    String subject = "title";
    String body = "See%20it";

    // temporary variables to store excel data for 1 row.
    // probably some way to make it more efficient than using nextCell. go by column
    // # or something
    String serial = "";
    String first = "";
    String last = "";
    String location = "";
    // could be changed to boolean
    String done = "";
    int jump;
    // create arraylist to store excel file

    try {

        // initialize reading of excel file
        FileInputStream excelFile = new FileInputStream(new File(fileName));

        // opens workbook for java to read
        Workbook workbook = new XSSFWorkbook(excelFile);

        // gets workbook sheet, usually 0, can be other values in case of 
multiple
        // sheets
        Sheet datatypeSheet = workbook.getSheetAt(0);

        // initialize iterator for new rows
        Iterator<Row> iterator = datatypeSheet.iterator();

        // while there is a row with data it will keep going
        while (iterator.hasNext()) {

            // increment var reset. stores iterator position
            jump = 0;

            // adds temp data to arraylist
            a.getList().add(new person(serial, first, last, location,done));

            // since only some columns of 'completed' are populated, the 'done' variable has
            // to be reset to null
            // otherwise the status will remain as yes, after the first yes, since that is
            // the only other variable state
            done = "";

            // initialize row iterator
            Row currentRow = iterator.next();

            // initialize cell iterator
            Iterator<Cell> cellIterator = currentRow.iterator();

            // while there is data in a cell, it will keep iterating to the right to the
            // next cell
            while (cellIterator.hasNext()) {

                // increments column/cell value
                jump++;

                // create cell object
                Cell currentCell = cellIterator.next();

                // if cell contains string & isnt null value, will jump to subgroup of if
                // statements
                if (currentCell.getCellType() == CellType.STRING && 
currentCell.getCellType() != null) {

                    // depending on position, store in corresponding temp value
                    // probably a better way to do this
                    // System.out.println(jump);

                    if (jump == 1) {
                        serial = currentCell.getStringCellValue();
                    }
                    if (jump == 2) {
                        first = currentCell.getStringCellValue();
                    }
                    if (jump == 3) {
                        last = currentCell.getStringCellValue();
                    }
                    if (jump == 4) {
                        location = currentCell.getStringCellValue();
                    }
                    if (jump == 5) {
                        done = currentCell.getStringCellValue();
                    }
                } else if (currentCell.getCellType() == CellType.NUMERIC) {
                    // no numeric values so nothing needed
                }

                // if cellIncrementer encounters empty cell it breaks loop and moves down to
                // next row
                else if (currentCell.getCellType() == null) {
                    break;
                }
            }

        }
        // removes column headers row from list
        a.getList().remove(0);
        a.getList().remove(0);

        // error catch messages
    } catch (FileNotFoundException e) {
        e.printStackTrace();
    } catch (IOException e) {
        e.printStackTrace();
    }
    /////////////////////////////////////////
    System.out.println(a.getList());
    // ISSUE
    // 'a' arraylist in is not getting transferred to gui
    w.fill(a);
    w.run();

    /////////////////////////////////////////////
}

// create outlook email.
// Desktop.getDesktop().mail( new URI(
// "mailto:prrout@inautix.co.in?subject="+subject+"&body="+body) );

}

класс окна, который генерирует графический интерфейс пользователя

public class Window extends JFrame {

private JPanel contentPane;

/**
 * Launch the application.
 */

/**
 * Create the frame.
 */

DefaultListModel<person> listModel = new DefaultListModel<>();

public Window() {

    JList<person> list = new JList<person>(listModel);

    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(100, 100, 450, 300);
    contentPane = new JPanel();
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    contentPane.setLayout(new BorderLayout(0, 0));
    setContentPane(contentPane);
    getContentPane().setLayout(new BorderLayout(0, 0));

    JButton button = new JButton("Generate Email");
    button.setFont(new Font("Tahoma", Font.PLAIN, 12));
    button.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent arg0) {
        }
    });
    getContentPane().add(button, BorderLayout.SOUTH);
    JScrollPane scrollPane = new JScrollPane(list);
    getContentPane().add(scrollPane, BorderLayout.CENTER);

    JLabel lblSelectPeople = new JLabel("Select Person(s)");
    lblSelectPeople.setFont(new Font("Tahoma", Font.PLAIN, 12));
    lblSelectPeople.setHorizontalAlignment(SwingConstants.CENTER);
    scrollPane.setColumnHeaderView(lblSelectPeople);
}

// needs to be variable 'a' from Main class
public void fill(Arr a) {
    for (int i = 0; i < a.getList().size(); i++) {
        listModel.addElement(a.getList().get(i));
    }
}

public void run() {
    EventQueue.invokeLater(new Runnable() {
        public void run() {
            try {
                Window frame = new Window();

                frame.setVisible(true);
                frame.getContentPane().setSize(800, 400);
                frame.setBounds(200, 50, 630, 500);

            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    });
}}

my arraylist class

public class Arr {
// create arraylist instance
ArrayList<person> arr = new ArrayList<person>();

// create getter unnecessary
public Arr() {
}

public ArrayList<person> getList() {
    return this.arr;
}}

1 Ответ

0 голосов
/ 30 ноября 2018

Вы создаете более одной переменной Window, что легко проверить, если поискать в приведенном выше коде new Window().Это должно появиться один раз, а вы звоните дважды.В одном окне вы вызываете fill(...) и передаете ему данные, в другом вы создаете метод run(), вы не вызываете fill(...) и вместо этого отображаете его без данные.Решение: не делай этого.Создайте один объект, заполните его и отобразите его.

Мой MCVE, подтверждающий точку:

import java.awt.*;
import java.awt.event.*;
import java.util.ArrayList;
import javax.swing.*;
import javax.swing.border.*;

public class Main extends JFrame {
    public static Arr a = new Arr();
    public static Window w = new Window();

    public static void main(String[] args) {

        // fill the gui with dummy data for our purposes without need of
        // outside excel libraries
        for (int i = 0; i < 20; i++) {
            String fName1 = "first name " + i;
            String lName1 = "last name " + i;
            a.getList().add(new Person(fName1, lName1));
        }
        a.getList().remove(0);
        a.getList().remove(0);
        System.out.println(a.getList());
        w.fill(a);
        w.run();
    }
}

class Window extends JFrame {
    private JPanel contentPane;
    DefaultListModel<Person> listModel = new DefaultListModel<>();

    public Window() {
        JList<Person> list = new JList<Person>(listModel);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 450, 300);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        contentPane.setLayout(new BorderLayout(0, 0));
        setContentPane(contentPane);
        getContentPane().setLayout(new BorderLayout(0, 0));
        JButton button = new JButton("Generate Email");
        button.setFont(new Font("Tahoma", Font.PLAIN, 12));
        button.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
            }
        });
        getContentPane().add(button, BorderLayout.SOUTH);
        JScrollPane scrollPane = new JScrollPane(list);
        getContentPane().add(scrollPane, BorderLayout.CENTER);
        JLabel lblSelectPeople = new JLabel("Select Person(s)");
        lblSelectPeople.setFont(new Font("Tahoma", Font.PLAIN, 12));
        lblSelectPeople.setHorizontalAlignment(SwingConstants.CENTER);
        scrollPane.setColumnHeaderView(lblSelectPeople);
    }

    public void fill(Arr a) {
        for (int i = 0; i < a.getList().size(); i++) {
            listModel.addElement(a.getList().get(i));
        }
    }

    public void run() {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    // ******Don't create a new Window object!!!!!!!!!!!! **********

                    // Window frame = new Window();
                    // frame.setVisible(true);
                    // frame.getContentPane().setSize(800, 400);
                    // frame.setBounds(200, 50, 630, 500);

                    Window frame = Window.this;
                    frame.setVisible(true);
                    frame.getContentPane().setSize(800, 400);
                    frame.setBounds(200, 50, 630, 500);

                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }
}

class Arr {
    ArrayList<Person> arr = new ArrayList<Person>();

    public ArrayList<Person> getList() {
        return this.arr;
    }
}

// Should be named Person, not person
class Person {
    private String fName, lName;

    public Person() {
        fName = "default";
        lName = "default";
    }

    public Person(String fName1, String lName1) {
        this.fName = fName1;
        this.lName = lName1;
    }

    public void setFirst(String newFirst) {
        this.fName = newFirst;
    }

    public String getFirst() {
        return fName;
    }

    public void setLast(String newLast) {
        this.lName = newLast;
    }

    public String getLast() {
        return lName;
    }

    @Override
    public String toString() {
        return (" |  Name: " + fName + " " + lName);
    }

    public String label() {
        return " Name:                    " + fName + " " + lName + "                 done: "
                + "\n";
    }

    public String printScript() {
        return "Hello " + fName + " " + lName + "\n\n";
    }
}

Другие не связанные с этим проблемы:

  • Избегайте установки размеров / границ и т. Д. Вместо этого используйте более чистые способы установки размеров, включая предоставление JList видимого количества строк и вызов setPrototypeCellValue(...) в вашем JList, чтобы установить его ширину
  • Переименуйте класс person в Person.Ваш код должен соответствовать соглашениям об именах Java - имена переменных должны начинаться со строчной буквы, а имена классов - с заглавной.Изучение этого и последующее это позволят нам лучше понять ваш код и позволят вам лучше понять код других.
  • Ваш класс Arr не служит никакой полезной цели, которая не может быть достигнута простым использованием ArrayList<Person> объект.Избавьтесь от этого класса.
  • Ваши данные о персоне будут лучше отображаться в JTable, а не в JList.Попробуйте использовать DefaultTableModel и JTable вместо JList.
  • Избегайте создания классов, расширяющих JFrame, так как это заставляет вас создавать и отображать JFrames, когда часто требуется большая гибкость.На самом деле, я бы рискнул заявить, что создаваемое вами окно, скорее всего, не окно основного приложения, а подокно или окно диалога.Если это так, вам гораздо лучше создавать и отображать данные в JDialog, а не в JFrame.
  • Избегайте использования имен классов, которые конфликтуют с именами основных классов Java, таких как «Окно».Используйте что-то более конкретное и описательное.
...