鱼壳，测试命令的行为不起作用

Question

我写了以下文件，但是 它无法正常工作。 如果通过使用Test命令选项的组合而不是...

function touch
  /usr/bin/touch $argv
  set -l ext (echo $argv | grep -Eo "\.(.*)\$" | grep -oE '[a-zA-Z]+')
  set -l fname (echo (pwd)/$argv)
  if not test -e $fname; and test "$ext" = "xml"
      echo "detection xml format"
      cat /home/mizuiro/template/xml/template.xml >> $argv
  end
end

请帮助我，我已经设置了测试命令以运行以运行。

Answer 1

我不明白您的代码应该做什么，您还不清楚要达到的目标或所遇到的错误。

让我们一键浏览一下：

功能触摸

此功能创建一个称为touch的函数。虽然它允许函数具有与触摸之类的知名命令相同的名称，但绝对不建议它基本上是相同的（例如，诸如GREP function之类的东西> grep -color = auto可能还可以）。在您运行touch的任何地方，这都可以尝试进行特殊的XML处理，包括在钓鱼已经运送的功能中。鱼功能并非以任何方式范围范围内或隔离，这使它成为一个坏主意。

我建议您将名称更改为“ mytouch”或“ newxml”之类的东西。

/usr/bin/touch $ argv

这会创建一个或多个文件（或将选项传递到touch程序），或调整其当日和MTIME。

set -l ext (echo $argv | grep -Eo "\.(.*)\$" | grep -oE '[a-zA-Z]+')

据推测，这可以在您给出的文件上获得扩展。如果您传递多个文件，则会断开，因为echo打印了参数分开的参数，但是grep希望它们每行一条。

set -l fname（echo（pwd）/$ argv）

这给文件提供了完整的路径 - 如果您给出了相对文件名。如果您给出一个绝对的文件名，则将打印Gibberish。

例如做touch/tmp/foo当您进入目录/home/rivi时，这将打印/home/rivi // tmp/foo 。

它也将使用多个文件名打破。您可以将其替换为

set -l fname $PWD/$argv
# or
set -l fname (pwd)/$argv

可以处理多个参数的，尽管它仍然会通过绝对路径打破。

我也不知道为什么在这里需要绝对路径，因为您已经为上述touch提供了相对路径。

如果不是测试-E $ fname;并测试“ $ ext” =“ xml”

这两个件事 - 首先，它检查不测试-e $ fname，如果文件不存在，则将是正确的。如果那是真的，它将检查扩展名是否为“ XML”。如果两个测试都是错误的，则不会进入IF块（由于和）。

第一个测试几乎可以保证是错误的，因为您在该文件上运行touch。因此，只有您没有创建该文件或类似内容的权限，这才是真的。

我相信问题是您在检查之前运行touch - 创建文件。

您可能想要类似的东西

if not test -e "$fname"; and test "$ext" = xml
   # do your thing
else
   # not a non-existent xml file, just run touch
   /usr/bin/touch $argv
end

（假设$ fname有意义 - 就像我说的那样，我不明白您为什么在那里使用绝对路径）

这也绝对不准备处理多个参数。