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

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

ただいまの
回答率

90.86%

  • C#

    6244questions

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

奇数と偶数で発生した乱数を振り分けたいです

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 1,327

57svu

score 14

前提・実現したいこと

始めまして
先日プログラミングをはじめたものです。
c#で配列変数を作成し
値が1から100までの乱数ときめて
配列の大きさは10
で実行した10個の乱数を
偶数だったら
string 1に追加
奇数だったら
string 2に追加
としたいです

問題URL
http://csharp.sevendays-study.com/problem4.html#problem5

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

エラーメッセージ

未割当のローカル変数が使われましたとOdd,Evenで出ます

該当のソースコード

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace prob5_3
{
class Program
{
static void Main(string[] args)
{
Random rnd = new Random();
int [] data = new int[10];

data[0] = rnd.Next(1, 101);
data[1] = rnd.Next(1, 101);
data[2] = rnd.Next(1, 101);
data[3] = rnd.Next(1, 101);
data[4] = rnd.Next(1, 101);
data[5] = rnd.Next(1, 101);
data[6] = rnd.Next(1, 101);
data[7] = rnd.Next(1, 101);
data[8] = rnd.Next(1, 101);
data[9] = rnd.Next(1, 101);

int i; // 文字列 i を宣言
string Odd ;     // 文字列 Oddを宣言
string Even; // 文字列 Evenを宣言

for (i = 0; i < data.Length; i++) // i は0で初期化 i がa の大きさ未満ならtrue

{
Console.Write(data[i] + "  ");  // 配列変数 data[i]と空白を表示
if (data[i] % 2 != 0)  // 配列変数 data[i] が偶数で割れないのであれば
{
int a = int.Parse(Odd);  // データ型変換 文字列oddは int 型の変数 aに置き換える
Odd += data[i];  // data[i] は奇数の aに代入

}
if (data[i] % 2 == 0)           // 配列変数 data[i] が偶数で割りきれるのであれば
{
int b = int.Parse(Even);
Even += data[i];
}

}
Console.WriteLine();
Console.Write(" 奇数 : " + Odd);
Console.WriteLine();
Console.Write("  偶数 : " + Even);
System.Threading.Thread.Sleep(100000);
}
}
}

試したこと

stringを使って変換作業を試みるもうまくいかないです

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

より詳細な情報

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

+3

最近ちょっと自信消失気味ですが・・・

stringで宣言した変数が初期化されていないのではないでしょうか?
stringであれば""で空文字を宣言するなど・・・

あと、そのstringにintを入れようとしているので・・・
そこもきちんとしておいたほうがよいと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/02 22:52

    ご指摘ありがとうございます。
    初期化されていない点に気がつき
    ひとまず、stringで空白を初期化し
    stringをそのままint にいれようとしていたので、そこの変換も気がつけました!
    ありがとうございます😊

    キャンセル

checkベストアンサー

+1

要するに偶数奇数で分けて、それらを文字列にしてスペース区切りで結合したいということなので、

string even = string.Join(" ", data.Where(x => x % 2 == 0));
string odd  = string.Join(" ", data.Where(x => x % 2 != 0));

つっこみどころ

if (data[i] % 2 != 0)  // 配列変数 data[i] が偶数で割れないのであれば
{
    int a = int.Parse(Odd);  // データ型変換 文字列oddは int 型の変数 aに置き換える
    Odd += data[i];  // data[i] は奇数の aに代入
} 

aどこ行った?

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/02 23:02

    aさん、せっかくつくったのに使ってなかったですね。
    ツッコミありがとうございます。
    あの後自分で調べた方法は
    intを .ToStringでstringに変換しstringに合算していく方法をとりましたが
    この方法でもやってみたいと思います
    ありがとうございます!

    キャンセル

0

偶数かどうかを判断して、奇数の処理はelse{}に入れてやったほうがエラーの原因がわかりやすいと思います。
未割当のローカル変数が使われましたってのは、要は( ´∀`)<ぬるぽ

あまりよくありませんが、一応同じプログラムを書いてみました。

using System;

class a{

    static void Main(){

        Random rnd = new Random();
        int[] data = new int[10];

        for(int i = 0; i < data.Length; i ++){

            data[i] = rnd.Next(1, 101);
        }

        string Odd = "";
        string Even = "";

        for(int i = 0; i < data.Length; i ++){

            if((double)data[i] % 2.0 == 0) Even += data[i] + " ";
            else Odd += data[i] + " ";
        }

        Console.WriteLine("奇数: " + Odd + "\n偶数: " + Even);
        Console.ReadLine();
    }
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/02 16:49

    > 偶数と奇数の判断は、if (data[i] % 2 != 0)をif ((double)data[i] % 2.0 != 0)にするのが一番手っ取り早いと思います。

    何が手っ取り早いんですか?

    キャンセル

  • 2017/05/02 17:35

    javaと混同してました。

    キャンセル

  • 2017/05/02 17:52

    javaだと手っ取り早いんですか?

    キャンセル

  • 2017/05/02 18:36

    偶奇を判定するのに整数のままで剰余演算子を用いるのが素直な方法と思います。
    が、otftroughさんの回答にちょっと興味を持ちました。なぜ浮動小数点数へ変換するのがよいと考えたか考えを述べてみると何かが得られると思います。自分も過去の回答で間違ったことを言ったとき閲覧者の皆さんに指摘していただき勘違いに気づけたことがありましたよ。

    キャンセル

  • 2017/05/02 22:59

    みなさんたくさんのご意見ありがとうございます。
    ぬるぽの箇所はひとまず初期化をすることで改善できました
    ただ
    なぜdoubleを使うのかがピンときていないです😔

    キャンセル

  • 2017/05/03 00:10

    単にいろいろ考えすぎて混乱して間違ってただけでした。
    普段自分は除算を使っていて、今回の場合ならif((double)(data[i] / 2) == (double)data[i] / 2.0)とかしてるので、%を使い慣れていないせいで混同してました。%使うなら少数関係ないですね。

    キャンセル

  • 2017/05/03 03:55

    なるほど、割り切れるかどうかをi1/i2 == (double)i1/i2としていたということですかね。
    そういうことならやはり整数のまま「i1 % i2 == 0」を使う方が素直だと思います。
    細かいようですが割り算や剰余は整数でも実数でも演算速度が大変遅いので2回除算するよりは1回の剰余の方が好まれると思います。

    キャンセル

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

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

関連した質問

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

  • C#

    6244questions

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