Show/Hide window problems

This code I wrote will hide or show windows that have a one-word title (such as the Run dialog), but it stops responding until I press Ctrl+C if I try to do a window with more than one word("My Documents"). Why is this happening? Here is the code:

Code:

---

#include <windows.h>
#include <iostream.h>

int main()
{
        char* winName = new char;
        cout<<"Type in the window name:"<<endl;
        cin>>winName;
        HWND h = FindWindow(NULL, winName);
        if (h == NULL)
                MessageBox(NULL, "Could not find window", "Error", MB_OK | MB_ICONERROR);
        else
        {
                if (ShowWindow(h, SW_HIDE) == 0) // window was hidden, show it
                        ShowWindow(h, 1);
                else //window was visible, hide it
                        ShowWindow(h, 0);
        }
        return 0;
}

---

Code:

---

char* winName = new char;
cout<<"Type in the window name:"<<endl;
cin>>winName;

---

This is why it doesn't work. You're only allocating space for one character. Perhaps:

Code:

---

string sName;
cout << "Type in the window name: " << endl;
cin.readline(sName);

---

Doesn't work:

Error E2316 test.cpp 8: 'readline' is not a member of 'istream' in function main()
Error E2034 test.cpp 10: Cannot convert 'string' to 'const char *' in function main()
Error E2342 test.cpp 10: Type mismatch in parameter 'lpWindowName' (wanted 'const char *', got 'string') in function main()

I don't think I can use the string in the FindWindow function, is there a way I can convert it to what it needs?

Code:

---

getline(cin, sName);

// ...

FindWindow(sName.c_str(), ...);

---

Oops...I really shouldn't drink before coding :eek:

So, it's not the monitor's fault this time? :p

Nope :p Totally my fault :)

Although that problem could have been solved with a simple search on Google but oh well... :rolleyes:

Thanks parksie, it works now :)

Code:

---

#include <windows.h>
#include <iostream.h>

int main()
{
        string sName;
        cout << "Type in the window name: " << endl;
        getline(cin, sName);
        HWND h = FindWindow(NULL, sName.c_str());
        if (h == NULL)
                MessageBox(NULL, "Could not find window", "Error", MB_OK | MB_ICONERROR);
        else
        {
                if (ShowWindow(h, SW_HIDE) == 0) // window was hidden, show it
                        ShowWindow(h, 1);
                else //window was visible, hide it
                        ShowWindow(h, 0);
        }
        return 0;
}

---

Woo..So you always drink before coding Parskie?

No, I drink full stop :)