The Weird and The Wonderful

BobJanova wrote:

Yes, or at least bind the items to an object, of which this magic number was a property.

public class Display
{
    public static readonly Display OpeningSplashdisplay =
        new Display(1, "Opening Splash display");
    public static readonly Display StartUpDateConfig =
        new Display(2, "start up date config");

    //
    // Skip a lot of Displays
    //

    /// <summary>
    /// Enumerate the Display objects.
    /// </summary>
    public static IEnumerable<Display> Values
    {
        get
        {
            object temp = null;
            FieldInfo[] fields = typeof(Display).GetFields(); // Obtain all fields
            foreach (FieldInfo field in fields)
            {
                temp = field.GetValue(null);
                if (temp is Display)
                    yield return (Display)temp;
            }
        }
    }

    private readonly byte value;
    private readonly string name;

    /// <summary>
    /// Enum constructor.
    /// </summary>
    /// <param name="value">The numeric value of this object.</param>
    /// <param name="name">The string representation of this object.</param>
    Display(byte value, string name)
    {
        this.value = value;
        this.name = name;
    }

    /// <summary>
    /// Get the numeric value of this object.
    /// </summary>
    /// <returns>The value as byte.</returns>
    public byte ToByte() { return value; }

    /// <summary>
    /// Get the string representation of this object.
    /// </summary>
    /// <returns>A string.</returns>
    public override string ToString() { return name; }

    /// <summary>
    /// Convert byte representation to Language object.
    /// </summary>
    /// <param name="value">A byte.</param>
    /// <returns>A Display enumerated object.</returns>
    public static Display ParseByte(byte value)
    {
        foreach (Display d in Display.Values)
        {
            if (d.ToByte() == value)
                return d;
        }

        // If we reach here return default
        return Display.OpeningSplashdisplay;
    }
}

cmbDisplay.Items.AddRange(new List<Display>(Display.Values).ToArray());

private void cmbDisplay_SelectedIndexChanged(object sender, EventArgs e)
{
    ComboBox c = (ComboBox)sender;
    Display selected = (Display)c.SelectedItem;
    _RawData[2] = selected.ToByte();
}

AspDotNetDev wrote:

It's hard to find anything NOT wrong with this

Dim oDAdapt As SqlClient.SqlDataAdapter

Dim cmdSQL As String = sSQL

Dim oRs As New DataSet
' ...not so many lines later...
oRs = New DataSet

AspDotNetDev wrote:

I really don't like when the parens aren't put after calling a constructor

using (resource) { }