Argument

[musefan]

perhaps try creating your own eventargs class that inherits from EventArgs then you can put any properties you want to use in it and pass that to the button click handler

Life goes very fast. Tomorrow, today is already yesterday.

[musefan]

yep, that possible. But if you were testing with null then your button click handler would surely never make use of EventArgs and you would never need to check it. anyway if you want the functionality called manually then you should have it as a separate function that the button click handler also calls.

Life goes very fast. Tomorrow, today is already yesterday.

[Henry Minute]

Agreed, but since EventArgs is essentially empty, nothing that I can think of actually makes any use of it.

Henry Minute Do not read medical books! You could die of a misprint. - Mark Twain Girl: (staring) "Why do you need an icy cucumber?" “I want to report a fraud. The government is lying to us all.”

[Luc Pattyn]

Hi, split your code into two methods: 1. one contains the "bunsiness logic" and no GUI stuff (No MessageBox); it does not take sender or eventargs as parameters, and it does return a calculated value and outputs (through out parameter) a string which should be shown to the user. 2. the buttonClickHandler which mainly calls the other method and shows the MessageBox when there is a need to.

private void btnAddEntry_Click(object sender, EventArgs e) {
string s;
double result=calculate(out s);
if (s==null) textBox.Text="The result is "+result;
else MessageBox.Show(s);
}

private double calculate(out string message) {
message=null;
double result=...;
if (...) message="Something went wrong";
...
return result;
}

That way you can easily perform unit tests on the business logic, and there probably won't be a need to test the actual buttonClickHandler anymore. :)

Luc Pattyn [Forum Guidelines] [My Articles]

---

Avoiding unwanted divs (as in "articles needing approval") with the help of this FireFox add-in

---

[CPallini]

musefan wrote:

well EventArgs does not have any useful information so you wouldn't need to check if its null or not as you would never use the reference

One useful information, maybe, for instance, if EventArgs reference is null or not. :)

If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler. -- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong. -- Iain Clarke
[My articles]

[musefan]

do you mean you could use it to represent a boolean value for something else such as 'isUserClick'?

Life goes very fast. Tomorrow, today is already yesterday.

[CPallini]

Yes, it may represent a boolean value, ugly, I should admit. :)

If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler. -- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong. -- Iain Clarke
[My articles]

[musefan]

yes very ugly, but i guess that is an answer to my question of what it could be used for so good answer.

Life goes very fast. Tomorrow, today is already yesterday.