I am making a card game in Lite-C. The player gets five cards to choose from. After they start I use a random number generator to decide what cards the player gets. Based on the number I want to change the button bmaps. I was wondering how I would change the bmaps without having to have lots of lines of codes.

Just tell me if you need to see my script and I can show you what I've already tried to do.

At the manual:
pan_setbmap(PANEL*,var type,var num,BMAP*)

The only problem with pan_setbmap is I have 25 different possibilities for what the bmap could be. I also need to change all three of the images for the button.

Also pan_setbmap does allow you to use any variables for the button number. This means I would have to do it for one button and then copy it three more times for the other buttons.

My fear is that having to do it this way will add lots of lines to my code.

If you don't understand what I mean tell me if you need to see what my script looks like.

there's also pan_setbutton
pan_setbutton(PANEL*, var num, var type, var x, var y, BMAP* bmapOn, BMAP* bmapOff, BMAP* bmapOver, BMAP* bmapOverOff, void* functionClick, void* functionLeave, void* functionOver);
so you can change all the buttons at once, in one line of code

I tried to do pan_setbutton but when I tried to compile it gave me an error.

Here is the code using pan_setbmap:

pan_setbmap(PlayersHand,3,1.0,SetUpTentOne);
pan_setbmap(PlayersHand,3,1.1,SetUpTentOne);
pan_setbmap(PlayersHand,3,1.2,SetUpTentOne);

could you help me change that so it uses pan_setbutton.

well unfortunately you need to pass all the parameters

if you're trying to just shorten that just create a simple function

When I try and run this code it says crash in main

for (Count; Count <= 24; Count++)
{
if (Count <= 2)
{
CardCountUnc[Count] = 18;
}
else if (Count == 3) // Setup of unchanged card number array
{
CardCountUnc[Count] = 10;
}
else
{
CardCountUnc[Count] = 2;
}
}

Count = 0;

for (Count; Count <= 24; Count++)
{
CardCountCha[Count] = CardCountUnc[Count]; // Setup of changed card number array
}

random_seed(0);

Count2 = 0;
Count = 0;
CardNumberZero = 1;

for (Count2; Count2 <= 4; Count2++)
{
while (CardNumberZero == 1)
{
CardDrawNumber = rand() % 24; // Deal out players hand

for (Count; Count <= 24; Count++)
{
if (CardDrawNumber == Count)
{
if (CardCountCha[Count] > 0)
{
PlayerHand[Count2] = CardDrawNumber;
CardCountCha[Count]--;
CardNumberZero = 0;
// ChangeCardMap(CardDrawNumber,Count2);
PanelName = PanelChangeName(Count2);
CaPicture = FindCardPicture(CardDrawNumber);
ChangeCardMap(PanelName, CaPicture);
}
}
}
}
CardNumberZero = 1;
Count = 0;
}
}

function PanelChangeName(var Number)
{
STRING* PlayerCardPanelName;

if (Number == 0)
{
PlayerCardPanelName = "PlayerCardOne";
}
if (Number == 1)
{
PlayerCardPanelName = "PlayerCardTwo";
}
if (Number == 2)
{
PlayerCardPanelName = "PlayerCardThree";
}
if (Number == 3)
{
PlayerCardPanelName = "PlayerCardFour";
}
if (Number == 4)
{
PlayerCardPanelName = "PlayerCardFive";
}
return (PlayerCardPanelName);
}

function FindCardPicture(var Card)
{
BMAP* CardPicture;

if (Card == 1)
{
CardPicture = SetUpTentOne;
}

return (CardPicture);
}

function ChangeCardMap(Name,BMAP* Picture)
{
pan_setbmap(Name,3,1.0,Picture);
}

I am trying to pass the panel name and the name of the bmap to a function that will change the button based on this information. I would appreciate any help I can get.

change your for loops to


instead of all the if else use switch


A7 can't return strings too well, you'll need to set it inside the function


you shouldn't be getting the name of the panel, just return it's pointer


would it not be easier for you to have just 1 image of all the cards, then use a window to display the card, rather than having them all as seperate images?

and use [ code ] [/code] tags, makes it easier to read

hope this helps

I have the different windows working now and they display the card pictures right. The last question I have is if I'm doing it in the shortest amount of lines possible.

PANEL* PlayersHand =
{
layer = 5;
window(180,790,140,218,GameCards2,382,0);
window(370,790,140,218,GameCards2,PictureX[1],0);
window(560,790,140,218,GameCards2,PictureX[2],0);
window(750,790,140,218,GameCards2,PictureX[3],0);
window(940,790,140,218,GameCards2,PictureX[4],0);
}

function main()
{
video_mode = 9;
video_screen = 1;

mouse_map = MousePointer;
mouse_mode = 4;
mouse_sync = 1;

set (PlayingBackground, SHOW);
set (OpponentsField, SHOW);
set (PlayersField, SHOW);
set (PlayersHand, SHOW);


}

function GameStart()
{

Count = 0;

for (Count; Count <= 24; Count++)
{
if (Count <= 2)
{
CardCountUnc[Count] = 18;
}
else if (Count == 3) // Setup of unchanged card number array
{
CardCountUnc[Count] = 10;
}
else
{
CardCountUnc[Count] = 2;
}
}

Count = 0;

for (Count; Count <= 24; Count++)
{
CardCountCha[Count] = CardCountUnc[Count]; // Setup of changed card number array
}

random_seed(0);

Count2 = 0;
Count = 0;
CardNumberZero = 1;

for (Count2; Count2 <= 4; Count2++)
{
while (CardNumberZero == 1)
{
CardDrawNumber = rand() % 24; // Deal out players hand

for (Count; Count <= 24; Count++)
{
if (CardDrawNumber == Count)
{
if (CardCountCha[Count] > 0)
{
PlayerHand[Count2] = CardDrawNumber;
CardCountCha[Count]--;
CardNumberZero = 0;
ChangeCardImage(CardDrawNumber,Count2);
}
}
}
}
CardNumberZero = 1;
Count = 0;
}

}
function ChangeCardImage(Number,HandNumber)
{
switch (Number)
{
case 0:
PictureX[HandNumber] = 135;
break;
case 1:
PictureX[HandNumber] = 263;
break;
case 2:
PictureX[HandNumber] = 382;
break;
}
}