Having some annoying issues with API calls

Quite simply, trying to get the window size of another window. Quite easy research told me to use GetWindowRect (and subsequently GetClientRect for other stuff I want to do).

Code:

---

[DllImport("user32.dll")]
private static extern bool GetWindowRect(int hwnd, ref RECT Rect);

...

// down here somewhere I have successfully tracked the program window I want to look up (and in future affect) y = process from a list

int hWnd = y.Handle.ToInt32(); // works, I get the correct handle
RECT r = new RECT(); // created with blank/zero values
GetWindowRect(hWnd, out r);

// get to here and r is still a bunch of zero's...

...

[StructLayout(LayoutKind.Sequential)]
public struct RECT
{
// ignore the values as such, i've found several different ways of doing this, if you know a way that works, let me know!
  public int X;
  public int Y;
  public int Width;
  public int Height;
}

---

So I successfully track and locate the window I want to move, via a system of adding a program to a listview by it's full path, and then searching the processes for a match which works successfully. But it seems anything I try with API stuff (and not just GetWindowRect) just seems to do nothing. I would be grateful if someone could point out what's going wrong, or even give me a numpties crash course in API stuff...!

Code:

---

static extern bool GetWindowRect(System.IntPtr hWnd, out RECT rect);

---

Changed, still zero's.

Quote:

---

y = process from a list

---

A process is NOT a window, thus not having a location and a size.
If y is indeed a window (form), then why not just call y.Bounds?

What I'm doing is getting the process that is running that I want to affect (which becomes y) and then I get it's handle by

Code:

---

IntPtr hWnd = y.handle;

---

This is then used for the hwnd in GetWindowRect

WinAPI has handles to many things, including processes and windows. You cannot pass a process handle to GetWindowRect which expects a window handle.

Ah! I'm now using FindWindow using this kind of method now. I wasn't using it before as I thought I had found the correct handle! Thanks!

Well, all that is working completely fine now. It even grabs the right window I'm looking for if there are multiple windows.

But, there is something going on with the other API stuff that I can't quite put my finger on. This is the current code I've got :

Code:

---

int GWL_STYLE = -16;
long WS_BORDER = -0x800000;
long WS_DLGFRAME = -0x400000;
long WS_THICKFRAME = -0x40000;
long WS_SIZEBOX = -0x00040000;
long WS_SYSMENU = -0x00080000;
long WS_POPUP = -0x80000000;
uint SWP_NOMOVE = 0x0002;
uint SWP_NOZORDER = 0x0004;
uint SWP_FRAMECHANGED = 0x0020;  /* The frame changed: send WM_NCCALCSIZE */
long WS_CAPTION = WS_BORDER | WS_DLGFRAME;
IntPtr hWndPtr = FindWindow(null, y.MainWindowTitle);
Rectangle ScreenSize = myScreens[x.Monitor - 1].Bounds;

long code = GetWindowLong(hWndPtr, GWL_STYLE);
code= (code & WS_CAPTION & WS_BORDER & WS_THICKFRAME & WS_SIZEBOX & WS_SYSMENU) | WS_POPUP;
// this line does the proper moving!
MoveWindow(hWndPtr, ScreenSize.Left, ScreenSize.Top, ScreenSize.Right, ScreenSize.Bottom, 1);

SetWindowLong(hWndPtr, GWL_STYLE, (int)code);
SetWindowPos(hWndPtr, new IntPtr(-1), ScreenSize.Left, ScreenSize.Top, ScreenSize.Right, ScreenSize.Bottom, SWP_NOMOVE | SWP_NOZORDER | SWP_FRAMECHANGED);

---

It all works as it should. It grabs the correct window, gets the correct info for the destination screen (multi-monitor stuff), gets the window's style information and modifies it. MoveWindow moves the window to the selected monitor, setwindowlong removes the border stuff, and setwindowpos makes it fill that monitor. So you have a pseudo full screen.

The strange thing is it doesn't work correctly the first time it moves the window. The contained controls in the window do not resize to fit the new dimensions (obvious by a test window mdi-child control not going all the way to the right and the bottom, by the same size as the now hidden borders). Except when you shift the window to another monitor selection, then it's perfect. But even looping this code twice to try and force it again doesn't work (well, it does, but by god is it ugly...).

So um... Why won't it work first time round, and how do I make it work?