my switch wont do its cases but it does with default??

jcylam (104)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
	switch (*pclick_function)
	{
	case 1:
			Left_click_once (*pGT_X, *pGT_Y);
			cout << "\tLeft clicked\r";
			break;
	case 2:
			Double_click (*pGT_X, *pGT_Y);
			cout << "\tDouble clicked\r";
			break;
	default:
		cout << "ERROR at clicking function\n";
		break;
	}
firedraco (6247)
What is pclick_function? Is it an int*?
kempofighter (1183)
Well then I guess the value passed to the switch wasn't a 1 or a 2. Any value other than 1 or 2 will cause the default to execute. Feel free to post a more complete example if you want any further analysis.
jcylam (104)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
//Function variables
	bool flag = TRUE;
	int click_function;
	int *pclick_function = &click_function;

// Get screen resolution
	long Screen_Res_X = GetSystemMetrics ( SM_CXSCREEN );		//Screen size in x-axis
	long Screen_Res_Y = GetSystemMetrics ( SM_CYSCREEN );		//Screen size in y-axis

	cout << "Screen size x-axis: " << Screen_Res_X << endl;		//Prints x-axis size
	cout << "Screen size y-axis: " << Screen_Res_Y << endl;		//Prints y-axis size
	
	long GT_X = Screen_Res_X/2;		//Declare Gaze Tracker X coordinate
	long GT_Y = Screen_Res_Y/2;		//Declare Gaze Tracker Y coordinate

//Declare pointers
	long *pGT_X;
	long *pGT_Y;
	long *pScreen_Res_X;
	long *pScreen_Res_Y;

//Assign pointers
	pGT_X = &GT_X;
	pGT_Y = &GT_Y;
	pScreen_Res_X = &Screen_Res_X;
	pScreen_Res_Y = &Screen_Res_Y;

		cout << "Press \"ESC\" to end program\n";



	cout << "Press any key to continue\n";
	getchar();
	
	do
	{
		cout << "\nInsert value:\n1.Left click\n2.Double click\n";
		cin >> *pclick_function;

		get_XY (pGT_X, pGT_Y, pScreen_Res_X, pScreen_Res_Y, pclick_function);

jumps to the function
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
 
void get_XY (long *pGT_X, long *pGT_Y, long *pScreen_Res_X, long *pScreen_Res_Y, int *pclick_function)
{
//DECLARE VARIABLES for function
	bool flag = TRUE;
	bool click = FALSE;
	int click_counter = 0;

//DECLARATION OF VARIABLES for use in detecting automated click
	long lower_x, lower_y, upper_x, upper_y;

//DECLARE pointers
	long *plower_x = &lower_x;
	long *plower_y = &lower_y;
	long *pupper_x = &upper_x;
	long *pupper_y = &upper_y;

	//Setting lower and upper limit of cursor
	//For X axis
		if ( (*pGT_X -5) < 0 )
			lower_x = 0;
		else
			lower_x = (*pGT_X - 5);//end else if

		if ( (*pGT_X + 5) > *pScreen_Res_X )
			upper_x = *pScreen_Res_X;
		else
			upper_x = (*pGT_X + 5);//end else if

	//For Y axis
		if ( (*pGT_Y -5) < 0 )
			lower_y = 0;
		else
			lower_y = (*pGT_Y - 5);//end else if

		if ( (*pGT_Y + 5) > *pScreen_Res_Y )
			upper_y = *pScreen_Res_Y;
		else
			upper_y = (*pGT_Y + 5);//end else if
	//End setting limits

	printf ("\nStart: gt x %d, gt y %d, lower x %d, lower y %d, upper x %d, upper y %d\n", *pGT_X, *pGT_Y, *plower_x, *plower_y, *pupper_x, *pupper_y);

	int n = 2;	//sets the sampling rate before a click
	for (int j = 0; j < n; ++j)
	{
		loop_for_control (pGT_X, pGT_Y, n);

		if (check_position (pGT_X, pGT_Y, plower_x, plower_y, pupper_x, pupper_y))
		{
			++click_counter;
			if (click_counter == n)
			{
				click = TRUE;

			}
			cout << "counter: " << click_counter << endl;
		}
		else
		{
			cout << "not same" << endl;
		}

	}
cout << *pclick_function << "click_function\n";

	switch (*pclick_function)
	{
	case 1:
			Left_click_once (*pGT_X, *pGT_Y);
			cout << "\tLeft clicked\r";
			break;
	case 2:
			Double_click (*pGT_X, *pGT_Y);
			cout << "\tDouble clicked\r";
			break;
	default:
		cout << "ERROR at clicking function\n";
		break;
	}

	cout << "**********************************************\n";
}


when i enter 1 or 2 it prints out 1 clicked or 2 clicked but doesnt print left clicked or double clicked. when i enter sthg else the default works?????? does it hav to do with the pointer??
jsmith (5804)
Left_click_once and Double_click are being called. Your prints aren't displaying because
they don't flush stdout (cout) and there is no endl or \n to explicitly flush them.

Add a << std::endl or << '\n' to the end of lines 70 and 74.
jcylam (104)
fixed it.....somehow without altering the code, and doing like 10 times recompile it decides to work, donno what went wrong but its working now.

Thanks guys
Topic archived. No new replies allowed.