関数とfmt.Printlnの出力を結合して出力する方法

Question

package main

import (
	"fmt"
)

func main() {
	fuga()
}

func fuga(){
    hoge1()
    hoge2()
}

func hoge1(){
    common()
	fmt.Println("hoge1")
}

func hoge2(){
    common()
	fmt.Println("hoge2")
}

func common(){
    fmt.Println("common")
}

fuga()を並列で繰り返し動作させていると
common
hoge1
common
hoge2
ではなく
common
common
hoge1
hoge2
のようになることもあるのですが、ならないように
common() + fmt.Println("hoge1")と言った感じにくっつける事はできないのでしょうか？

Answer 1

fugaの内容をMutexで再入不可能にするとfugaの処理がシーケンシャルになるので
ご希望のように出力が入り混じることはなくなります。
ただし、fugaが並列に実行されることはなくなりますのでgoroutineに載せることによる効率化はありません。

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

package main

import (
	"fmt"
	"sync"
)

func main() {
	ch := make(chan bool, 10)
	for i := 0; i < 100; i++ {
		ch <- true
		go func() {
			fuga()
			<-ch
		}()
	}
}

var mu sync.Mutex

func fuga() {
	mu.Lock()
	defer mu.Unlock()
	hoge1()
	hoge2()
}

func hoge1() {
	common()
	fmt.Println("hoge1")
}

func hoge2() {
	common()
	fmt.Println("hoge2")
}

func common() {
	fmt.Println("common")
}

大切なのは出力がcommonが決めた内容とhoge#が決めた内容が「一塊」であるということ。
実装例のひとつは以下の通り。
messageQueueには文字列スライスを投入可能にして、

• common決定した文字列とhoge1が決定した文字列をセットでキューに投げ込む
• common決定した文字列とhoge2が決定した文字列をセットでキューに投げ込む

別のgoroutineでmessageQueueの内容をひたすらfmt.Printする。
https://play.golang.org/p/DyFpZfb1YLb

package main

import (
	"fmt"
)

func main() {
	messageQueue := make(chan []string, 10)
	go func() {
		for v := range messageQueue {
			for _, s := range v {
			fmt.Println(s)
			}
		}
	}()
	ch := make(chan bool, 10)
	for i := 0; i < 100; i++ {
		ch <- true
		go func() {
			fuga(messageQueue)
			<-ch
		}()
	}
}

func fuga(msg chan<- []string) {
	hoge1(msg)
	hoge2(msg)
}

func hoge1(msg chan<- []string) {
	msg <- []string{common(), "hoge1"}
}

func hoge2(msg chan<- []string) {
	msg <- []string{common(), "hoge2"}
}

func common() string {
	return "common"
}

Comments

[Palo_Punte]

なるほど！
スライスでまとめてからプリントをすれば良かったのですね
少し手間にはなるのですけど
ご回答ありがとうございます