golangの標準入力をgoroutineにて動かしたときの挙動について

Question

概要

初めまして。
Golang初心者です。

Gopher道場の課題の「タイピングゲームを作ろう」にて、標準入力を行う際に、挙動がどうなっているかわからないため、どなたかに教えていただきたく、質問させていただきました。

自分が考えたゲームの流れ

ゲーム性としてはおかしいですが、以下の感じで考えました。

名前入力 -> 制限時間入力 -> カウントダウン開始 -> タイピングゲーム

コード

以下ざっくりと現状作ったコードになります。
まだ、上記のタイピングゲームの部分は完成していません。

package main

import (
	"bufio"
	"fmt"
	"io"
	"os"
	"strconv"
	"time"
)

// TypingGameを扱う情報
type GameInfo struct {
	timeLimit time.Duration
	score 		int
	name			string
}

func main() {
	// 初期化
 	g := GameInfo{time.Duration(10 * time.Second), 0, "defalut",}
	stdin := os.Stdin

	// 名前入力
	done := makeDoneChan()
	fmt.Print("名前を入力してね！(何も入力しない場合は defalut になります)\n > ")
	name := input(stdin, done)
	res := <-name
	if res != "" {
		g.name = res
	}
	fmt.Println(g.name, "さんようこそ！")
	// サブルーチンを終了させる
	close(done)

	// タイムリミットの設定
	fmt.Print("タイムリミットを設定してね！(正しい値が入力されない場合は 10秒 になります) \n > ")
	done = makeDoneChan()
	timeLimit := input(stdin, done)
	num := <-timeLimit
	// サブルーチンを終了させる
	close(done)
	if v, err := strconv.Atoi(num); err == nil && v > 0 {
		g.timeLimit = time.Duration(v)
	}
	fmt.Printf("タイムリミットは %v です！\n", g.timeLimit)


	// カウントダウン開始
	countdown()
	
	// ゲーム開始

}
// goroutine終了用のチャネルを返す
func makeDoneChan() (chan struct{}) {
	return make(chan struct{})
}

// 3秒間カウントダウンを行うコード
func countdown() {
	for i := 3; i > 0; i-- {
		fmt.Printf("%d ", i)
		time.Sleep(time.Second)
	}
	fmt.Println("Go!")
}

// func game() {
// 	words, err := questions()
// 	if err != nil {
// 		fmt.Println("エラーが起きたため、ゲームを終了します。")
// 		os.Exit(1)
// 	}

// 	// words
// }

// 自分で作成したtextファイルを読み込む
func questions() ([]string, error) {
	var words []string
	f, err := os.Open("question.txt")
	if err != nil {
		return nil, fmt.Errorf("%v", err)
	}
	s := bufio.NewScanner(f)
	for s.Scan() {
		words = append(words, s.Text())
	}
	return words, nil
}

func input(r io.Reader, done chan struct{}) <-chan string {
	ch := make(chan string)
	go func() {
		s := bufio.NewScanner(r)
		defer close(ch)
		for s.Scan() {
			select {
			case <-done:
				return
			case ch <- s.Text():
			}
		}
	}()
	return ch
}

実行結果とわからない箇所

パターン1と2から、タイムリミット入力時には一度改行しないと、欲しい入力結果を得ることができません。
なぜ、このようなことが起こるのかを教えていただければ幸いです。
よろしくお願い致します。

パターン1

名前を入力してね！(何も入力しない場合は defalut になります)
 > nanashi
nanashi さんようこそ！
タイムリミットを設定してね！(正しい値が入力されない場合は 10秒 になります) 
 > 20

タイムリミットは 10s です！
3 2 1 Go!

パターン2

名前を入力してね！(何も入力しない場合は defalut になります)
 > nanashi
nanashi さんようこそ！
タイムリミットを設定してね！(正しい値が入力されない場合は 10秒 になります) 
 > 
20
タイムリミットは 20s です！
3 2 1 Go!

最後に

go始めたばかりなんで、かなりコードが汚いと思いますが、その部分は見逃していただきたいです。
また、きたないと思う部分も同時に指摘していただけると助かります。

Comments

[melian]

> なぜ、このようなことが起こるのか
input 関数内にある goroutine で、
case ch <- s.Text():
としていますが、この直後に for ループに戻って s.Scan() が実行されるので入力待ちになっています。
そして、その時点で何かを入力すると done が受信されて channel が閉じます(goroutine が終了)。
この場合、読み込むのは一行なので channel に入力内容を送信したら return する必要があります(goroutine を終了させる)。

case ch <- s.Text():
return

[Elmogawa]

s.Scanの挙動がわからなかったのですが、melianさんのコメントを見て、スッキリしました！
ありがとうございます！
解決できたので、ぜひベストアンサーに選ばせてください！

[melian]

ありがとうございます。回答に転記しました。

Answer 1

※ コメント欄より転記

なぜ、このようなことが起こるのか

input 関数の goroutine 内で、

  case ch <- s.Text():

としていますが、この直後に for ループに戻って s.Scan() が実行されるので入力待ちになっています。そして、その時点で何かを入力すると select が実行されて done が受信されます。ここで channel が閉じます(goroutine が終了)。
この場合、読み込むのは一行なので channel に入力内容を送信したら return する必要があります(goroutine を終了させる)。

  case ch <- s.Text():
      return

※ 実を言いますと、この場合の処理では goroutine も channel も必要ないのではないかな〜と思います

Comments

[Elmogawa]

ご回答・ご指摘ありがとうございます！
無理矢理goroutineを使っていましたが、順に処理しているだけなので、確かにいらないですね。。