数字以外の文字がある行を取得する ＋ α

Question

前提

数字以外を発見したら「数字以外が〇行目で見つかりました - InputMismatchException Error」と出力するプログラムです。

やりたいこと

上に示したように数字以外が見つかったときには、その旨の文章を出力するのですが、見つかった行で検査が止まってしまうので、そのまま続いて検査するようにしたいのですが、どうすれば良いのかわかりません。どなたか教えて頂けませんか？

入力されたデータの行数を取得するにはどうすれば良いのでしょうか？何か方法があれば教えてください。

該当のソースコード

import java.util.Scanner;
import java.util.InputMismatchException;
import java.util.NoSuchElementException;

/*
 * - How To Use -
 * 1. 1行目に検査する行数を入力する
 * 2. 2行目からデータを入力する
 */

public class Main {
    public static void main(String[] args){
        Scanner sc = new Scanner(System.in);
        int a = 0;
        try{
            int times = sc.nextInt();
            System.out.println(times + "回検査します");
            System.out.println("===== 出力結果 =====");
            try{
                for(a = 0;a < times; a++){
                    int line = sc.nextInt();
                    int code = Integer.valueOf(line);
                }
            } catch(InputMismatchException e) {
                System.out.println("数字以外が" + (a+1) + "行目で見つかりました - InputMismatchException Error");
            } finally{ }
        } catch(NoSuchElementException e) {
            System.out.println("検査する回数を入力してください");
        } finally { }
    }
}

試したこと

力不足で、何をどのようにすれば良いのかわかりませんでした。

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

JDKをインストールできない状態で、paiza.ioを使用しています。

最後になりましたが、前回の質問で、不快にさせてしまった方々、申し訳ありませんでした。

Answer 1

Javaを学び始めの方が標準ライブラリーにどんな機能があるかを殆ど知らずにいきなりpaizaの問題を解き始めると、「入力はjava.util.Scannerを使うもんだ」と思い込んでしまうのかも知れません。Scannerを使うことがいけないというわけではないですが、それはあくまで入力用の機能の一つです。Scannerのnext/nextIntなどのメソッドが便利に使える前提を考えてみますと・・・

• 入力データはスペースなど都合のよい区切り文字で文字列や数値(の文字列表現)が分割されている
• 入力データが数値(の文字列表現)なのか文字列なのかあらかじめわかっている
• 今読み込んでいるのは何行目かは意識しなくてよい
• 上の条件に合致しない場合は即座に「エラー」としてプログラムを終えていい

など。

また質問者さんは「やりたいことを正確に述べる」という点が若干不足しています。例えば

2
10 a b 20
c

なんて入力があったとき10を読んだ後でnextIntはaを読もうとします。そこでエラーを検出したとして「3行目に数字でないものがある」と表示するとそれは間違いということになると思います。また、エラーの後継続するとしてその後どうしたいのでしょう？"b 20"という部分をスキップして次の行から継続したいのですか？それとも"b"もまたエラーであるというメッセージを出したいのでしょうか？どちらにするかで結果は変わってきますよ？

---

入力データの何行目なのかを正確に表示するためにはScannerクラスのnextLine以外を使うのは論理を難しくすることになります。特に「数値でないものが存在する可能性がある」「一行に空白文字で区切られた数値や文字列が複数個存在する可能性」を前提にするならなおのことです。

なんらかの前提を追加してnextIntを使える条件を整えるならよいですがそれは少々難しいので、素直に考えるなら以下の方針の方がよさそうです。

(A) Scanner#nextIntで直接数値を読もうとせずにまず行単位で読み込む
(B) 読み込む度に「今何行目を読んだか」を数えておく
(C) 読み込んだ行に対して「そこに数字以外の文字があるかどうか」を検査する

---

細かい点で恐縮ですが・・・「数字以外の文字があるか」の検査方法については解釈のしかたが微妙にわかれると思います。

S1: 「数字」とは「'0'～'9'の数字文字(=digit)」のこと
S2: 「数字」とは「整数値の文字列表現」のこと

質問者さんはおそらくS2を意図しておられるでしょう。しかし「数字以外の行」と聞いたとき何人かのプログラマーはS1であると解釈するのではないでしょうか？

例えば、以下のような行があったとするとS2の仕様ではNGですが、S1の仕様ではOKとせねばなりません。

111111111111111111111111111111111111111111

S2でよいならInteger#parseIntで検査できます。当然ながら-123なんて行もOKになります。これはintと言えますから。

---

回答なんだか指摘なんだか曖昧なコメントになってしまいましたが、「正確にどういうことをさせたいか」を詳しく考えるというのもプログラミングの大事な練習だと思いますのでそういう意味合いで受け取っていただければと思います。

Answer 2

... 真にやりたかったたことは、数字以外の文字を消すこと ...

stream を使って書いてみました。

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class Trance {
    public static void main(String[] args) {
        try (BufferedReader br = new BufferedReader(new InputStreamReader(System.in))) {
            br.lines()
              .map(line -> line.replaceAll("\D", ""))
              .forEach(System.out::println);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

eclipse での実行例

参考情報

• Javaで1行ずつテキストデータを読み込むイディオムの変遷

http://argius.hatenablog.jp/entry/2015/06/06/234315

• Java 8 Streams: An Intro to Filter, Map and Reduce Operations

https://www.sitepoint.com/java-8-streams-filter-map-reduce/

Answer 3

見つかった行で検査が止まってしまうので、そのまま続いて検査するようにしたい

tryブロックをfor文の中に放り込めば良いです。

for(a = 0; a < times; a++) {
    int line = sc.nextInt();
    try {
        int code = Integer.valueOf(line);
    } catch(InputMismatchException e) {
        System.out.println("数字以外が" + (a+1) + "行目で見つかりました - InputMismatchException Error");
    }
}

入力されたデータの行数を取得するにはどうすれば良いのでしょうか？

前もって全行読み取って、その配列／リスト長を調べるのが一番楽かと。
ファイルが巨大な場合はメモリを圧迫しないように工夫が必要ですが。

Answer 4

私が真にやりたかったたことは、数字以外の文字を消すことであったので、よく考えてみると、下記のソースコードのようにすれば良いのかと思い、もう一度書いてみると、見事動作しました。

KSwordOfHasteさんの指摘通り、質問に「やりたいことを正確に述べる」ということが不足していました。回答してくださった御三方、非常に申し訳ありませんでした。また、そんな質問に対して、解決する方法や指摘を回答してくださり、ありがとうございました。

---

[ソースコード]

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.ArrayList;

/*
 * - How To Use -
 * 検査したいデータを入力して実行するだけ
 */

public class Main { 
	public static void main(String[] args)throws Exception{
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		ArrayList<String> array = new ArrayList<String>();
		
        while(true){
            String line = br.readLine();
            if(line == null || line == " "){
                break;
            } else {
                if(line.matches("[0-9]{1,}")){
                    array.add(line);
                } else {
                    String replaced_line = line.replaceAll("[A-Za-z!#\$%&'\[\]\(\)=~\|\-\^`\{\}\*\+<>\?_@;,\.\/',]","");
                    array.add(replaced_line);
                }
            }
		}
		
		for(String data : array){
            System.out.println(data);
		}
        
	}
}

---

[入力例]

841
1e813
28130

12730
agja
381920
9qwd
182y38
3891u20jqdwio

[入力例に対する出力]

841
1813
28130

12730

381920
9
18238
389120

Answer 5

forループ内でtry～catchブロックを追加するといいのでは？

try{
    for(a = 0;a < times; a++){
        try{
            int line = sc.nextInt();
            int code = Integer.valueOf(line);
        } catch(InputMismatchException e) {
            System.out.println("数字以外が" + (a+1) + "行目で見つかりました - InputMismatchException Error");
        } finally{ }
    }
} catch(NoSuchElementException e) {
    System.out.println("検査する回数を入力してください");
} finally { }