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")
}