return confusions - GameCreators Forum

Author

Message

chan

12

Years of Service

User Offline

Joined: 14th Nov 2011

Location:

Posted: 22nd Nov 2011 10:04

Link

i made a test if im doing a "return" right, but its obviously i didnt, here is the codes.

main

+ Code Snippet

while(LoopGDK())
	{
		dbSync();
		dbCLS();

		open_menu(phase);
		if(open_menu(phase) == 2)
		{
			phase = 2;
		}
		else if(open_menu(phase) == 1)
		{
			phase =1;
		}
		if(phase == 2)
		{
			dbText(50, 50, "phase 2");
		}
		else if(phase == 1)
		{
			dbText(40, 40, "phase 1");
		}


		if(dbEscapeKey())
		{
			break;
		}
	}

return

+ Code Snippet

int open_menu(int phase)
{
	if(dbSpaceKey() == 1 && phase == 2)
	{
		return 1;
	}

	if(dbMouseClick() == 1 && phase == 1)
	{
		return 2;
	}


}

so here what i wanted to do is, whenever i press space, the phase will return 1 (1 is the default). and whenever i click mouse it will return 2.

but what happened is it is continuously changing from one phase to another whenever i click or press.

so confused. i was trying to solve it myself but took me hours already and don't want to waste time.

-Excellence deals with weak people-

Back to top

Profile PM Email

Orac

17

Years of Service

User Offline

Joined: 16th May 2006

Location: Billingham

Posted: 22nd Nov 2011 13:22

Link

You have defined your function open_phase() to return a value int open_phase(phase) but if neither condition is satisfied you are not returning anything. You should modify it to something like:

+ Code Snippet

int open_menu(int phase)
{
  if(dbSpaceKey() == 1 && phase == 2)
  {
    return 1;
  }

  if(dbMouseClick() == 1 && phase == 1)
  {
    return 2;
  }

  return 0; /* default if neither conditions are true */

}

Also in your main function when you call open_phase() you are not assigning the return value to anything. You should do something like:

+ Code Snippet


[b]int retval;[/b]
while(LoopGDK())
{
  dbSync();
  dbCLS();

  [b]retval = 0;   /* reset return value */[/b]
  [b]retval[/b] = open_menu(phase);
  if([b]retval[/b] == 2)
  {
    phase = 2;
  }
  else if([b]retval[/b] == 1)
  {
    phase =1;
  }
  if(phase == 2)
  {
    dbText(50, 50, "phase 2");
  }
  else if(phase == 1)
  {
    dbText(40, 40, "phase 1");
  }

  if(dbEscapeKey())
  {
    break;
  }
}

Of all the things I've lost .. I miss my marbles most of all.

Back to top

Profile PM

Orac

17

Years of Service

User Offline

Joined: 16th May 2006

Location: Billingham

Posted: 22nd Nov 2011 13:24

Link

I just noticed that the HTML tags don't appear to work inside a code snippet, just ignore the bold/endbold tags.

Of all the things I've lost .. I miss my marbles most of all.

Back to top

Profile PM

chan

12

Years of Service

User Offline

Joined: 14th Nov 2011

Location:

Posted: 22nd Nov 2011 13:33

Link

thanks for the help!

it works now.

-Excellence deals with weak people-

Back to top

Profile PM Email

JTK

14

Years of Service

User Offline

Joined: 10th Feb 2010

Location:

Posted: 22nd Nov 2011 16:15

Link

Or even cleaner:

+ Code Snippet

int phase=1; //default to keyboard
phase=open_menu(phase);

if (phase==1) 
{
  ...
}

...

JTK

Back to top

Profile PM

Sorry your browser is not supported!

Dark GDK / return confusions