Làm sao ẩn cửa sổ Command Prompt khi thực thi lệnh DOS trong VBA

Liên hệ QC

ndu96081631

Huyền thoại GPE
Thành viên BQT
Super Moderator
Tham gia
5/6/08
Bài viết
30,703
Được thích
53,952
Để đọc kết quả của lệnh DOS, tôi dùng StdOut properties
Ví dụ: Để đọc kết quả của lệnh IPCONFIG, tôi viết code VBA như sau:
PHP:
Sub Test()
  With CreateObject("WScript.Shell")
    MsgBox .Exec("IPCONFIG").StdOut.ReadAll
  End With
End Sub
Vấn đề ở đây là khi chạy code, 1 cửa sổ Command Prompt sẽ xuất hiện (rồi biến mất) khiến cho ta có cảm giác hơi khó chịu
Xin hỏi các cao thủ: Với cách dùng code như trên, có cách nào ẩn được cửa sổ Command Prompt hay không?
Cảm ơn!
 
Để đọc kết quả của lệnh DOS, tôi dùng StdOut properties
Ví dụ: Để đọc kết quả của lệnh IPCONFIG, tôi viết code VBA như sau:
PHP:
Sub Test()
  With CreateObject("WScript.Shell")
    MsgBox .Exec("IPCONFIG").StdOut.ReadAll
  End With
End Sub
Vấn đề ở đây là khi chạy code, 1 cửa sổ Command Prompt sẽ xuất hiện (rồi biến mất) khiến cho ta có cảm giác hơi khó chịu
Xin hỏi các cao thủ: Với cách dùng code như trên, có cách nào ẩn được cửa sổ Command Prompt hay không?
Cảm ơn!

Dưới đây là code viết trên Delphi, các hàm đều là API nên có thể dùng trên VB/VBA.
Mã:
function GetDosOutput(CommandLine: string; Work: string = 'C:\'): string;
var
  SA: TSecurityAttributes;
  SI: TStartupInfo;
  PI: TProcessInformation;
  StdOutPipeRead, StdOutPipeWrite: THandle;
  WasOK: Boolean;
  Buffer: array[0..255] of AnsiChar;
  BytesRead: Cardinal;
  WorkDir: string;
  Handle: Boolean;
begin
  Result := '';
  with SA do begin
    nLength := SizeOf(SA);
    bInheritHandle := True;
    lpSecurityDescriptor := nil;
  end;
  CreatePipe(StdOutPipeRead, StdOutPipeWrite, @SA, 0);
  try
    with SI do
    begin
      FillChar(SI, SizeOf(SI), 0);
      cb := SizeOf(SI);
      dwFlags := STARTF_USESHOWWINDOW or STARTF_USESTDHANDLES;
      wShowWindow := SW_HIDE;
      hStdInput := GetStdHandle(STD_INPUT_HANDLE); // don't redirect stdin
      hStdOutput := StdOutPipeWrite;
      hStdError := StdOutPipeWrite;
    end;
    WorkDir := Work;
    Handle := CreateProcess(nil, PChar('cmd.exe /C ' + CommandLine),
                            nil, nil, True, 0, nil,
                            PChar(WorkDir), SI, PI);
    CloseHandle(StdOutPipeWrite);
    if Handle then
      try
        repeat
          WasOK := ReadFile(StdOutPipeRead, Buffer, 255, BytesRead, nil);
          if BytesRead > 0 then
          begin
            Buffer[BytesRead] := #0;
            Result := Result + Buffer;
          end;
        until not WasOK or (BytesRead = 0);
        WaitForSingleObject(PI.hProcess, INFINITE);
      finally
        CloseHandle(PI.hThread);
        CloseHandle(PI.hProcess);
      end;
  finally
    CloseHandle(StdOutPipeRead);
  end;
end;
 
Upvote 0
Để đọc kết quả của lệnh DOS, tôi dùng StdOut properties
Ví dụ: Để đọc kết quả của lệnh IPCONFIG, tôi viết code VBA như sau:
PHP:
Sub Test()
  With CreateObject("WScript.Shell")
    MsgBox .Exec("IPCONFIG").StdOut.ReadAll
  End With
End Sub
Vấn đề ở đây là khi chạy code, 1 cửa sổ Command Prompt sẽ xuất hiện (rồi biến mất) khiến cho ta có cảm giác hơi khó chịu
Xin hỏi các cao thủ: Với cách dùng code như trên, có cách nào ẩn được cửa sổ Command Prompt hay không?
Cảm ơn!

Dưới đây là code viết trên Delphi, các hàm đều là API nên có thể dùng trên VB/VBA.
Mã:
function GetDosOutput(CommandLine: string; Work: string = 'C:\'): string;
var
  SA: TSecurityAttributes;
  SI: TStartupInfo;
  PI: TProcessInformation;
  StdOutPipeRead, StdOutPipeWrite: THandle;
  WasOK: Boolean;
  Buffer: array[0..255] of AnsiChar;
  BytesRead: Cardinal;
  WorkDir: string;
  Handle: Boolean;
begin
  Result := '';
  with SA do begin
    nLength := SizeOf(SA);
    bInheritHandle := True;
    lpSecurityDescriptor := nil;
  end;
  CreatePipe(StdOutPipeRead, StdOutPipeWrite, @SA, 0);
  try
    with SI do
    begin
      FillChar(SI, SizeOf(SI), 0);
      cb := SizeOf(SI);
      dwFlags := STARTF_USESHOWWINDOW or STARTF_USESTDHANDLES;
      wShowWindow := SW_HIDE;
      hStdInput := GetStdHandle(STD_INPUT_HANDLE); // don't redirect stdin
      hStdOutput := StdOutPipeWrite;
      hStdError := StdOutPipeWrite;
    end;
    WorkDir := Work;
    Handle := CreateProcess(nil, PChar('cmd.exe /C ' + CommandLine),
                            nil, nil, True, 0, nil,
                            PChar(WorkDir), SI, PI);
    CloseHandle(StdOutPipeWrite);
    if Handle then
      try
        repeat
          WasOK := ReadFile(StdOutPipeRead, Buffer, 255, BytesRead, nil);
          if BytesRead > 0 then
          begin
            Buffer[BytesRead] := #0;
            Result := Result + Buffer;
          end;
        until not WasOK or (BytesRead = 0);
        WaitForSingleObject(PI.hProcess, INFINITE);
      finally
        CloseHandle(PI.hThread);
        CloseHandle(PI.hProcess);
      end;
  finally
    CloseHandle(StdOutPipeRead);
  end;
end;
 
Upvote 0
Tuân ơi, cuối là là đưa code vào VBA như thế nào? Mong bạn giúp cho trót!
 
Upvote 0
Để đọc kết quả của lệnh DOS...
Code anh em ruột với VBA: (nguồn http://allapi.mentalis.org/apilist/DBE61037E261CDE165E2120438BB65CE.html

PHP:
'Redirects output from console program to textbox.
'Requires two textboxes and one command button.
'Set MultiLine property of Text2 to true.
'
'Original bcx version of this program was made by
' dl
'VB port was made by Jernej Simoncic
'Visit Jernejs site at http://www2.arnes.si/~sopjsimo/
'
'Note: don't run plain DOS programs with this example
'under Windows 95,98 and ME, as the program freezes when
'execution of program is finnished.

Option Explicit
Private Declare Function CreatePipe Lib "kernel32" (phReadPipe As Long, phWritePipe As Long, lpPipeAttributes As SECURITY_ATTRIBUTES, ByVal nSize As Long) As Long
Private Declare Sub GetStartupInfo Lib "kernel32" Alias "GetStartupInfoA" (lpStartupInfo As STARTUPINFO)
Private Declare Function CreateProcess Lib "kernel32" Alias "CreateProcessA" (ByVal lpApplicationName As String, ByVal lpCommandLine As String, lpProcessAttributes As Any, lpThreadAttributes As Any, ByVal bInheritHandles As Long, ByVal dwCreationFlags As Long, lpEnvironment As Any, ByVal lpCurrentDriectory As String, lpStartupInfo As STARTUPINFO, lpProcessInformation As PROCESS_INFORMATION) As Long
Private Declare Function SetWindowText Lib "user32" Alias "SetWindowTextA" (ByVal hwnd As Long, ByVal lpString As String) As Long
Private Declare Function ReadFile Lib "kernel32" (ByVal hFile As Long, lpBuffer As Any, ByVal nNumberOfBytesToRead As Long, lpNumberOfBytesRead As Long, lpOverlapped As Any) As Long
Private Declare Function SendMessage Lib "user32" Alias "SendMessageA" (ByVal hwnd As Long, ByVal wMsg As Long, ByVal wParam As Long, lParam As Any) As Long
Private Declare Function CloseHandle Lib "kernel32" (ByVal hObject As Long) As Long

Private Type SECURITY_ATTRIBUTES
  nLength As Long
  lpSecurityDescriptor As Long
  bInheritHandle As Long
End Type

Private Type PROCESS_INFORMATION
  hProcess As Long
  hThread As Long
  dwProcessId As Long
  dwThreadId As Long
End Type

Private Type STARTUPINFO
  cb As Long
  lpReserved As Long
  lpDesktop As Long
  lpTitle As Long
  dwX As Long
  dwY As Long
  dwXSize As Long
  dwYSize As Long
  dwXCountChars As Long
  dwYCountChars As Long
  dwFillAttribute As Long
  dwFlags As Long
  wShowWindow As Integer
  cbReserved2 As Integer
  lpReserved2 As Byte
  hStdInput As Long
  hStdOutput As Long
  hStdError As Long
End Type

Private Type OVERLAPPED
    ternal As Long
    ternalHigh As Long
    offset As Long
    OffsetHigh As Long
    hEvent As Long
End Type

Private Const STARTF_USESHOWWINDOW = &H1
Private Const STARTF_USESTDHANDLES = &H100
Private Const SW_HIDE = 0
Private Const EM_SETSEL = &HB1
Private Const EM_REPLACESEL = &HC2

Private Sub Command1_Click()
  Command1.Enabled = False
  Redirect Text1.Text, Text2
  Command1.Enabled = True
End Sub
Private Sub Form_Load()
    Text1.Text = "ping"
End Sub
Private Sub Form_QueryUnload(Cancel As Integer, UnloadMode As Integer)
  If Command1.Enabled = False Then Cancel = True
End Sub

Sub Redirect(cmdLine As String, objTarget As Object)
  Dim i%, t$
  Dim pa As SECURITY_ATTRIBUTES
  Dim pra As SECURITY_ATTRIBUTES
  Dim tra As SECURITY_ATTRIBUTES
  Dim pi As PROCESS_INFORMATION
  Dim sui As STARTUPINFO
  Dim hRead As Long
  Dim hWrite As Long
  Dim bRead As Long
  Dim lpBuffer(1024) As Byte
  pa.nLength = Len(pa)
  pa.lpSecurityDescriptor = 0
  pa.bInheritHandle = True
 
  pra.nLength = Len(pra)
  tra.nLength = Len(tra)

  If CreatePipe(hRead, hWrite, pa, 0) <> 0 Then
    sui.cb = Len(sui)
    GetStartupInfo sui
    sui.hStdOutput = hWrite
    sui.hStdError = hWrite
    sui.dwFlags = STARTF_USESHOWWINDOW Or STARTF_USESTDHANDLES
    sui.wShowWindow = SW_HIDE
    If CreateProcess(vbNullString, cmdLine, pra, tra, True, 0, Null, vbNullString, sui, pi) <> 0 Then
      SetWindowText objTarget.hwnd, ""
      Do
        Erase lpBuffer()
        If ReadFile(hRead, lpBuffer(0), 1023, bRead, ByVal 0&) Then
          SendMessage objTarget.hwnd, EM_SETSEL, -1, 0
          SendMessage objTarget.hwnd, EM_REPLACESEL, False, lpBuffer(0)
          DoEvents
        Else
          CloseHandle pi.hThread
          CloseHandle pi.hProcess
          Exit Do
        End If
        CloseHandle hWrite
      Loop
      CloseHandle hRead
    End If
  End If
End Sub
 
Upvote 0
Code anh em ruột với VBA: (nguồn http://allapi.mentalis.org/apilist/DBE61037E261CDE165E2120438BB65CE.html

PHP:
'Redirects output from console program to textbox.
'Requires two textboxes and one command button.
'Set MultiLine property of Text2 to true.
'
'Original bcx version of this program was made by
' dl
'VB port was made by Jernej Simoncic
'Visit Jernejs site at http://www2.arnes.si/~sopjsimo/
'
'Note: don't run plain DOS programs with this example
'under Windows 95,98 and ME, as the program freezes when
'execution of program is finnished.

Option Explicit
Private Declare Function CreatePipe Lib "kernel32" (phReadPipe As Long, phWritePipe As Long, lpPipeAttributes As SECURITY_ATTRIBUTES, ByVal nSize As Long) As Long
Private Declare Sub GetStartupInfo Lib "kernel32" Alias "GetStartupInfoA" (lpStartupInfo As STARTUPINFO)
Private Declare Function CreateProcess Lib "kernel32" Alias "CreateProcessA" (ByVal lpApplicationName As String, ByVal lpCommandLine As String, lpProcessAttributes As Any, lpThreadAttributes As Any, ByVal bInheritHandles As Long, ByVal dwCreationFlags As Long, lpEnvironment As Any, ByVal lpCurrentDriectory As String, lpStartupInfo As STARTUPINFO, lpProcessInformation As PROCESS_INFORMATION) As Long
Private Declare Function SetWindowText Lib "user32" Alias "SetWindowTextA" (ByVal hwnd As Long, ByVal lpString As String) As Long
Private Declare Function ReadFile Lib "kernel32" (ByVal hFile As Long, lpBuffer As Any, ByVal nNumberOfBytesToRead As Long, lpNumberOfBytesRead As Long, lpOverlapped As Any) As Long
Private Declare Function SendMessage Lib "user32" Alias "SendMessageA" (ByVal hwnd As Long, ByVal wMsg As Long, ByVal wParam As Long, lParam As Any) As Long
Private Declare Function CloseHandle Lib "kernel32" (ByVal hObject As Long) As Long

Private Type SECURITY_ATTRIBUTES
  nLength As Long
  lpSecurityDescriptor As Long
  bInheritHandle As Long
End Type

Private Type PROCESS_INFORMATION
  hProcess As Long
  hThread As Long
  dwProcessId As Long
  dwThreadId As Long
End Type

Private Type STARTUPINFO
  cb As Long
  lpReserved As Long
  lpDesktop As Long
  lpTitle As Long
  dwX As Long
  dwY As Long
  dwXSize As Long
  dwYSize As Long
  dwXCountChars As Long
  dwYCountChars As Long
  dwFillAttribute As Long
  dwFlags As Long
  wShowWindow As Integer
  cbReserved2 As Integer
  lpReserved2 As Byte
  hStdInput As Long
  hStdOutput As Long
  hStdError As Long
End Type

Private Type OVERLAPPED
    ternal As Long
    ternalHigh As Long
    offset As Long
    OffsetHigh As Long
    hEvent As Long
End Type

Private Const STARTF_USESHOWWINDOW = &H1
Private Const STARTF_USESTDHANDLES = &H100
Private Const SW_HIDE = 0
Private Const EM_SETSEL = &HB1
Private Const EM_REPLACESEL = &HC2

Private Sub Command1_Click()
  Command1.Enabled = False
  Redirect Text1.Text, Text2
  Command1.Enabled = True
End Sub
Private Sub Form_Load()
    Text1.Text = "ping"
End Sub
Private Sub Form_QueryUnload(Cancel As Integer, UnloadMode As Integer)
  If Command1.Enabled = False Then Cancel = True
End Sub

Sub Redirect(cmdLine As String, objTarget As Object)
  Dim i%, t$
  Dim pa As SECURITY_ATTRIBUTES
  Dim pra As SECURITY_ATTRIBUTES
  Dim tra As SECURITY_ATTRIBUTES
  Dim pi As PROCESS_INFORMATION
  Dim sui As STARTUPINFO
  Dim hRead As Long
  Dim hWrite As Long
  Dim bRead As Long
  Dim lpBuffer(1024) As Byte
  pa.nLength = Len(pa)
  pa.lpSecurityDescriptor = 0
  pa.bInheritHandle = True
 
  pra.nLength = Len(pra)
  tra.nLength = Len(tra)

  If CreatePipe(hRead, hWrite, pa, 0) <> 0 Then
    sui.cb = Len(sui)
    GetStartupInfo sui
    sui.hStdOutput = hWrite
    sui.hStdError = hWrite
    sui.dwFlags = STARTF_USESHOWWINDOW Or STARTF_USESTDHANDLES
    sui.wShowWindow = SW_HIDE
    If CreateProcess(vbNullString, cmdLine, pra, tra, True, 0, Null, vbNullString, sui, pi) <> 0 Then
      SetWindowText objTarget.hwnd, ""
      Do
        Erase lpBuffer()
        If ReadFile(hRead, lpBuffer(0), 1023, bRead, ByVal 0&) Then
          SendMessage objTarget.hwnd, EM_SETSEL, -1, 0
          SendMessage objTarget.hwnd, EM_REPLACESEL, False, lpBuffer(0)
          DoEvents
        Else
          CloseHandle pi.hThread
          CloseHandle pi.hProcess
          Exit Do
        End If
        CloseHandle hWrite
      Loop
      CloseHandle hRead
    End If
  End If
End Sub

Mình mới thử trên máy chạy Windows Vista thì không chạy. Đang cố tìm nguyên nhân nhưng chưa được.
 
Upvote 0
Upvote 0
Mình mới thử trên máy chạy Windows Vista thì không chạy. Đang cố tìm nguyên nhân nhưng chưa được.

API có nhược điểm đấy mà anh.

@ndu96081631: Chuyển sang VBA cũng không khó lắm đâu. Bỏ SetWindowText... và bỏ 2 cái SendMessage... đi, thay vào đó là xử lý mảng byte lpBuffer sang string thôi.
 
Upvote 0
Tôi thấy nếu vấn đề quá phức tạp như vậy thì có thể chuyển qua 1 hướng khác đơn giản hơn như sau, vẫn đạt được mục đích, tuy nhiên tuỳ từng trường hợp cụ thể có thể phải xử lý thêm.
Mã:
Declare Sub Sleep Lib "kernel32" (ByVal dwMilliseconds As Long)

Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
    Sleep 500
    MsgBox GetText("C:\_Result.txt")
End Sub

Function GetText(sFile As String) As String
   Dim nSourceFile As Integer, sText As String
   Close
   nSourceFile = FreeFile
   Open sFile For Input As #nSourceFile
   sText = Input$(LOF(1), 1)
   Close
   GetText = sText
End Function
 
Upvote 0
PHP:
Sub Test()
  With CreateObject("WScript.Shell")
    MsgBox .Exec("IPCONFIG").StdOut.ReadAll
  End With
End Sub
Anh NDU ơi, làm sao sau Msgbox em châm một cái không thấy hiện list gì nhỉ? (hiện list ví dụ để ta chọn exec ý

Cám ơn Anh
 
Upvote 0
Tôi thấy nếu vấn đề quá phức tạp như vậy thì có thể chuyển qua 1 hướng khác đơn giản hơn như sau, vẫn đạt được mục đích, tuy nhiên tuỳ từng trường hợp cụ thể có thể phải xử lý thêm.
Mã:
Declare Sub Sleep Lib "kernel32" (ByVal dwMilliseconds As Long)

Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
    Sleep 500
    MsgBox GetText("C:\_Result.txt")
End Sub

Function GetText(sFile As String) As String
   Dim nSourceFile As Integer, sText As String
   Close
   nSourceFile = FreeFile
   Open sFile For Input As #nSourceFile
   sText = Input$(LOF(1), 1)
   Close
   GetText = sText
End Function
Woa!... VÔ ĐỊCH
Ẹc.. Ẹc.. sao mình không nghĩ ra vụ lưu thành textfile nhỉ?
Đúng là rollover79 luôn có những giải pháp đơn giản và thú vị
Cảm ơn bạn!
-------------------------------
PHP:
Sub Test()
  With CreateObject("WScript.Shell")
    MsgBox .Exec("IPCONFIG").StdOut.ReadAll
  End With
End Sub
Anh NDU ơi, làm sao sau Msgbox em châm một cái không thấy hiện list gì nhỉ? (hiện list ví dụ để ta chọn exec ý

Cám ơn Anh
Dùng kiểu CreateObject(....) nó có hạn chế vậy đấy
Nếu bạn làm theo kiểu này:
- Vào menu Tools\Preferences và check vào mục Windows Script Host Object Model
- Gõ code theo cú pháp sau:
PHP:
With New WshShell
  .....
End With
- Ở giữa đoạn With.. End With này, bạn cứ.. chấm 1 cái xem
Nói tóm lại:
- Dùng CreateObject(....) có cái bất tiện về việc hổ trợ các gợi ý, nhưng được cái là ta chẳng cần phải thêm các thành phần trong Preferences
- Dùng CreateObject(....) thì gần như mang đến máy nào cũng dùng được (khỏi phải nhắc người dùng đăng ký thành phần trong Preferences)
vân vân...
 
Lần chỉnh sửa cuối:
Upvote 0
Tôi thấy nếu vấn đề quá phức tạp như vậy thì có thể chuyển qua 1 hướng khác đơn giản hơn như sau, vẫn đạt được mục đích, tuy nhiên tuỳ từng trường hợp cụ thể có thể phải xử lý thêm.
Mã:
Declare Sub Sleep Lib "kernel32" (ByVal dwMilliseconds As Long)

Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
    Sleep 500
    MsgBox GetText("C:\_Result.txt")
End Sub

Function GetText(sFile As String) As String
   Dim nSourceFile As Integer, sText As String
   Close
   nSourceFile = FreeFile
   Open sFile For Input As #nSourceFile
   sText = Input$(LOF(1), 1)
   Close
   GetText = sText
End Function
Tôi biết rằng bạn rollover79 dùng Sleep ở đây với ý đồ "chờ" cho file _Result.txt được tạo xong rồi mới tiến hành lấy kết quả
Ấy thế mà tôi sửa code thành:
PHP:
Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
Stp1:
    If Len(Dir("C:\_Result.txt")) = 0 Then GoTo Stp1
    MsgBox GetText("C:\_Result.txt")
End Sub
Với mục đích bỏ hàm Sleep
Nhìn vào code thấy rất hợp lý, ấy thế mà nó lại không ra kết quả
Sao thế nhỉ?
 
Upvote 0
Tôi biết rằng bạn rollover79 dùng Sleep ở đây với ý đồ "chờ" cho file _Result.txt được tạo xong rồi mới tiến hành lấy kết quả
Ấy thế mà tôi sửa code thành:
Mã:
Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
Stp1:
    If Len(Dir("C:\_Result.txt")) = 0 Then GoTo Stp1
    MsgBox GetText("C:\_Result.txt")
End Sub
Với mục đích bỏ hàm Sleep
Nhìn vào code thấy rất hợp lý, ấy thế mà nó lại không ra kết quả
Sao thế nhỉ?

Tạo file nhưng còn phải có thời gian ghi dữ liệu nữa mà.

Mã:
Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
Stp1:
    If (Len(Dir("C:\_Result.txt")) = 0)[COLOR="Blue"] Or (FileLen("C:\_Result.txt") = 0) [/COLOR]Then GoTo Stp1
    MsgBox GetText("C:\_Result.txt")
End Sub

Cần dùng thêm FileLen để kiểm tra dữ liệu đã được ghi vào file hay chưa.
 
Upvote 0
Tạo file nhưng còn phải có thời gian ghi dữ liệu nữa mà.

Mã:
Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
Stp1:
    If (Len(Dir("C:\_Result.txt")) = 0)[COLOR=Blue] Or (FileLen("C:\_Result.txt") = 0) [/COLOR]Then GoTo Stp1
    MsgBox GetText("C:\_Result.txt")
End Sub
Cần dùng thêm FileLen để kiểm tra dữ liệu đã được ghi vào file hay chưa.
Cảm ơn bạn đã gợi ý! Tôi sửa code thành vầy:
PHP:
Sub DoCommand()
  Dim lFile As Double
  On Error Resume Next
  Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
  Do
    lFile = FileLen("C:\_Result.txt")
  Loop Until lFile <> 0
  MsgBox GetText("C:\_Result.txt")
End Sub
Thêm On Error Resume Next vì bạn đầu file chưa tồn tại nên sẽ lỗi tại hàm FileLen()
----------------------
Lạ 1 điều là lần đầu chạy code, thời gian "đợi" khá lâu so với khi dùng hàm Sleep ---> chẳng hiểu tại sao nó lại đợi lâu như thế?
 
Upvote 0
Cảm ơn bạn đã gợi ý! Tôi sửa code thành vầy:
PHP:
Sub DoCommand()
  Dim lFile As Double
  On Error Resume Next
  Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
  Do
    lFile = FileLen("C:\_Result.txt")
  Loop Until lFile <> 0
  MsgBox GetText("C:\_Result.txt")
End Sub
Thêm On Error Resume Next vì bạn đầu file chưa tồn tại nên sẽ lỗi tại hàm FileLen()
----------------------
Lạ 1 điều là lần đầu chạy code, thời gian "đợi" khá lâu so với khi dùng hàm Sleep ---> chẳng hiểu tại sao nó lại đợi lâu như thế?

Điều kiện
(Len(Dir("C:\_Result.txt")) = 0) Or (FileLen("C:\_Result.txt") = 0)

là đảm bảo không lỗi mà. Nếu đỏ thỏa mãn thì VB sẽ không chạy xanh
Khi đỏ không thỏa mãn (đã có file) thì VB chạy xanh

Với sự kết hợp (OR) 2 logic trên là đảm bảo chạy không lỗi
TuanVNUNI đã viết:
Mã:
Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
Stp1:
    If (Len(Dir("C:\_Result.txt")) = 0) Or (FileLen("C:\_Result.txt") = 0) Then GoTo Stp1
    MsgBox GetText("C:\_Result.txt")
End Sub
 
Upvote 0
Điều kiện
(Len(Dir("C:\_Result.txt")) = 0) Or (FileLen("C:\_Result.txt") = 0)

là đảm bảo không lỗi mà. Nếu đỏ thỏa mãn thì VB sẽ không chạy xanh
Khi đỏ không thỏa mãn (đã có file) thì VB chạy xanh

Với sự kết hợp (OR) 2 logic trên là đảm bảo chạy không lỗi
Có lỗi đấy:

untitled.JPG

Chạy file đính kèm này là biết liền
(Chạy tiếp lần 2 code mới không bị lỗi)
------------------------
Tuy nhiên, vấn đề mình muốn hỏi ở đây là: Lý ra, code này phải tối ưu hơn so với dùng Sleep mới đúng chứ! Vì ta đã "canh" vừa vặn lúc file _Result.txt được hình thành là lấy kết quả luôn ---> Ấy thế mà tốc độ xử lý lại chậm hơn mới lạ chứ? (dùng Sleep ta cảm thấy code xử lý nhanh hơn)
 

File đính kèm

Lần chỉnh sửa cuối:
Upvote 0
Cảm ơn bạn đã gợi ý! Tôi sửa code thành vầy:
PHP:
Sub DoCommand()
  Dim lFile As Double
  On Error Resume Next
  Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
  Do
    lFile = FileLen("C:\_Result.txt")
  Loop Until lFile <> 0
  MsgBox GetText("C:\_Result.txt")
End Sub
Thêm On Error Resume Next vì bạn đầu file chưa tồn tại nên sẽ lỗi tại hàm FileLen()
----------------------
Lạ 1 điều là lần đầu chạy code, thời gian "đợi" khá lâu so với khi dùng hàm Sleep ---> chẳng hiểu tại sao nó lại đợi lâu như thế?

Điều kiện
(Len(Dir("C:\_Result.txt")) = 0) Or (FileLen("C:\_Result.txt") = 0)

là đảm bảo không lỗi mà. Nếu đỏ thỏa mãn thì VB sẽ không chạy xanh
Khi đỏ không thỏa mãn (đã có file) thì VB chạy xanh

Với sự kết hợp (OR) 2 logic trên là đảm bảo chạy không lỗi
[/QUOTE=TuanVNUNI]
Mã:
Sub DoCommand()
    Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
Stp1:
    If (Len(Dir("C:\_Result.txt")) = 0) Or (FileLen("C:\_Result.txt") = 0) Then GoTo Stp1
    MsgBox GetText("C:\_Result.txt")
End Sub
[/QUOTE]
 
Upvote 0
Có lỗi đấy:

View attachment 47243

Chạy file đính kèm này là biết liền
(Chạy tiếp lần 2 code mới không bị lỗi)
------------------------
Tuy nhiên, vấn đề mình muốn hỏi ở đây là: Lý ra, code này phải tối ưu hơn so với dùng Sleep mới đúng chứ! Vì ta đã "canh" vừa vặn lúc file _Result.txt được hình thành là lấy kết quả luôn ---> Ấy thế mà tốc độ xử lý lại chậm hơn mới lạ chứ? (dùng Sleep ta cảm thấy code xử lý nhanh hơn)

Em gửi ví dụ lên đã chạy tốt trên VB6 (sp6), trong VBA Office 2007 (VBA v 6.5), HĐH Vista, thấy không lỗi gì chạy rất nhanh.

Không biết anh chạy Office gì. Nếu lỗi là do logic test IF của VB bị sai, có thể họ đã fix ở sp6, VBA 6.5.

Nếu dùng vòng lặp Do để test thì viết như sau. Trên máy em chạy nhanh.
Mã:
Sub DoCommand()
  Dim lFile As Double
  On Error Resume Next
  Shell "cmd.exe /c IPCONFIG > C:\_Result.txt", vbHide
  Do
    If (Len(Dir("C:\_Result.txt")) <> 0) Then
        lFile = FileLen("C:\_Result.txt")
    End If
  Loop Until lFile <> 0
  MsgBox GetText("C:\_Result.txt")
End Sub
 
Upvote 0
Web KT

Bài viết mới nhất

Back
Top Bottom