minetest_modding_book/_it/quality/common_mistakes.md

---
title: Errori comuni
layout: default
root: ../..
idx: 8.1
redirect_from: /it/chapters/common_mistakes.html
---

## Introduzione <!-- omit in toc -->

Questo capitolo illustra gli errori più comuni e come evitarli.

- [Fai attenzione quando salvi gli ObjectRef in delle variabili (giocatori ed entità)](#fai-attenzione-quando-salvi-gli-objectref-in-delle-variabili-giocatori-ed-entità)
- [Non fidarti dei campi dei formspec](#non-fidarti-dei-campi-dei-formspec)
- [Imposta gli ItemStack dopo averli modificati](#imposta-gli-itemstack-dopo-averli-modificati)

## Fai attenzione quando salvi gli ObjectRef in delle variabili (giocatori ed entità)

Un ObjectRef viene invalidato quando il giocatore o l'entità che esso rappresenta abbandona il gioco.
Ciò si verifica quando un giocatore si disconnette, o quando un'entità viene rimossa dalla memoria.

Da Minetest 5.2, i metodi degli ObjectRef ritorneranno sempre `nil` quando non validi.
In altre parole, ogni chiamata verrà ignorata.

Si dovrebbe evitare quanto possibile di immagazzinare gli ObjectRef in delle variabili.
Se ciò tuttavia accade, assicurati di controllare se esiste ancora, come illustrato qui di seguito:

```lua
-- questo codice funziona solo da Minetest 5.2 in poi
if obj:get_pos() then
	-- è valido!
end
```

## Non fidarti dei campi dei formspec

Client malevoli possono compilare i campi nei formspec quando vogliono, con qualsiasi contenuto vogliono.

Per esempio, il seguente codice presenta una vulnerabilità che permette ai giocatori di assegnarsi da soli il privilegio di moderatore:

```lua
local function show_formspec(name)
    if not minetest.check_player_privs(name, { privs = true }) then
        return false
    end

    minetest.show_formspec(name, "modman:modman", [[
        size[3,2]
        field[0,0;3,1;target;Nome;]
        button_exit[0,1;3,1;sub;Promuovi]
    ]])
    return true
})

minetest.register_on_player_receive_fields(function(player,
        formname, fields)
    -- MALE! Manca il controllo dei privilegi!

    local privs = minetest.get_player_privs(fields.target)
    privs.kick  = true
    privs.ban   = true
    minetest.set_player_privs(fields.target, privs)
    return true
end)
```

Aggiungi un controllo dei privilegi per ovviare:

```lua
minetest.register_on_player_receive_fields(function(player,
        formname, fields)
    if not minetest.check_player_privs(name, { privs = true }) then
        return false
    end

    -- code
end)
```

## Imposta gli ItemStack dopo averli modificati

Se ci si fa caso, nella documentazione si parla di `ItemStack` e non `ItemStackRef`.
Questo perché gli ItemStack NON sono un riferimento, bensì una copia.
Questo vuol dire che modificando la copia, non si modificherà in automatico anche l'originale.

Sbagliato:

```lua
local inv = player:get_inventory()
local pila = inv:get_stack("main", 1)  -- lo copio
pila:get_meta():set_string("description", "Un po' smangiucchiato")
-- MALE! Le modifiche saranno perse
```

Giusto:

```lua
local inv = player:get_inventory()
local pila = inv:get_stack("main", 1)  -- lo copio
pila:get_meta():set_string("description", "Un po' smangiucchiato")
inv:set_stack("main", 1, pila)
-- Corretto! L'ItemStack è stato cambiato con la copia
```

Il comportamento dei callback è leggermente più complicato.

```lua
minetest.register_on_item_eat(function(hp_change, replace_with_item,
        itemstack, user, pointed_thing)
    itemstack:get_meta():set_string("description", "Un po' smangiucchiato")
    -- Quasi corretto! I dati saranno persi se un altro callback annulla questa chiamata
end)
```

Se nessun callback cancella l'operazione, la pila sarà impostata e la descrizione aggiornata; ma se un callback effettivamente cancella l'operazione, l'aggiornamento potrebbe andar perduto.

È meglio quindi fare così:

```lua
minetest.register_on_item_eat(function(hp_change, replace_with_item,
        itemstack, user, pointed_thing)
    itemstack:get_meta():set_string("description", "Un po' smangiucchiato")
    user:get_inventory():set_stack("main", user:get_wield_index(),
            itemstack)
    -- Corretto! La descrizione verrà sempre aggiornata
end)
```