Помогите Как хранить ключи от кейсов игроков в Map

Bomber123

Bomber123

Активный участник
Пользователь
Баллы
76
Привет форумчане!
Не давно столкнулся с такой проблемой. Когда я запрашиваю ключи, и открываю кейс с другим игроком то количество ключей у обоих одинаково. Как исправить?
код
 
Последнее редактирование:
Сортировать по дате Сортировать по голосам
Объясняй нормально. а такой троллинг тонкий, что картинку с нета спер - топ. Хорошо, что я удосужился проверить, что это за творение гугла
 
За 0 Против
iD3LSY сказал(а):
Объясняй нормально. а такой троллинг тонкий, что картинку с нета спер - топ. Хорошо, что я удосужился проверить, что это за творение гугла
Нажмите, чтобы раскрыть...
Картинка это ссылка на паст бин.

Теперь вопрос в ТС.
HashMap<String, Integer> keys = new HashMap<>();

void add(Player p, int key){
keys.put(p.getNmae(), key)
}

void remove(Player p, int key) {
int _key = keys.get(p.getName());
_key -= key;
keys.put(p.getName(), _key);
}

Что из этого тебе не понятно?
 
За 0 Против
Exception_Prototype сказал(а):
Картинка это ссылка на паст бин.

Теперь вопрос в ТС.
HashMap<String, Integer> keys = new HashMap<>();

void add(Player p, int key){
keys.put(p.getNmae(), key)
}

void remove(Player p, int key) {
int _key = keys.get(p.getName());
_key -= key;
keys.put(p.getName(), _key);
}

Что из этого тебе не понятно?
Нажмите, чтобы раскрыть...
я написал, что удосужился проверить, а дальше - в коде он использует в addKey,removeKey переменную keys. Видимо не дошло использовать keys.put(p.getname(),keys.getOrDefault(p.getname(),0)-1);
 
За 0 Против
Exception_Prototype сказал(а):
Картинка это ссылка на паст бин.

Теперь вопрос в ТС.
HashMap<String, Integer> keys = new HashMap<>();

void add(Player p, int key){
keys.put(p.getNmae(), key)
}

void remove(Player p, int key) {
int _key = keys.get(p.getName());
_key -= key;
keys.put(p.getName(), _key);
}
Что из этого тебе не понятно?
Нажмите, чтобы раскрыть...
Посмотри внимательно на пасте бине.
 
За 0 Против
Exception_Prototype сказал(а):
HashMap<String, Integer> keys = new HashMap<>();

void add(Player p, int key){
keys.put(p.getNmae(), key)
}

void remove(Player p, int key) {
int _key = keys.get(p.getName());
_key -= key;
keys.put(p.getName(), _key);
}
Нажмите, чтобы раскрыть...
Небезопасный код. Лучше так:
Java: 
HashMap<String, Integer> keys = new HashMap<>();

int get(Player p) {
    Integer i = keys.get(p.getName());
    return i == null ? 0 : i;
}

void add(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i == null)
            keys.put(name, key);
        else
            keys.put(name, i + key);
    }
}

void remove(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i != null) {
            i -= key;
            if (i > 0)
                keys.put(name, i);
            else
                keys.remove(name);
        }
    }
}
 
За 0 Против
gamerforEA сказал(а):
Небезопасный код. Лучше так:
Java: 
HashMap<String, Integer> keys = new HashMap<>();

int get(Player p) {
    Integer i = keys.get(p.getName());
    return i == null ? 0 : i;
}

void add(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i == null)
            keys.put(name, key);
        else
            keys.put(name, i + key);
    }
}

void remove(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i != null) {
            i -= key;
            if (i > 0)
                keys.put(name, i);
            else
                keys.remove(name);
        }
    }
}
Нажмите, чтобы раскрыть...
Ребята ну если помогаете то посмотрите на паст бине я для кого код оставлял.
Там написано что метод add принимает два аргумента игрока и количество ключей а remove сокращает их на 1.
И метод add статический чтобы можно было управлять им в другом классе. А класс этот для выдачи этих ключей с помощью команды /getkey [nickname] [amount]. Усё. Мне надо чтобы у каждого игрока было своё количество ключей. Чтобы не было вот так
kakpng_5005008_27379396.png
kakpng_5005008_27379396.png

а потом и у обоих по 20.
 
За 0 Против
gamerforEA сказал(а):
Небезопасный код. Лучше так:
Java: 
HashMap<String, Integer> keys = new HashMap<>();

int get(Player p) {
    Integer i = keys.get(p.getName());
    return i == null ? 0 : i;
}

void add(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i == null)
            keys.put(name, key);
        else
            keys.put(name, i + key);
    }
}

void remove(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i != null) {
            i -= key;
            if (i > 0)
                keys.put(name, i);
            else
                keys.remove(name);
        }
    }
}
Нажмите, чтобы раскрыть...
И скинул не инкапсулированный код, норм.
 
За 0 Против
Bomber123 сказал(а):
Ребята ну если помогаете то посмотрите на паст бине я для кого код оставлял.
Там написано что метод add принимает два аргумента игрока и количество ключей а remove сокращает их на 1.
И метод add статический чтобы можно было управлять им в другом классе. А класс этот для выдачи этих ключей с помощью команды /getkey [nickname] [amount]. Усё. Мне надо чтобы у каждого игрока было своё количество ключей. Чтобы не было вот так
kakpng_5005008_27379396.png
kakpng_5005008_27379396.png

а потом и у обоих по 20.
Нажмите, чтобы раскрыть...
Ну ясен пень у обоих 20 потом... Ты ж используешь глобальную переменную для хранения ключей, причем для всех игроков.

Возьми этот код, инкапсулируй его, если хочешь, и переделай под себя.
gamerforEA сказал(а):
Небезопасный код. Лучше так:
Java: 
HashMap<String, Integer> keys = new HashMap<>();

int get(Player p) {
    Integer i = keys.get(p.getName());
    return i == null ? 0 : i;
}

void add(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i == null)
            keys.put(name, key);
        else
            keys.put(name, i + key);
    }
}

void remove(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i != null) {
            i -= key;
            if (i > 0)
                keys.put(name, i);
            else
                keys.remove(name);
        }
    }
}
Нажмите, чтобы раскрыть...
 
За 0 Против
И поменяй уже название темы хотя бы на "Как хранить ключи от кейсов игроков в Map", а то сразу не понятно, что ты хочешь.
 
За 0 Против
gamerforEA сказал(а):
Небезопасный код. Лучше так:
Java: 
HashMap<String, Integer> keys = new HashMap<>();

int get(Player p) {
    Integer i = keys.get(p.getName());
    return i == null ? 0 : i;
}

void add(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i == null)
            keys.put(name, key);
        else
            keys.put(name, i + key);
    }
}

void remove(Player p, int key) {
    if (key > 0) {
        String name = p.getName();
        Integer i = keys.get(name);
        if (i != null) {
            i -= key;
            if (i > 0)
                keys.put(name, i);
            else
                keys.remove(name);
        }
    }
}
Нажмите, чтобы раскрыть...
getOrDefault для кого придумали?
 
За 0 Против
Код: 
    private static Map<String, Integer> player_keys = new HashMap<>();
    
    
    public static int get(String name) {
        return player_keys.getOrDefault(name, 0);
    }
    
    public static void add(String name, int count) {
        player_keys.put(name, get(name)+count);
    }
    
    public static int remove(String name) {
        int count = get(name);
        if(count <= 0)return 0;
        return player_keys.put(name, count-1);
    }
 
За 0 Против
Bomber123 сказал(а):
Привет форумчане!
Не давно столкнулся с такой проблемой. Когда я запрашиваю ключи, и открываю кейс с другим игроком то количество ключей у обоих одинаково. Как исправить?
код
Нажмите, чтобы раскрыть...
А тебе не насрать, что при перезагрузке сервера всё пропадёт?
 
За 0 Против
Rider3217 сказал(а):
Код: 
    private static Map<String, Integer> player_keys = new HashMap<>();
   
   
    public static int get(String name) {
        return player_keys.getOrDefault(name, 0);
    }
   
    public static void add(String name, int count) {
        player_keys.put(name, get(name)+count);
    }
   
    public static int remove(String name) {
        int count = get(name);
        if(count <= 0)return 0;
        return player_keys.put(name, count-1);
    }
Нажмите, чтобы раскрыть...
Мб так будет правильнее:
private static Map<String, Integer> player_keys = new HashMap<String, Integer>();
 
За 0 Против
Для ответа нужно войти/зарегистрироваться
Назад
Верх