[jeffery30162]

I have a list of saved houses and I want it to when a player logs in it checks the list of houses to see if he owns one. But when I run the code I have it only checks the first house on the list when there is more to be checked. Please help me.

Here is what I have: (there are no spaces in the actual coding)

Код:

   			for(new idxx=1; idxx<MAX_HOUSE; idxx++)
			{
			    new name[MAX_PLAYER_NAME+1], string[24+MAX_PLAYER_NAME+1];
				GetPlayerName(playerid, name, sizeof(name));
				if(strcmp(HouseInfo[idxx][hOwner],name,true)!=0) //does not own
				{
				    SendClientMessage(playerid, COLOR_ORANGE, "YOUR HOUSE HAS BEEN SOLD DUE TO INACTIVITY");
				    PlayerInfo[playerid][Houses] = 0;
				    return 1;
				}
				else
				{
				    SendClientMessage(playerid, COLOR_ORANGE, "YOUR HOUSE AUTO-SELL TIME HAS BEEN SET TO 14 DAYS FROM NOW");
                                    ///code will soon be placed here
				    return 1;
				}

    		}

please help me

[Hanger]

arrays start with a zero!

HouseInfo[0][Owner] = FirstHouse

In your loop it starts to count from 1

Код:

for(new idxx=0; idxx<MAX_HOUSE; idxx++)

[jeffery30162]

it does start at one, this is not the problem

[Vince]

Do not return inside a loop. How do you know a player doesn't own a house, before having gone through the entire list? You don't. That's why that part is supposed to be placed after and outside the loop.

[jeffery30162]

yes, but when I do that it displays the messages for each house on the list.
I need it to go down the list, when it finds a name match it does something. If it gets to the end of the list and it doesnt find a match. it also does something

[R0]

try this:

pawn Код:

for(new idxx=1; idxx<MAX_HOUSE; idxx++)
            {
                new name[MAX_PLAYER_NAME+1], string[24+MAX_PLAYER_NAME+1];
                GetPlayerName(playerid, name, sizeof(name));
                if(PlayerInfo[playerid][Houses] > 0)
                {
                    if(strcmp(HouseInfo[idxx][hOwner],name,true)!=0) //does not own
                    {
                        SendClientMessage(playerid, COLOR_ORANGE, "YOUR HOUSE HAS BEEN SOLD DUE TO INACTIVITY");
                        PlayerInfo[playerid][Houses] = 0;
                        return 1;
                    }
                    else
                    {
                        SendClientMessage(playerid, COLOR_ORANGE, "YOUR HOUSE AUTO-SELL TIME HAS BEEN SET TO 14 DAYS FROM NOW");
                                        ///code will soon be placed here
                        return 1;
                    }
                }

            }

[Hanger]

Or just dont loop then

Код:

new name[MAX_PLAYER_NAME+1], string[24+MAX_PLAYER_NAME+1];
                GetPlayerName(playerid, name, sizeof(name));
                if(PlayerInfo[playerid][Houses] > 0)
                {
                    if(strcmp(HouseInfo[PlayerInfo[playerid][Houses]][hOwner],name,true)!=0) //does not own
                    {
                        SendClientMessage(playerid, COLOR_ORANGE, "YOUR HOUSE HAS BEEN SOLD DUE TO INACTIVITY");
                        PlayerInfo[playerid][Houses] = 0;
                        return 1;
                    }
                    else
                    {
                        SendClientMessage(playerid, COLOR_ORANGE, "YOUR HOUSE AUTO-SELL TIME HAS BEEN SET TO 14 DAYS FROM NOW");
                                        ///code will soon be placed here
                        return 1;
                    }
                }