C#

Pete O'Hanlon wrote:

While this may seem to be a reasonable solution, on the surface, there are several issues with this design. The biggest problem I have with it is that you are allocating a larger object than you need to. Even if we were to change this to a struct, it's still an unreasonably large surface area for a simple value.

public class Parse<T>
{
  bool HasValue { get; set; }
  T Value { get; set; }
  string AdditionalInformation { get; set; }
}

public abstract class Parse<T>
{
    public abstract bool HasValue { get; }
    public abstract T Value { get; }
    public abstract string AdditionalInformation { get; }
    
    private sealed class ParseSome : Parse<T>
    {
        public ParseSome(T value)
        {
            Value = value;
        }
        
        public override bool HasValue => true;
        public override string AdditionalInformation => null;
        public override T Value { get; }
    }
    
    private sealed class ParseNone : Parse<T>
    {
        public ParseNone(string additionalInformation)
        {
            AdditionalInformation = additionalInformation;
        }
        
        public override bool HasValue => false;
        public override string AdditionalInformation { get; }
        
        public override T Value
        {
            get { throw new InvalidOperationException(AdditionalInformation); }
        }
    }
    
    public static Parse<T> Some(T value)
    {
        return new ParseSome(value);
    }
    
    public static Parse<T> None(string additionalInformation)
    {
        return new ParseNone(additionalInformation);
    }
}

// Let's say that you run a search to see if a particular product was on sale
public bool IsProductOnSale(guid productId, out Guid[] productResellerIds)
{
  // This function runs a query against the database to see if this particular product has references in the 'on sale' reference table.
  // If it has references, it returns true also sets the array of resellers that sell that product.
  // The data for which seller has the item on sale is returned in the product query.
}

class Product
{
    private Collection<Seller> resellers;
    public Collection<Seller> Resellers
    {
        if (null == resellers)
        {
           // Do your look-up, and set resellers accordingly (using interlock if necessary)
        }
        return resellers;
    }

    public bool IsProductOnSale()
    {
        return (Resellers.Count > 0);
    }

    // ...
}

private void btnDemo_MouseDown(object sender, MouseEventArgs e)
{
    demoThread = new Thread(ThreadProcSafe);
    btnDemo.Text = "Stop Demo";
    demoThread.Start();
}

private void btnDemo_MouseUp(object sender, MouseEventArgs e)
{
    if (!stopDemo)
    {
        stopDemo = true;
        if (!demoThread.Join(1000))
        {
            demoThread.Abort();
        }
        SetText("Stopped");
        btnDemo.Text = "Start Demo";
    }
}

delegate void SetTextCallback(string text);
private Thread demoThread;
private volatile bool stopDemo;

private void ThreadProcSafe()
{
    var x = 0;
    while (!stopDemo)
    {
        SetText((x++).ToString());
        Thread.Sleep(600);
    }
}
private void SetText(string text)
{
    try
    {
        if (lbl_right.Disposing) return;

        if (lblDemo.InvokeRequired)
        {
            var d = new SetTextCallback(SetText);
            BeginInvoke(d, text);
        }
        else
            lblDemo.Text = text;
    }
    catch (Exception e)
    {
        MessageBox.Show(e.Message);
    }
}

protected override void OnClosed(EventArgs e)
{
    stopDemo = true;

    if (!demoThread.Join(1000))
    {
        demoThread.Abort();
    }
    base.OnClosed(e);
}

myDeleteButton = myTableCell.Controls[0];

myDeleteButton = (LinkButton)myTableCell.Controls[0];

myDeleteButton = myTableCell.Controls[0] as LinkButton;
if (myDeleteButton != null) { ... }