C / C++ / MFC

15-Feb-11 20:07

Hi im creating cotrols for various dialogs in my applcaition.So for each control one ID will create in resource.h .Sometimes waht happened same numeric ID is assigned to two differnt reource.It happens many of the time.
How can i avoid that?
Because of this i cannot add the resource to accelerator to assign fucntion.Every time when i add it shows resource id already exists..

Example:

#define IDC_STATE0OP 1171
#define ID_CLRPAGE 1171

#define IDC_BUTTON3 1173
#define ID_ALRMDETAIL 1173

#define IDC_ACKBUTTON 1175
#define ID_BLOCKDET 1175

Pls help me to sort out this problem.

Anu

Hans Dietrich15-Feb-11 21:10

Hans Dietrich

15-Feb-11 21:10

Menu and taskbar IDs (for example, ID_CLRPAGE) by convention are assigned numbers greater than 32771. How did ID_CLRPAGE and ID_BLOCKDET get assigned their values in the first place?

If you used the VS resource editor, it will automatically assign the values, and it will use the lines at the bottom of resource.h that look like:

// Next default values for new objects
// 
#ifdef APSTUDIO_INVOKED
#ifndef APSTUDIO_READONLY_SYMBOLS
#define _APS_NEXT_RESOURCE_VALUE        131
#define _APS_NEXT_COMMAND_VALUE         32771
#define _APS_NEXT_CONTROL_VALUE         1041
#define _APS_NEXT_SYMED_VALUE           104
#endif
#endif

These lines tell the resource editor what the next available value is, so there won't be collisions.

Best wishes,
Hans

[Hans Dietrich Software]

ThatsAlok15-Feb-11 21:12

ThatsAlok

15-Feb-11 21:12

See everything is in your resource file only, if you see these macro/decalartion govern the numbering of controls

#ifdef APSTUDIO_INVOKED
#ifndef APSTUDIO_READONLY_SYMBOLS

#define _APS_NEXT_RESOURCE_VALUE	129
#define _APS_NEXT_CONTROL_VALUE		1000
#define _APS_NEXT_SYMED_VALUE		101
#define _APS_NEXT_COMMAND_VALUE		32771
#endif
#endif

check and correct there next count

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow
Never mind - my own stupidity is the source of every "problem" - Mixture

cheers,
Alok Gupta
VC Forum Q&A :- I/IV
Support CRY- Child Relief and You

Anu_Bala15-Feb-11 21:46

15-Feb-11 21:46

In my resource.h it shows like

#ifdef APSTUDIO_INVOKED
#ifndef APSTUDIO_READONLY_SYMBOLS
#define _APS_NEXT_RESOURCE_VALUE        252
#define _APS_NEXT_COMMAND_VALUE         33078
#define _APS_NEXT_CONTROL_VALUE         1140
#define _APS_NEXT_SYMED_VALUE           115
#endif
#endif

Here how can i change..Can u gice ay idea?

Anu

chevu15-Feb-11 22:01

chevu

15-Feb-11 22:01

Make sure that
1. all the resourses ids (like dialog) that are being used in these files are less than _APS_NEXT_RESOURCE_VALUE
2. all the control ids (i.e. button, textbox) are less than _APS_NEXT_CONTROL_VALUE value.
3. all commands (keyboard shortcuts) you have added should less than _APS_NEXT_COMMAND_VALUE

Hope it will solve your problem.

ThatsAlok15-Feb-11 22:52

ThatsAlok

15-Feb-11 22:52

Hi Anu,

Sometime VS misbehave while assigning ids. i believe either try installing latest SP for you visualstudio or manually change all the ID's.

cheers,
Alok Gupta
VC Forum Q&A :- I/IV
Support CRY- Child Relief and You

Anu_Bala15-Feb-11 22:54

Closing childview in MDI [SOLVED]

15-Feb-11 22:54

Yes,i did like that.I changed the IDs manually.

Anu

Array Issue.

Mike Certini15-Feb-11 19:13

Mike Certini

15-Feb-11 19:13

I put together a program that takes a list of numbers in an array and sorts them by copying out of the first array into a second array when the value in the first array matches the maximum array element in the first array. What I cannot figure out is why when I look at the array elements saved in the final array there is a skip in element #10 to #11. In other words the second array correctly saves nine 10's and then skips element #10 and then saves the 9's starting in element #11.

#include <stdio.h>
#include <stdlib.h>
#define LIMIT 100

int roll[100];
int fin[100];
int num = 0;


int main(void)
	{
	int cyc = 0;
	int top = 0;
	int rotate = 0;
	int seek = 0;
	int loop = 0;
	int temp = 0;
		
	while(num < LIMIT)
		{
		roll[num] = rand() % 10 + 1;
		printf("%i",roll[num]);
		printf("\n");
		num++;
		}
	
	for(cyc = 0; cyc < num; cyc++)
		{
		if(roll[loop] > roll[loop+1] && roll[loop] > temp)
			{
			temp = roll[loop];
			}
		loop++;
		}
	
	for(cyc = 0; cyc < num; cyc++)
		{
		loop = 0;			        //Zero loop for second number to be evaluated
		while(loop < num)	                //Go through entire list
			{
			if(roll[loop] == temp)	        //Evaluate whether item matches maximum number
				{																			
				fin[cyc] = roll[loop];	//Copy item in the first array into the final array.
				cyc++;			//Increment cycle by 1.
				}
			loop++;				//Increment loop to pass through entire list.
			}
		temp--;					//Decrement number to be evaluated.	
		}
	}

Solution To Array Issue.

Mike Certini15-Feb-11 20:07

Mike Certini

15-Feb-11 20:07

Posted on different site:

You are incrementing cyc unnecessarily in your for loop when you are copying.

try this

for(cyc = 0; cyc < num;)

or a while loop

cyc=0;
while(cyc < num)

Re: Array Issue.

User 742933815-Feb-11 20:07

User 7429338

15-Feb-11 20:07

In the second for-loop, you increase cyc in the if-statement. It's no longer necesary to also increase it in the loop:

for(cyc = 0; cyc < num;){
    loop = 0;                   //Zero loop for second number to be evaluated
    while(loop < num){               //Go through entire list
        if(roll[loop] == temp){         //Evaluate whether item matches maximum number
            fin[cyc] = roll[loop];      //Copy item in the first array into the final array.
            cyc++;              //Increment cycle by 1.
        }
        loop++;                 //Increment loop to pass through entire list.
    }
    temp--;                     //Decrement number to be evaluated.
}

modified 13-Sep-18 21:01pm.

Re: Array Issue.

Stefan_Lang16-Feb-11 0:18

Stefan_Lang

16-Feb-11 0:18

Your while loop initializes the array roll with values between 1 and 10. Note that there is no guarantee every value between 1 and 10 will actually be stored in roll somewhere; for all you know it might be an array full of 1s or full of 10s!

In your first for loop you increment loop num times, then access roll[loop+1]; on the last loop you will read past the end of the array, invalidating the comparison, so if the biggest value in your array happens to be stored in the last element, it may or may not be stored in temp. In Debug this probably won't cause a problem, as likely roll[100] would just access fin[0] instead, and that would be initialized to 0. In Release mode however there is no guarantee that the array fin will be initialized to 0 - in my experience it most likely will not.

The intention of the first for loop appears to be to find the largest value in roll and store it in temp. Note that this value may be anything between 1 and 10!

The error you noticed, however, is due to what the others already pointed out above: you shouldn't increment cyc in the outer loop! What happens is this: the inner loop finds all items in roll that correspond to the temp value, store it in fin, and increment (correctly) the index cyc. Once you're at the end of the loop, temp gets decremented, then cyc gets incremented again, causing the gap you noticed, then you go on searching for occurences of your now decreased value of temp.

The reason for this confusion is that the outer loop is really all about temp, not cyc! The clearest way to write your last loop would be like this:

cyc = 0;
for (int compare = temp; compare > 0; compare--) {
   for (loop = 0; loop < LIMIT; loop++) {
      if (roll[loop] == compare) {
         fin[cyc] = compare;
         cyc++;
      }
   }
}

Note that I replaced num with LIMIT. This doesn't change anything in your code, but it is easier to read, as a reader unfamiliar with your code doesn't have to interpret all of it just to find out what value num would hold. Also it's less likely to accidentally mess up if you later change your code and possibly change the value of num.

Now, while the above is clearer, it is also potentially less efficient; you used the condition cyc < num, which would lead the outer loop to terminate, even if temp is still > 0, e.g. when there are no 1s in the array roll. So you could replace compare > 0 in the above loop with cyc < num, or, preferably, cyc < LIMIT.

You could of course also avoid the introduction of yet another variable (compare) and just write the outer for loop like this, trading clarity for shorter, and potentially more efficient, code:

for (cyc = 0; cyc < LIMIT; temp--) {
   //... (use temp in comparison instead of compare)
}

Anu_Bala15-Feb-11 18:23

Re: Closing childview in MDI

15-Feb-11 18:23

Hi,In my MDI application,i want to close each child window with confirm message.
For this in each view,OnClose() fucntion i coded like

void CAlarmView::OnClose()
{
	if(AfxMessageBox("Do you want to close the Alarm?",MB_YESNO|MB_DEFBUTTON2,NULL) == IDNO)
	        return;	
	CView::OnClose();
}

But when i close the child window it does not go this OnClose() function itself.
So i gave this OnDestroy() fucntion.

void CAlarmView::OnDestroy() 
{

	if(AfxMessageBox("Do you want to close the Alarm?",MB_YESNO|MB_DEFBUTTON2,NULL) == IDNO)
		return;	
       CView::OnDestroy();
}

When the application shows this confirm message,when i press NO , the child window is getting closed.Actually if i press YES then the child window should close,if i press NO it doesnot get close.

Pls help me in this concern

Anu

modified on Wednesday, February 16, 2011 2:02 AM

tagopi15-Feb-11 18:53

tagopi

15-Feb-11 18:53

hi,

actually, your condition is wrong. change like this.

if(AfxMessageBox("Do you want to close the Alarm?",MB_YESNO|MB_DEFBUTTON2,NULL) == IDNO)
return;
else
CView::OnDestroy();

or

if(AfxMessageBox("Do you want to close the Alarm?",MB_YESNO|MB_DEFBUTTON2,NULL) == IDYES)
CView::OnDestroy();

Hope this will work..
Regards.
A. Gopinath.

Re: Closing childview in MDI

_AnsHUMAN_ 15-Feb-11 19:02

_AnsHUMAN_

15-Feb-11 19:02

OnDestroy[^] is called by the framework. Can you debug to check the issue with OnClose?

I am a HUMAN. I have that keyword in my name........
Wink | ;-)

_AnsHUMAN_

Re: Closing childview in MDI

Anu_Bala15-Feb-11 20:01

How to bring an App to front, from another App

15-Feb-11 20:01

I changed the code to childframe OnClose(),there im checking each window and gave this AfxMessageBox and it works fine.Thank for ur support.

void CChildFrame::OnClose()
{
	if(iWindowNumber == 1)
	{
		if(AfxMessageBox("Do you want to close the Alarm?",MB_YESNO|MB_DEFBUTTON2,NULL) == IDNO)
			return;
	}
	CMDIChildWnd::OnClose();
}

Anu

timbk15-Feb-11 15:50

15-Feb-11 15:50

Hello , I'm programing in VC++ 6, and the entire project runs in a computer without keyboard but with touch screen.
The thing is that a have two app runnig at the same time, and I want the user to be able to switch between these apps by touching in a third app (called "Switcher",wich is always on top), this third app is a dialog based app, dialog's size is small and it has only two buttons, one for bring the first app to front and the other button to bring to the second app.
The project has to run over win 98 an over win XP.
How can i do to get wich app is The "Switcher" and bring the other to front but under the "switcher"?

David Crow15-Feb-11 16:54

David Crow

15-Feb-11 16:54

Have you considered SetWindowPos()?

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"Man who follows car will be exhausted." - Confucius

Anthony Mushrow15-Feb-11 16:58

Anthony Mushrow

15-Feb-11 16:58

You can use SetWindowPos to move, resize and change the z-index of a window, SetWindowPos[^]

So you could use this:

SetWindowPos(targetWindow, switcherHandle, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE);

To bring the window to the front, by passing the handle to your switcher window as the second parameter the target application should appear just behind it. Rather than at the front of all windows.

My current favourite phrase: I've seen better!

-SK Genius

Source Indexing and Symbol Servers

Vehicle Simulation Demo - Mostly Works

timbk16-Feb-11 0:32

16-Feb-11 0:32

Thanks, So I have to use this instruccion SetWindowPos(targetWindow, switcherHandle, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE); on my "Switcher" dialog, ok, my last doubt is how can I get the handler to the target window from the "Switcher" dialog?

Anthony Mushrow16-Feb-11 3:31

Anthony Mushrow

16-Feb-11 3:31

If you know the title of the window, or its class name then you can use FindWindow[^].

If you don't have the name of the window, but do have a HINSTANCE then you can search through all of the windows until you find the one you need:

HWND tmpWnd;

bool foundWindow = false;

//Get topmost window (which would probably be switcher in this case but it doesn't really matter)
tepWnd = FindWindow(NULL, NULL)
   
while(tmpWnd != NULL)
{
  HINSTANCE hInstance = ProcIDFromWnd(tmpWnd);
  if(hInstance == windowInst)
  {
    foundWindow = true;
    break;
  }
  else
  {
    tmpWnd = GetWindow(tmpWnd, GW_HWNDNEXT);
  }
}

Without either a HINSTANCE the class name, or the window name then I don't think there's much you could do.

My current favourite phrase: I've seen better!

-SK Genius

Source Indexing and Symbol Servers

Vehicle Simulation Demo - Mostly Works

timbk16-Feb-11 11:06

16-Feb-11 11:06

Ok, it's working for the app1 that I know his title, let say from the "Switcher" app it's possible bring the other app1(the one i know his title) behind the Switcher, but with the other app2 , (wich is no-modal dialog) it doesn't work , i do not know the name of his class and the dialog it has not title, so i went whith the code that you wrote, but compiler complains with ProcIDFromWnd, it says that is undeclared identifier, it seems that is not a valid intruction for VC++ 6 , ??
Anyway , since in the final projet will be runnig only these three apps, i thoth that the Switcher can bring the app 1(the one working) behind it and send it to bottom when the user wants to see the app2, mfc library says that it's possible to use the parameter HWND_BOTTOM instead hWndSwitcher in SetWindowPos(hWndTarget, hWndSwitcher, 0, 0, 0, 0, SWP_NOSIZE | SWP_NOMOVE); to do that, but it's doesn't work neither. What do you think?

Anthony Mushrow16-Feb-11 12:03

Anthony Mushrow

16-Feb-11 12:03

I'm not sure why SetWindowPos wouldn't work, unless you have the wrong hWnd, but as for finding a window by its HINSTANCE this code should actually work. The sample I originally found was in VB so I guess that ProcIDFromWnd is specific to that, GetWindowThreadProcessId seems to be what we really need.

HWND tmpWnd;

bool foundWindow = false;

//Get topmost window (which would probably be switcher in this case but it doesn't really matter)
tepWnd = FindWindow(NULL, NULL)
DWORD procID = GetProcessId(hInstace);

while(tmpWnd != NULL)
{
  DWORD id = 0;
  GetWindowThreadProcessId(tmpWnd, &id);
  if(id == procID)
  {
    foundWindow = true;
    break;
  }
  else
  {
    tmpWnd = GetWindow(tmpWnd, GW_HWNDNEXT);
  }
}

Bear in mind that this code would find all windows that a process owns, so if it has more than one then you'll have to try and figure out which one you really need.

My current favourite phrase: I've seen better!

-SK Genius

Source Indexing and Symbol Servers

Vehicle Simulation Demo - Mostly Works

timbk16-Feb-11 16:54