Question

private void delete_Click(object sender, EventArgs e)
{
convertedText.Text = “”;
}

private void copy_Click(object sender, EventArgs e)
{
    if (convertedText.Text != "")
        Clipboard.SetText(convertedText.Text);
        convertedText.Text = Clipboard.GetText();
    else... what to put here?
}

The program has two buttons (copy and delete) and one textbox. If I click the ‘Copy’ button, it copies the text from convertedText.Text without any problem. The ‘delete’ button also clears the textbox fine.

But if there’s nothing in the textbox, the ‘copy’ button still attempts to copy it which is causing an unexpected behaviour.

So, what code do I add to the ‘else’ statement…? What I want is that if the textbox has nothing in it, then the clipboard operation won’t be used. How to do that?

Thanks in advance!

Answer 1

Don’t add an else clause, just have the if by itself, e.g.

private void copy_Click(object sender, EventArgs e)
{
    if (!string.IsNullOrEmpty(convertedText.Text))
    {
        Clipboard.SetText(convertedText.Text);
        convertedText.Text = Clipboard.GetText();
    }
}

Also, is there any reason why you’re copying the text box text to the clipboard and then using the clipboard text to update the text box text? Unless I’m missing something, this should have no effect on the text box, so the code can be simpler:

private void copy_Click(object sender, EventArgs e)
{
    if (!string.IsNullOrEmpty(convertedText.Text))
        Clipboard.SetText(convertedText.Text);
}