[Pottus]

Suggestions for some improvement

1) There is no need for rCreated in your enum initialize rObject entries to INVALID_OBJECT_ID
2) This code is written incorrectly it will always do one iteration

pawn Code:

for(new i = 0; i < sizeof(RoadBlockInfo); i++)
        {
            if(IsPlayerInRangeOfPoint(playerid,5.0,RoadBlockInfo[i][rX],RoadBlockInfo[i][rY],RoadBlockInfo[i][rZ]))
            {
                SendClientMessage(playerid,-1,"You have succesfully removed the closest roadblock");
                DeleteClosestRB(playerid);
                return 1;
            }
            else
            {
                SendClientMessage(playerid,-1,"You are not near to any roadblock!");
                return 1;
            }
        }
        return 1;

You need to do it like this

pawn Code:

for(new i = 0; i < sizeof(RoadBlockInfo); i++)
        {
            if(IsPlayerInRangeOfPoint(playerid,5.0,RoadBlockInfo[i][rX],RoadBlockInfo[i][rY],RoadBlockInfo[i][rZ]))
            {
                SendClientMessage(playerid,-1,"You have succesfully removed the closest roadblock");
                DeleteClosestRB(playerid);
                return 1;
            }
        }
                SendClientMessage(playerid,-1,"You are not near to any roadblock!");
        return 1;

3) Return the index of the road block when being created instead of returning 1 that way it can be referenced if needed return INVALID_ROAD_BLOCK if failed.... #define INVALID_ROAD_BLOCK (0xFFFF)

4) Provide an option to use incognito streamer dynamic objects instead of create object

pawn Code:

#define USE_STREAMER

#if defined USE_STREAMER
RoadBlockInfo[i][rObject] = CreateDynamicObject(973, x, y, z-0.9, 0, 0, Angle-90);
#else
RoadBlockInfo[i][rObject] = CreateObject(973, x, y, z-0.9, 0, 0, Angle-90);
#endif

5) Provide an optional parameter for the modelid

pawn Code:

stock CreateRB(Float:x,Float:y,Float:z,Float:Angle,modelid = 973)