Go语言循环退出异常：输入'y'无法终止循环问题排查

问题根源 #

你的代码存在三个关键问题导致输入"y"无法触发循环退出：

1. 循环判断时机错位：原for循环的条件判断在每次循环开始时执行，但第一次判断时order是空字符串；即使后续输入了"y"，也要等到下一次循环启动才会校验，且如果输入包含其他空白字符，会导致匹配失败。
2. strings.Trim的范围过窄：strings.Trim(order, "\r\n")仅能去除两端的\r和\n，如果输入时不小心带了空格（比如" y"或"y "），就无法匹配目标字符串"y"。
3. 重复创建bufio.Reader：每次循环都新建Reader属于不必要的性能损耗，虽然不影响功能，但不符合Go的最佳实践。

修正后的代码 #

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

type Cuisine struct{}

func (c Cuisine) getOrders() {
    fmt.Println("Enter Orders [Type \"y\" to exit]:")
    // 复用Reader，避免重复初始化
    reader := bufio.NewReader(os.Stdin)
    
    for {
        fmt.Print("Enter Item Name: ")
        order, err := reader.ReadString('\n')
        // 处理输入读取错误
        if err != nil {
            fmt.Println("读取输入出错:", err)
            continue
        }
        // 去除所有空白字符，覆盖换行、空格、制表符等场景
        trimmedOrder := strings.TrimSpace(order)
        if trimmedOrder == "y" {
            fmt.Println("退出订单输入...")
            break
        }
        fmt.Println("收到订单:", trimmedOrder)
    }
}

关键修正说明 #

• 复用bufio.Reader：将Reader初始化移到循环外部，避免重复创建IO资源。
• 使用strings.TrimSpace()：替代原有的strings.Trim()，可以去除字符串两端所有空白字符，覆盖更多输入场景。
• 调整循环逻辑：改用无限循环，读取输入后立即校验退出条件，逻辑更直观，彻底解决判断时机的问题。
• 增加错误处理：不再忽略ReadString返回的错误，提升代码健壮性。

内容的提问来源于stack exchange，提问作者Oshank Kashyap