Как более эффективно и быстро научится кодить

[Deazer]

Все потому что на вас нет Бьёрна Страуструпа

[Zubastic]

Не забываем о косяках вроде расхода памяти, указателей и тд.

[OneThunder]

Модераторы, Можно я в эту тему иногда буду задавать вопросы по кодингу? Кто сможет ответит )

[flopix]

А почему называете "кодинг", почему не "программирование"?

"Кодинг" в момем понятии это написание программы по четкому сценарию, когда заказчиком решено все вплоть до используемых для реализации алгоритмов.

[OneThunder]

А почему называете "кодинг", почему не "программирование"?

"Кодинг" в момем понятии это написание программы по четкому сценарию, когда заказчиком решено все вплоть до используемых для реализации алгоритмов.

Слово короче ХД

Дабы не постить еще 1 мессагу)) В общем дело пошло поехало))) к конце недели покажу мини игру который я делаю))) Нашёлся хороший учитель)

[OneThunder]

день хз какой: HashMap

Такой вопросик

Код:

public class DataDefinition
{

    private Vector2[] vectorList2;
    private Vector3[] vectorList3;
    private Vector4[] vectorList4;
    
    private  HashMap<Integer, Vector2>  _mapElementsV2 =  new HashMap<Integer, Vector2>();
    private  HashMap<Integer, Vector3>  _mapElementsV3 = null;
    private  HashMap<Integer, Vector4>  _mapElementsV4 = null;
    public int key=0;
    
    public boolean addMapValueV2 (Vector2 v)
    {     ++key;
    
        //Checkers
        if(_mapElementsV2 == null)
        {
            System.out.println("_mapElementsV2 not initialized!");
            return false;
        }
        
        if (_mapElementsV2.put(key, v) == null)
        {
            return false;
        }

        return true;
    }
        
    public boolean replaceMapValueV2 (int k, Vector2 v)
    {
        
        if(_mapElementsV2.replace(k, v) == null)
        {
            return false;
        }
        _mapElementsV2.replace(k, v);
        return true;
    }
    
    /**
     *
     * @param vector2
     * @return
     */

    public boolean removeMapValueV2 (int keyId)
    {
        if(_mapElementsV2.remove(keyId) == null)
        {
            return false;
        }
        //System.out.print("size="+_mapElementsV2.size());
        
        return true;
        
    }
    
    
    // PRINTERS
    public void printMapV2()
    {
        int keyIdIncrement = 0;
        for (Vector2 v: getMapElementsV2().values())
        {
        
            System.out.println("[ ID:" + keyIdIncrement++  + " , X:" + v.getX() + " , Y:" + v.getY() + " ]");
        }
    }

Код:

package commands;

import java.util.Random;
import java.util.Scanner;

import main.DataDefinition;
import main.Inputs;
import main.MainGame;
import util.Vector2;

public class CommandCollection
{
    
    public MainGame _mainGameHandler = null;                     // this is handler for MainGame object.
    
    private int _x;
    private int _y;
    

    public CommandCollection(MainGame mainGameHandler)
    {
        _mainGameHandler = mainGameHandler;
    }
    
    public void addNewLocationV2()
    {
        if(_mainGameHandler == null || _mainGameHandler.getDatastorage() == null)
        {
            System.out.println("_mainGameHandler is not initialized!");
            return;
        }

        //@TODO Checker for symbols
        Scanner scn = new Scanner(System.in);

        {
            System.out.println("Enter Vector2 Location:");    
            System.out.println("Enter Coordinates for : x , y:");    
            
            _x = scn.nextInt();        //Read X Value
            _y = scn.nextInt();        //Read Y Value
            
            _mainGameHandler.getDatastorage().addMapValueV2(new Vector2(_x, _y));
            System.out.println("Data was succesfully inserted:");
            _mainGameHandler.getDatastorage().printMapV2();
                
        }

        System.out.println("\n\n");
        _mainGameHandler.input.keyboardInput();
        scn.close();
    }
    
    public void removeLocationV2()
    {
        if(_mainGameHandler == null || _mainGameHandler.getDatastorage() == null)
        {
            System.out.println("_mainGameHandler is not initialized!");
            return;
        }

        //@TODO Checker for symbols
        Scanner scn = new Scanner(System.in);

        {
            System.out.println("Enter Vector2 Location ID you want to remove:");    
            int removableKeyId;
            removableKeyId = scn.nextInt();
            _mainGameHandler.getDatastorage().removeMapValueV2(removableKeyId); // here the method knows what id user want to removesec gimme
            System.out.println("Data was succesfully removed:");
            _mainGameHandler.getDatastorage().printMapV2();
                
        }


        System.out.println("\n\n");
        _mainGameHandler.input.keyboardInput();
        scn.close();
    }

......

public void showAllStoredData()
    {
        
        Inputs input = new Inputs(_mainGameHandler);
        DataDefinition dataStorage = new DataDefinition();
        Scanner scn = new Scanner(System.in);
        
        //Null Checker
        if (dataStorage.getMapElementsV2() == null)
        {
            System.out.println("Error: Data is null");
            input.keyboardInput();
            scn.close();
        }
        
        dataStorage.printMapV2();
        dataStorage.getMapElementsV2().get(_mainGameHandler);
        
        _mainGameHandler.getDatastorage().printMapV2();
        System.out.println("\n");
        //Returning to Inputs
        input.keyboardInput();
        scn.close();
    }

Значит фишка вот в чем

Я ввожу данные в хэш мапу.

Он их вводит с кейИд начиная с 1 2 3 4...

Я ввожу 3 к примеру числовых данных. [10 10] , [20 20] , [30 30]

Удаляю первый идшник. ОК работает

Добавляю еще 1 Дату, 555 555 к примеру

И при удаление еще раз он не сортирует их заного с 1 2 3 4 а берет к примеру 2 3 4 и так далее т.е ничего не происходит ибо он не находит нужный ид

Мне нужен метод пересортировки после удаления. Правильно? Подскажите куда копать










////UPDATE

Сделал небольшой дебажичег с КейСет...

Вот каковы реальные значение ид в мапе



А мне надо при каждом удаление их пересобрать в 1 2 3 4 5 и так далее

[n3k0nation]

Пост не читал@Click-clack

[OneThunder]

Код:

    public void RefreshKeys()
    {
        HashMap<Integer, Vector2> _localmapElementsV2 = new HashMap<Integer, Vector2>();
        int localKeyVar = 0;
        for(Vector2 v : _mapElementsV2.values())
        {
            _localmapElementsV2.put(++localKeyVar, v);
        }
        _mapElementsV2 = _localmapElementsV2;
        
    }

Всем спасибо на покачто)) Буду учится дальше))

[Donatte]

Не рекомендую использовать без необходимости увеличение на единицу до выполнения операции типа ++i.

В вашем примере вы объявили значение переменной localKeyVar = 0, но оно не используется. В итоге у вас карта начинается с 1.

На мой взгляд правильнее было бы так:
[SRC="java"] public void RefreshKeys()
{
List<Vector2> _cashVectors = _mapElementsV2.values();
Map<Integer, Vector2> _mapElementsV2 = new HashMap<>(_cashVectors.size());

int localKeyVar = 0;
for (Vector2 v : _cashVectors)
_mapElementsV2.put(localKeyVar++, v);
}[/SRC]