c# 文字数を決めるコード(乱数に近いかな？)の続き

Question

前回の質問
動作確認をした所

～省略～
int pos = r.Next(randomInput.Length);
            char c = randomInput[pos];
            //↓文字重複させたくない場合
            //if (sb.ToString().IndexOf(c) >= 0) continue;
            sb.Append(c);
～省略～

の"//if (sb.ToString().IndexOf(c) >= 0) continue;"を有効にすると何故か応答なしになります
解決方法を教えて下さい(_ _)

※追記

private static readonly string randomInput = "10101010101010101";  //本当は文字ですが都合上数字に変えています(元の例:あいあいあいあい...←こんな感じです)
        static string GetHoge()
        {
            StringBuilder sb = new StringBuilder();
            Random r = new Random();
            //↓文字数固定であればこれを固定値にする
            int length = r.Next(5, 17);  //◯～◯文字
            while (sb.Length <= length)
            {
                int pos = r.Next(randomInput.Length);
                char c = randomInput[pos];
                //↓文字重複させたくない場合
                if (sb.ToString().IndexOf(c) >= 0)continue;
                sb.Append(c);
            }
            return sb.ToString();
        }

Comments

[Tak1wa]

既にコメントアウトされてる行ですが、有効行に変更したという意味でしょうか？いずれかの理由で無限ループしていますね。
「randomInput」の内容と、前の質問の「int length = r.Next(3, 5); 」の部分が現在どういう実装になっているか教えてください。

[come25136]

すみません、そうです有効にしたのです

[Tak1wa]

あー、すみません。無限ループになり得ますねこれは…。
修正するのでお待ちを。

Answer 1

文字数だけであれば、以下だけでいいことになります。

static string GetHoge()
{
    Random r = new Random();
    int length = r.Next(5, 18);
    return randomInput.Substring(0, length);
}

＃ちなみにNextメソッドの第二引数は最大桁数-1が正しかったです。
＃上の例では5~17までがランダムで取得されます。

Comments

[come25136]

ありがとうございます！！
無事解決しました
今回は多分ですが・・・"乱数"と言う言葉を使ったことにより解釈を間違ったのだと思います
次からは最初からわかりやすく質問しますのでこれからもよろしくお願いします(_ _)

Answer 2

こんにちは。

ガラリと変わりますが以下のように変更しました。
重複してなくて良いとのことなので、文字列をランダムソートし、ランダムな長さだけ先頭から切り出しました。

static string GetHoge()
{
    StringBuilder sb = new StringBuilder();
    Random r = new Random();
    int length = r.Next(5, 17);  //◯～◯文字
    var shuffle = randomInput.OrderBy(x => Guid.NewGuid()).ToArray();
    for (int i = 0; i < length; i++)
    {
        sb.Append(shuffle[i]);
    }
    return sb.ToString();
}

Comments

[come25136]

うーんやはり無限ループになってしまいます
(コメントアウトを有効にした時だけ)

[Tak1wa]

このロジックのコメント部分は全て削除して良いです。以前のロジックは重複した文字があれば再度ループして乱数から文字列を取得していました。
しかし「あいあいあいあいあい」の場合は重複を考慮してしまうと２文字までにしかなりません。

今回のロジックは重複かどうかに関係なく、最初に与えられた文字をソートしますのでその問題は解決されています。
GetHogeメソッドの中を入れ替えてみてください。
コメント行は削除しておきます。

[come25136]

再度試しましたが今度は"11100101111100010"みたいになってしまいます
夜遅くまですみません(_ _)
(理想:10101010101010101から10101010←文字数だけランダムで変えれればいい)

[Tak1wa]

すみません。そもそもの質問を解釈違っていたかもしれません。
最初に与えられた文字列自体はランダム化しなくても良かったのですね。
上記の場合だと開始位置は先頭からですか？ランダムですか？

[come25136]

先頭からです(_ _)