[BlackRaven]

pawn Код:

public OnPlayerPickUpPickup(playerid, pickupid)
{
    for(new i; i < MAX_HOUSES; i++)
    {
        if(pickupid == housepickup[i])
        {
            new str[120];
            format(str, sizeof(str),"You are standing on %s house",houseinfo[i][owner]);
            printf("%s",houseinfo[i][owner]);
            SendClientMessage(playerid,COLOR_GREEN,str);
        }
    }
    return 1;
}

The loop prints 500 times the message , what is wrong?

[Da_Noob]

My guess is that you have 500 housepickups. Everytime the server detects a housepickup it will send a message (the message you formatted).

[EpicNinja]

pawn Код:

public OnPlayerPickUpPickup(playerid, pickupid)
{
    for(new i; i < MAX_HOUSES; i++)
    {
        if(pickupid == housepickup[i])
        {
            new str[120];
            format(str, sizeof(str),"You are standing on %s house",houseinfo[i][owner]);
            printf("%s",houseinfo[i][owner]);
        }
    }
    SendClientMessage(playerid,COLOR_GREEN,str);
    return 1;
}

Sending message to playerid inside loop sends him the message about "MAX_HOUSES" times.

[coakiddo]

Use break; to stop the loop.

[BlackRaven]

now it doesnt print the owner of house

[Babul]

the reason is the SendClientMessage being outside of the loop. i assume there is only 1 house for a player, so if you break/return from the nested loop (good idea btw), it wont call the SCM. try this, slightly changed:

Код:

public OnPlayerPickUpPickup(playerid, pickupid)
{
	for(new i; i < MAX_HOUSES; i++)
	{
		if(pickupid == housepickup[i])
		{
			new str[120];
			format(str, sizeof(str),"You are standing on %s house",houseinfo[i][owner]);
			printf("%s",houseinfo[i][owner]);
			SendClientMessage(playerid,COLOR_GREEN,str);
			return 1;
		}
	}
	return 1;
}

[BlackRaven]

Thanks Babul. But now it doesnt display the name of the house owner.