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

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

ただいまの
回答率

88.32%

Go goroutinで取得した値を配列にしたい

解決済

回答 1

投稿 編集

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

pokerStars

score 45

package controller

import (
    "github.com/gin-gonic/gin"
    "github.com/sclevine/agouti"
    "log"
    "sync"
)

func chrome() *agouti.Page {
    agoutiDriver := agouti.ChromeDriver(
        agouti.ChromeOptions("args", []string{
            "--headless", // headlessモードの指定 サイト読み込み時の起動をなくす
        }),
    )
    agoutiDriver.Start()
    // defer agoutiDriver.Stop()
    page, _ := agoutiDriver.NewPage()
    return page
}

func DisplayAction(c *gin.Context) {

    wg := &sync.WaitGroup{}

    messages := make([]string, 0, 6)

    test := make(chan []string)

    wg.Add(2)

    go func(){
        page := chrome()

        // アイウエオ店
        page.Navigate("https://www.アイウエオ.co.jp/")
        title1, _ := page.Title()//アイウエオ

        y := make([]string, 0, len(messages)+2)
        y = append(y, messages...)
        y = append(y, title1)

        test <- y

        wg.Done()
    }()

    go func() {
        page := chrome()

        // かきくけこ店
        page.Navigate("https://www.かきくけこ.co.jp/")

        title2, _ := page.Title()//かきくけこ

        a := make([]string, 0, len(messages)+2)
        a = append(a, messages...)
        a = append(a, title2)

        test <- a

        wg.Done()
    }()

        wg.Wait()//これを入れるとここで止まってしまう

        msgs := <-test

    c.HTML(200, "hoge.html", gin.H{
        "titles":  msgs,

    })
}
hoge.html

<!DOCTYPE html>
<html lang="ja">
  <head>
    <meta charset="UTF-8" />
  </head>
  <body>
    <ul>
    <!-- {{define "index"}} -->
    <br>
    <h1>{{range .titles}}</h1>
      <p>{{.}}</p>
     {{end}}
     <!-- {{end}} -->
    </ul>
  </body>
</html>


期待する結果
msgs = [{アイウエオ} {かきくけこ}]

参考にした記事

現在配列に値を入れようとしてもtitle1またはtitle2のどちらか片方のみmsgsに値が入ってしまいます。 (例)msgs = [アイウエオ]
そこで&sync.WaitGroup{}を使ってgoroutinの処理が終わってから次の処理にいくことで問題を解決できるのではないかと考えたのですが、
wg.Wait()を入れると止まってしまいます。(ずっとロード中)
&sync.WaitGroup{}の使い方でどこかおかしな点はございますでしょうか?

もしくは他にgoroutinで取得したそれぞれの値(title1,title2)をうまく配列にする方法はございますでしょうか?
お力をお借りしたいです。何卒よろしくお願い致します。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

WaitGroupとtestチャネルがどちらも待ち受けしていてチャネルの動作条件が満たされません。
チャネルはバッファ無しの場合、送信待ちと受信待ちが双方スタンバイになるまでブロックします。

解決方法のひとつはtestチャネルをバッファ付きでサイズを2以上にすることです。
(結果の受け取りは1度ではなく2度にする必要はあります。)

余談ですが、明確な理由なくスライスのキャパシティを指定するのはやめましょう。
(多くのケースで自動拡張に任せるほうが良いことが多いと思います)

追記

UMLのアクティビティ図でいうところの
フォークに相当するのが「go」キーワード、
ジョインに相当するのは「chan」か「sync.WaitGroup」が使えます。
ジョインの目的のために両方を使う必要はありません。

Gopherの書き方

以下の様にすると目的をはたしつつ、WaitGroupも不要です。
(この書き方がgoroutineから戻り値のある場合のジョインの仕方です。)

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

package main

import "fmt"

func main() {

    messages := []string{}
    test := make(chan []string)

    go func() {
        // アイウエオ店
        // page := chrome()
        // page.Navigate("https://www.アイウエオ.co.jp/")
        title1 := "アイウエオ"

        y := []string{}
        // ...
        y = append(y, title1)

        test <- y
    }()

    go func() {
        // かきくけこ店
        // page := chrome()
        // page.Navigate("https://www.かきくけこ.co.jp/")

        title2 := "かきくけこ"

        a := []string{}
        // ...
        a = append(a, title2)

        test <- a
    }()

    messages = append(messages, <-test...)
    messages = append(messages, <-test...)
    fmt.Println("titles:", messages)
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2021/01/14 06:54

    つまり、現状のコードではwg.Done()が呼ばれる事はありません。

    キャンセル

  • 2021/01/15 22:30 編集

    返信遅くなりすみません。
    バッファ付きでサイズを2以上にすることでwg.Wait()で止まってしまう問題を解決できました。
    sync.WaitGroup問題が解決し一歩進めましたのでベストアンサーにさせていただきます!
    しかし、変わらず配列に2つの値を入れることができていないためジョイン含め見直しています...

    ご教授ありがとうございました!誠に感謝致します!

    キャンセル

  • 2021/01/16 22:24

    配列にapendするロジックがないです。testから2度受け取った配列をくっつけてください。

    キャンセル

  • 2021/01/19 16:16

    コメントありがとうございます!
    append(messages, <-test...)この書き方で、次の処理に行かずにブロックしながら配列の追加が行えるのですね!
    とても勉強になりました!!

    chanelはgoroutine間のデータを送受信するもの。
    WaitGroupは「goroutineが処理を終えるまで待ってくれる機能」。
    という認識止まりだったのですが、<- chanelで何か値が入るまで次の処理に行かずにブロックさせることができるんですね!
    sync.Mutexなどと組み合わせる以外ではchanelで書いた方が見やすくなりそうですね!

    キャンセル

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

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

関連した質問

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