Question

With this code:

private void PlatypusMainForm_FormClosing(object sender, FormClosingEventArgs e) {
    if ((UnsavedChanges()) && (!(UserWantsToMoveOnWithoutSaving(CONFIRM_CLOSE_UNSAVED_CHANGES_LOST, "Close Without Saving?")))) {
        e.Cancel = true;
        return;
    }
    if (oracleConnectionMainForm.State == ConnectionState.Open) {
        oracleConnectionMainForm.Close();
        oracleConnectionMainForm.Dispose();
    }
}

…if e.Cancel = true is commented out, the form closes anyway.

…if return is commented out, the rest of the code (Close and Dispose) runs (so, if I then try to save changes, I get the err msg that the connection is not open).

So, I have to do both (cancel and return) to get the code to work as I think it should with either one.

Is this normal/as expected?

Answer 1

Yes, that’s as expected. The e.Cancel tells the framework that you’ve handled the event, and you don’t want the automatic behavior. Without it, after your method returns, the framework will continue to handle the event, and close the window.

The return aborts the execution of this current method, so the stuff at the end doesn’t execute.