Creating and populating a generic list
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.