|
There is a paper that discusses this subject, at Martin Fowler's website. See
http://martinfowler.com/ieeeSoftware/dataAccessRoutines.pdf
|
|
|
|
|
Interesting paper indeed, thanks for the resource.
---------------------------
He who knows that enough is enough will always have enough.
-Lao Tsu
|
|
|
|
|
I like your solution better.
His argument about that the coupling is incorrect. His solution is no more or less coupled than yours. Your solution breaks the coupling by using events which is a version of the observer design pattern developed to break the coupling of two classes in situations like this.
You can tweak you solution to make the inner workings of the Foo class just as concealed as his by creating a private nested subclass of FooCollection inside Foo. You can then override the Add method to do the necessary validation. This would also eliminate the need for the event as well and results in a very elegant design.
Aside from the nonstanard interface of his solution, which is awkward to me, the real problem I have with his code is the O(n) operation required to return a reference to the child collection.
|
|
|
|
|
His main argument is that my solution exposes the inner workings of the children, allowing for the possibility of breaking the whole thing, which is something I've discussed with Heath above. It is a valid point if I allow assignment of items from the collection.
I totally agree as far as the collection returning goes. That's gotta be terrible for performance for a tree with thousands of Foo items in it.
Good thinking on the subclass, I'd then be able to access the private methods of the parent from the subclass, which would eliminate the need for events as you mentioned.
Thanks for your input Brian.
---------------------------
He who knows that enough is enough will always have enough.
-Lao Tsu
|
|
|
|
|
I prefer your solution (#1).
I don't buy the argument that external objects could break the inner workings of Foo children. Foo is a class that happens to have property named Children. It's up that object (Children) to maintain it's own integrity, just as any other property of Foo. Foo's responsibility is to make sure all it's property values make sense as a whole, not that they make sense within themselves. If the Foo class is interested in knowing about special events in the Children object, it can listen to them using an eventhandler, just as you have done in your design.
I would say #2 is a bad design, because Foo must know too much about the Children object type (class). That's the FooCollection's responsibilities, and can be easily done with indexers, properties and events. If your friend has a problem with your solution not being OO enough, I would say his solution is less OO.
Is there a special circumstance in that the Children object is of type FooCollection, making this a tree structure? Really not. I think the same design patterns should apply.
A sligthly better solution would be that FooCollection contains objects that implement the IFoo interface. Possibly overkill, but could be useful.
Regards,
/Björn Morén
|
|
|
|
|
I am using c# to create web forms in .net and am trying to add hotkeys. I have been unsuccessful in locating any reference material. If anyone knows of any good reference sights on this subject it would be greatly appreciated or if it is a simple process and explanation would be even better.
Thanks,
Rick
|
|
|
|
|
Here's some sample code for you.
private void MyControl_KeyDown(object sender, System.Windows.Forms.KeyEventArgs e)
{
switch (e.KeyCode)
{
case Keys.Insert:
if (e.Modifiers == Keys.Shift)
{
e.Handled = true;
}
return;
case Keys.Enter:
e.Handled = true;
return;
case Keys.Up:
if (e.Modifiers == Keys.Shift)
{
if (CanMoveItemUp)
{
MoveItemUp();
e.Handled = true;
}
}
return;
default:
return;
}
}
Tom Clement
Apptero, Inc.
|
|
|
|
|
You can also implenent hotkeys in your menus easily by simply assigning the MenuItem.Shortcut property to the key you want.
For global hotkeys in your application, see the IMessageFilter interface. Implement that and catch the WM_KEYDOWN message (0x0100), then use the Message.WParam member to get which key was pressed (matches up with the Keys enumeration). Using this concept, hotkeys are available throughout your application (if you desire) and you could even implement a simple key mapping to make them easy to change (for example, from a user interface for the benefit of a user). This is common in games, for example, like Quake that allow you to bind functions to practically any key you want.
You can add your implementation at any time using Application.AddMessageFilter.
Microsoft MVP, Visual C#
My Articles
|
|
|
|
|
You and other CPains didn't consider that original poster want to do that in a WEB form. 
Mazy
"I think that only daring speculation can lead us further and not accumulation of facts." - Albert Einstein
|
|
|
|
|
True. Far be it for us to assume that someone would use the right forums. 
Microsoft MVP, Visual C#
My Articles
|
|
|
|
|
Mazy is correct I am talking about a Web application not a Window application.
|
|
|
|
|
I would like to thank everyone for their input. I was able to work out my Hotkey issue. If anyone is interested, here is an example of a button control that is executed by Alt + Q
<asp:button id="Button2" style="Z-INDEX: 103; LEFT: 316px; POSITION: absolute; TOP: 166px" accesskey="Q" runat="server" text="Button">
Rick
|
|
|
|
|
You should use AccessKey property to assign hotkey if you are using asp button control. This way you don't have to code anything.
Cheers
Never comment ur code. If it was hard to write, it should be hard to understand !!!
|
|
|
|
|
I sort of hesitate to say this, but I think we've run across a Winforms bug. I suppose the alternative is that I have a brain bug. Anyway, here's the situation:
Create a trivial application (Form1) with a button that launches a modeless dialog using:
Form2 frm = new Form2();
frm.Owner = this;
frm.Show();
In Form2, a button launches Form3 modally like this:
Form2 frm = new Form3();
frm.ShowDialog();
another button hides Form2 like this:
this.Hide();
Now, if you run the application (Form1), then switch focus to some other application (e.g. Outlook). Then click the button that launches Form2 (modeless), then click the button that launches Form3 (modal). Close Form3, and Hide Form2. Form1 goes into the background and Outlook gets focus.
Does anyone know what underlying mechanism is happening here?
Thanks as always for your help.
Tom Clement
Apptero, Inc.
|
|
|
|
|
if I have a generic class like this for example:
public class GenericExample<T>
{
public GenericExample(T parameter) {}
//...
}
when I instantiate it then:
GenericExample<string> ge1 = new GenericExample<string>(String.Empty);
GenericExample<SolidBrush> ge2 = new GenericExample<SolidBrush>(null);
so here is my question:
how can I verify, whether parameter is null,
or whether it is string.Empty, although I don't know,
if T is an value-Type or reference-type. (yes...I know string is a reference type but I had no other example in mind  ).
|
|
|
|
|
[EDIT: Note, this was posted while the < and > brackets were not appearing in the root thread. I leave it here for posterity.]
This is not a template class, which would look like:
public class GenericExmaple<T>
{
public GenericExample(T parameter) {}
}And would be declared as a variable like so:
GenericExample<string> ge = new GenericExample<string>(null); If you want a class that takes a "generic" parameter, use object:
public class GenericExample
{
public GenericExample(object parameter)
{
if (parameter == null) throw new ArgumentNullException("parameter");
}
}When you define the parameter as an object, even if it's a value type it will be boxed as an object and can be compared to null, although it will never be null since value types can't be null. A reference could and thus is subject to the exception if it is null.
Microsoft MVP, Visual C#
My Articles
|
|
|
|
|
I see you fixed your code, which changes things a little bit...
You can always use conditions like so to check if it's null or empty in your constructor regardless of the type (so long as it's a reference type):
if (parameter == null) throw new ArgumentNullException("parameter");
if (parameter.Equals(string.Empty))
throw new ArgumentException("Error", "parameter");If you want value types to also be valid for your generic class, then you can use something like this in your constructor:
Type t = typeof(T);
if (!t.IsValueType)
{
}
Microsoft MVP, Visual C#
My Articles
|
|
|
|
|
thanks, Heath.
the brackets '<' and '>' were missing because I forgot to
check the option 'Don't treat <'s as html-tags' and recognized just later that the brackets were not shown.
|
|
|
|
|
It also helps to put code between <pre></pre> tags, which usually translates the brackets automatically (though not always). This also makes for a nicer fixed-width format (including spaces and tabs you may use) for display code in the forums (and in articles, for that matter).
Microsoft MVP, Visual C#
My Articles
|
|
|
|
|
thanks for that tip.....I guess a good one
|
|
|
|
|
how to load 3d objects in open gl and is it possible to make these models with 3d studio max and import them to opengl(using C#)?
MTSHEDID
|
|
|
|
|
here[^] and here[^]
---------------------------
He who knows that enough is enough will always have enough.
-Lao Tsu
|
|
|
|
|
thank u very much
i couldn't 've started my assignment without it
|
|
|
|
|
hi all,
I have a windows form which is used to create a user in a database. In this form I have several textfields to enter info such as first name, last name, etc...and I have 2 options using radio buttons, one is the classic "male-female" choice and the other is the classic "yes-no" choice. Only I am not able to separate the groups, I have included each pair in a separate groupBox but I don't know what else to do to make them work as 2 separate entities  ...for now, only 1 choice over 4 can be selected.
help anyone?
thanks,
Talal
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."
--Rich Cook
|
|
|
|
|
Make sure that the RadioButtons are actually added to the container's (i.e., GroupBox, Panel, etc.) Controls collection property. If they are added to the same container (i.e., your Form) but only positioned inside the separate containers, they will act as a single group.
Microsoft MVP, Visual C#
My Articles
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
