indexOf()で部分文字列を検出できない

Question

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

public class ipaddr {
  public static void main(String[] args) throws Exception {
    
    int i = 0;
    BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
    String line = br.readLine();
    
    while ((line = br.readLine()) != null) {
      if(line.indexOf(".") == -1){
        System.out.println("FALSE");
        break;
      }  

上記コードなのですが、indexOf()で標準入力にIPアドレスを入力しても、FALSEが返ってしまいます。どのへんに原因がありますでしょうか？

追記：
質問にはありませんが、IPアドレスの分割をしたかったのですが、その問題は

String[] value = line.split("\\.", -1);

でうまくいきました。ありがとうございました。

Answer 1

whileに入る前に1行読み飛ばしているからでは？
標準入力の仕様にもよると思いますが、最初に入力したIPアドレスは

String line = br.readLine();

ここで読み飛ばされて、次のwhileに入った時の読み込みでは空の文字列が渡ったせいで、
空の文字列に"."は無いのでindexOfは-1を返し、ifに入ってFALSEを出力したのではないでしょうか？

Comments

[tixure55]

回答ありがとうございます、コンソール上でjavac IPADDR.javaとしてコンパイルして、java IPADDRとして標準入力しているのですが、原因はおっしゃるとおり、カンマを入れない入力のせいでした。

Answer 2

まずコードのString line = br.readLine();は使っていないので String line = ""; でいいです。

おそらくソースコードと入力しているipの仕様がおかしいのだと思います。

たとえば198.168.11.11をipとした場合に198 と 168 と 11 と 11を入れようとしているのではないでしょうか？whileが行でループさせようとしているので、そう感じました。
ただif分はそのつもりはなく「198.168.11.11」が来ることを想定している書き方です。

質問の回答ですが、「198.168.11.11」を入力時にいれてみていただければドットが入っているのでFALSEという結果にはなりません。入力で区切ってまず198でエンターの場合ドットがないのでFALSEになります。

コードでやろうとしていることを理解して、入力するipアドレスも考えて作ってください。

Comments

[tixure55]

ドット演算子が入っている入力ばかりしていると追われないので空エンターの入力でFALSEになっていたようです。入力としては192.168.11.11というように入力していたので、ドット演算子を含む行はFALSEが返っていませんでした。