java ファイル読み込みでファイル内の数字と異なるエラー

Question

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
javaで部活動の名前と部員数が記録されているファイルを読み込み、部活動の名前をリストに表示し、クリックすると選択した部活動の部員数をテキストフィールドに表示するプログラムです。

結果は出力されているが、その結果が期待したものとは異なっていました。
運動部、文化部ごとに用意されたファイルの部員数を降順に並べることを実現したいです。

ファイルを読み込む機能を実装中に以下のエラーメッセージが発生しました。

###発生している問題・エラーメッセージ

ファイルは読み込めるのですが、画像の通りになってしまいます

###該当のソースコード

package reidai; 
import java.awt.BorderLayout; 
import java.awt.EventQueue; 
import javax.swing.ButtonGroup; 
import javax.swing.JFrame; 
import javax.swing.JPanel; 
import javax.swing.border.EmptyBorder; 
import javax.swing.JScrollPane; 
import javax.swing.JList; 
import javax.swing.JTextField; 
import java.awt.FlowLayout; 
import javax.swing.JLabel; 
import java.awt.Dimension; 
import javax.swing.JButton; 
import java.awt.event.ActionListener; 
import java.awt.event.ActionEvent; 
import java.io.BufferedReader; 
import java.io.File; 
import java.io.FileReader; 
import java.io.IOException; 
import javax.swing.event.ListSelectionListener; 
import javax.swing.event.ListSelectionEvent; 
import java.awt.GridLayout; 
import java.awt.Component; 
import javax.swing.SwingConstants; 
import javax.swing.ScrollPaneConstants; 
import java.util.ArrayList; 
import java.awt.BorderLayout; 
import java.awt.EventQueue; 
import java.awt.event.ActionEvent; 
import javax.swing.JRadioButton; 
import javax.swing.JComboBox; 
public class Rei4_09 extends JFrame { 
private static final long serialVersionUID = 1L; 
private JPanel contentPane; 
private JPanel panelCenter; 
private JTextField textField01; 
private JTextField textField02; 
private JTextField textField03; 
private JLabel label01; 
private JLabel label02; 
private JScrollPane scrollPane; 
private JList list; 
private JList list1; 
private JLabel label03; 
private JButton button; 
private JPanel panel4; 
private JPanel panel3; 
private JPanel panel2; 
private JPanel panel1; 
private JRadioButton radioButton01; 
private JRadioButton radioButton02; 
private JComboBox comboBox; 
private JRadioButton aBunkabu; 
private JRadioButton aUndoubu; 
private final static int MAX=60; 
private int buinsu[]; 
private int buinsu1[]; 
private String[] aseito=new String[MAX]; 
private String[] bseito=new String[MAX]; 
  
public static void main(String[] args) { 
EventQueue.invokeLater(new Runnable() { 
public void run() { 
try { 
Rei4_09 frame = new Rei4_09(); 
frame.setVisible(true); 
} catch (Exception e) { 
e.printStackTrace(); 
} 
} 
}); 
}

public Rei4_09() { 
  
setTitle("リストとファイル"); 
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
setBounds(100, 100, 700,250); 
contentPane = new JPanel(); 
contentPane.setBorder(new EmptyBorder(5, 5, 5, 5)); 
contentPane.setLayout(new BorderLayout(0, 0)); 
setContentPane(contentPane); 
panelCenter = new JPanel(); 
contentPane.add(panelCenter, BorderLayout.CENTER); 
panelCenter.setLayout(new GridLayout(2, 1, 0, 0)); 
   
   
panel1 = new JPanel(); 
panel1.setAlignmentY(Component.BOTTOM_ALIGNMENT); 
panelCenter.add(panel1); 
panel1.setLayout(new GridLayout(6, 1, 0, 0)); 
   
label01 = new JLabel("クラブ名"); 
panel1.add(label01); 
label01.setHorizontalAlignment(SwingConstants.LEFT); 
textField01 = new JTextField(); 
panel1.add(textField01); 
textField01.setColumns(10); 
label02 = new JLabel("部員数"); 
label02.setHorizontalAlignment(SwingConstants.LEFT); 
panel1.add(label02); 
textField02 = new JTextField(); 
panel1.add(textField02); 
textField02.setColumns(10); 
   
label03 = new JLabel("クラブ名2"); 
panel1.add(label03); 
label03.setHorizontalAlignment(SwingConstants.LEFT); 
textField03 = new JTextField(); 
panel1.add(textField03); 
textField03.setColumns(10); 
   
panel2 = new JPanel(); 
FlowLayout fl_panel2 = (FlowLayout) panel2.getLayout(); 
fl_panel2.setAlignment(FlowLayout.RIGHT); 
panelCenter.add(panel2); 	
radioButton01 = new JRadioButton("運動部");
		radioButton01.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
					ArrayList<String> tmpAL=new ArrayList<String>(MAX);
				buinsu=new int[MAX];
				int i=0;
				File fl = new File("./undoubu.txt");
				try{
					BufferedReader br = new BufferedReader(new FileReader(fl));
					String cItem;
					  while((cItem=br.readLine())!=null && i<MAX){
						 tmpAL.add(cItem);
						 if((cItem=br.readLine())==null)
							 break;
						 buinsu[i]=Integer.parseInt(cItem);
						 i++;
					  }
					  br.close();
				}catch(IOException fe){
					  textField01.setText(fe.toString());
				}
				
				tmpAL.trimToSize();
				list.setListData(tmpAL.toArray(new String[tmpAL.size()]));
		String[] tmpS=tmpAL.toArray(new String[tmpAL.size()]);
				String suuA;
				int suuB;
				for(int j=0;j<i-1;j++){
					for(int k=j+1;k<i;k++){
						if(buinsu[j]<buinsu[k]){
							suuB=buinsu[j];
							buinsu[j]=buinsu[k];
							buinsu[k]=suuB;
							suuA=tmpS[j];
							tmpS[j]=tmpS[k];
							tmpS[k]=suuA;
						}
					}
				}		
           	}
	});
			panelCenter.add(radioButton01);
			
		radioButton02 = new JRadioButton("文化部");
			radioButton02.addActionListener(new ActionListener() {
		public void actionPerformed(ActionEvent e) {
					ArrayList<String> tmpAL=new ArrayList<String>(MAX);
				buinsu=new int[MAX];
				int i=0;
				File fl = new File("./bunkabu.txt");
				try{
					BufferedReader br = new BufferedReader(new FileReader(fl));
					String cItem;
					  while((cItem=br.readLine())!=null && i<MAX){
						 tmpAL.add(cItem);
						 if((cItem=br.readLine())==null)
							 break;
						 buinsu[i]=Integer.parseInt(cItem);
						 i++;
					  }
					  br.close();
				}catch(IOException fe){
					  textField01.setText(fe.toString());
				}
				
				tmpAL.trimToSize();
				list.setListData(tmpAL.toArray(new String[tmpAL.size()]));
		String[] tmpS=tmpAL.toArray(new String[tmpAL.size()]);
				String suuA;
				int suuB;
				for(int j=0;j<i-1;j++){
					for(int k=j+1;k<i;k++){
						if(buinsu[j]<buinsu[k]){
							suuB=buinsu[j];
							buinsu[j]=buinsu[k];
							buinsu[k]=suuB;
							suuA=tmpS[j];
							tmpS[j]=tmpS[k];
							tmpS[k]=suuA;
						}
					}
				}		

	}
				});
				
		panelCenter.add(radioButton02);

Comments

[swordone]

画像が例外のスタックトレースのようですが、肝心の例外クラスが見えません。この上の行はありませんか？

Answer 1

たぶん同一人物だと思うので、過去の質問を解決してから質問し直した方がいいですよ。

また、ソートするならArrays.sortを使った方がいいですね。
思い通りの結果にならない時はログ出力を行い、想定した値と実際の値を比較してください。

Comments

[yugdm2]

ありがとうございます
cItem=br.readLine()を何に書き換えればいいかがわからないので前の投稿より画像などをいれてもう一度投稿した次第です
明日になるのですが、もし、指摘された部分を直してもわからなければ質問させていただこうと思っています

[yona]

回答の意味がわからないなら質問にコメントするべきですね。
cItem=br.readLine()は行を読み取り、次回の読み取り位置を次の行に写します。そのためあなたのコードでは奇数行に対してしか処理が行われていません。
cItem=br.readLine()==nullではなく、cItem==nullにしてください。

[yugdm2]

誠に失礼なことをなしてしまいましたこと深くお詫び申し上げます。
ご丁寧に教えて頂きありがとうございます。今後は充分注意いたします