1.ページを進める方法

Question

前提・実現したいこと

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

エラーメッセージ

該当のソースコード

検索

   public void actionPerformed(ActionEvent e) {  
       // TODO 自動生成されたメソッド・スタブ  
 
       readBook();  
 
       //オブジェクト型の配列生成  
               Book bk[] = new Book[4];  
               bk[0] = new Book(data[0][0], data[0][1], data[0][2], data[0][3], data[0][4]);  
               bk[1] = new Book(data[1][0], data[1][1], data[1][2], data[1][3], data[1][4]);  
               bk[2] = new Book(data[2][0], data[2][1], data[2][2], data[2][3], data[2][4]);  
               bk[3] = new Book(data[0][0], data[0][1], data[0][2], data[0][3], data[0][4]);  
 
       //検索文字列を取得  
       String gStr = text1.getText();  
 
       //部分文字列検索  
       int result1 = book1.getTitle().indexOf(gStr);  
       int result2 = book1.getAuthor().indexOf(gStr);  
       int result3 = book1.getPublisher().indexOf(gStr);  
       int result4 = book1.getYop().indexOf(gStr);  
       int result5 = book1.getIsbn().indexOf(gStr);  
       int result6 = book2.getTitle().indexOf(gStr);  
       int result7 = book2.getAuthor().indexOf(gStr);  
       int result8 = book2.getPublisher().indexOf(gStr);  
       int result9 = book2.getYop().indexOf(gStr);  
       int result10 = book2.getIsbn().indexOf(gStr);  
       int result11 = book3.getTitle().indexOf(gStr);  
       int result12 = book3.getAuthor().indexOf(gStr);  
       int result13 = book3.getPublisher().indexOf(gStr);  
       int result14 = book3.getYop().indexOf(gStr);  
       int result15 = book3.getIsbn().indexOf(gStr);  
       int result16 = book4.getTitle().indexOf(gStr);  
       int result17 = book4.getAuthor().indexOf(gStr);  
       int result18 = book4.getPublisher().indexOf(gStr);  
       int result19 = book4.getYop().indexOf(gStr);  
       int result20 = book4.getIsbn().indexOf(gStr);  
 
       if (e.getSource() == button1) { //検索ボタン  
           if (result1 != -1 || result2 != -1 || result3 != -1 || result4 != -1 || result5 != -1) {  
               text2.setText(book1.getTitle());  
               text3.setText(book1.getAuthor());  
               text4.setText(book1.getPublisher());  
               text5.setText(book1.getYop());  
               text6.setText(book1.getIsbn());  
           }  
 
       } else if (e.getSource() == button3) { //進むボタン  
           for (int i = 0; i < 3; i++) {  
               if ((result6 != -1 || result7 != -1 || result8 != -1 || result9 != -1 || result10 != -1) && i == 0) {  
                   text2.setText(book2.getTitle());  
                   text3.setText(book2.getAuthor());  
                   text4.setText(book2.getPublisher());  
                   text5.setText(book2.getYop());  
                   text6.setText(book2.getIsbn());  
               } else if ((result11 != -1 || result12 != -1 || result13 != -1 || result14 != -1 || result15 != -1)  
                       && i == 1) {  
                   text2.setText(book3.getTitle());  
                   text3.setText(book3.getAuthor());  
                   text4.setText(book3.getPublisher());  
                   text5.setText(book3.getYop());  
                   text6.setText(book3.getIsbn());  
               }  
           }  
       } else if (e.getSource() == button2) { //戻るボタン  
           if ((result11 != -1 || result2 != -1 || result3 != -1 || result4 != -1 || result5 != -1)) {  
               text2.setText(book1.getTitle());  
               text3.setText(book1.getAuthor());  
               text4.setText(book1.getPublisher());  
               text5.setText(book1.getYop());  
               text6.setText(book1.getIsbn());  
           }  
 
       }  
   }  
}  
 

試したこと

補足情報（FW/ツールのバージョンなど）

Comments

[jimbe]

まず、「オブジェクトも配列が出来る」ということはご存じでしょうか。
actionPerformed 内で 「//インスタンス生成」として book1～4 を生成していますが、このように「番号で区別する変数」が出来たということはその一連のデータは配列的に管理できるということを表しています。
ですのでこれを(まず)配列にしてみては如何でしょうか。

[退会済みユーザー]

ご指摘ありがどうございます。

[jimbe]

コメント欄にはコードを書く「```」等が使えませんので、コードはご質問を編集して追加なり修正なりしてください。

さて、ここから必要な処理が二つ出来ます。一つは「条件に合った蔵書を見つける事」もう一つは「進む・戻るボタンでそれらを1つずつ表示すること」です。
プログラミングの進め方は人それぞれですが、一度に二つ以上のことを進めるのは慣れないうちは大変です。結果が分かり易いほうがモチベが保ち易いこともあるので、ひとまず条件による絞り込みは無視して全部の蔵書を「進む・戻るボタン」を駆使して見れるようにしては如何でしょう。

（「部分文字列検索」～の result に代入している部分や「検索ボタン」、進む・戻るボタンの所にある result を使っている if 文等をバッサリ消してしまって、「進む」「戻る」ボタンの機能のみとしてしまったほうが良いかと思います。勿体ないと思うのでしたら「メモ帳」等にカット＆ペーストで取っておいても良いです。）

[jimbe]

蛇足ですが、BookData のタイトルのスペル間違ってませんか。

[退会済みユーザー]

ご指摘ありがとうございます。

[jimbe]

配列化することで最初の書が( book1 では無く)bk[0], (book2 では無く)bk[1] といった具合に [] の中の数値によって n 番目の書を対象とすることが出来るようになりました。そして、[] の中には変数が使用できますね。
readBook 内で data[index] や data[i] 等としていることと同じです。

さて、仮に「 n 番目の書を表示する」為の変数として(int 型の) bookIndex なる変数を作ることにします。
bookIndex が 0 の状態で「 bk[bookIndex] 」という部分を通ると、それは bk[0] と同じ、すなわち最初の書を指します。
bookIndex が 1 の状態で同じく「 bk[bookIndex] 」という部分を通ると、それは bk[1] と同じ、すなわち2番目の書を指します。
では、2番目の書を示している状態から、最初の書を示すようにするにはどうすればいいのかといいますと…
bookIndex から 1 を引いたうえで、bk[bookIndex] を通ればよいことになります。
実装的には、まず、int bookIndex をフィールドとして宣言します。
次に bk[bookIndex] で示される書の情報を各テキストフィールドに設定(表示)するメソッド(仮に printBook() )を作ります。して
次の書を表示する場合には bookIndex を +1 して printBook() を呼び、前の書を表示する場合には bookIndex を -1 して printBook() を呼ぶことで、前後へのページ移動が出来ます。
なお、bookIndex がマイナスになったり、書数以上にならないようにしておかないと、異常終了してしまいますのでお気を付けください。

[退会済みユーザー]

解決することができました。ありがとうございました。

[jimbe]

解決したからといって内容を消さないでください。
このサイトは、後から来る人への情報のデータベースの役割も想定されています。

Answer 1

個数があらかじめわかっている場合は配列もいいですが、個数が不明な場合はリストを使いましょう。
ArrayList (Java Platform SE 8 )

せっかくBookクラスを作ったんですから、最初からBookクラスに入れてしまいましょう（多次元配列はややこしくなりがちなので、できるだけ避けたほうがいいです）

//String data[][];
ArrayList<Book> books;

この時、複数の蔵書がマッチした場合、進むボタンを押すと2個目、3個目の蔵書情報がテキストフィールドに表示でき、戻るボタンを押すと前の蔵書情報を表示する方法を教えて頂きたいです。

if分でどうにかやろうと思いましたが全くできませんでした。

最初は1件だけの表示だったんでしょうか？（それを拡張しようとして今こうなっている？）
何件ヒットするかわからない以上、このアプローチ（if文）はどう考えても無理がありますよね？

よく考えてください。
棚に複数の本があって、今そのうちの一冊を手に取っています。
次に進むにはそれを棚に返し、隣の本を手に持ちますね？

つまり蔵書とは別で「今手に取っている本」という変数がいるのです。
本自体を変数にする（Book currentのような）場合もありますが、今回は棚の左から何番目という「インデックス番号」のほうがいいでしょう。

進むにはインデックスを増やし、戻るときは減らします（もちろん行き過ぎないようにする必要があります）

検索も同じように蔵書とは別の、「検索にヒットした本のリスト」変数を用意すればいいでしょう。

import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.BufferedReader;
import java.io.FileReader;
import java.util.ArrayList;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;


public class BookData extends JFrame implements ActionListener {
    public static void main(String[] args) {
        new BookData("Search Book").setVisible(true);
    }

    private JButton searchButton = new JButton("検索");
    private JButton previousButton = new JButton("＜");
    private JButton nextButton = new JButton("＞");
    private JButton saveButton = new JButton("検索結果保存");

    private JTextField searchText = new JTextField();
    private JLabel resultText = new JLabel();
    private JLabel indexText = new JLabel();

    private JTextField titleText = new JTextField();
    private JTextField authorText = new JTextField();
    private JTextField publisherText = new JTextField();
    private JTextField yopText = new JTextField();
    private JTextField isbnText = new JTextField();

    private ArrayList<Book> books = new ArrayList<Book>(); // 全部の本
    private ArrayList<Book> searchBooks = new ArrayList<Book>(); // 検索にヒットした本
    private int index; // 今表示中の本の（searchBooks中の）インデックス番号

    BookData(String title) {
        super(title);
        setSize(800, 400);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(EXIT_ON_CLOSE);

        initialize();
        readCsv("booklist.csv");
        updateText();
    }

    private void initialize() {
        JPanel panel = new JPanel(new GridBagLayout());
        panel.setBorder(BorderFactory.createEmptyBorder(10, 20, 10, 20));
        GridBagConstraints gbc = new GridBagConstraints();
        gbc.fill = GridBagConstraints.HORIZONTAL;
        gbc.weighty = 1;

        gbc.gridy = 0;
        panel.add(new JLabel("検索語："), gbc);
        gbc.gridy = 1;
        panel.add(new JLabel("結果："), gbc);
        gbc.gridy = 2;
        panel.add(new JLabel("タイトル："), gbc);
        gbc.gridy = 3;
        panel.add(new JLabel("著者："), gbc);
        gbc.gridy = 4;
        panel.add(new JLabel("出版社："), gbc);
        gbc.gridy = 5;
        panel.add(new JLabel("出版年："), gbc);
        gbc.gridy = 6;
        panel.add(new JLabel("ISBN："), gbc);

        gbc.gridx = 1;
        gbc.gridy = 0;
        gbc.weightx = 1;
        panel.add(searchText, gbc);
        gbc.gridy = 1;
        panel.add(resultText, gbc);
        gbc.gridy = 2;
        panel.add(titleText, gbc);
        gbc.gridy = 3;
        panel.add(authorText, gbc);
        gbc.gridy = 4;
        panel.add(publisherText, gbc);
        gbc.gridy = 5;
        panel.add(yopText, gbc);
        gbc.gridy = 6;
        panel.add(isbnText, gbc);

        gbc.anchor = GridBagConstraints.CENTER;
        gbc.fill = GridBagConstraints.NONE;
        gbc.gridx = 0;
        gbc.gridy = 7;
        gbc.gridwidth = 2;
        JPanel panel2 = new JPanel();
        panel2.add(previousButton);
        panel2.add(indexText);
        panel2.add(nextButton);
        panel.add(panel2, gbc);

        gbc.anchor = GridBagConstraints.EAST;
        gbc.gridy = 7;
        panel.add(saveButton, gbc);

        gbc.anchor = GridBagConstraints.CENTER;
        gbc.gridx = 2;
        gbc.gridy = 0;
        gbc.gridwidth = 1;
        gbc.weightx = 0;
        panel.add(searchButton, gbc);

        add(panel);

        searchText.addActionListener(this); // Enterキーを押したとき
        searchButton.addActionListener(this);
        previousButton.addActionListener(this);
        nextButton.addActionListener(this);
//        saveButton.addActionListener(this);
    }

    private void readCsv(String filePath) {
        try (BufferedReader br = new BufferedReader(new FileReader(filePath))) {
            String line;
            while ((line = br.readLine()) != null) {
                String[] split = line.split(",");
                Book book = new Book(split[0], split[1], split[2], split[3], split[4]);
                books.add(book);
                searchBooks.add(book);
            }
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    @Override public void actionPerformed(ActionEvent e) {
        if (e.getSource() == searchButton || e.getSource() == searchText) {
            searchBooks.clear();
            index = 0;
            String s = searchText.getText();
            for (Book book : books) {
                // sが空文字列 "" だった場合true なので空で再検索すると全件表示
                if (book.contains(s)) {
                    searchBooks.add(book);
                }
            }
        } else if (e.getSource() == nextButton) {
            if (index < searchBooks.size() - 1) {
                index++;
            }
        } else if (e.getSource() == previousButton) {
            if (0 < index) {
                index--;
            }
        }

        updateText(); // 表示の更新
    }

    private void updateText() {
        Book book;
        if (searchBooks.isEmpty()) { // 検索結果がなかった時
            // titleText.setText("") のようにするのは
            // コードが重複するのでカラのBookを与えちゃう
            book = Book.empty;
        } else {
            book = searchBooks.get(index);
        }

        resultText.setText(searchBooks.size() + " / " + books.size() + " 件");
        indexText.setText(String.valueOf(index + 1));

        titleText.setText(book.getTitle());
        authorText.setText(book.getAuthor());
        publisherText.setText(book.getPublisher());
        yopText.setText(book.getYop());
        isbnText.setText(book.getIsbn());
    }
}

class Book {
    private String title;
    private String author;
    private String publisher;
    private String yop;
    private String isbn;

    public static Book empty = new Book(); // カラのBook

    private Book() {} // empty用

    public Book(String title, String author, String publisher, String yop, String isbn) {
        this.title = title;
        this.author = author;
        this.publisher = publisher;
        this.yop = yop;
        this.isbn = isbn;
    }

    public boolean contains(String s) { // どれかのメンバに部分一致しているかどうか
        return title.contains(s) || author.contains(s) || publisher.contains(s)
                || yop.contains(s) || isbn.contains(s);
    }

    public String getTitle() { return title; }

    public void setTitle(String title) { this.title = title; }

    public String getAuthor() { return author; }

    public void setAuthor(String author) { this.author = author; }

    public String getPublisher() { return publisher; }

    public void setPublisher(String publisher) { this.publisher = publisher; }

    public String getYop() { return yop; }

    public void setYop(String yop) { this.yop = yop; }

    public String getIsbn() { return isbn; }

    public void setIsbn(String isbn) { this.isbn = isbn; }
}

---

以降本題とは関係ない細かい点。

button1) { //検索ボタン

コメントをつけるくらいなら、わかりやすい変数名にしましょう。

文字列が含まれているかどうかは、indexOfよりcontainsのほうがわかりやすいです。
String#contains (Java Platform SE 8 )

一覧性を考えると、JTableを検討されてもよさそうです（結構難しいですが^^;
JTable (Java Platform SE 8 )

How to Use Tables (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)

Comments

[退会済みユーザー]

回答ありがとうございます。
とても分かりやすいコードで理解することができました。
ベストアンサーにさせて頂きます。本当にありがとうございました。