use whence constant for Seek() (#746)

quii · Mar 19, 2024 · 22780a6 · 22780a6
1 parent 2746e89
commit 22780a6
Show file tree

Hide file tree

Showing 40 changed files with 91 additions and 69 deletions.
diff --git a/command-line/v1/file_system_store.go b/command-line/v1/file_system_store.go
@@ -3,6 +3,7 @@ package poker
 import (
 	"encoding/json"
 	"fmt"
+	"io"
 	"os"
 	"sort"
 )
@@ -35,7 +36,7 @@ func NewFileSystemPlayerStore(file *os.File) (*FileSystemPlayerStore, error) {
 }
 
 func initialisePlayerDBFile(file *os.File) error {
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 
 	info, err := file.Stat()
 
@@ -45,7 +46,7 @@ func initialisePlayerDBFile(file *os.File) error {
 
 	if info.Size() == 0 {
 		file.Write([]byte("[]"))
-		file.Seek(0, 0)
+		file.Seek(0, io.SeekStart)
 	}
 
 	return nil

diff --git a/command-line/v1/tape.go b/command-line/v1/tape.go
@@ -1,6 +1,7 @@
 package poker
 
 import (
+	"io"
 	"os"
 )
 
@@ -10,6 +11,6 @@ type tape struct {
 
 func (t *tape) Write(p []byte) (n int, err error) {
 	t.file.Truncate(0)
-	t.file.Seek(0, 0)
+	t.file.Seek(0, io.SeekStart)
 	return t.file.Write(p)
 }
diff --git a/command-line/v1/tape_test.go b/command-line/v1/tape_test.go
@@ -13,7 +13,7 @@ func TestTape_Write(t *testing.T) {
 
 	tape.Write([]byte("abc"))
 
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 	newFileContents, _ := io.ReadAll(file)
 
 	got := string(newFileContents)

diff --git a/command-line/v2/file_system_store.go b/command-line/v2/file_system_store.go
@@ -3,6 +3,7 @@ package poker
 import (
 	"encoding/json"
 	"fmt"
+	"io"
 	"os"
 	"sort"
 )
@@ -35,7 +36,7 @@ func NewFileSystemPlayerStore(file *os.File) (*FileSystemPlayerStore, error) {
 }
 
 func initialisePlayerDBFile(file *os.File) error {
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 
 	info, err := file.Stat()
 
@@ -45,7 +46,7 @@ func initialisePlayerDBFile(file *os.File) error {
 
 	if info.Size() == 0 {
 		file.Write([]byte("[]"))
-		file.Seek(0, 0)
+		file.Seek(0, io.SeekStart)
 	}
 
 	return nil

diff --git a/command-line/v2/tape.go b/command-line/v2/tape.go
@@ -1,6 +1,7 @@
 package poker
 
 import (
+	"io"
 	"os"
 )
 
@@ -10,6 +11,6 @@ type tape struct {
 
 func (t *tape) Write(p []byte) (n int, err error) {
 	t.file.Truncate(0)
-	t.file.Seek(0, 0)
+	t.file.Seek(0, io.SeekStart)
 	return t.file.Write(p)
 }
diff --git a/command-line/v2/tape_test.go b/command-line/v2/tape_test.go
@@ -13,7 +13,7 @@ func TestTape_Write(t *testing.T) {
 
 	tape.Write([]byte("abc"))
 
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 	newFileContents, _ := io.ReadAll(file)
 
 	got := string(newFileContents)

diff --git a/command-line/v3/file_system_store.go b/command-line/v3/file_system_store.go
@@ -3,6 +3,7 @@ package poker
 import (
 	"encoding/json"
 	"fmt"
+	"io"
 	"os"
 	"sort"
 )
@@ -57,7 +58,7 @@ func FileSystemPlayerStoreFromFile(path string) (*FileSystemPlayerStore, func(),
 }
 
 func initialisePlayerDBFile(file *os.File) error {
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 
 	info, err := file.Stat()
 
@@ -67,7 +68,7 @@ func initialisePlayerDBFile(file *os.File) error {
 
 	if info.Size() == 0 {
 		file.Write([]byte("[]"))
-		file.Seek(0, 0)
+		file.Seek(0, io.SeekStart)
 	}
 
 	return nil

diff --git a/command-line/v3/tape.go b/command-line/v3/tape.go
@@ -1,6 +1,7 @@
 package poker
 
 import (
+	"io"
 	"os"
 )
 
@@ -10,6 +11,6 @@ type tape struct {
 
 func (t *tape) Write(p []byte) (n int, err error) {
 	t.file.Truncate(0)
-	t.file.Seek(0, 0)
+	t.file.Seek(0, io.SeekStart)
 	return t.file.Write(p)
 }
diff --git a/command-line/v3/tape_test.go b/command-line/v3/tape_test.go
@@ -13,7 +13,7 @@ func TestTape_Write(t *testing.T) {
 
 	tape.Write([]byte("abc"))
 
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 	newFileContents, _ := io.ReadAll(file)
 
 	got := string(newFileContents)

diff --git a/io.md b/io.md
@@ -319,7 +319,7 @@ type FileSystemPlayerStore struct {
 }
 
 func (f *FileSystemPlayerStore) GetLeague() []Player {
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(f.database)
 	return league
 }
@@ -586,7 +586,7 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		}
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(league)
 }
 ```
@@ -646,7 +646,7 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		player.Wins++
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(league)
 }
 ```
@@ -699,7 +699,7 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		league = append(league, Player{name, 1})
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(league)
 }
 ```
@@ -765,7 +765,7 @@ type FileSystemPlayerStore struct {
 }
 
 func NewFileSystemPlayerStore(database io.ReadWriteSeeker) *FileSystemPlayerStore {
-	database.Seek(0, 0)
+	database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(database)
 	return &FileSystemPlayerStore{
 		database: database,
@@ -802,7 +802,7 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		f.league = append(f.league, Player{name, 1})
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(f.league)
 }
 ```
@@ -832,7 +832,7 @@ type tape struct {
 }
 
 func (t *tape) Write(p []byte) (n int, err error) {
-	t.file.Seek(0, 0)
+	t.file.Seek(0, io.SeekStart)
 	return t.file.Write(p)
 }
 ```
@@ -852,7 +852,7 @@ Update the constructor to use `Tape`
 ```go
 //file_system_store.go
 func NewFileSystemPlayerStore(database io.ReadWriteSeeker) *FileSystemPlayerStore {
-	database.Seek(0, 0)
+	database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(database)
 
 	return &FileSystemPlayerStore{
@@ -880,7 +880,7 @@ func TestTape_Write(t *testing.T) {
 
 	tape.Write([]byte("abc"))
 
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 	newFileContents, _ := io.ReadAll(file)
 
 	got := string(newFileContents)
@@ -916,7 +916,7 @@ type tape struct {
 
 func (t *tape) Write(p []byte) (n int, err error) {
 	t.file.Truncate(0)
-	t.file.Seek(0, 0)
+	t.file.Seek(0, io.SeekStart)
 	return t.file.Write(p)
 }
 ```
@@ -941,7 +941,7 @@ type FileSystemPlayerStore struct {
 }
 
 func NewFileSystemPlayerStore(file *os.File) *FileSystemPlayerStore {
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 	league, _ := NewLeague(file)
 
 	return &FileSystemPlayerStore{
@@ -1011,7 +1011,7 @@ Let's make it so our constructor is capable of returning an error.
 ```go
 //file_system_store.go
 func NewFileSystemPlayerStore(file *os.File) (*FileSystemPlayerStore, error) {
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 	league, err := NewLeague(file)
 
 	if err != nil {
@@ -1121,7 +1121,7 @@ Change our constructor to the following
 //file_system_store.go
 func NewFileSystemPlayerStore(file *os.File) (*FileSystemPlayerStore, error) {
 
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 
 	info, err := file.Stat()
 
@@ -1131,7 +1131,7 @@ func NewFileSystemPlayerStore(file *os.File) (*FileSystemPlayerStore, error) {
 
 	if info.Size() == 0 {
 		file.Write([]byte("[]"))
-		file.Seek(0, 0)
+		file.Seek(0, io.SeekStart)
 	}
 
 	league, err := NewLeague(file)
@@ -1156,7 +1156,7 @@ Our constructor is a bit messy now, so let's extract the initialise code into a
 ```go
 //file_system_store.go
 func initialisePlayerDBFile(file *os.File) error {
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 
 	info, err := file.Stat()
 
@@ -1166,7 +1166,7 @@ func initialisePlayerDBFile(file *os.File) error {
 
 	if info.Size() == 0 {
 		file.Write([]byte("[]"))
-		file.Seek(0, 0)
+		file.Seek(0, io.SeekStart)
 	}
 
 	return nil

diff --git a/io/v1/file_system_store.go b/io/v1/file_system_store.go
@@ -11,7 +11,7 @@ type FileSystemPlayerStore struct {
 
 // GetLeague returns the scores of all the players.
 func (f *FileSystemPlayerStore) GetLeague() []Player {
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(f.database)
 	return league
 }
diff --git a/io/v2/file_system_store.go b/io/v2/file_system_store.go
@@ -11,7 +11,7 @@ type FileSystemPlayerStore struct {
 
 // GetLeague returns the scores of all the players.
 func (f *FileSystemPlayerStore) GetLeague() []Player {
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(f.database)
 	return league
 }

diff --git a/io/v3/file_system_store.go b/io/v3/file_system_store.go
@@ -12,7 +12,7 @@ type FileSystemPlayerStore struct {
 
 // GetLeague returns the scores of all the players.
 func (f *FileSystemPlayerStore) GetLeague() League {
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(f.database)
 	return league
 }
@@ -38,6 +38,6 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		player.Wins++
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(league)
 }
diff --git a/io/v4/file_system_store.go b/io/v4/file_system_store.go
@@ -12,7 +12,7 @@ type FileSystemPlayerStore struct {
 
 // GetLeague returns the scores of all the players.
 func (f *FileSystemPlayerStore) GetLeague() League {
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(f.database)
 	return league
 }
@@ -40,6 +40,6 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		league = append(league, Player{name, 1})
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(league)
 }
diff --git a/io/v5/file_system_store.go b/io/v5/file_system_store.go
@@ -12,7 +12,7 @@ type FileSystemPlayerStore struct {
 
 // GetLeague returns the scores of all the players.
 func (f *FileSystemPlayerStore) GetLeague() League {
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(f.database)
 	return league
 }
@@ -40,6 +40,6 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		league = append(league, Player{name, 1})
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(league)
 }
diff --git a/io/v6/file_system_store.go b/io/v6/file_system_store.go
@@ -13,7 +13,7 @@ type FileSystemPlayerStore struct {
 
 // NewFileSystemPlayerStore creates a FileSystemPlayerStore.
 func NewFileSystemPlayerStore(database io.ReadWriteSeeker) *FileSystemPlayerStore {
-	database.Seek(0, 0)
+	database.Seek(0, io.SeekStart)
 	league, _ := NewLeague(database)
 
 	return &FileSystemPlayerStore{
@@ -49,6 +49,6 @@ func (f *FileSystemPlayerStore) RecordWin(name string) {
 		f.league = append(f.league, Player{name, 1})
 	}
 
-	f.database.Seek(0, 0)
+	f.database.Seek(0, io.SeekStart)
 	json.NewEncoder(f.database).Encode(f.league)
 }
diff --git a/io/v7/file_system_store.go b/io/v7/file_system_store.go
@@ -2,6 +2,7 @@ package main
 
 import (
 	"encoding/json"
+	"io"
 	"os"
 )
 
@@ -13,7 +14,7 @@ type FileSystemPlayerStore struct {
 
 // NewFileSystemPlayerStore creates a FileSystemPlayerStore.
 func NewFileSystemPlayerStore(file *os.File) *FileSystemPlayerStore {
-	file.Seek(0, 0)
+	file.Seek(0, io.SeekStart)
 	league, _ := NewLeague(file)
 
 	return &FileSystemPlayerStore{