Карта сети

[LeonidM/Dasperal]

private void map_Scan(World world, int X, int Y, int Z)
{
    int lastCaunt = 0;
    int Count = 1;
    for(int radius = 1; Count != lastCaunt; radius++)
    {
        lastCaunt = Count;
        Count = 0;
        map.clear();
        for(int x = -radius; x == radius; x++)
        {
            for(int y = -radius; y == radius; y++)
            {
                for(int z = -radius; z == radius; z++)
                {
                    TileEntity t = world.getTileEntity(X + x, Y + y, Z + z);
                    if(t instanceof IPowerProvider && ((IPowerProvider)t).isProvider()){ map.add(new Point(x, y, z, PointType.Provaider)); Count++;}
                    else if(t instanceof IPowerEmitter && ((IPowerEmitter)t).isEmitter()){ map.add(new Point(x, y, z, PointType.Emiter)); Count++;}
                    else if(t instanceof IPowerRequester && ((IPowerRequester)t).isRequester()){ map.add(new Point(x, y, z, PointType.Requester)); Count++;}
                    else{map.add(new Point(x, y, z, PointType.Null)); Count++;}
                }
            }
        }
    }
}

Почему List map пустой?
p.s. Как этот метод можно оптимизировать?

 

[laz2727]

Ну, во-первых, iirc, ключ в карте идет первым...

 

[Dragon2488]

Ты вообще понимаешь код? У него карта своих блоков-устройств.

 

[LeonidM/Dasperal]

Переменную поправил. Карта сети проводов по которым передаются данные.

 

[Nao]

У тебя в первом же цикле на каждой итерации в начале идет map.clear(). Соответственно, если на последней итерации ни одно из условий внутренних циклов не выполнено, то твой список будет пустым.

 

[Oldestkon]

        for(int x = -radius; x == radius; x++)
        {
            for(int y = -radius; y == radius; y++)
            {
                for(int z = -radius; z == radius; z++)
                {

Я совсем уже отупел, или не я?
Читай:
Пусть radius = 5;
[Начало цикла]
Пусть x = -5;
[Условие срабатывания цикла]
Пока x == 5;
[Итерация цикла]
x++;

Сколько раз твоё условие в цикле там сработает?
[merge_posts_bbcode]Добавлено: 02.02.2015 04:01:31[/merge_posts_bbcode]

А я такой всмотрелся в код, чуть его переоформил, логику оставил той же, только пофиксил логическую ошибку (поменял == на <=, что, видимо, подозревалось), и попрощался с логикой:

private void map_Scan(World world, int X, int Y, int Z) {
        int lastCaunt = 0;
        int Count = 1;
        
        for(int radius = 1; Count != lastCaunt; radius++) {
            lastCaunt = Count;
            Count = 0;
            map.clear();
            
            for(int x = -radius; x <= radius; x++)
                for(int y = -radius; y <= radius; y++)
                    for(int z = -radius; z <= radius; z++) {
                        TileEntity t = world.getTileEntity(X + x, Y + y, Z + z);
                        if (t instanceof IPowerProvider && ((IPowerProvider)t).isProvider())
                            map.add(new Point(x, y, z, PointType.Provaider));     //provaider, ok
                        else if(t instanceof IPowerEmitter && ((IPowerEmitter)t).isEmitter())
                            map.add(new Point(x, y, z, PointType.Emiter)); 
                        else if(t instanceof IPowerRequester && ((IPowerRequester)t).isRequester())
                            map.add(new Point(x, y, z, PointType.Requester)); 
                        else 
                            map.add(new Point(x, y, z, PointType.Null)); 
                        
                        Count++; 
                        //Всё тлен, уходим в бесконечность, уходим в небесаааа, 
                        //улетая в космос и залетая в чёёёрную дырууу;
                        //заметив сингулярность, сбиваем обороты, оглядываемся, и не видим нифига 
                        //толкового
                    }
        }
}

На самом деле, если мы увеличиваем радиус каждый раз, то как бы логично, что
число блоков в этой итерации не будет равно числу блоков в предыдущей итерации.
Да?

 

[LeonidM/Dasperal]

Да, если бы я считал блоки, но я считаю тайлы, наследующие IPowerProvider и рано или поздно они закончатся. И число найденных объектов в этой итерации будет равно количеству объектов в прошлой.  В коде, который я выложил в первом посте, была ошибка, что Count увеличивался даже если тайл не подходил. 

Как можно оптимизировать этот метод, а то при радиусе 90+начинает существенно подлагивать?

 

[Dragon2488]

Выполняй его не каждый тик.

 

[LeonidM/Dasperal]

Он и так выполняется 1 раз.

 

[laz2727]

Выполняй не в один тик тогда.