Проблема с ArrayLists и чтением файла

Question

У меня трудности со следующим методом. Я не могу понять, если моя проблема, но я сузил ее, чтобы не заполнять список массивов из файла. Любая помощь с благодарностью.

private void searchButtonActionPerformed(java.awt.event.ActionEvent evt) {

//create arraylists
ArrayList<String> model = new ArrayList<String>();
ArrayList<String> length = new ArrayList<String>();
ArrayList<String> width = new ArrayList<String>();
ArrayList<String> radius = new ArrayList<String>();
ArrayList<String> depth = new ArrayList<String>();
ArrayList<String> volume = new ArrayList<String>();
ArrayList<String> shape = new ArrayList<String>();

//fill arraylists from file
try {
    String outputline = "";

    BufferedReader fin = new BufferedReader(new FileReader("stock.dat"));
    while((outputline = fin.readLine()) != null)    {
       // for(int i = 0; i < outputline.length(); i++)    {
       int i = 0;

            //model
            boolean flag = false;
            String pass = "";
            while(flag = false) {
                if(outputline.charAt(i) != ',')
                    pass.concat(Character.toString(outputline.charAt(i)));

                else
                    flag = true;
                i++;
            }
            model.add(pass);

            //length
            flag = false;
            pass = "";
            while(flag = false) {
                if(outputline.charAt(i) != ',') 
                    pass.concat(Character.toString(outputline.charAt(i)));
                else
                    flag = true;
            }
            length.add(pass);

            //width
            flag = false;
            pass = "";
            while(flag = false) {
                if(outputline.charAt(i) != ',') 
                    pass.concat(Character.toString(outputline.charAt(i)));
                else
                    flag = true;
            }
            width.add(pass);

            //radius
            flag = false;
            pass = "";
            while(flag = false) {
                if(outputline.charAt(i) != ',') 
                    pass.concat(Character.toString(outputline.charAt(i)));
                else
                    flag = true;
            }
            radius.add(pass);

            //depth
            flag = false;
            pass = "";
            while(flag = false) {
                if(outputline.charAt(i) != ',') 
                    pass.concat(Character.toString(outputline.charAt(i)));
                else
                    flag = true;
            }
            depth.add(pass);

            //volume
            flag = false;
            pass = "";
            while(flag = false) {
                if(outputline.charAt(i) != ',') 
                    pass.concat(Character.toString(outputline.charAt(i)));
                else
                    flag = true;
            }
            volume.add(pass);

            //shape
            pass = "";
            for(int j = i; j < outputline.length(); j++)
                pass.concat(Character.toString(outputline.charAt(i)));
            shape.add(pass);
        }
    fin.close();
    }
catch(IOException e)    {
    System.err.print("Unable to read from file");
    System.exit(-1);

}

int at = -1;
for(int i = 0; i < model.size(); i++)   {
    if(model.get(i).equals(searchIn.getText())) {
        at = i;
        i = model.size();
    }
}
    Component frame = null;

if(at != -1)    {
    searchDepthOut.setText(depth.get(at));
    searchLengthOut.setText(length.get(at));
    searchRadiusOut.setText(radius.get(at));
    searchVolumeOut.setText(volume.get(at));
    searchWidthOut.setText(width.get(at));

}
else
    JOptionPane.showMessageDialog(null, "Your search did not return any results", "ERORR", JOptionPane.ERROR_MESSAGE);

}

Answer 1

Разделите строку чтения запятой и покончите с этим. Я бы также создал объект для модели, длины, ширины и т. Д., А затем имел бы 1 массив данных этого объекта.

while((outputline = fin.readLine()) != null)    {

    String[] tokens = outputline.split(",");
    if(tokens.length == 7){
        SObj o = new SObj; //Some Object

        o.model = tokens[0];
        o.length = tokens[1];
        //and so on

        oList.add(o);
    }
}

Answer 2

Помимо всех других проблем, перечисленных людьми ...

String pass = "";
while(flag = false) {
if(outputline.charAt(i) != ',')
   pass.concat(Character.toString(outputline.charAt(i)));

проход - это строка. Строки неизменны. Вы хотите

   pass = pass.concat(.....)

Answer 3

while(flag = false) никогда не будет запущен - он всегда оценивается как false. Попробуйте while (!flag)

Answer 4

Я бы предложил вам реструктурировать код. Проблема не только в том, что есть какая-то ошибка синтаксического анализатора, но и в том, что трудно сказать, что происходит - код явно делает предположения о структуре строки ввода, но вы должны как бы прочитать и проследить способ реструктурировать это в вашем уме.

Что-то вроде

/** Expect a line of the form model, length, ...,
  return a list of ... 
*/
private String[] parse (String inputLine)
{
  //check input line charachteristics-not null, length, ...
  String out=  inputLine.split(",");
  if (out.length()!= ... 
  //whatever sanity checking...

}

private List<String[]> extract(BufferedReader fin)
{
  while((outputline = fin.readLine()) != null) 
 {
    //do something with parse(outputline);
  }
}

Полезно будет разделить чтение файла и разбор строки, чтобы вместо того, чтобы делать все это в длинной последовательности, вы могли видеть, что происходит, скорее всего, предположение о структуре строки, скрытой в коде. Нужно ли 4 целых числа через запятую? 5? как насчет, если они дополняются пробелами? с префиксом пустой строки?