这是基于OTP发布版本的正确代码执行方式吗？
您不应检查OTP版本，而应检查应用程序（在本例中为crypto）版本。因为OTP版本可能与应用程序版本不同。
有没有更好的方法来解决这个问题？
是，检查给定函数是否导出，而不是检查OTP/应用程序的版本。
在运行时，有两种方法可以实现这一点：

def hmac(key, data) do
  if Code.ensure_loaded?(:crypto) and function_exported?(:crypto, :mac, 4) do
    :crypto.mac(:mac, :sha256, key, data)
  else
    :crypto.hmac(:sha256, key, data)
  end
end

或在编译期间：

if Code.ensure_loaded?(:crypto) and function_exported?(:crypto, :mac, 4) do
  def hmac(key, data), do: :crypto.mac(:mac, :sha256, key, data)
else
  def hmac(key, data), do: :crypto.hmac(:sha256, key, data)
end

宏是编译时的产物，编译后没有宏的踪迹;相反，它注入的AST发生在释放中。
也就是说，如果您希望它是编译时的，这意味着您需要许多环境来编译，并且每个版本都只在编译时的同一平台上工作，这是正确的方法。
如果您希望一次组装一个发行版并使其在不同的目标平台上工作，最好使用Kernel.apply/3作为

def hmac(key, data) do
  if String.to_integer(System.otp_release) >= 22,
    do: apply(:crypto, :mac, [:hmac, :sha256, key, data]),
    else: apply(:crypto, :hmac, [:sha256, key, data])
end

• 旁注：* 即使你希望它是编译时的，声明不同的函数比声明一个宏更干净。

if String.to_integer(System.otp_release) >= 22 do
  def hmac(key, data),
    do: :crypto.mac(:hmac, :sha256, key, data)
else
  def hmac(key, data),
    do: :crypto.hmac(:sha256, key, data)
end

这将在编译时进行评估，相应的功能将在生成的BEAM代码中发生。