'Return Window handle by it's name / title
I can't solve this problem. I get an error:
The name 'hWnd' does not exist in the current context
It sounds very easy and probably is... sorry for asking so obvious questions.
Here's my code:
public static IntPtr WinGetHandle(string wName)
{
foreach (Process pList in Process.GetProcesses())
{
if (pList.MainWindowTitle.Contains(wName))
{
IntPtr hWnd = pList.MainWindowHandle;
}
}
return hWnd;
}
I tried with many different ways and each fails. Thanks in advance.
Solution 1:[1]
Update: See Richard's Answer for a more elegant approach.
Don't forget you're declaring you hWnd
inside the loop - which means it's only visible inside the loop. What happens if the window title doesn't exist? If you want to do it with a for
you should declare it outside your loop, set it inside the loop then return it...
IntPtr hWnd = IntPtr.Zero;
foreach (Process pList in Process.GetProcesses())
{
if (pList.MainWindowTitle.Contains(wName))
{
hWnd = pList.MainWindowHandle;
}
}
return hWnd; //Should contain the handle but may be zero if the title doesn't match
Or in a more LINQ-y way....
IntPtr? handle = Process
.GetProcesses()
.SingleOrDefault(x => x.MainWindowTitle.Contains(wName))
?.Handle;
return handle.HasValue ? handle.Value : IntPtr.Zero
Solution 2:[2]
As an option to solve this problem:
[DllImport("user32.dll")]
private static extern IntPtr FindWindow(string lpClassName, string lpWindowName);
public IntPtr GetHandleWindow(string title)
{
return FindWindow(null, title);
}
Solution 3:[3]
Coming several years late to this but, as others have mentioned, the scope of hWnd
is only in the foreach
loop.
However it's worth noting that, assuming you're doing nothing else with the function, there are two issues with the answers others have provided:
- The variable
hWnd
is actually unnecessary since it's only being used for one thing (as the variable for thereturn
) - The
foreach
loop is inefficient as, even after you've found a match, you continue to search the rest of the processes. In actual fact, it'll return the last process it finds that matches.
Assuming that you don't want to match the last process (point #2), then this is a cleaner and more efficient function:
public static IntPtr WinGetHandle(string wName)
{
foreach (Process pList in Process.GetProcesses())
if (pList.MainWindowTitle.Contains(wName))
return pList.MainWindowHandle;
return IntPtr.Zero;
}
Solution 4:[4]
Because you are declaring hWnd
inside the if block, it is inaccessible to the return statement which is outside it. See http://www.blackwasp.co.uk/CSharpVariableScopes.aspx for clarification.
The code you've provided can be fixed by moving the declaration of the hWnd variable:
public static IntPtr GetWindowHandleByTitle(string wName)
{
IntPtr hWnd = IntPtr.Zero;
foreach (Process pList in Process.GetProcesses())
{
if (pList.MainWindowTitle.Contains(wName))
{
hWnd = pList.MainWindowHandle;
}
}
return hWnd;
}
Solution 5:[5]
hWnd
is declared in the foreach
loop. Its context is inside foeach
loop. To get its value declare it outside foreach
loop.
Use it like this,
public static IntPtr WinGetHandle(string wName){
IntPtr hWnd = NULL;
foreach (Process pList in Process.GetProcesses())
if (pList.MainWindowTitle.Contains(wName))
hWnd = pList.MainWindowHandle;
return hWnd;
}
Solution 6:[6]
#include <windows.h>
#using <System.dll>
using namespace System;
using namespace System::Diagnostics;
using namespace System::ComponentModel;
// get window handle from part of window title
public static IntPtr WinGetHandle(string wName)
{
IntPtr hwnd = IntPtr.Zero;
foreach (Process pList in Process.GetProcesses())
{
if (pList.MainWindowTitle.Contains(wName))
{
hWnd = pList.MainWindowHandle;
return hWnd;
}
}
return hWnd;
}
// get window handle from exact window title
[DllImport("user32.dll")]
private static extern IntPtr FindWindow(string lpClassName, string lpWindowName);
public IntPtr GetHandleWindow(string title)
{
return FindWindow(null, title);
}
Solution 7:[7]
For a method that allows you to search, case insensitive, by part of the window title (that will search all windows, not just the first window for each process)
using System.Runtime.InteropServices;
using System.Text;
var hwnd = GetHandleByTitle("Chrome");
Console.WriteLine(hwnd);
[DllImport("USER32.DLL")]
static extern IntPtr GetShellWindow();
[DllImport("USER32.DLL")]
static extern bool EnumWindows(EnumWindowsProc enumFunc, int lParam);
[DllImport("user32.dll")]
static extern int GetWindowText(IntPtr hWnd, StringBuilder text, int count);
static IntPtr GetHandleByTitle(string windowTitle)
{
const int nChars = 256;
IntPtr shellWindow = GetShellWindow();
IntPtr found = IntPtr.Zero;
EnumWindows(
delegate (IntPtr hWnd, int lParam)
{
//ignore shell window
if (hWnd == shellWindow) return true;
//get Window Title
StringBuilder Buff = new StringBuilder(nChars);
if (GetWindowText(hWnd, Buff, nChars) > 0)
{
//Case insensitive match
if (Buff.ToString().Contains(windowTitle, StringComparison.InvariantCultureIgnoreCase))
{
found = hWnd;
return true;
}
}
return true;
}, 0
);
return found;
}
delegate bool EnumWindowsProc(IntPtr hWnd, int lParam);
Sources
This article follows the attribution requirements of Stack Overflow and is licensed under CC BY-SA 3.0.
Source: Stack Overflow
Solution | Source |
---|---|
Solution 1 | |
Solution 2 | |
Solution 3 | |
Solution 4 | Mukesh Methaniya |
Solution 5 | Shiplu Mokaddim |
Solution 6 | |
Solution 7 | Daniel Barnes |