在播放器清单中堆叠项目(Java,Array / Arraylist)

时间:2011-12-09 17:12:23

标签: java arrays arraylist

编辑:我使用ArrayList

获得了工作版本
void addItem(Item item, int count){

        int stacks;

        for (int i = 0; i < useableSlots && count != 0; i++){
            if (itemSlot[i].stack.size() > 0){
                if (itemSlot[i].stack.get(0) == item) {
                    if(itemSlot[i].stack.size() < item.stackLimit){

                        int n = itemSlot[i].stack.size();

                        for(; n < item.stackLimit && count > 0; count--, n++) {
                            itemSlot[i].stack.add(item);
                        }
                    }
                }
            }
            if (i == (useableSlots - 1) && count > 0){
                for(int n = 0; n < useableSlots && count != 0; n++){

                    stacks = ((count - (count % item.stackLimit)) / item.stackLimit);

                    if(itemSlot[n].occupied == false){
                        if(stacks == 0){
                            for(int j = 0; j < count; j++){
                                itemSlot[n].stack.add(item);
                                itemSlot[n].occupied = true;
                            }
                            count = 0;
                        }
                        else {
                            for(int j = 0; j < item.stackLimit; j++){
                                itemSlot[n].stack.add(item);
                            }
                            count -= item.stackLimit;
                            itemSlot[n].occupied = true;
                        }
                    }
                    if (n == (useableSlots - 1)){
                        println("You don't have any room in your inventory");
                    }
                }
            }
        }
    }

package com.projects.aoa;

import java.util.*;

public class Itemslot extends Item{

    List<Item> stack = new ArrayList<Item>();
    Item oi = new Item();
    boolean occupied, isArray, full;

}

public class Inventory {

    int useableSlots, slots = 50;
    Itemslot[] itemSlot = new Itemslot[slots];


...}

我有一个由Item个对象组成的清单。我想要在占用另一个库存空间之前将相似的物品堆叠到一定限度。我必须以完全错误的方式解决这个问题,因为我的每次尝试都遇到了同样的问题。 hp药水stackLimit是25,(27 hpPotions需要两个库存空间,一个25个,另一个有2个。)

主要

playerOne.backPack.addItem(hpPotion, 20);
playerOne.backPack.printInventory();
playerOne.backPack.addItem(hpPotion, 15);
playerOne.backPack.printInventory();

Item.java

package com.projects.aoa;

import static com.projects.aoa.Print.*;

import java.util.*;
import java.io.*;

class Item {

//Item
String name, type, category;
int id;
int hp, mp, str, def, duration;

//Inventory
public boolean filled;
public int count, stackLimit;

static void getAllStats(Item[] e){
    for(Item i : e){
        getItemStats(i);
    }
}

static void getItemStats(Item i){
    i.getStats();
}

void getStats(){
    try {
        //System.out.println(System.getProperty("user.dir"));

        FileInputStream fstream = new FileInputStream(System.getProperty("user.dir") 
                + "/src/com/projects/aoa/" + this.type + "_" + this.name + ".txt");

        DataInputStream in = new DataInputStream(fstream);

        BufferedReader br = new BufferedReader(new InputStreamReader(in));

        String line;
        int counter = 0;

        while ((line = br.readLine()) != null) {
            if (line.length() == 0){
                break;
            }

            switch (counter) {
            case 0:
                this.hp = Integer.parseInt(line);
                counter++;
                break;
            case 1:
                this.mp = Integer.parseInt(line);
                counter++;
                break;
            case 2:
                this.def = Integer.parseInt(line);
                counter++;
                break;
            case 3:
                this.str = Integer.parseInt(line);
                counter++;
                break;
            case 4:
                this.stackLimit = Integer.parseInt(line);
                counter++;
                break;
            case 5:
                this.duration = Integer.parseInt(line);
                counter++;
                break;
            }   
        }


        in.close();
    } catch (Exception e) {
        e.printStackTrace();
    } 
}

void printStats(){
    println("[" + name + "]");
    println("Type: " + type);
    println("Duration: " + duration);
    println("HP:  " + hp);
    println("MP:  " + mp);
    println("Def: " + def);
    println("Str: " + str);
}
}

库存输出:第三次尝试

addItem(hpPotion,20);

[1]  Mallet              [2]  BronzeHelmet        [3]  hpPotion(20)        [4]  Empty               [5]  Empty               

[6]  Empty               [7]  Empty               [8]  Empty               [9]  Empty               [10] Empty               

[11] Empty               [12] Empty               [13] Empty               [14] Empty               [15] Empty               

[16] Empty               [17] Empty               [18] Empty               [19] Empty               [20] Empty  

addItem(hpPotion,15);

[1]  Mallet              [2]  BronzeHelmet        [3]  hpPotion(10)        [4]  hpPotion(10)        [5]  Empty               

[6]  Empty               [7]  Empty               [8]  Empty               [9]  Empty               [10] Empty               

[11] Empty               [12] Empty               [13] Empty               [14] Empty               [15] Empty               

[16] Empty               [17] Empty               [18] Empty               [19] Empty               [20] Empty 

同样的事情发生在第二次尝试中,除了它显示添加两个

[3]  hpPotion(35)        [4]  hpPotion(35)

它似乎只是一个简单的修复(也许是),但我不知道它可能是什么。任何帮助将不胜感激。

第三次尝试

void addItem(Item item, int count){

    boolean newStack = false;

    int room, totalCount = 0;

    for(int i = 0; i < 20; i++){
        if(itemSlot[i] == item && count > 0){
            room = (item.stackLimit - itemSlot[i].count);
            if(room > 0){
                if(count >= room){
                    itemSlot[i].count += room;
                    count -= room;
                }
                else if(count > 0 && count < room){
                    itemSlot[i].count += count;
                    count = 0;
                    break;
                }
            }
        }

        if(i >= 19 && count > 0){
            for(int n = 0; n < 20; n++){
                if(itemSlot[n].filled == false){
                    int stacks = ((count - (count % item.stackLimit)) / 25);
                    println(stacks);
                    if (stacks == 0){
                        itemSlot[n] = item;
                        itemSlot[n].filled = true;
                        itemSlot[n].count = count;
                        count = 0;
                        break;
                    }
                    else {
                        itemSlot[n] = item;
                        itemSlot[n].filled = true;
                        itemSlot[n].count = item.stackLimit;
                        count -= item.stackLimit;
                    }
                }
            }
        }
    }
}

第二次尝试

    void addItem(Item item, int count){

    boolean newStack = false;

    int room, totalCount = 0;

    outerLoopCurrentStack:
    for(int i = 0; i < 20; i++) {
        if(itemSlot[i].name == item.name){
            if(itemSlot[i].count < itemSlot[i].stackLimit){

                while(count > 0){
                    count--;
                    itemSlot[i].count++;

                    if(itemSlot[i].count == itemSlot[i].stackLimit) {
                        break outerLoopCurrentStack;
                    }
                }
            }
        }
        else if((i >= 19) && (count > 0)){
            newStack = true;
        }
    }

    if(newStack = true){

        outerLoopNewStack:
        for(int i = 0; i < 20; i++){
            if(itemSlot[i].filled == false){

                itemSlot[i] = item;
                itemSlot[i].filled = true;

                while(count > 0){
                    count--;
                    itemSlot[i].count++;

                    if(count == 0){
                        newItem = false;
                        count = 0;
                        break outerLoopNewStack;
                    }
                }
            }
        }
    }
}
     }  

第一次尝试

    void addItem(Item item, int count){

    boolean newStack = false;

    int room, totalCount = 0;

    for (int i = 0; i < 20; i++){
        if(itemSlot[i].name == item.name) {
            if(itemSlot[i].count < item.stackLimit){
                room = (item.stackLimit - itemSlot[i].count);
                if (count > room){
                    itemSlot[i].count += room;
                    itemSlot[i].filled = true;
                    count -= room;
                }
                else {
                    itemSlot[i].count += count;
                    break;
                }
            }
        }
        else if(i == 19){
            newStack = true;
        }
    }

    if (newStack == true){
        for(int i = 0; i < 20; i++){
             if(itemSlot[i].filled == false) {
                if(count > item.stackLimit){
                    itemSlot[i] = item;
                    itemSlot[i].count = item.stackLimit;
                    itemSlot[i].filled = true;
                    count -= item.stackLimit;
                }
                else{
                    itemSlot[i] = item;
                    itemSlot[i].count = count;
                    itemSlot[i].filled = true;
                    break;
                }
            }
        }
    }
     }

3 个答案:

答案 0 :(得分:2)

看起来您将同一个对象放入两个广告资源位置,因此当您更改.count成员时,它会在两个广告位中更改它。

第一个标志是您将现有项目与使用==插入的项目进行比较。当您满溢时,请尝试更改

itemslot[i] = new Item(item);

而不是

itemslot[i] = item;

假设你有一个复制构造函数。

答案 1 :(得分:1)

Look into the Composite Pattern,关注项目是广告资源“内容”的一部分。

然后你可以把一个包含在用户内容中的包,同时包含在包中的内容。

计算体重等顶级资料会总结当前容器中每件物品的重量。如果这涉及询问作为容器的物品的重量,则容器物品将使用相同的技术计算它的重量(直到最终非容器物品仅报告其未计算的重量)。

无论哪种方式,你都希望摆脱表现出行为的集合的数组,因为你不能将行为添加到带有数组的集合中,并且将它放在外面(在一堆方法中)将最终使你的代码难以维护/修复/改变。

答案 2 :(得分:1)

好的,我认为有两种可能的方法可以做到这一点,我可能会采取另一种方式。

替代方法
我会将背包技工从实际的数据存储中抽象出来。我会有一个数组或字典来做到这一点。我可能会使用字典来做{ item => count },但我知道如果您在每个项目的每个实例中都存储了特定数据,这可能不实用。

从这个数据存储中,当通过界面呈现给用户,或者可能检查背包是否已满的其他代码时,您可以只是迭代字典,生成占用了多少空格的运行计数。我认为这是最好的方式。

编辑:我认为这种方法模糊不清,@ EdwinBuck在答案中提到了这一点。

但是,它对您的应用程序(游戏)可能不实用,所以我希望以下基本算法可能对您有所帮助。我不知道它是否完美无缺,并且可能效率不高,但是我怀疑背包的重量是否足够大。

def addItem(item, count)

    foreach (slot in backpack)
        if (slot.type = Array<Item>)
            if (slot.count < max)

                // Enough space to add at least one more
                if (count > 1)
                    foreach (count) addItem(item, 1)
                    // yes this is recursive to get the checking
                else
                    slot.add(item)
                end

            else

                // Not enough space, create a new slot
                var newSlot
                backpack.add(newSlot)

                if (count > 1)
                    foreach (count) addItem(item, 1)
                    // yes this is recursive to get the checking
                else
                    newSlot.add(item)
                end 

            end
        end
    end
end