Kadai3-1: nagaa052 by nagaa052 · Pull Request #25 · gopherdojo/dojo5

nagaa052 · 2019-06-17T12:22:45Z

タイピングゲーム

要件

標準出力に英単語を出す（出すものは自由）
標準入力から1行受け取る
制限時間内に何問解けたか表示する

コマンド/オプション

$ tgame -h

tgame is a Typing Game
Usage:
  tgame [option]
Options:
  -c	Print Color
  -t int
    	Timeout Seconds (default 30)

パッケージ構成

├── main.go
└── pkg
    ├── game
    └── questions

main: 起動/コマンドオプション
pkg/game : タイピングゲームのメインロジック
pkg/questions : 問題の管理/取得

やってみたこと

export_test.goを使って、非公開の機能テストも書いてみた
interfaceを使って問題の取得先を切り替えやすくした(つもり)
ExampleTestを書いてみた

できなかったこと

メインロジックのテストが書けなかった
- テストからだと入力と出力の制御が行えず、テスト実行中、入力と出力を繰り替えしてしまう
- 「エラーが発生するか」のテストは書けるが、大量の書き込みが発生してしまうのでやめた

tenntenn · 2019-06-21T09:43:43Z

+	flag.Usage = usage
+	flag.Parse()
+
+	g, err := game.New(game.Options{


基本ポインタにしておく。

tenntenn · 2019-06-21T09:43:55Z

+	}, os.Stdin, os.Stdout, os.Stderr)
+
+	if err != nil {
+		log.Fatal("Failed to start the game")


log.Fatalは使わない

tenntenn · 2019-06-21T09:45:10Z

+)
+
+const (
+	ExitOK = iota


プログラムの外部に出る情報にはiotaを使わない。
例：DBに保存されるもの、ネットワークを介して送られるもの、コマンドライツールの出力になるもの

tenntenn · 2019-06-21T09:45:27Z

+// Game is manages game information
+type Game struct {
+	qs *questions.Questions
+	*Result


埋め込む必要性は？

tenntenn · 2019-06-21T09:45:55Z

+// DefaultOptions is the default value of Options.
+var DefaultOptions = Options{
+	TimeUpSecond: 30,
+	IsColor:      false,


falseなら別に初期化する必要ないのでは？

tenntenn · 2019-06-21T09:46:11Z

+// Options is a specifiable option.
+type Options struct {
+	TimeUpSecond int
+	IsColor      bool


tenntenn · 2019-06-21T09:46:37Z

+
+// Options is a specifiable option.
+type Options struct {
+	TimeUpSecond int


time.Durationを使う

tenntenn · 2019-06-21T09:47:58Z

+}
+
+// Print is Output the result.
+func (r *Result) Print(out io.Writer) {


io.Writerの変数はwと書くことが多い

tenntenn · 2019-06-21T09:48:23Z

+
+	return &Game{
+		qs:        qs,
+		Result:    &Result{},


埋め込むならポインタじゃなくてもいい。そしたらゼロ値で使える。

tenntenn · 2019-06-21T09:49:25Z

+		scanner := bufio.NewScanner(g.inStream)
+		defer close(dst)
+
+		for scanner.Scan() {


エラー処理

tenntenn · 2019-06-21T09:49:58Z

+
+func (g *Game) getQuestion() (*questions.Question, error) {
+
+	rand.Seed(time.Now().UnixNano())


呼び出しの度にSeedをセットしない

tenntenn · 2019-06-21T09:50:06Z

+func (g *Game) getQuestion() (*questions.Question, error) {
+
+	rand.Seed(time.Now().UnixNano())
+	index := rand.Intn(g.qs.GetSize())


同じ問題が出る

tenntenn · 2019-06-21T09:50:58Z

+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {


これはt.Paralellしない？

tenntenn · 2019-06-21T09:51:10Z

+		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
+
+			inStream := &bytes.Buffer{}


ゼロ値でよい

tenntenn · 2019-06-21T09:51:47Z

+			_, err := game.New(tt.opt, inStream, outStream, errStream)
+			if (err != nil) != tt.wantErr {
+				t.Errorf("New() error = %v, wantErr %v", err, tt.wantErr)
+				return


returnする必要性ある？

tenntenn · 2019-06-21T09:52:05Z

+			got, err := game.ExportGetQuestion(g)
+			if (err != nil) != tt.wantErr {
+				t.Errorf("Game.getQuestion() error = %v, wantErr %v", err, tt.wantErr)
+				return


returnするならt.Fatalfを使う

tenntenn · 2019-06-21T09:52:34Z

+
+type inMemQ struct{}
+
+var words = []string{


外部ファイルからもらうようにした方がよいのでは？

あ、なるほど、テスト用の実装？

tenntenn · 2019-06-21T09:53:26Z

+		want    *qs.Questions
+		wantErr bool
+	}{
+		{


テストケースが少なすぎる

tenntenn · 2019-06-21T09:54:06Z

+		want   bool
+	}{
+		{
+			name: "Is correct test",


もっと名前をシンプルにして、パット見で分かるようにする。

nagaa052 added 4 commits June 15, 2019 18:43

Add Makefile

b7067c0

Add pkg questions

b1ef90b

Add typing game

55aceb2

Add Usage

2098fcd

tenntenn added the kadai3-1 label Jun 21, 2019

tenntenn reviewed Jun 21, 2019

View reviewed changes


		func (g Game) getQuestion() (questions.Question, error) {

		rand.Seed(time.Now().UnixNano())

Uh oh!

Conversation

nagaa052 commented Jun 17, 2019

タイピングゲーム

要件

コマンド/オプション

パッケージ構成

やってみたこと

できなかったこと

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tenntenn Jun 21, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

tenntenn Jun 21, 2019 •

edited

Loading