Gut das du fragst.
Ich habe erstmal alle IF Abfragen gekürzt
ocmd:getincar(playerid,params[])
{
new pID,string[128],Float:x,Float:y,Float:z;
if(sscanf(params,"u",pID))return SendClientMessage(playerid,HELLBLAU,"/giv [ID]");
if(!isPlayerInFrakt(playerid,1))return SendClientMessage(playerid,GRAU,"Du bist kein Cop!");
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return SendClientMessage(playerid,HELLBLAU,"Du bist nicht der Fahrer!"); //Hier
if(GetPlayerState(pID) != PLAYER_STATE_ONFOOT) return SendClientMessage(playerid,HELLBLAU,"Dieser Spieler ist nicht am Laufen!"); //Hier
if(!IsPlayerInRangeOfPoint(pID,10,x,y,z))return SendClientMessage(playerid,HELLBLAU,"Dieser Spieler ist nicht in deiner Nähe!"); //Hier habe Ich es nach oben gesetzt, damit vehiclieid nicht unnltig gesetzt wird falls er nicht in der Nähe ist.
new vehicleid = GetPlayerVehicleID(playerid);
GetPlayerPos(playerid,x,y,z);
PutPlayerInVehicle(pID, vehicleid, 3);
format(string,sizeof(string),"Du hast %s in dein Auto gezogen!Vergesse nicht ihn zu fesseln!(/cuff)",SpielerName(pID));
SendClientMessage(playerid,HELLBLAU,string);
format(string,sizeof(string),"Du wurdest von %s in sein Auto gezogen!",SpielerName(playerid));
SendClientMessage(pID,GRAU,string);
return 1;
}
Ich bin nur die Kriterien durchgegangen.
Man sollte wenn möglich if-Abfragen ohne else nutzen.
Zu deinen Fehler:
ocmd:getincar(playerid,params[])
{
new pID,string[128],Float:x,Float:y,Float:z;
if(sscanf(params,"u",pID))return SendClientMessage(playerid,HELLBLAU,"/giv [ID]");
if(GetPlayerState(playerid) == PLAYER_STATE_DRIVER) //"{" Fehlt, hat Ja [P4L]Chris@Key schon erwähnt
if(!isPlayerInFrakt(playerid,1))return SendClientMessage(playerid,GRAU,"Du bist kein Cop!");
{
if(GetPlayerState(pID) == PLAYER_STATE_ONFOOT)
{
new vehicleid = GetPlayerVehicleID(playerid);
GetPlayerPos(playerid,x,y,z);
if(!IsPlayerInRangeOfPoint(pID,10,x,y,z))return SendClientMessage(playerid,HELLBLAU,"Dieser Spieler ist nicht in deiner Nähe!"); // Das wie schon erwähnt nach oben setzen, damit nicht unnötige Variablen gesetzt werden.
PutPlayerInVehicle(pID, vehicleid, 3);
format(string,sizeof(string),"Du hast %s in dein Auto gezogen!Vergesse nicht ihn zu fesseln!(/cuff)",SpielerName(pID));
SendClientMessage(playerid,HELLBLAU,string);
format(string,sizeof(string),"Du wurdest von %s in sein Auto gezogen!",SpielerName(playerid));
SendClientMessage(pID,GRAU,string);
return 1;
}
else
{
SendClientMessage(playerid,HELLBLAU,"Dieser Spieler ist nicht am Laufen!");
return 1;
}
} //<- das hat gefehlt.
else //HIER SOLL DER EROOR SEIN
{
SendClientMessage(playerid,HELLBLAU,"Du bist nicht der Fahrer!");
return 1;
}
}
}
Hoffe das diese Erklärung ausreicht 
EDIT:
Zu deiner Frage:
Einfach Freeze wenn er im Auto sitzt:
TogglePlayerControllable