Go 中 if 语句与标准输入字符串比较的正确处理方式

在 Go 里，如果用 bufio.NewReader.ReadString 读取用户输入，拿到的结果通常会把末尾的换行符 n 一并带上。这样一来，直接拿它和 "hi" 做比较，结果往往对不上。要想判断准确，得先把尾部换行去掉，常见做法包括使用 strings.TrimRight、bufio.Scanner，或者改用 ReadLine 这类方式处理。

在 Go 里用 `bufio.NewReader.ReadString` 读取用户输入时，有个很容易踩到的细节：返回的字符串默认会把换行符 `n` 一并带上，所以拿它直接去和 `"hi"` 做比较，结果通常会判断失败。要让判断准确生效，就得先把末尾的换行处理掉，比如用 `strings.TrimRight`，或者干脆改用 `bufio.Scanner`、`ReadLine` 这类方式，再进行比较。

你最初的代码逻辑清晰，但未能考虑到 Go 标准库中 ReadString('n') 的行为细节：它会将分隔符（即 n）一并包含在返回的字符串中。因此，当用户输入 hi 并按下回车后，实际读到的是 "hin"，而非 "hi"，导致 if text == "hi" 判断恒为 false。

以下是几种推荐的修复方案，兼顾可读性、健壮性与 Go 最佳实践：

✅ 推荐方案：使用 bufio.Scanner（最简洁、最常用）

package main

import (
"bufio"
"fmt"
"os"
)

func main() {
scanner := bufio.NewScanner(os.Stdin)
fmt.Println("Simple Shell")
fmt.Println("---------------------")

for {
fmt.Print("-> ")
if !scanner.Scan() {
break // 处理 EOF 或读取错误
}
text := scanner.Text() // 自动去除换行符，返回纯文本
if text == "hi" {
fmt.Println("hiii")
} else if text == "quit" || text == "exit" {
fmt.Println("Goodbye!")
break
}
}
}

✅ 优势：scanner.Text() 默认按行读取且自动剥离 rn 或 n；内置错误处理（scanner.Err() 可检查）；内存高效，适合交互式输入。

⚠️ 替代方案：手动清理换行符（适用于 ReadString/ReadBytes）

若需保留 bufio.Reader（例如需混合读取不同格式），请显式裁剪：

import "strings"

// ... 在循环内
text, err := reader.ReadString('n')
if err != nil {
fmt.Fprintln(os.Stderr, "读取输入失败:", err)
continue
}
text = strings.TrimSpace(text) // 推荐用 TrimSpace 而非 TrimRight，兼容 rn 和空格
if text == "hi" {
fmt.Println("hiii")
}

? strings.TrimSpace 比 TrimRight(text, "n") 更健壮——它同时移除首尾空白符（包括 r, n, t, 空格），适配 Windows/Linux/macOS 换行差异。

❌ 不推荐：忽略错误与裸用 _

原始代码中 text, _ := reader.ReadString('n') 隐藏了潜在 I/O 错误（如终端关闭、信号中断），应始终检查 err；否则程序可能静默崩溃或陷入死循环。

? 补充说明：ReadLine 的注意事项

虽然 reader.ReadLine() 返回 []byte 且不包含换行符，但它不保证返回完整行（在缓冲区不足时可能返回部分数据并设置 isPrefix=true）。因此，除非明确处理分片逻辑，否则优先选用 Scanner 或 TrimSpace。

✅ 最终建议

• 交互式命令行首选 bufio.Scanner —— 简洁、安全、符合 Go idioms；
• 批量/流式处理大文本时，可结合 Reader 与 strings.TrimSpace；
• 始终检查 I/O 错误，避免 err 被丢弃；
• 使用 == 比较字符串是安全的（Go 字符串不可变且支持字节级相等判断）。

这样，你的 shell 就能真正响应 "hi" 并输出 "hiii" 了。