質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.47%

  • Java

    14116questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

動作時間の短縮(複数行入力の最適化)

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 204

kanetugu

score 2

 前提・実現したいこと

動作自体は問題ないのですが、処理時間をより短縮したいです。
また、複数行入力はfor文やwhile文以外に方法があるのでしょうか?
なお、入力条件及び出力は以下の通りです。

【入力条件】
・3行分、"cat"又は"dog"のいずれかが入力される。

【出力】
・入力が多いほうを表示する。

 該当のソースコード

・使用言語 Java
・ソースコード
import java.util.*;

public class Main {
public static void main(String[] args) {
int cat=0,dog=0,i=0;
while(i<3){
Scanner sc = new Scanner(System.in);
String line=sc.nextLine();
if(line.equals("cat")){
cat++;
}
else{
dog++;
}
i++;
}

if(cat>dog){
System.out.println("cat");
}
else{
System.out.println("dog");
}
}
}
```

 試したこと

入力を1度配列に格納してから判定を行っていたので、String型変数の再利用で判定に利用してみましたが、改善しませんでした。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • umyu

    2018/06/06 19:17

    キーボードからの文字入力待ちが実行時間を占めるコードに見受けられますが、あくまでもサンプルコードということでしょうか?

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2018/06/06 20:03

    処理時間の測定方法を記載してください。

    キャンセル

回答 2

checkベストアンサー

+2

処理時間をより短縮したい

↑提示されているコードではたいして効果がないかもしれませんが、

while(i<3){
    Scanner sc = new Scanner(System.in);

↑この書き方では、ループのたびにScannerインスタンスを生成していることになります。
インスタンス生成はそれなりのオーバーヘッドになるので、ループの外で宣言する方がいいですね。

複数行入力はfor文やwhile文以外に方法があるのでしょうか?

↑ループ処理は必須だと思います。

ちなみに私が書くとこんな風に。

import java.util.*; 

public class Main {

    public static void main(String[] args) {

        int cat=0,dog=0,i=0;

        Scanner sc = new Scanner(System.in);

        while(i++ < 3){

            String line = sc.nextLine();

            if(line.equals("cat")){
                cat++;
                continue;
            }
            dog++;
        } 

        System.out.println( (cat > dog) ? "cat" : "dog" );
    }
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/06 20:05

    Scannerインスタンスの配置には気がつきませんでした。
    複数行入力については、やはりループで行うしかないのですね。
    ご回答頂きありがとうございました。

    キャンセル

+1

【入力条件】
・3行分、"cat"又は"dog"のいずれかが入力される。 

この条件があるということはcat又はdogのどちらが必ず2以上になります
この事実を使うと短くなると思います。

import java.util.*;

public class Main {
    public static void main(String[] args) {

        Scanner sc = new Scanner(System.in);
        int []data = new int[2];
        String []text = {"cat","dog"};
        for(int i = 0; i < 3; i++){
            String line = sc.nextLine();
            for(int j = 0; j < 2; j++){
                if(line.equals(text[j])){
                    data[j]++;
                }
            }
        }
        for(int i = 0; i < 2; i++){
            if(data[i] >= 2){
                System.out.println(text[i]);
            }
        }
    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/06 20:07

    "cat"/"dog"それぞれのカウントをするのではなく、いずれか片方のみのカウントをするということでしょうか?

    キャンセル

  • 2018/06/06 23:41

    追記として載せたコードのようにfor文を用いて判定しています。カウントする変数は配列にするのが
    いいと思います。

    キャンセル

  • 2018/06/07 00:10

    確かに、カウント変数は配列で纏めたほうがわかりやすいですね。
    ご回答ありがとうございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.47%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 解決済

    javaの繰り返しに関する問題です

    java初心者です。 forの繰り返しについての質問です。

  • 解決済

    java 北ソフト工房

    java 北ソフト工房なんですが 最後の問題8-4がわかりません dogのところでシンボルが見つかりませんと出ます 解答例から引っ張ってきたものでも通らず なにをどうすればいい

  • 解決済

    占いの機能 Javaの学習中

    ただいま、Javaの初歩の本を利用し、占いの機能を製作したのですがエラーがでてしまいました。 その理由がわからないため質問させていただきたいと思います。 ソースコード publ

  • 解決済

    暗証番号入力のプログラム

    初めてこのサイトを利用するのとjava初心者なので、もし至らない点があればご指摘ください。 暗証番号をキーボードから入力し、正しければ「暗証番号が確認できました」間違っていれば「

  • 受付中

    csvの抽出条件について質問です

    csvファイルの読み込みにキーボード入力をした文字と一致するものを抽出させるプログラムをつくっています。 ただ読み込むだけならできるのですが、それに条件をつけると読み込めなくなりま

  • 解決済

    InputMismatchExceptionが発生します。useDelimiterには指定してありま...

    前提・実現したいこと InputMismatchExceptionが発生します。 入力は"3,3,3"と,カンマ区切りで与えているのですが、 ソースコードの変数sizeを入力

  • 受付中

    ジャンケンゲーム

    ジャンケンゲームを作ってます。 <ルール> コンピュータに3回負けたらゲーム終了! 負けるまでゲームは続く! 数字を入力するとループが止まりません。 あと、winlo

  • 受付中

    Java 初心者です

    import java.util.Scanner; class car{ private int speed; public void speedUp(){ if(speed 

同じタグがついた質問を見る

  • Java

    14116questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。