[问题求助] [已解决]结束进程的VBS代码中for each语句为何会报错呢？

秋风·飞扬

strComputer="."
Set wmi = GetObject("winmgmts:\\" & strComputer & "\root\cimv2")
Set Get_Process=wmi.instancesof("win32_process")


For Each Process In Get_Process
count=count+1
ReDim Preserve Array_Name(count-1)
Array_Name(count-1)=Process.name
Next

Input= InputBox("请输入要结束的进程名称：")

For i=0 To count
ReDim Preserve Array_Name(count)
If Input=Array_Name(i) Then 
Set New_Process = wmi.execquery("select * from win32_process where name ="&  Array_Name(i) )
For Each Over_Process In New_Process
Over_Process.terminate()
Next
Set New_Process = Nothing
Set Over_Process= Nothing
End If 
Next 

Set wmi = Nothing 
Set Get_Process = Nothing 
Set Process =  Nothing 
复制代码

报错是在上面的第18行

代码的目的是，想实现用inputbox输入一个进程的名字，例如输入taskmgr.exe ，然后结束它。

powerbat

因为要报错，所以要报错！^_^
你把具体出错信息贴出来不行吗？
for each x in x
vbs是不区分大小写的！
另外，你对变量的命名挺奇怪的，不是任何一种常见风格，倒像把几种风格糅合起来了，显得不伦不类。

broly

楼主，你想实现什么功能？
你的代码缩进风格要改善。特别if ， for之类的嵌套时，如果你没有良好的缩进风格，要看清楚你的代码很费劲的。

If pro_name=LCase(Arry_Name(i)) Then Set New_Process = wmi.execquery("select * from win32_process where name ="& LCase(Arry_Name(i)))

红字部分，难以理解

秋风·飞扬

回复 2# powerbat


    OK,我以后好好改善一下

秋风·飞扬

回复 3# broly


    恩，我会改进的

秋风·飞扬

回复 3# broly


    嘿嘿 ，那个红字部分就是我输入的值，和原来数组里面的值对比，如果相等那么就进行下一步，我稍微改一下吧

broly

回复 6# 秋风·飞扬


    我理解错误了。我忘记了VBS用=号比较的。我碰到的语言用==比较的情况比较多，就习惯性的以为这里是个赋值语句。囧了

秋风·飞扬

回复 7# broly


    哈哈 ，知识都学杂了

broly

要用数组干嘛？

strComputer = "."
Set objWMIService = GetObject("winmgmts:\\" & strComputer & "\root\cimv2")
set colProcessList = objWMIService.ExecQuery("SELECT * FROM Win32_Process where name='cmd.exe'") 
For Each objProcess in colProcessList
  objProcess.Terminate()
Next
复制代码

秋风·飞扬

回复 9# broly


    数组存放了所有进程的名称，起初有个查询进程的功能，我本来的目的就是我知道进程名字，我就可以在inputbox里面输入，然后结束它，所以我让where name = &  Array_Name(i) 这数组就用到这里面了，也不知道这样的用法对不对

powerbat

请学会在cmd中用cscript.exe运行脚本，以便查看和调试。
WQL中，文本值需要用引号括起来（单引号或双引号皆可，配对使用），数字值可用可不用。

Set New_Process = wmi.execquery("select * from win32_process where name ='" &  Array_Name(i) & "'")
或
Set New_Process = wmi.execquery("select * from win32_process where name ="""&  Array_Name(i) & """")
或
Set New_Process = wmi.execquery("select * from win32_process where name ="""&  Array_Name(i) & chr(34))
复制代码

powerbat

回复 10# 秋风·飞扬


可以，但这样做没什么必要。
如果你想列出特定进程名并显示给用户供其选择，则有点意义。
否则像9楼那样直接查询即可，不需要先获取所有进程列表，wmi不一定比你的数组逐个比较慢，而且如果有新进程产生，你原来获取的列表完全没用。

秋风·飞扬

回复 12# powerbat


    听君一席话，胜读十年书。

秋风·飞扬

回复 11# powerbat


    cscript.exe调试，说实话，还真没有用这个调试的习惯，多谢指点。