Setting Two Registry Keys

Question

1.00/5 (2 votes)

See more:

I'm trying to set two registry keys at once without running this code twice. How can I do this? Thanks!

C++

HKEY hKey; DWORD buffersize = 1024; DWORD dwErr = NO_ERROR; 
#define BUFFER_SIZE 1024
dwErr = RegOpenKeyEx (HKEY_CURRENT_USER,"Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3",NULL,KEY_SET_VALUE|KEY_QUERY_VALUE,&hKey);
fwErr = RegOpenKeyEx (HKEY_LOCAL_MACHINE,"SOFTWARE\Microsoft\Windows\CurrentVersion\Internet Settings\Zones\3",NULL,KEY_SET_VALUE|KEY_QUERY_VALUE,&hKey);
if(ERROR_SUCCESS == dwErr)
	{
	BYTE data[BUFFER_SIZE] = {0};
	DWORD dwType = REG_NONE;
	dwErr = RegQueryValueExW(hKey,L"1803",NULL,&dwType,data,&buffersize);
	if(ERROR_SUCCESS == dwErr && REG_DWORD == dwType)
		{
		*((LPDWORD)data) = 3; //0 is Enable 3 is Disable
		dwErr = RegSetValueExW(hKey,L"1803",0,REG_DWORD,data,sizeof(DWORD));
		}
		RegCloseKey (hKey);
		SendNotifyMessage(HWND_BROADCAST, WM_SETTINGCHANGE, 0, (LPARAM)_T("HKEY_CURRENT_USER\\Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3"));
	}

Posted 18-Sep-11 9:11am

Member 7766180

Updated 20-Sep-11 13:34pm

v3

Add a Solution

Comments

Philippe Mori 20-Sep-11 20:30pm

Using space character and new line more often in your code would improve readability...

By the way the are some \ that are not properly doubled...

Member 7766180 20-Sep-11 21:21pm

Thank you Phillipe. I fixed that. "/". I will also take your suggestion.

2 solutions

Add a Solution

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sergey Alexandrovich Kryukov · Answer 1 · 2011-09-18T11:27:00

You need to master such a basic method of abstraction as "function with parameters". The registry keys and a values to set should be passed as parameters to the code.

The fact you show this code with access to OS API and immediate constants hard-coded in this fragment (which is really bad in general) but without showing a single function looks disturbing. It looks like you develop your programming skills in a very disproportional manner. You're getting deep enough into Windows API without basic understanding of how to write code. Are you really in such rush? Then the usefulness of your activity is questionable.

You claim you're using C++, but in fact not only you're not using OOP, you need to get familiar with the basics of programming which go well before OOP and other advanced topics. I would advise to go along the full course of C++ and using of basic language features in practice, without skipping of any topics, before you start any program using OS APIs, UI and the like.

—SA

Member 7766180 · Answer 2 · 2011-09-20T09:00:00

Does this work?

C++

HKEY hKey; DWORD buffersize = 1024;
DWORD dwErr = NO_ERROR;
#define BUFFER_SIZE 1024
dwErr = RegOpenKeyEx HKEY_CURRENT_USER,"Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3",NULL,KEY_SET_VALUE|KEY_QUERY_VALUE,&hKey)&& RegOpenKeyEx (HKEY_LOCAL_MACHINE,"SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3",NULL,KEY_SET_VALUE|KEY_QUERY_VALUE,&hKey);
	if(ERROR_SUCCESS == dwErr)
	{
	BYTE data[BUFFER_SIZE] = {0};
	DWORD dwType = REG_NONE;
dwErr = RegQueryValueExW(hKey,L"1803",NULL,&dwType,data,&buffersize);
	if(ERROR_SUCCESS == dwErr && REG_DWORD == dwType && *((LPDWORD)data) != 0)	
	{
	*((LPDWORD)data) = 0; //0 is Enable 3 is Disable
	dwErr = RegSetValueExW(hKey,L"1803",0,REG_DWORD,data,sizeof(DWORD));
	}
	RegCloseKey (hKey);
SendNotifyMessage(HWND_BROADCAST, WM_SETTINGCHANGE, 0, (LPARAM)_T    ("HKEY_CURRENT_USER\\Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3"));
SendNotifyMessage(HWND_BROADCAST, WM_SETTINGCHANGE, 0, (LPARAM)_T("HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Internet Settings\\Zones\\3"));
}

Setting Two Registry Keys

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0

Setting Two Registry Keys

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0

Existing Members

...or Join us