Review by tenntenn

buffer.go

@@ -30,15 +30,19 @@ func NewLocation(l, c int) *Location {
 	return &Location{x: l, y: c}
 }
 
+// by tenntenn *os.Fileではなく、io.ReadSeekerの方がいいかも
 func (buf *Buffer) Read(f *os.File) error {
+	// by tenntenn バッファリングはするべきだけど、全部のデータをメモリ上に展開するとでかいファイルをひらけなくなるのでは？エディタを作ったことがないのでなんとも言えないけど
 	b := make([]byte, BUFSIZE)
+	// by tenntenn 全部読み込むならioutil.ReadAllを使ったほうがいい。
 	n, err := f.Read(b)
 	if err != nil {
 		return err
 	}
 	b = b[:n]
 
 	var s []rune
+	// by tenntenn bufio.Scannerを使ったほうが良さそう
 	for _, v := range b {
 		if v == byte(10) { // 改行のバイト
 			s = append(s, rune(v))
@@ -53,6 +57,7 @@ func (buf *Buffer) Read(f *os.File) error {
 }
 
 func (buf *Buffer) open() error {
+	// by tenntenn os.Openなので、新規作成ができない
 	f, err := os.Open(buf.path)
 	if err != nil {
 		return err

cursor.go

@@ -2,6 +2,9 @@ package main
 
 import "github.com/gdamore/tcell"
 
+// by tenntenn もうちょい抽象化したほうが良さそう。tcellに依存し過ぎな気がする。
+// キーコードを使うのではなく、それを対応付けたものを定義したほうがいいかな
+// 例えば、ユーザが好きにキーバインドを変える機能を追加したときに変更点を減らすにはどうすればよいか？
 func (buf *Buffer) CursorMove(key MoveC) {
 	switch key {
 	case tcell.KeyBackspace, tcell.KeyBackspace2, tcell.KeyLeft, 'h':

event.go

@@ -2,8 +2,11 @@ package main
 
 import "github.com/gdamore/tcell"
 
+// by tenntenn これはなんのための型？
 type MoveC interface{}
 
+// by tenntenn よくあるイベント駆動モデルを作ったほうが良さそう。
+// 一箇所にイベントの処理を書くと大変そう。
 func (v *View) EventHandle(ev *tcell.EventKey) error {
 	switch v.mode {
 	case Normal:

init.go

@@ -5,6 +5,7 @@ import (
 	"github.com/gdamore/tcell/encoding"
 )
 
+// by tenntenn 初期化処理だとしても、極力パッケージ変数の初期化はしない
 func initScreen() error {
 	var err error
 	screen, err = tcell.NewScreen()
@@ -22,12 +23,15 @@ func initScreen() error {
 	screen.Clear()
 
 	screenWidth, screenHeight = screen.Size()
+	// by tenntenn この-2はなんだろう？
 	screen.Resize(0, 0, screenWidth, screenHeight-2)
 
 	return nil
 }
 
 func initEvent() {
+	// by tenntenn このゴールーチンを終わらせるための仕組みがない
+	// See: https://qiita.com/tenntenn/items/dd6041d630af7feeec52
 	go func() {
 		for {
 			if screen == nil {
@@ -39,11 +43,11 @@ func initEvent() {
 }
 
 func initBuffer(path string) (*Buffer, error) {
-	buf := new(Buffer)
+	buf := new(Buffer) // by tenntenn var buf Bufferでも使える
 	if err := buf.fileManage(path); err != nil {
 		return nil, err
 	}
 	buf.Cursor = NewLocation(0, 0)
 
-	return buf, nil
+	return buf, nil // by tenntenn var buf Bufferとした場合は&buf
 }

main.go

@@ -7,8 +7,10 @@ import (
 	"github.com/gdamore/tcell"
 )
 
+// by tenntenn 構造体にすることでパッケージ変数を減らせないか考えてみてくdさい
 var (
-	Events                    = make(chan tcell.Event)
+	Events = make(chan tcell.Event)
+	// by tenntenn 変数は明示的に初期化しなくてもゼロ値で初期化されている
 	screen       tcell.Screen = nil
 	screenWidth               = 0
 	screenHeight              = 0
@@ -28,7 +30,7 @@ func run(args []string) int {
 		fmt.Fprintln(os.Stderr, err)
 		return 1
 	}
-	defer screen.Fini()
+	defer screen.Fini() // by tenntenn 略さない方がいい
 
 	initEvent()
 
@@ -44,6 +46,7 @@ func run(args []string) int {
 		return 1
 	}
 
+	// by tenntenn 別の関数にしてreturnで抜けたほうがわかりやすいかも
 loop:
 	for {
 		select {

mode.go

@@ -8,6 +8,7 @@ const (
 	Visual
 )
 
+// by tenntenn 構造体のフィールドにしてテストしやすくしよう
 var (
 	mode Mode
 )