1

我已经采用了很多不同的方式，并查看了很多关于高效 Powershell 脚本的网站，但我仍然可以使用帮助来提高这个脚本的效率……它使用 3-4% 的 CPU。处理器上的 Main Drag 是 MainLogic 函数。它嵌套在我的脚本主体开始处的 While($true) 循环中，并且一直运行...当 MainLogic 函数中的某些参数更改函数中断并且某些 if then 语句在主体中进行测试时剧本。我正在努力学习如何编写更高效的代码......我正在练习成为一名 Lisp 开发人员，我认为关于 Powershell 效率的好课程对我们所有人都有好处。我已经重写了这个脚本 3 次，我仍然需要帮助使它变得更好......我看到大多数 Windows 进程几乎不使用 CPU，它们一直在运行并且是非常聪明的代码。我' 我很想变得那么好。我需要脚本中的所有内容\我必须测试脚本中测试的所有内容。我评论了很多以帮助有人帮助我

我试过这些东西：

  • 我是否在这种情况下使用了正确的构造？我有很多方法...我想我是但是可能有更好的数学方法...我正在学习线性代数来帮助我解决这个问题。
  • 我是否做了太多或不必要的操作？这是我的主要问题....可以使用新想法的帮助
  • 我是否使用了太多的对象？不确定是否有其他方法....我想我使用了最少的对象，但 Getwmiobject 比 Get-Process 更好吗？等等
  • 管道是最好的方法吗？我认为我的流水线很好
  • 是否有另一种可能效果更好的命令模型？我用谷歌搜索但找不到与 Powershell 相关的命令模型
  • 我是否正确地考虑对象？似乎无法在 Google 上找到一个好的 PowerShell 对象课程我已经阅读了 1 1\2 Powershell 书籍，但其中没有类似的代码可以挖掘
  • 我是否使用 -Filter（如果可用）？不适用于 Get-Process
  • 我是否不必要地重新初始化变量？没有重新初始化
  • 我的循环设计是否有效？我认为我以最好的方式设计了它们 - 一个 while($true) 嵌套在另一个 while($true) 中，所以我不会在每个循环中都运行我的整个脚本......我需要 MainLogic 函数中的所有东西（我的循环一直在运行）
  • 我是否在使用 Foreach-Object 时也可以正常工作？不适用
  • 我是否检索一次值，如果以后需要再次保存在变量中？...我这样做
  • 您是否将大型集合传递到管道而不是存储在对象中？...不这样做
  • 我是否在不必要地测试条件？我是吗？

这只是一个大脚本还是我没有看到编码方面的大局......请引用在线资源来帮助我编写更有效的代码，如果你看到我的逻辑中的漏洞。

这是我的脚本：MainLogic 函数是处理器猪和所有需要查看的内容......我添加了其余部分以帮助我的帮助。提前致谢

#this function is ran by itself in a while($true) loop and is the main place in my script where information 
about Firefox.Skype and VPN Processes is gathered.
function global:MainLogic()
{

#Monitor Skype WorkingSet for any changes - If change is detected the change it is processed under the
#"Skype Logic" Section to find out if a Skype call is in progress. 
$SkypeProcess1 = get-process | where-object {$_.ProcessName -eq "skype"}
ss -s 1
$SkypeProcess2 = get-process | where-object {$_.ProcessName -eq "skype"}

$SkypeWsDiff = $SkypeProcess2.workingset - $SkypeProcess1.workingset

###Skype Logic###

#Test Skype Call Status and if changed break from this function in the ###SKype Break### sectiom below and 
#check all parameters in body of script.

$y = $x
wo "`$x `= $x"
wo "`$y `= $y"

#Testing Information gathered in above section to see if Skype call in progress.

if($SkypeWsDiff -gt 8000000){$x = 1}
elseif($SkypeWsDiff -lt -11000000){$x = 2}
wo "Function2 `$x `= $x"
if ($x -eq 1){wo "Skype Telephone Call in Progress"
}
if ($x -eq 2){wo "Skype Telephone Call not in Progress"
}
wo "`$SkypeWsDiff `= $SkypeWsDiff"

###SKype Break###
if($y -ne $x){wo "Skype changed - break"
break}
wo "`$x `= $x"
wo "`$y `= $y"
#Skype Logic End

#Firefox Status gathered below and saved to variable $a is saved to variable $b here and stored to compare later 
#with variable $a to see if Firefox Status changed.

$b = $a
wo "`$a `= $a"
wo "`$b `= $b"


#VPN Status gathered below and saved to variable $c is saved to variable $d here and stored to compare later 
#with variable $c to see if VPN Status changed.

$d = $c
wo "`$c `= $c"
wo "`$d `= $d"

#Firefox and VPN status are saved here to variables for comparison later and above this section in the script.
if(ps firefox -ea 0){$a = 3 ; wo "Firefox is Open"
}
if(-not(ps firefox -ea 0)){$a = 4 ; wo "Firefox is Closed"
}
if(ipconfig | Select-String 'PPP adapter'){$c = 5 ; wo "VPN is Connected"
}
if((ipconfig | Select-String 'PPP adapter') -eq $null){$c = 6 ; wo "VPN is not connected"
}

#This is where the Firefox Variable comparison mentioned above actually happens. If change is detected the script
#breaks from this function and subsequently if\then logic is tested in body of script.
if($b -ne $a){wo "Firefox changed - break"
break}
wo "`$a `= $a"
wo "`$b `= $b"

#This is where the VPN Variable comparison mentioned above actually happens. If change is detected the script
#breaks from this function and subsequently if\then logic is tested in body of script.
if($d -ne $c){wo "VPN changed - break"
ss -s 2
break}
wo "`$c `= $c"
wo "`$d `= $d"

}

#Functions


#If Firefox is Open and VPN is Connected - Disconnect VPN

if ((ps firefox -ea 0) -and (ipconfig | Select-String 'PPP adapter')) {
rasdial "BTGuard VPN" /DISCONNECT
ss -s 5
}

#If Firefox is not Open and VPN is not Connected - Connect VPN
if ((-not(ps firefox -ea 0)) -and ((ipconfig | Select-String 'PPP adapter') -eq $null)) {
rasdial "BTGuard VPN" joe 11223344
}

while($true){

while($true){. MainLogic}
#If Skype call not in progress(variable $x) and Firefox is open and VPN is not connected - do nothing
if($x -eq 2 -and (ps firefox -ea 0) -and (ipconfig | Select-String 'PPP adapter') -eq $null)
{

}
#ElseIf Skype call not in progress(variable $x) and Firefox is not open and VPN is connected - do nothing
elseif($x -eq 2 -and (-not(ps firefox -ea 0)) -and (ipconfig | Select-String 'PPP adapter')){

}
#ElseIf Skype call not in progress(variable $x) and Firefox is open and VPN is connected - Disconnect VPN
elseif($x -eq 2 -and (ps firefox -ea 0) -and (ipconfig | Select-String 'PPP adapter')){

ss -s 1
rasdial "BTGuard VPN" /DISCONNECT
 }
#ElseIf Skype call not in progress(variable $x) and Firefox is not open and VPN is not connected - Connect to VPN
elseif($x -eq 2 -and (-not(ps firefox -ea 0)) -and (ipconfig | Select-String 'PPP adapter') -eq $null){

rasdial "BTGuard VPN" joe 11223344
}
#ElseIf Skype call in progress(variable $x) - Do Nothing
elseif($x -eq 1){
}}
4

1 回答 1

4

3-4% 的 CPU 使用率并不算多。话虽如此，用于Measure-Command查看哪些语句消耗时间，哪些语句不消耗时间。

一般来说，尽量避免不必要的工作。也就是说，一些命令行开关提供内置过滤。为此，管道输出?是浪费的。考虑Get-Process：

Measure-Command -Expression { get-process | ? { $_.processname -eq "iexplore" } }
Measure-Command -Expression { get-process -name "iexplore"  }

在我的系统中，第一个平均需要大约 13 毫秒。第二个大约需要 1.5 毫秒。改进大约十年。

此外，考虑将命令结果存储到变量中，而不是一直重新运行命令。例如，代码中充斥着诸如ipconfig | Select-String. 取而代之的是，将输出存储到变量中并仅在必要时更新变量。

于 2013-05-14T13:24:47.470 回答