Bugs and Suggestions

/// <summary><br />
/// Converts a List to a DataTable by converting each public property to<br />
/// a DataColumn<br />
/// </summary><br />
/// <remarks><br />
/// To stop a column becoming part of the DataTable, set it's Browsable <br />
/// attribute to false:<br />
///         [Browsable(false)]<br />
///         public TrackStatus Status { get; set; }<br />
/// </remarks><br />
/// <typeparam name="T"></typeparam><br />
/// <param name="data"></param><br />
/// <returns></returns><br />
public static DataTable ToDataTable<T>(this IList<T> data)<br />
    {<br />
    PropertyDescriptorCollection properties = TypeDescriptor.GetProperties(typeof(T));<br />
    bool[] visibility = new bool[properties.Count];<br />
    DataTable dt = new DataTable();<br />
    for (int i = 0; i < properties.Count; i++)<br />
        {<br />
        PropertyDescriptor property = properties[i];<br />
        AttributeCollection ac = property.Attributes;<br />
        visibility[i] = true;<br />
        if (ac.Contains(BrowsableAttribute.No))<br />
            {<br />
            visibility[i] = false;<br />
            continue;<br />
            }<br />
        dt.Columns.Add(property.Name, property.PropertyType);<br />
        }<br />
    foreach (T item in data)<br />
        {<br />
        List<object> values = new List<object>();<br />
        for (int i = 0; i < properties.Count; i++)<br />
            {<br />
            if (visibility[i])<br />
                {<br />
                values.Add(properties[i].GetValue(item));<br />
                }<br />
            }<br />
        dt.Rows.Add(values.ToArray());<br />
        }<br />
    return dt;<br />
    }

/// <summary>
/// Converts a List to a DataTable by converting each public property to
/// a DataColumn
/// </summary>
/// <remarks>
/// To stop a column becoming part of the DataTable, set it's Browsable
/// attribute to false:
///         [Browsable(false)]
///         public TrackStatus Status { get; set; }
/// </remarks>
/// <typeparam name="T"></typeparam>
/// <param name="data"></param>
/// <returns></returns>
public static DataTable ToDataTable<T>(this IList<T> data)
    {
    PropertyDescriptorCollection properties = TypeDescriptor.GetProperties(typeof(T));
    bool[] visibility = new bool[properties.Count];
    DataTable dt = new DataTable();
    for (int i = 0; i < properties.Count; i++)
        {
        PropertyDescriptor property = properties[i];
        AttributeCollection ac = property.Attributes;
        visibility[i] = true;
        if (ac.Contains(BrowsableAttribute.No))
            {
            visibility[i] = false;
            continue;
            }
        dt.Columns.Add(property.Name, property.PropertyType);
        }
    foreach (T item in data)
        {
        List<object> values = new List<object>();
        for (int i = 0; i < properties.Count; i++)
            {
            if (visibility[i])
                {
                values.Add(properties[i].GetValue(item));
                }
            }
        dt.Rows.Add(values.ToArray());
        }
    return dt;
    }