Question

我正在使用反射来做到这一点：

        method_name := CommandMap[command]
    thing := reflect.ValueOf(self).MethodByName(method_name)
    thing_s := fmt.Sprintf("%s", thing)
    if thing_s == "<invalid reflect.Value>" {
        self.writeMessage(550, "not allowed")
    } else {
        thing.Call([]reflect.Value{})
    }

但必须有更好的方法。上面代码的要点是摆脱：

if command == "this" {
  runThis()
} else if command == "that" {
  runThat()
} etc. etc.

我希望没有一个大的if else树。这是一个ftp服务器http://github.com/andrewarrow/paradise_ftp，我需要处理15个左右的命令，如“USER”，“PASS”和“QUIT”等。我希望每个人都能调用像“handleUser”或“handlePass”这样的方法”。但使用反射是一个坏主意。这太慢了。

Answer 1

从链接的用例中，看起来有一个字符串映射到函数将最有效。例如：

func MakeCommandMap() map[string]func(*Paradise) {
    m := map[string]func(*Paradise){
        "USER": (*Paradise).HandleUser,
        "PASS": (*Paradise).HandlePass,
    }
    // ...
    return m
}

调用函数：

name := "USER"
if fn := m[name]; fn != nil {
    fn(self) // where self is *Paradise
}

如何在不使用反射的情况下在golang中优雅地调用方法？

1 个答案: