[Hunud]

Hey,

I have problem when i want to delete bussiness its say You are not close to bussiness but im standing there! +REP if solved

Код:

CMD:deletebis(playerid, params[])
{
	// Setup local variables
	new file[100], Msg[128];

	// Send the command to all admins so they can see it
	SendAdminText(playerid, "/deletebis", params);

	// Check if the player has logged in
	if (APlayerData[playerid][LoggedIn] == true)
	{
		// Check if the player's admin-level is at least 5
		if (APlayerData[playerid][PlayerLevel] >= 7)
		{
			// Make sure the player isn't inside a vehicle
			if (GetPlayerVehicleID(playerid) == 0)
			{
				// Loop through all player-owned businesses
				for (new BusID = 1; BusID < MAX_BUSINESS; BusID++)
				{
					// Check if the business exists
					if (ABusinessData[BusID][PickupID] != 0)
					{
						// Check if the business has no owner
						if (ABusinessData[BusID][Owned] == false)
						{
							// Check if the player is in range of the business-pickup
							if (IsPlayerInRangeOfPoint(playerid, 2.5, ABusinessData[BusID][BusinessX], ABusinessData[BusID][BusinessY], ABusinessData[BusID][BusinessZ]))
							{
								// Clear all data of the business
	                            ABusinessData[BusID][BusinessName] = 0;
	                            ABusinessData[BusID][BusinessX] = 0.0;
	                            ABusinessData[BusID][BusinessY] = 0.0;
	                            ABusinessData[BusID][BusinessZ] = 0.0;
	                            ABusinessData[BusID][BusinessType] = 0;
	                            ABusinessData[BusID][BusinessLevel] = 0;
	                            ABusinessData[BusID][LastTransaction] = 0;
	                            ABusinessData[BusID][Owned] = false;
	                            ABusinessData[BusID][Owner] = 0;
								// Destroy the mapicon, 3DText and pickup for the house
								DestroyDynamicPickup(ABusinessData[BusID][PickupID]);
								DestroyDynamicMapIcon(ABusinessData[BusID][MapIconID]);
								DestroyDynamic3DTextLabel(ABusinessData[BusID][DoorText]);
	                            ABusinessData[BusID][PickupID] = 0;
	                            ABusinessData[BusID][MapIconID] = 0;

								// Delete the business-file
								format(file, sizeof(file), BusinessFile, BusID); // Construct the complete filename for this business-file
								if (fexist(file)) // Make sure the file exists
									fremove(file); // Delete the file

								// Also let the player know he deleted the business
								format(Msg, 128, "{00FF00}Izbrisao si biz sa ID: {FFFF00}%i", BusID);
								SendClientMessage(playerid, 0xFFFFFFFF, Msg);

								// Exit the function
								return 1;
							}
						}
					}
				}

				// There was no house in range, so let the player know about it
				SendClientMessage(playerid, 0xFFFFFFFF, "{FF0000}You are not close bussiness");
			}
			else
			    SendClientMessage(playerid, 0xFFFFFFFF, "{FF0000}Nemoћeљ brisat biznis dok si u autu");
		}
		else
		    return 0;
	}
	else
	    return 0;

	// Let the server know that this was a valid command
	return 1;
}

[Hunud]

Please help for +REP

[Kaliber]

Quote:

for (new BusID = 1; BusID < MAX_BUSINESS; BusID++)

Well, actually an Array starts at zero not one.

So do:

PHP код:

new BusID = 0; 


Greekz

[Hunud]

Same problem its say "There is no business in your range" but im standing to business

[Kaliber]

Then...check whats happening?

...use print to check what values they have...

Are admin level >= 7..is pickup != 0...are the coords right..is there no owner..

Check all this...

We can't help you..cause we have no data...

[PrO.GameR]

Make debug msgs and make it print your pos and the bizzpos, you might not have saved the XYZ properly.

Also as a suggestion I suggest when checking player related conditions like their adminlvl or w/e, check if they don't have it I.E:

PHP код:

if (APlayerData[playerid][LoggedIn] != true) return SendClientMessage(playerid,-1,"Not logged in."); 


This way you will avoid so many unnecessary braces and make your code more readable, at least thats my idea.

[Hunud]

Solved my myself! Thanks for repling!