C#. Работа с списками

[Лакира]

Не могу понять, как работать со списками одномерными.Как написать адекватный индексатор?

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace L_2
{

    class LinkedListVector
    {

        class Node
        {
             private double a = 0;
             private Node next = null;  

             public double A
             {
                 get;
                 set;
             }
             public Node Next
             {
                 get;
                 set;
             }

             public Node(double a, Node next)
             {
                 Next = next;
                 A = a;
             }
             public Node()
             {
             }
        }


        private Node first;
        private Node last;

        public LinkedListVector(int a)
        {
            Node tec = new Node();
            if (first != null)
            {
                for (int i = 0; i < a; i++)
                {
                    tec.Next = last;
                    last = tec;
                }
            }
        }
       

        public double this[int i]
        {
            
            get
            {
                
                if (i == 0)
                {
                    Node tec = new Node();
                    first = last = tec;
                    tec =first.Next ;
                    return tec.A;
                }
                else
                {
                    Node tec = new Node();
                    last.Next = tec;
                    last = tec;
                    return tec.A;

                }
            }
            set
            {
                
                last.Next = tec;
                last = tec;
                value = tec.A;
               
                
            }
        }

        public void Print()
        {
            Node tec = first;
            while (tec != null)
            {
                Console.Write("\t" + tec.A);
                tec = tec.Next;
            }
        }

        

    }
}

[Лакира]

Вот 2 вариант:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace L_2
{

    class LinkedListVector
    {

        class Node
        {
           

             public double A
             {
                 get;
                 set;
             }
             public Node Next
             {
                 get;
                 set;
             }

             public Node(double a, Node next)
             {
                 Next = next;
                 A = a;
             }
            public Node ()
            {
                }
        }


        private Node first;
        private Node last;

        public LinkedListVector(int a)
        {
            Node tec = new Node();
            if (first != null)
            {
                for (int i = 0; i < a; i++)
                {
                    tec.Next = last;
                    last = tec;
                }
            }
        }
       

        public double this[int i]
        {
            
            get
            {
                    Node tec = new Node();
                    if(tec.Next!=null)
                    
                    tec.Next = tec;
                    return tec.A;
                
                }
                set
                {
                    Node tec = new Node();
                    if (tec.Next != null)

                    tec.Next = tec;
                    value= tec.A;
                }
                
            }
          
        public void Print()
        {
            Node tec = first;
            while (tec!= null)
            {
                Console.Write("\t" + tec.A);
                tec =tec.Next ;
            }
        }

        

    }
}

[New man]

Сообщение от Лакира

public LinkedListVector(int a)
        { .... if (first != null) ....

Вот это условие лишнее. И да, ты в конструкторе не инициализируешь first, а это плохо.
Тебе там не нужен tec, для заполнения стоит юзануть first. Как-то так.

for(int i = 0; i< a;++i)
{
   first = new Node(0, first);
}

Для индексации надо так или иначе перейти к i элементу. А значит там должен быть цикл, а у тебя ничего.
Держи. Переход к i узлу вынес в отдельный метод.

private Node getNodeByIndex(int index)
{
   Node iterator = first;
   if(0>=index)
            throw new System.ArgumentOutOfRangeException("Index out of Range");
   if(null==iterator)
       throw new System.ArgumentOutOfRangeException("Tried to index empty list");
   for(int i = 0; i<index;++i)
   {
      if(null==iterator.Next)
            throw new System.ArgumentOutOfRangeException("Index out of Range");
      iterator = iterator.Next;
   }
   return iterator;
}

public double this[int i]
{
            get { return getNodeByIndex(i).A;}
            set{ getNodeByIndex(i).A = value;}
}

[Лакира]

Cпасибо огромное. Подскажите, что происходит во второй строчке кода

for(int i = 0; i< a;++i)
{
   first = new Node(0, first);
}

[Лакира]

Почему именно передается значение 0?

[Лакира]

Я поняла зачем.Спасибо