Java HashMap Создание, редактирование и удаление - PullRequest
Новая
       37

Java HashMap Создание, редактирование и удаление

1 голос
/ 13 февраля 2012

Я пытаюсь обновить HashMap, используйте его непосредственно в следующем методе, но он не работает.Из того, что я прочитал, я не мог найти решение.Некоторые говорят, что это невозможно, а некоторые говорят, что используют итератор, но даже с итератором он не работает.ошибка в том, что метод печати не печатается и даже не попадает в цикл while, потому что он пустой, но я не могу найти, почему

Это два метода, которые я пытаюсь обновить и напечатать некоторую информацию.* 

  import java.io.File;
     import java.util.ArrayList;
     import java.util.HashMap;
     import java.util.Iterator;
     import java.util.Scanner;
     import java.util.Enumeration;
     import java.util.Hashtable;
     import java.util.Iterator;
     import java.util.Map;
     import java.util.Set;

    public class OrderList {
    // Storage for an arbitrary number of details.

    private HashMap<String, Order> orderList = new HashMap<String, Order>();

    /**
     * Perform any initialization .
     */
    public OrderList() {
        orderList = new HashMap<String, Order>();
    }

    public HashMap<String, Order> getOrders() {
        return orderList;

    }

    public void readOrderFile(String OrderListPath) {
        try {
            File file = new File(OrderListPath);
            Scanner scan = new Scanner(file);
            while (scan.hasNextLine()) {
                String readLine = scan.nextLine();

                if (readLine != null) {

                    getSplitLinesOrders(readLine);
                }
            }

        } catch (Exception e) {
        }
    }

    public void getSplitLinesOrders(String readLine) {
        String id = "";
        String customerId = "";
        String itemId = "";
        int quantity = 0;
        try {
            String[] splitedLine = readLine.split(",");

            if (splitedLine.length == 4) {
                id = splitedLine[0];
                customerId = splitedLine[1];
                itemId = splitedLine[2];
                quantity = Integer.parseInt(splitedLine[3]);
                Order newOrder = new Order(id, customerId, itemId, quantity);
                orderList.put(id, newOrder);
            }
        } catch (Exception e) {
        }


    }

    /**
     * Add a new set of details to the list
     * @param details The details of the staff
     */
//    public void addDetails(Order details) {
//        orderList.add(details);
//    }
    public boolean hasOrder() {
        return orderList.size() != 0;
    }

    public Order getNextOrder() {
        Order order = orderList.remove(0);
        return order;

    }

    /**
     * @return All the details
     */
    public String listDetails() {
        StringBuffer allEntries = new StringBuffer();
        for (Map.Entry<String, Order> details : orderList.entrySet()) {
            String Key = details.getKey();
            Object value = details.getValue();
            allEntries.append(Key + " " + value);
        }
        return allEntries.toString();
    }

    public void PrintListOfOrders() {
        Iterator it = getOrders().entrySet().iterator();
        try {

            while (it.hasNext()) {
                Order value = (Order) it.next();
                System.out.println(value.getOrderId() + " " + value.getCustomerId() + " " +     value.getItemId() + " " + value.getQuantity());
            }




        } catch (Exception e) {
            System.out.println(e);
        }
    }
}

Ответы [ 2 ]

2 голосов
/ 13 февраля 2012

Вы, вероятно, получаете NullPointerException? В следующий раз расскажите нам, что происходит не так и предоставьте трассировки стека, если применимо.

Код, который вы разместили, не создает экземпляр orderList, поэтому, если это не будет сделано в другом месте, код выдаст NullPointerException

Попробуйте добавить: 

 private HashMap<String, Order> orderList = new HashMap<String, Order>;

Глотание Exception как это: 

} catch (Exception e) {
}

не является хорошей практикой, поскольку она будет скрывать всю информацию о том, что идет не так, по крайней мере: 

catch (Exception e) {
   e.printStacktrace();

}

0 голосов
/ 13 февраля 2012

Вы можете сделать что-то вроде этого: 

Set<String> s = List.keySet();
Iterator<String> i = s.iterator();

Это инициализация итератора, и затем вы можете перебирать ключи, используя i.next(), каждый раз получая строку и спрашивая orderList.get(thatString)чтобы получить значение.

...