How Unsafe/Safe is this code?

I needed a way to store lists of data of different types in a list, so I have created a list like type that can store any type that conforms to

CollectionElement

CollectionElement

. It works well and doesn't have any memory errors, but I wanted to get a second opinion on if it seems safe enough to use.
One issue it has is that I can't check the type when you append or get from it since it does not store type info. Currently I just store the types size and compare to that.

# simple example:
    var table = List[Column](Column(), Column(), Column())
    table[0].append[Vector2](Vector2(0, 9))
    table[1].append[Int64](10)
    table[2].append[Float64](3.14)
    
    var val = table[1].get[Int64](idx=0)
    table[2].get[Float64](idx=0) *= 2

# simple example:
    var table = List[Column](Column(), Column(), Column())
    table[0].append[Vector2](Vector2(0, 9))
    table[1].append[Int64](10)
    table[2].append[Float64](3.14)
    
    var val = table[1].get[Int64](idx=0)
    table[2].get[Float64](idx=0) *= 2

The code was a bit too long to fit in one message, so here is a link to it.
https://gist.github.com/RyanLeber/d9ca608a3334482283c81771ff106485

Gist

Column

Column. GitHub Gist: instantly share code, notes, and snippets.

Modular•17mo ago•

4 replies

Ryan

How Unsafe/Safe is this code?

I needed a way to store lists of data of different types in a list, so I have created a list like type that can store any type that conforms to

CollectionElement

CollectionElement

# simple example:
    var table = List[Column](Column(), Column(), Column())
    table[0].append[Vector2](Vector2(0, 9))
    table[1].append[Int64](10)
    table[2].append[Float64](3.14)
    
    var val = table[1].get[Int64](idx=0)
    table[2].get[Float64](idx=0) *= 2

# simple example:
    var table = List[Column](Column(), Column(), Column())
    table[0].append[Vector2](Vector2(0, 9))
    table[1].append[Int64](10)
    table[2].append[Float64](3.14)
    
    var val = table[1].get[Int64](idx=0)
    table[2].get[Float64](idx=0) *= 2

The code was a bit too long to fit in one message, so here is a link to it.
https://gist.github.com/RyanLeber/d9ca608a3334482283c81771ff106485

Gist

Column

Column. GitHub Gist: instantly share code, notes, and snippets.

How Unsafe/Safe is this code?

Similar Threads

How Unsafe/Safe is this code?

Similar Threads

Similar Threads

Similar Threads