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

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

ただいまの
回答率

90.52%

  • Go

    505questions

    Go(golang)は、Googleで開発されたオープンソースのプログラミング言語です。

forの中でselectを回し続ける場合のチャネルの使い方の例を教えてください!

解決済

回答 1

投稿 編集

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

DDxlk

score 115

例文と質問

例えば

func test() {
    for {
        select {
        case v, ok := <-ch:
            if ok {
                fmt.Println(v)
            } else {
                fmt.Println("close")
            }
        default:
        }
    }
}


このようなtest関数を作り、ループで待機させてここに別のgoroutine(例えばmain関数)からどんどんchannel経由で値を送っていき、それを裁かせるようにしたいのですが、そういう場合他のgoroutineからの値の送信はどのようにすれば一番合理的でしょうか。

考えたこと

グローバル変数を使って

var ch = make(chan string)

func main() {
    go test()
    ch <- "Hello"
    ch <- "2"
    ch <- "teratail"
}

func test() {
    for {
        select {
        case v, ok := <-ch:
            if ok {
                fmt.Println(v)
            } else {
                return
            }
        default:
        }
    }
}

とすれば、test関数からもmain関数からもch変数を参照できて楽なのですが、どうもグローバル変数を簡単に作るのはよくない気がします。
このように複数のgoroutine同士のやりとりの際のチャネルの参照方法について、グローバル変数を使ってもいいのか、もしくはできるだけスコープを狭めるようにした実装がいいのか。また後者なら方法をご教授ください。
お願いいたします...

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

2点修正をおすすめします。

1点目はselectにdefaultケースをつけてしまうと、ほかのケースが成立しない間は
ビジーループ(CPU資源の浪費)になってしまうので避けましょう。

2点目はtest関数内に必要なものは引数で渡す方が望ましいと思います。

修正を反映すると以下のようになります。
これで広域変数に依存しなくなり、テストコードが書きやすくなるでしょう。

https://play.golang.org/p/zDGlCsEFiKa

func main() {
    ch := make(chan string)
    go test(ch)
    ch <- "Hello"
    ch <- "2"
    ch <- "teratail"
}

func test(ch <-chan string) { // 読み出し専用と宣言する時、<-をchanの前につける
    for {
        select {
        case v, ok := <-ch:
            if ok {
                fmt.Println(v)
            } else {
                return
            }
        }
    }
}

ただし、selectにケースが1つしかない場合はselect構文を使う必要はありません。

for文以下を整理すると以下のようになります。

for {
    v, ok := <-ch:
    if ok {
        fmt.Println(v)
    } else {
        return
    }
}

Goのifによる分岐はearlyリターンが推奨されているので更に整理すると・・・。

for {
    v, ok := <-ch:
    if !ok {
        return
    }
    fmt.Println(v)
}

Goのforには実は上記と等価な構文がありまして・・・。

for v := range ch {
    fmt.Println(v)
}

というように書けます。

 追記

また、goroutineとして動作を開始するtest関数が正常に終了するためには、
test関数以外のgoroutineからclose(ch)を呼ぶ必要があります。
現在の実装はmainのgoroutineが終了したことによりプロセスが終了することで
強制的に他のgoroutineが停止している状況です。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/21 19:13

    ちなみに広域変数と書いたのはわけがあり、GoではC言語で言うところのグローバル変数と言うものが作れません。必ずパッケージ単位で分離した名前空間にマップされます。

    キャンセル

  • 2018/05/22 07:03

    とても丁寧な回答ありがとうございます; _ ;
    仰るとおりdefaultをつけないならselectにしなくても問題ありませんね!
    また、チャネルをrangeで回せることも初めて知りました
    とてもわかりやすく勉強になりました。ありがとうございました!

    キャンセル

  • 2018/05/22 20:57

    closeを呼ばなかった場合(正常終了させなかった場合)どのような事態になりますか?

    キャンセル

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

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

関連した質問

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

  • Go

    505questions

    Go(golang)は、Googleで開発されたオープンソースのプログラミング言語です。