android studio CSVファイルを読み込んでおき、特定の文字が入力されたら対応する文字を出力したい

Question

PCスペック
Windows7 64bit
メモリ　8GB
プロセッサ　Intel Core i7-3610

Android Studio 3.4

android studioで

1.CSVファイルを読み込んでおく
2.画面右上で条件を選択
3.ExitTextで文字を入力して検索ボタンをクリック
4.入力された文字がCSVファイルの中にあれば、「その文字」＆「同じ行の違う列の対応する情報」をTextViewで出力

上のような機能を実装したいです。

3までは一応実装済みです。

写真はとりあえず、中身をすべて表示しているだけです。
![

具体的には、
ほしい情報として画面右上の選択で「気温」を選択しているときに、「1時」と入力すると、「1時」＆「21度」がTextViewで出力されるみたいな感じです。

現時点でのソースコードはこんな感じです。

package com.example.csvread3;

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.AdapterView;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.EditText;
import android.widget.ListView;
import android.widget.Spinner;
import android.widget.TextView;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import java.util.StringTokenizer;


public class MainActivity extends AppCompatActivity {

    //リストの宣言
    private ArrayList<String> timeList;
    private ArrayList<String> tempList;
    private ArrayList<String> amountList;

    //アダプターの宣言
    private ArrayAdapter arrayAdapter;
    private ArrayAdapter arrayAdapter2;
    private ArrayAdapter arrayAdapter3;

    private TextView textView;//適合した際に文章を表示
    private EditText editText;//時刻を入力
    String text;//入力内容
    String item;//気温or電車の本数


    private String spinnerItems[] = {"選択", "気温", "電車"};//スピナー
    //private TextView SpinnertextView;//現在のスピナーで選んでるやつの表示用

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        //フレームワーク作成
        timeList = new ArrayList<String>();
        tempList = new ArrayList<String>();
        amountList = new ArrayList<String>();

        //CSVを読み込んでおく
        readCSV();

        //アプリにCSVの中身を表示するためのインスタンスを作成
        ListView listView = (ListView) findViewById(R.id.list_view1);
        ListView listView2 = (ListView) findViewById(R.id.list_view2);
        ListView listView3 = (ListView) findViewById(R.id.list_view3);

        //時刻の情報
        arrayAdapter =
                new ArrayAdapter(
                        this, R.layout.list_item_layout , timeList);
        //気温の情報
        arrayAdapter2 =
                new ArrayAdapter(
                        this, R.layout.list_item_layout , tempList);
        //電車の本数
        arrayAdapter3 =
                new ArrayAdapter(
                        this, R.layout.list_item_layout , amountList);

        //画面内に表示
        listView.setAdapter(arrayAdapter);
        listView2.setAdapter(arrayAdapter2);
        listView3.setAdapter(arrayAdapter3);

        //文字を入力
        editText = findViewById(R.id.edit_text);
        //入力した文字を表示する為の準備
        textView = findViewById(R.id.text_view);
        //ボタンの作成
        Button button = findViewById(R.id.button);

        //検索条件(気温or電車の本数)を選択
        //SpinnertextView = findViewById(R.id.spinner_text_view);
        Spinner spinner = findViewById(R.id.spinner);

        // ArrayAdapter
        ArrayAdapter<String> adapter
                = new ArrayAdapter<>(this,
                android.R.layout.simple_spinner_item, spinnerItems);

        //ユーザーが選択しようとするときにドロップダウンによりリストを表示するのに使うレイアウトを指定
        adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);

        // spinner に adapter をセット
        spinner.setAdapter(adapter);

        // リスナーを登録
        spinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
            //　アイテムが選択された時
            @Override
            public void onItemSelected(AdapterView<?> parent,
                                       View view, int position, long id) {
                Spinner spinner = (Spinner)parent;
                item = (String)spinner.getSelectedItem();
                //SpinnertextView.setText(item);


            }
            //　アイテムが選択されなかった
            public void onNothingSelected(AdapterView<?> parent) {
                //
            }
        });


        //検索ボタンが押されたら
        button.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                // エディットテキストの文字を取得
                text = editText.getText().toString();
                //入力された時刻と適合する時刻を探すメソッド
                match(text);

            }
        });
    }

    /**
     * CSVの読み込み
     */
    public void readCSV() {
        try {
            InputStream inputStream =
                    getResources().getAssets().open("data1.csv");

            InputStreamReader inputStreamReader =
                    new InputStreamReader(inputStream);

            BufferedReader bufferReader =
                    new BufferedReader(inputStreamReader);

            String line = "";

            while ((line = bufferReader.readLine()) != null) {
                StringTokenizer stringTokenizer =
                        new StringTokenizer(line, ",");

                timeList.add(stringTokenizer.nextToken());
                tempList.add(stringTokenizer.nextToken());
                amountList.add(stringTokenizer.nextToken());
            }
            bufferReader.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    //入力された文字(時刻)と適合する時刻を探すメソッド
    public void match(String text){
        //時刻の情報を１番目から最後まで回す
        for(String s : timeList){
            //入力情報と一致したら
            if(text == s){
                if(item == spinnerItems[1]){//スピナーで選択した条件が「気温」だったら
                    textView.setText(text+"の"+spinnerItems[1]+"は○○です");
                }else if(item == spinnerItems[2]){//条件が「電車の本数」だったら
                    textView.setText(text+"の"+spinnerItems[2]+"は○○です");
                }
            }
        }
    }
}

現在のところ、edittextから受け取った文字をmatchメソッドにわたして、matchメソッドの中で、CSVファイルの時刻情報だけをfor文で回したときに、入力情報と一致すれば画面に、気温or電車の本数の情報をとりあえず表示させたいのですが、アルゴリズムがおかしいせいか「1時」と入力しても画面上に何も表示されません。。。

間違っている箇所がありましたら教えていただけませんでしょうか？

Comments

[jimbe]

> CSVの読み込み方すら分からない
そちらからまず調査・実装しては如何でしょうか.

[sekaikan_ozaki]

やってみましたが、
xmlファイルにてエラーが出てしまいまして、そちらに関して別に質問立てました

[jimbe]

大分進みましたですね.
>「気温」を選択しているときに、「1時」と入力すると
とありますが, 画面にもコードにも気温を選択している様子はありません. これは「とりあえず気温を選択していることにして」作られていると考えてよろしいでしょうか.

[sekaikan_ozaki]

その部分についても実装してみました。
「気温」と「電車の本数」をアプリ画面右上で選べるようにしました。

[sekaikan_ozaki]

質問のコードとアプリ画面の修正しました

[jimbe]

すいません, 機能が足りないと言いたかったわけではありませんでした. 説明とコードに差がありましたので, どちらが正しいのかということでした.
今さらですが, 動作しない状態で機能を追加するのはお勧めできませんので...

[sekaikan_ozaki]

確かにそうですよね。。

すみません、
以後気をつけます！

Answer 1

  //ボタンが押されたら
  button.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
      // エディットテキストの文字を取得
      text = editText.getText().toString();
      // 取得したテキストを TextView に張り付ける
      //textView.setText(text);
    }
  });

  //入力された時刻と適合する時刻を探すメソッド
  match(text);

となっていますが, なぜ match メソッドの呼び出しが onClick メソッドの外なのでしょうか.
ボタンが押されたら検索・表示を行うのではないのですか.

---

追記：

間違っている箇所がありましたら

これだけ出来る方なので, 落ち着いて確認されればお分かりになると思います.

  //入力情報と一致したら
  if(text == s){
    if(item == spinnerItems[1]){//スピナーで選択した条件が「気温」だったら
      textView.setText(text+"の"+spinnerItems[1]+"は○○です");
    }else if(item == spinnerItems[2]){//条件が「電車の本数」だったら
      textView.setText(text+"の"+spinnerItems[2]+"は○○です");
    }
  }

この中で「java としては」やりたいことと記述が違っている箇所はどこでしょうか.

Comments

[sekaikan_ozaki]

ご指摘ありがとうございます
確かに言われてみるとそうでした。。。

コードを加筆・修正したのですが、「1時」と入力しても、「気温」or「電車の本数」のテキストが表示されないです。

コード自体にエラーは無く実行できるので、おそらくmatchメソッドがちゃんと機能してないのかと思われますが、おかしい点などありませんでしょうか...?

[sekaikan_ozaki]

match メソッドの呼び出しの場所を onClick メソッドの中にしたのですが、表示がうまくいかないみたいです。。

[sekaikan_ozaki]

textviewの文章内の○○は後から追加する内容なので、ここは関係ないとしておきますと、「やりたいことと記述が違っている箇所」という意味ではどこが違っているのか30分ほど考えてみましたが、ちょっと見当がいまだにつかないです。。
jimbe様の、ご指摘を上手く解釈できてない自分がもどかしいです

[sekaikan_ozaki]

ちなみに、
//入力された文字(時刻)と適合する時刻を探すメソッド
public void match(String text){
//時刻の情報を１番目から最後まで回す
for(String time : timeList){
//入力情報と一致したら
if(text == time){
if(item == spinnerItems[1]){//スピナーで選択した条件が「気温」だったら
textView.setText(text+"の"+spinnerItems[1]+"は○○です");
}else if(item == spinnerItems[2]){//条件が「電車の本数」だったら
textView.setText(text+"の"+spinnerItems[2]+"は○○です");
}
}else{
textView.setText("時刻が間違っています");
}
}
}
メソッドをこのように書き換えた状態で、「1時」と入力すると、
「時刻が間違っています」と表示されてしまったので、そもそも入力情報とも一致してないことが判明してますますよくわからなくなってきました。。

[jimbe]

なかなか視点を変えるのも難しい時もありますね.
if で else に行ってしまうということは, 条件式があっているか, もしくは「判定方法があっているか」...ということを確認してみては如何でしょうか.

[sekaikan_ozaki]

jimbe様が思っているほど私は頭が賢くないので、
一旦、上のコードのアルゴリズムを日本語で整理してみたいと思います笑
①拡張for文を使って、timeListの要素を変数timeに入れて、回す。
②判別方法として、timeListの要素である変数timeと、入力情報である変数textの値(両方ともString型)を比べて一致したら、「○時の○は○○です」と画面に表示して、breakでfor文を抜ける

要するに線形探索アルゴリズムになってるはずなのですが、自分では正しいと思っている(頭が固すぎる)だけに、なぜこれでまず１個目のif文(何を入力してもelseで返ってくる)が正しく判別されないのか謎です。
if文では、判別するtimeとtextが両方ともString型の変数になっているし、何も問題ないはずだと思うのですが...

そういえばよく考えてみると、for文が終わってからでないと、テキスト表示したなと思い、for文の外で表示するように書き換えてみました。
```java
//入力された文字(時刻)と適合する時刻を探すメソッド
public void match(String text){
int find = -1;
//時刻の情報を１番目から最後まで回す
for(String time : timeList){
if(time == text){//入力情報と一致したら
find =1;
break;
}
}
if(find == 1) {
if(item == spinnerItems[1]) {//スピナーで選択した条件が「気温」だったら
textView.setText(text + "の" + spinnerItems[1] + "は○○です");
}else if(item == spinnerItems[2]) {//条件が「電車の本数」だったら
textView.setText(text + "の" + spinnerItems[2] + "は○○です");
}
}else if(find == -1){
textView.setText("時刻が間違っています");
}
}

```
これでも案の定」、for文の中のif文の判別式がおそらく正しくないせいで、「1時」と入力しても「時刻が間違っています」で返ってきます。。

難しいですね。
for文→拡張for文
配列→List
を使ったことで判別式のやり方が変わってきたりするのでしょうか。。。

[sekaikan_ozaki]

else ifはelseでも代用できますね笑

[jimbe]

> jimbe様が思っているほど私は頭が賢くないので
賢さは分かりませんが, 申し上げました通り「視点を変える」難しさです.
コメントされた文面から要所を抜き出しますと,

> timeListの要素である変数timeと、入力情報である変数textの値(両方ともString型)を比べて一致したら
> 自分では正しいと思っている

です.
文字列の一致を判定するには '==' でよかったでしょうか.

[sekaikan_ozaki]

jimbe様

このコメントを見た瞬間に衝撃が走りました。
完全に頭の中から抜け去っていたequalsメソッドを記述したことで、動作確認が取れました。

どこが原因かは推測できるのに、それがなぜダメなのかという理由をしっかり考え、すぐに間違いに気付けるようにならないといけないなと思いました。
今後もこういうことがたくさん起こると思うので、ちゃんと自力で気づけるようになっていきたいし、そのためにももっとプログラムに触れる機会を増やしていきたいと思います。

本当にありがとうございました

[jimbe]

equals のことは勿論ご存知だろうとの推測から, 気付いてさえ頂ければと少々意地の悪い回答/コメントが続いてしまってすいません. <(_ _;>
text と s が(表示上)同じ文字列の時に `text == s` が true になっているかと疑うのは, `1+1=2` を疑うようでなかなか難しいと思います.

自分の間違いは分かり難いですが, 他人の間違いは案外気付くものです.
teratail には(自称)初心者の方々のご質問もありますので, 試しに原因を究明してみますと, 「こんなこともある」「あんなこともある」と気付きの種を得られるのではと思います. その種がやがて「そういえばあの時こんな感じだった」と自身の気付きに成長するでしょう.
偉そうになってしまいますが, (機能追加の手際等)お力はあると思いますので, 今後も是非開発を続けて頂けたらと思います.
お疲れ様でした.

[sekaikan_ozaki]

jimbe様のご指摘の仕方はとてもありがたく、これからエンジニアとしてやっていく中で自分で考えて問題を解決できるようになっていきたいと思います。
まだまだAndroid開発が初心者で色々質問投稿することもあると思いますが、自分で考えられる範囲のことは、自力で考えてその分の成長に繋がていきたいと思います。
ありがとうございました。