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

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

ただいまの
回答率

90.32%

  • Java

    14437questions

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

無駄な記述をなくしたい

解決済

回答 8

投稿

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

nekocat

score 1

 前提・実現したいこと

お世話になります。
初めての質問でお見苦しい点多々あると思いますが、ご容赦下さい。
ネットで見かけたJavaの問題を解いております。

 改善したい点

一応コンパイル・実行ともにでき、やりたいことはできたのですが、
該当のソースコードの★の部分が同じ処理をしているため、
str.indexOf(args[1])を1回だけ使うように
書き換えることはできないかと思い、今回質問させていただきました。

 該当のソースコード

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

        String str = args[0];        

        //★
        int num = str.indexOf(args[1]);

        while(num != -1){
            str = (str.substring(0,num) + args[2] + str.substring(num + args[1].length()));

            //★
            num = str.indexOf(args[1]);
        }

        System.out.println(str);
    }
}

/*
コマンドライン引数↓
たけやぶにたけたてかけた たけ 竹
実行結果↓
竹やぶに竹たてかけた
*/

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

Eclipce

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • mts10806

    2018/05/30 17:23

    タイトルには要件を書いて下さい。 https://teratail.com/help/question-tips#questionTips3-1 タイトルから質問内容が想像できないもの、タイトルと質問内容が食い違っているものはスムーズな回答を得られなくなります。

    キャンセル

回答 8

+2

replaceを使わずに、省略したいなら、こんな感じ?

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

        String str = args[0];
        int num;

        while((num = str.indexOf(args[1])) != -1){
            str = (str.substring(0,num) + args[2] + str.substring(num + args[1].length()));
        }

        System.out.println(str);
    }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/30 17:29

    やはりnumの宣言を外に出すのがよさそうですね!
    ありがとうございます。

    キャンセル

+1

1回探索したところは2度探索しないように改良
ついでにコマンドライン引数の内容がどれがどれかわかりにくいため変数名を付けた

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

        String str = args[0];
        String target = args[1];
        String replacement = args[2];      

        int num = 0;

        while((num = str.indexOf(target, num)) != -1){
            str = (str.substring(0,num) + replacement + str.substring(num + target.length()));
            num += replacement.length();
        }

        System.out.println(str);
    }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

checkベストアンサー

0

breakすればいいじゃない。

while(true){
    int num = str.indexOf(args[1]);

    if(num == -1){
      break;
    }

    str = (str.substring(0,num) + args[2] 
         + str.substring(num + args[1].length()));

}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/30 17:32

    ご回答ありがとうございます。
    無限ループですね!(でもこの場合はbreakしているので無限ではないのでしょうか...???)
    また違った発想だったので面白かったです!ありがとうございます。

    キャンセル

0

String#replaceを使えば数行で全体を代替可能です。

System.out.println(
    args[0].replace(args[1], args[2])
);

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/30 17:06

    ご回答ありがとうございます。
    今回アルゴリズムを学ぼうと思っており、replaceは使わないように
    やってみようと試行錯誤中なのです。
    ご助言ありがとうございます!

    キャンセル

0

replace関数を使うのはダメなんでしょうか?

class Mondai3{
    public static void main(String[]args){
        System.out.println(args[0].replace(args[1], args[2]));
    }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/30 17:05

    ご回答ありがとうございます。
    今回アルゴリズムを学ぼうと思っており、replaceは使わないように
    やってみようと試行錯誤中なのです。
    ご助言ありがとうございます!

    キャンセル

0

Javaでやるのは文化的に受け入れられるかわからないですが、ループ自体を無限ループにしておいて、途中脱出にすればindexOfの記載は1箇所で済ませられます。

        while(true){
            int num = str.indexOf(args[1]);
            if(num == -1) break;
            str = (str.substring(0,num) + args[2] + str.substring(num + args[1].length()));
        }

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/30 17:35 編集

    ご回答ありがとうございます。
    Javaの教科書でも見たことがあったので大丈夫だと思います!
    (ただbreakの条件があるのに無限ループとは…?と見るたびに混乱します(笑))
    ありがとうございました!

    キャンセル

0

自分でループを回す場合、無限ループにならないようにする注意が必要です。
失眠分のコードに、
たけやぶにたけたてかけた たけ 竹たけ
をコマンドライン引数に指定したらどうなるでしょう?

ループさせずに済ます方法と、ループをさせる方法の2つを書いてみました。

import java.util.Arrays;
import java.util.List;

public class Mondai3 {
    public static void main(String[] args) {
        List<String> words = Arrays.asList(args[0].split(args[1]));
        System.out.println(String.join(args[2], words.toArray(new String[words.size()])));

        final int src_len = args[1].length();
        if (src_len == 0) {
            System.out.println(args[0]);
            return;
        }
        String ans = "";
        int p = 0;
        while (p < args[0].length()) {
            if (args[0].substring(p).startsWith(args[1])) {
                ans += args[2];
                p += src_len;
            } else {
                ans += args[0].charAt(p);
                p += 1;
            }
        }
        System.out.println(ans);
    }
}
/*
 * 入力: たけやぶにたけたてかけた たけ 竹 
 * 出力: 竹やぶに竹たてかけた
 * 
 * 入力: たけやぶにたけたてかけた たけ 竹たけ 
 * 出力: 竹たけやぶに竹たけたてかけた
 */

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

-1

ループの中でstrが変化する以上、2個めのindexOfは省略できないですね

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/30 17:33

    ご回答ありがとうございます。
    そうですよね...書き方に一工夫いりそうです。。。

    キャンセル

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

  • Java

    14437questions

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