[Résolu] Problème d'écriture dans une table

Il suffit juste de ne pas faire de table.insert dans ton onClientRender.

J'ai séparer ta fonction en 2 dont chacune à son rôle définit:

-- S'occupe juste de mettre à jour pVehicles (à appeler lors d'un achat et vente d'un veh) 
function updatePlayerVehicles() 
    pVehicles = {} 
    for i,v in ipairs (getElementsByType("vehicle")) do 
        local owner = getElementData(v, "owner") 
        local pName = getPlayerName(localPlayer) 
        if (owner == pName) then 
            table.insert(pVehicles, v) 
        end 
    end 
end 
addEvent("onPlayerVehiclesChanged", true) 
addEventHandler("onPlayerVehiclesChanged", localPlayer, updatePlayerVehicles) 
  
-- Va retoruner "Sanchez | Monser 3" 
function getPlayerVehiclesStr() 
    local vehStr = "" 
    for k, v in ipairs( pVehicles ) do 
        local vehName = getVehicleName(v) 
        vehStr = vehStr + vehName 
        if k < #pVehicles then 
            vehStr = vehStr + " | " 
        end 
    end 
    return vehStr 
end 

La première se charge juste de mettre à jour la table pVehicles et va contenir la liste des véhicules (et non pas leurs noms) que le joueur possède. Cette fonction est à appeler à chaque fois qu'il y aura une modification dans la liste des véhicules possédés par le joueur.

Comme j'imagine que ce genre de modification sera réalisé du côté serveur, j'ai fait en sorte que tu puisses appeler cette fonction, qui est du côté client via la ligne suivante:

triggerClientEvent(player, "onPlayerVehiclesChanged", player) 

la variable player doit évidemment être le joueur qui à réalisé la modification (achat, vente).

La deuxième se charge de construire ton "tableau" en parcourant la liste des véhicules et en les mettant à la suite dans un string et en les séparant via un pipe ("|"). Ça sera la méthode à appeler dans ton onClientRender:

function Infos() 
    --[...]-- 
    local vehStr = getPlayerVehiclesStr() 
    --[...]-- 
    dxDrawText("Véhicules: "..vehStr, screenW * 0.3968, screenH * 0.5664, screenW * 0.6032, screenH * 0.5924, tocolor(255, 255, 255, 255), 1.00, "default", "left", "center", false, false, false, false, false) 
    --[...]-- 
end 

Avec ces modifications là, ça devrait fonctionner comme tu le souhaites.

Oui ça aurait fonctionner mais au niveau perfs ça n'aurait pas été terrible.

Il faut éviter d'exécuter du code trop compliqué/long a exécuter dans un onClientRender.

En plus dans ton cas, la valeur afficher (la liste des véhicules) ne va pas changer souvent pour un joueur (aller, grand maximum 10 fois dans le mois ).

Imaginons qu'un joueur fait tourner son jeu à 30FPS, ton code allait boucler sur TOUS les véhicules présent sur le serveur (potentiellement >500 vehs), en récupérant le owner et reconstruisant la table et tout ça 30 fois par seconde !! (et 60fois/s pour 60FPS etc).

Tu aurais utilisé pas mal de ressource pour rien et qui aurait pu être utiliser pour autre chose.

Il vaut mieux recalculer la table que quand il y en a besoin (lorsque le joueur se connecte, lorsqu'il achète, et lorsqu'il vend ou échange un véhicule).

On aurait pu encore améliorer les perfs en évitant de reconstruire le string que retourne getPlayerVehiclesStr pour exactement le même raisons (tu peux le faire si tu veux en mettant le string en global que tu utiliseras directement dans ton dxDrawText). Mais le gain de perfs pour cette fonction sera minime.

Le onClientRender est très pratique mais il faut faire attention à ce qu'on lui fait faire. A utiliser avec précaution !

Cordialement,

Citizen