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

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

ただいまの
回答率

90.34%

  • C#

    7695questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Unity

    4405questions

    Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。

文字列の配列を、重複なしで乱数にするには

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 746

Buddha_san2

score 19

 前提・実現したいこと

C#とUnityで現在クイズゲームを作っています。クイズの問題を、一度解いたものが出ないように、ランダムで出したいと思っています。

 発生している問題・エラーメッセージ

配列を乱数にするところまでは出来たのですが、重複なしで行うことが出来ませんでした。よろしくお願いします。

 該当のソースコード

public void QuestionLabelSet()
      {
        //Unityに問題を渡す変数
        Text qLabel = GameObject.Find("").GetComponentInChildren<>();
 List<string> Q = new List<string>()
        {"問題1","問題2"


リストの並べ替えを使って、

Q = Q.OrderBy(a => Guid.NewGuid()).ToList();

        for (int i = 0; i < Q.Count; i++)
        {         
            qLabel.text = Q[i];  
        }


これでも重複してしまったので、Randomを使ってみました。

      var random = new System.Random();

        for (int i = 0; i < Q.Count; i++)
        {         
           var up = Q[random.Next(0, Q.Count)];

            qLabel.text = up;


しかし、これも重複してしまいました。

 試したこと

var random = new System.Random();

        for (int i = 0; i < Q.Count; i++)
        {         
           var up = Q[random.Next(0, Q.Count)];

           //↓ここに入れてみましたが、エラー(stringからintに変換できない)が起きました
            Q.RemoveAt(up);
            qLabel.text = up;

「追記」

public class QuizMgr : MonoBehaviour
{
    void Start()
    {
        QuestionLabelSet();
    }
public void QuestionLabelSet()
    {
        //問題をUnityに渡す
        Text qLabel = GameObject.Find("A").GetComponentInChildren<B>();

        //問題を置く
        List<string> Q = new List<string>()
        {"問題1","問題2"};

    //ランダムに問題を出す
        for (int i = 0; i < Q.Count; i++)
        {
            var num = UnityEngine.Random.Range(0, Q.Count);
            var up = Q[num];
            qLabel.text = up;
            Q.RemoveAt(num);   
        }     
}


ランダムに問題は出たのですが、重複してしまいます。すみません、確認お願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • negitama

    2018/05/04 00:19

    質問確認中ですが、コードを貼り付けるときは<code>のアイコンをクリックして表示される```と```の間に入れると読みやすくなりますよ。

    キャンセル

回答 2

checkベストアンサー

+3

リストからランダムに取り出す方法は2通りあると思います。
A) リストをランダムに並べ替えたあと、順番に取り出す。取り出したものを消す
B) リスト内のランダムな位置の要素を取り出し、その要素番号の要素を消す

質問者さんが「試したこと」でやろうとしているのは B の方法かと思います。
エラーメッセージの意味は、RemoveAtの引数には要素番号(int)を指定する必要があるのに、要素の中身そのもの(string)を指定してしまっている、ということだと思います。

var num = Random.Range(0, Q.Count);
var up = Q[num];
qLabel.text = up;
Q.RemoveAt(num);


のようにしてみてください。

なお、上記コードでは Random.Range(x, y) を使っています。これは(引数が整数なら)x 以上 y 未満のランダムな値を返すもので、UnityEngine.Random クラスのメソッドです。
もしコードの冒頭で using System; を書いている場合は System.Random クラスと UnityEngine.Random クラスのどちらを使うのかプログラムが解釈できずにエラーになります。その場合は、using System; の行を削除または // でコメントアウトするか、上記コードで

var num = UnityEngine.Random.Range(0, Q.Count);


のようにしてください。

【追記】
あと、質問者さんのコードでは for 文で、Q.Count を上限とする繰り返し処理になっていますので、for 文の中で リスト Q の要素数を削ってしまうとまずそうです。
クイズの出題は1問ずつだと思うので、ここは for 文 をやめて、(仮置きで)何かボタンを押すごとに処理されるようにしてみてはいかがでしょうか。
たとえば、

if(Input.GetMouseButtonDown(0))
{
    // 処理の中身
}


のようにすると、マウスの左クリックを押すたびに処理が行われます。

 参考コード

シンプルに書くとこういうことになります。上の【追記】で書いた理由から、for文での処理にはしていません。

using System.Collections.Generic;
using UnityEngine;

public class RandomPickUpFromList : MonoBehaviour
{
    // インスペクターから設定
    [SerializeField] List<string> questionList;


    private void Update ()
    {
        // マウスの左クリックが押されるたびに処理を行う
        if ( Input.GetMouseButtonDown ( 0 ) )
        {
            // 問題が残っていない場合は処理を行わない
            if(questionList.Count == 0 )
            {
                print ( "もう問題がない" );
                return;
            }

            // 問題番号をランダムに選ぶ
            var questionNum = Random.Range(0, questionList.Count);

            // 出題する(Consoleに出力する例)
            print ( questionList[questionNum] );

            // 問題番号の要素をリストから削除する
            questionList.RemoveAt ( questionNum );
        }
    }
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/04 01:13

    非常に分かりやすい回答、本当にありがとうございます!
    エラーは無くなったのですが、現在も、同じ問題が連続して発生してしてしまいます。
    根本的にコードが間違っていたのでしょうか・・?

    キャンセル

  • 2018/05/04 01:16

    元の質問に、「追記」とヘッダーとつけて、変更したコードを貼り付けてみてください。確認しますので。

    キャンセル

  • 2018/05/04 03:27

    参考コードを追記してみたので確認してみてください。不明点などあればコメントください。

    キャンセル

  • 2018/05/04 03:44

    ありがとうございます!頑張ってみます!

    キャンセル

  • 2018/05/04 19:52

    自分でも整理しておきたかったので記事にしました。参考まで。

    リストからランダムに要素を取り出す2種類の方法 : ねぎたまらぼ(Unity, C#, VR)
    http://negi-lab.blog.jp/RandomPickUpFromList

    キャンセル

  • 2018/05/04 22:45

    了解です!ありがとうございます!

    キャンセル

0

こちらの質問 を参考にしてみてください。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

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

  • C#

    7695questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Unity

    4405questions

    Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。