Как правильно создать очередь внутри класса? - PullRequest
Новая
       91

Как правильно создать очередь внутри класса?

0 голосов
/ 14 февраля 2019

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

Отладка кода, который я вижу, что, когда значение ставится в очередь, оно переопределяет все остальные значения в очереди.

class Program
{

    static void Main(string[] args)
    {
        Console.WriteLine("Starting...");
        byte[] dataInit = new byte[] { 0x00 };
        Client clientTest = new Client();


        for (int i = 0; i <= 5; i++)
        {
            dataInit[0]++;
            Console.WriteLine("Adding Order -  i = {0}; Order: {1}.", i, BitConverter.ToString(dataInit));
            clientTest.AddOrder(dataInit);
            Console.WriteLine("Peeking Order - i = {0}; Order: {1}", i, BitConverter.ToString(clientTest.PeekOrder()));
        }

        for (int i = 0; i <= 5; i++)
        {
            Console.WriteLine("Removing order - i = {0}; Order: {1}.", i, BitConverter.ToString(clientTest.RemoveOrder()));
        }

        Console.WriteLine("Press Any Key...");
        Console.Read();

    }

    class ClientOrder
    {
        public byte[] Order;


        public ClientOrder(byte[] data)
        {
            Order = data;
        }


    }

    class Client
    {
        public Queue<ClientOrder> ClientOrders = new Queue<ClientOrder>();

        public void AddOrder(byte[] orderToAdd)
        {
            ClientOrders.Enqueue(new ClientOrder(orderToAdd));
        }

        public byte[] RemoveOrder()
        {

            ClientOrder toReturn = ClientOrders.Dequeue();
            return toReturn.Order;
        }

        public byte[] PeekOrder()
        {
            ClientOrder toReturn = ClientOrders.Peek();
            return toReturn.Order;
        }
    }
}

Я ожидал, что очередь была в порядке [0-6].но фактический результат равен {06,06,06,06,06,06} (последнее добавленное значение).

1 Ответ

0 голосов
/ 14 февраля 2019

Вы фактически используете одну и ту же ссылку на byte[], а затем с каждым Enqueue вы фактически заменяете все элементы в очереди, так как они все ссылаются на один и тот же массив.Вы должны сделать копию при создании ClientOrder.Самый простой способ - использовать Linq, но есть и другие возможности . 

public ClientOrder(byte[] data)
{
    Order = data.ToArray();
}

или наоборот, как Джефф сказал

...