Creating and populating a generic list I would like to know if this is being done correctly.

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

public class InnerJoinTables
{

    public List&lt;InnerJoinTable&gt; Items = new List&lt;InnerJoinTable&gt;();

}

public class InnerJoinTable
{
    public int ID { get; set; }
    public string RightTable { get; set; }
    public string LeftTable { get; set; }

    public InnerJoinTable(int ID, string RightTable, string LeftTable)
    {
        this.ID = ID;
        this.RightTable = RightTable;
        this.LeftTable = LeftTable;
    }
}


Calling code:

InnerJoinTables m = new InnerJoinTables();

m.Items.Add(new InnerJoinTable(1, "A", "B"));
The code will work, so from that point of view, it is correct.

But there are also some issues with it:


I don't see any reason for the InnerJoinTables class to exist. You should just use List&lt;InnerJoinTable&gt; directly in the calling code. That's unless there's something else going on that you're not showing.
Public fields should not be used. Use a property instead.