innerFile.WriteString("sup") should be preferred to the innerFile.Write([]byte("sup"))
35 innerFile, err := os.Create(filepath.Join(dir, "outer-dir", "inner-dir", "some-file"))
36 Expect(err).NotTo(HaveOccurred())
37
38 _, err = innerFile.Write([]byte("sup")) 39 Expect(err).NotTo(HaveOccurred())
40
41 err = os.Symlink("some-file", filepath.Join(dir, "outer-dir", "inner-dir", "some-symlink"))innerFile.WriteString("sup") should be preferred to the innerFile.Write([]byte("sup"))
35 Expect(err).NotTo(HaveOccurred())
36 defer innerFile.Close()
37
38 _, err = innerFile.Write([]byte("sup")) 39 Expect(err).NotTo(HaveOccurred())
40
41 err = os.Symlink("some-file", filepath.Join(dir, "outer-dir", "inner-dir", "some-symlink"))
Description
It is recommended to use (io.StringWriter).WriteString instead of
(io.StringWriter).Write or io.WriteString for writing strings as it decreases
the number of allocations required, therefore improving performance.
Bad practice
package main
import (
"io"
)
func foo(w io.StringWriter) {
w.Write([]byte("foo"))
io.WriteString(w, "bar")
}
Recommended
package main
import (
"io"
)
func foo(w io.StringWriter) {
w.WriteString("foo")
w.WriteString("bar")
}