Как отличить нормальный код от быдлокода:
1. Лишние скобки, которые ничего не выделяют.
2. Использование запятых в коде, когда они не нужны
3. Частичная оптимизация кода

Вот оптимизированная версия:

Код

forward OnVehicleDamageStatusUpdate(vehicleid, playerid);
public OnVehicleDamageStatusUpdate(vehicleid, playerid)  
{
  new vehid = GetPlayerVehicleID(playerid),
   Float: carhp;  
  GetVehicleHealth(vehicleid, carhp);  
  if(vehid == caridhouse[playerid] && proccar[vehid]++ > 16)  
  {  
   proccar[vehid] = 0,  
   PlayerInfo[playerid][pProcents]--;  
  }
  if(carhp < 400 || PlayerInfo[playerid][pProcents] < 2 && vehid == caridhouse[playerid])   
  {  
   GameTextForPlayer(playerid,"~r~ENGINE HAS BROKENED", 5000, 3);
   enginebrokened[vehicleid] = 1;
   player_NoCheckTimeVeh[playerid] = 1;  
   GetVehicleParamsEx(vehicleid,engine,lights,alarm,doors,bonnet,boot,objective);  
   SetVehicleParamsEx(vehicleid,VEHICLE_PARAMS_OFF,VEHICLE_PARAMS_OFF,alarm,doors,bonnet,boot,objective);  
   zavodis[vehid] = 0;  
  }
  return 1;  
}

Отличия:
1. Убрано использование макроса (Считаю некрасивым так делать).
2. Код написан в едином стиле (В двух строках скрипта использовалось ++ и -= 1)
3. Добавлена переменная vehid, чтобы несколько раз не вызывать функцию GetPlayerVehicleID
4. Убраны лишние детали
5. Значительно укорочено (Не смотря на дополнительную переменную и приводу кода к нормальному виду)