.NET (Core and Framework)

11-May-05 0:37

Hi,

Here i required some help in architecuring my Project.Here is whole senerio.

My project is already made in C# which is having around 60 .cs files.In my project,there is one main form and from that form i am calling and using all others .cs files of whole project.Currently there is somany coding available into main form.
Here another thing is that we are not using any database like sql or oracle but storing data into xml files as this is project requirement.
Now main thing is that as we had decided to build proper architecture of this project by which we can make it like library.Now from where to start i am confused so can anybody help me out here.

thanks.,

regards.,

Christian Graus11-May-05 13:54

11-May-05 13:54

Well, it sounds like all your code apart from the main form is just providing business logic for that form, and therefore could be moved into a library and imported into the project. Why do you want to build a library ? What benefit are you looking for ?

Christian Graus - Microsoft MVP - C++

montu337711-May-05 21:47

11-May-05 21:47

Hi,

By making the library our client can use that library and use that library into their other projects.That's main advantage of making that library.
so that's why we want to make proper architecture of our code.so now can you suggest me some good idea about how to architecte this.?

regards.,

montu337712-May-05 2:04

12-May-05 2:04

Christian Graus15-May-05 18:18

15-May-05 18:18

I don't see what's so hard. First of all, I don't know enough about your specific system to offer specific advice, secondly, all you need to do is pull all the non-UI code into a code library DLL, and import it into your project. I can't percieve of how I could give specific advice to help you to do that, unless you have no idea how to create a project that compiles to a dll.

Christian Graus - Microsoft MVP - C++

montu337717-May-05 0:08

17-May-05 0:08

Hi,

I tell you some more on my exact requirement here.You can assume my project just like paint in which one main form is there and user can create project folder,load image and create different different shapes from the menu items of the main form so for us the architecture & coding of this form is important.
Here main thing is that every menu items say when you click menuitems like file--> images goes to one methods and do required functionality same thing is for all menu items but as all those methods are in form itself and i don't want to depend on this form becoz i want to make layerwise architecture.

As all methods are very unique and calls only 1-2 times from mainform so if i put those forms methods into one class and make dll then is that good way to do that?
if i am doing that i have to pass controls as parameters of those methods.
can you tell me that if i put those forms methods into one class and make dll then is that good way?

if you have some more good idea here to build architecture than it's good.

regards

Christian Graus17-May-05 12:58

17-May-05 12:58

montu3377 wrote:
As all methods are very unique and calls only 1-2 times from mainform so if i put those forms methods into one class and make dll then is that good way to do that?

They don't need to go into the one class, no.

montu3377 wrote:
if i am doing that i have to pass controls as parameters of those methods.

Don't pass controls to middle tier methods, that ties them to the presentation tier. Perhaps create an enum, and a method that generates an enum value from a control value, as a helper function in the presentation tier.

montu3377 wrote:
can you tell me that if i put those forms methods into one class and make dll then is that good way?

If your product is a paint program, then creating a layer that is distinct from the presentation layer will be harder to do right, because it's all about what you display. Certainly don't put everything in one class, that will be more of a hinderance than having no middle tier is. To be honest, I've written some big paint programs, and I've focused on a good class structure, and not on loading anything via dlls. The only way that can be good is if you can write the main program to be generic, and to dynamically load tools from a dll, then you can distribute a new dll to add new tools, or fix tool bugs.

Christian Graus - Microsoft MVP - C++

montu337723-May-05 4:49

23-May-05 4:49

Hi,

If you had put some sort of article of that paint programme then can you pls. send me link or url of that?or can you pls. give me 1 sample code how to create that types of architecture?

Thanks.,

Christian Graus23-May-05 11:43

23-May-05 11:43

It was a commercial product. Basically, I wrote a base class like this:

toolbase
{
void OnLButtonDown
void OnKeyPress
void OnPaint
//etc, etc.

}

Then each tool would inherit from toolbase, and do whatever it should in those methods. Finally, my main window kept an instance of toolbase, which was then changed as the tool was changed, and it caught all these events, and called the methods on the tool when they occured.

Christian Graus - Microsoft MVP - C++

montu337724-May-05 7:50

24-May-05 7:50

Hi,

i had implemented as you said above,main problem is that i can't just declare method otherwise i have to define that method and class as abstract and if i am defining that class as abstract then in the form i can't make the new instance of that class becoz we can't create instance of abstract class.

second way,simply i can directly create instance of second class that inherited base class in that case why had created base class that is also question.Here i am sending u one simple code what i am doing.

//this is base class
public abstract class ToolBase
{

public abstract void mnuNewProj_Click(object sender, System.EventArgs e);

}

//second class that inherits toolbase class
public class InputComponent:ToolBase
{
public override void mnuNewProj_Click(object sender, EventArgs e)
{
//done implementation
}
}

but from main form i can't able to create new instance of Toolbase class becoz it's abstract.
Now can you tell me some more over here.I really like this design but need some more clearification.

thanks you a lot in advance.

Christian Graus24-May-05 11:45

24-May-05 11:45

montu3377 wrote:
we can't create instance of abstract class.

My 'toolbase' class is abstract, and I create an instance of it in my application, no problems. I never create an instance of toolbase though, that's not what it is for, the base class is useless by itself anyhow, that's not what it is for.

montu3377 wrote:
second way,simply i can directly create instance of second class that inherited base class in that case why had created base class that is also question.Here i am sending u one simple code what i am doing.

You create a base class because your app only needs to use one tool at a time, so you carry an instance of the base class, but you instantiate it as instances of the derived classes. i.e. if you click the FreeHandDraw button then my instance of toolbase gets instantiated as an instance of toolFreeHandDraw. The point is that catching UI events is common to all tools, so it's defined in a base class, so I don't need to switch between drawing modes with every UI event to figure out which control to send the event to.

Christian Graus - Microsoft MVP - C++

montu337724-May-05 23:26

24-May-05 23:26

HI,

I got your point what you means to say here i had implemented two classes and call it from main form as below.Just let me know is it ok or not.If any improvement is required then pls. tell me becoz after that i will do implement into whole project.

toolbase class
//base class

public abstract class ToolBase
{
public abstract void mnuNewProj_Click(object sender, System.EventArgs e);
}

//child class implemented that method
public class InputComponent:ToolBase
{
public override void mnuNewProj_Click(object sender, EventArgs e)
{
//doing implementation code here
}
}

//From the Main Form (presentation layer) calling the child class and that method into click event
//Main form
public class main
{
private void mnuNewApp_Click(object sender, EventArgs e)
{
InputComponent input = new InputComponent();
input.mnuNewProj_Click(sender, e);
}
}
Pls. check the above code and if any improvement need than tell me.

it's working fine but i am afraid that in later on stage i have to pass controls into both classes as return values so it gives effect on main form.

pls. let me know urs comments on above code.

Thanks a lot for your nice co-operation.
Really appriciated your efforts.

Christian Graus25-May-05 12:48

Incorrect value returned from this.getType ().Assembly.Location for C# COM-addin

25-May-05 12:48

montu3377 wrote:
it's working fine but i am afraid that in later on stage i have to pass controls into both classes as return values so it gives effect on main form.

Creating a new image is not a tool, the class that encapsulates an image window should be carrying the tool here, and the tools should be draw/fill/shape/text/etc.

montu3377 wrote:
Thanks a lot for your nice co-operation.
Really appriciated your efforts.

That's cool - you happened to ask in an area that I have a lot of experience, so glad to help Smile | :)

Overall, the class structure is fine, but I'd expect the tools to work like this: ( this is pseudocode, it won't compile )

class MyMainWindow
{
private ToolBase tool = new ToolNone; // Create an empty tool so the variable never has to be null

OnMenuClick()
{
switch(menuItemClicked)
{
case text:
tool.Dispose(); // Your tools will have resources you should delete in an IDisposable implimentation
tool = new ToolText(this); // The constructors will need a copy of the window if the tools are to call Invalidate() to make the window redraw when needed
break;
case freehand:
tool.Dispose();
too = new ToolFree(this);
break;
// etc
}
}

OnMouseClick(args)
{
tool.OnMouseClick(args);
}
// etc
}

Christian Graus - Microsoft MVP - C++

Barry G10-May-05 18:27

Barry G

10-May-05 18:27

Hi all,

I wonder if anyone can help, I'm at a complete loss about the behaviour I'm seeing!

I've created an Outlook COM-addin project in VS2003 (this is via the Wizard). The trouble is is that I'm trying to get the installation path because there are files that are installed that I want to load/access. (I don't really fancy using resource file(s) at this time)

To get the install path I'm using:

this.getType ().Assembly.Location

now the trouble is that on my local machine this is returning the path of where the .dll file has been created by VS2003 (in the bin\Debug directory) NOT the install path (which is different, and on a different partition to the build path).

Anyone any idea why?

What irks me is that if I create a vanilla addin (also via the Wizard) then when I install and load Outlook, the path returned is correct. It's only when I try and use my existing project that it returns the wrong path.

I even tried creating a completely new project and then cut-n-pasting the code from the old project into new class files since I wasn't sure if the registry has got "bunged-up" with invalid/old entries. I have noticed that when VS2003 registers the addin for COM interop it will create registry entries that have a CodeBase value that points to the build .dll file (which is pretty obvious since at that point it hasn't been installed!). However for the vanilla project it must do something similar so I don't think that's causing the problem.

Anyone any ideas on how I can stop/correct this problem? It's driving me nuts WTF | :WTF:

Thanks (in advance),

Gary

P.S. The addin did "do the right thing" in the past but I changed the ProgId value and then everything went belly up... Mad | :mad:

Re: Incorrect value returned from this.getType ().Assembly.Location for C# COM-addin

Barry G11-May-05 18:26

Barry G

11-May-05 18:26

I've found a way out of this little mess... Poke tongue | ;-P

The way to do this seems to be to get the MS installer to create a registry item with the install path when it does the install.

To do this just set the value of the item to [TARGETDIR] the square brackets are important and it's case sensitive. (More info on the installer properties can be found at: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/msi/setup/property_reference.asp)

The installer will then replace the [TARGETDIR] with the directory that the COM addin was installed to.

Given some of strange Confused | :confused:

things (i.e. things I don't understand at the moment Unsure | :~

) that the .NET framework will do for assembly loading this seems to be a more reliable method.

Gary

How to create a reminder notification in Pocket PC

Member 19358809-May-05 21:03

Member 1935880

9-May-05 21:03

I found a link in CodeProject about creating notification and time application. The source codes included is build from scretch which is too confusing to me. Anyone know any easier way to make a reminder notication in Pocket PC. Thanks for reply.

How to find a pattern in an image

Tad McClellan9-May-05 16:40

Tad McClellan

9-May-05 16:40

Hi all. I need to find a signature block in a scanned document. I can create the document so I can put a box around the signature or mark it in some way. I just don't know how to find it. I don't think I can depend on it always being in the same spot. Any ideas?

Re: How to find a pattern in an image

Christian Graus9-May-05 17:19