Suggestions On How To Refactor This - C# Discussion Boards

Hi all,
I have some html files (Custom Reports) and I want to show the print preview of them to the user.I don't want to use IE print preview or web browser because they are not embedded in my application and I can not control them the way I want. I was wondering if there is a way to preview a html file in PrintPreviewControl (or any other components like that)?
Please help me.
thank you.

Every new thing you learn,Gives you a new personality.

Assuming your app is a WinForms app, I see three possibilities:

1.
Use a WebBrowser control and have it navigate to your HTML document.

2.
Use any external app you choose (MS Word if you must), and make it show its main form inside your app, by calling SetParent (using P/Invoke).

3.
create your own UserControl, that parses and interprets HTML; it is quite a job, probably not worth the effort it would take.

Smile | :)

Luc Pattyn [My Articles] Nil Volentibus Arduum

The quality and detail of your question reflects on the effectiveness of the help you are likely to get.
Please use <PRE> tags for code snippets, they improve readability.
CP Vanity has been updated to V2.4

I tend to agree. I have numerous copy-books and some of them are pretty big. Take this one for instance:

<br />
01  X30002-FUNCTION-SEG.                                          		<br />
	05  X30002-MSG-LL                             PIC S9(04) COMP. 	Message length is variable depending on number of items in the set<br />
	05  X30002-FUNC-TAG                           PIC  X(08).                                      	ZPE0805F<br />
	05  X30002-VERSION                            PIC  9(02).          	Version Number. Must be set to 00<br />
	05  X30002-FUNCTION-INPUT.                                		<br />
        10  X30002-HEADER.                                    		<br />
            15  X30002-BATCH-TYPE                 PIC  X(03).       	P   = Payment batch<br />
																		T   = Transfer batch within a customers own profile<br />
																		PS  = Purchase of Services batch<br />
																		RTI = Real-time Clearing inward batch<br />
																		RTO = Real-time Clearing outward batch<br />
            15  X30002-SET-TYPE                   PIC  X(01).       	C = Consolidated batch. 1 Debit upto 50 credits <br />
																		I = Itemised batch. Upto 25 Dr/CR items<br />
            15  X30002-CUSTOMER-ID                PIC  X(20).     	A valid customer identifier from the channel. Not Validated<br />
            15  X30002-BATCH-ID                   PIC  X(20).     	A unique batch identifier. Not Validated<br />
            15  X30002-SET-NUMBER                 PIC  9(05).     	The number of the set contained within the batch. Start at 1 and increment for each set within the batch<br />
            15  X30002-NOMINATED-POST-IND         PIC  X(01).     	R = Real-time post<br />
																	  = defaults to Real-Time<br />
            15  X30002-HOMING-POST-IND            PIC  X(01).     	F = Force post<br />
																	R = Real-time post<br />
																	  = defaults to Real-Time where applicable<br />
            15  X30002-CHECK-AVAIL-BAL-IND        PIC  X(01).     	Set to Y if available balance must be checked prior to posting	<br />
            15  X30002-TRAN-CODE-IND              PIC  X(01).     	C = Channel<br />
																	D = Default <br />
																	  = Defaults to Channel <br />
            15  X30002-NO-OF-TXNS                 PIC  9(03).     	The number of transactions contained in the homing table<br />
            15  FILLER                            PIC  9(08).     	<br />
        10  X30002-REVERSAL-DETAILS.                        		<br />
            15  X30002-NOM-REV-PROD-CODE          PIC  X(03).     	Reversal Product Code eg DDA / TCS<br />
            15  X30002-NOM-REV-ACCOUNT            PIC  X(23).     	Reversal Account Number. Right justified, zero filled<br />
            15  X30002-REVERSAL-TRAN-CODE         PIC  X(05).     	Reversal Tran Code, can be supplied or resolved by PE<br />
            15  X30002-MATCH-REVERSAL-IND         PIC  X(01).     	Y if matched reversals are required. If set the transaction does not appear on the statement<br />
        10  X30002-NOMINATED-DETAILS.                       	Details of the FNB account to be debited	<br />
            15  X30002-NOM-CO-ID                  PIC  9(05).     	FNB company identifier<br />
            15  X30002-NOM-PROD-CODE              PIC  X(03).     	Nominated Product code eg DDA / TCS / MLS / GLB<br />
            15  X30002-NOM-ACCOUNT-NO             PIC  X(23).     	Account Number to be debited. Right justified, zero filled<br />
            15  X30002-NOM-ACCOUNT-NAME           PIC  X(30).     	Account Name. Not Validated<br />
            15  X30002-NOM-REF                    PIC  X(20).     	Mandatory when batch is Consolidated	 <br />
            15  X30002-NOM-DESCR                  PIC  X(47).     	Used when batch is Consolidated If not supplied the default description for the transaction code will be applied 	 <br />
            15  X30002-NOM-AMOUNT                 PIC  9(15)V99.  	Mandatory when batch is Consolidated Right justified, zero filled, 2 decimal places<br />
            15  X30002-NOM-TRAN-CODE              PIC  X(05).     	If not supplied the transaction code will be resolved by the PE	 <br />
            15  X30002-NOM-SERIAL-NO              PIC  9(10).     	Only applicable to IDS, optional cheque serial number	 <br />
        10  FILLER                                PIC  X(26).     		<br />
        10  X30002-GL-DETAILS.                              	Mandatory when CONSOLIDATED and SET-NUMBER is > 1            	<br />
            15  X30002-GL-AMOUNT                  PIC  9(15)V99.  	Right justified, zero filled, 2 decimal places	<br />
            15  X30002-GL-SIGN                    PIC  X(01).     	D or C	<br />
        10  FILLER                                PIC  X(01).     		<br />
        10  X30002-AMOUNT-CURRENCY                PIC  X(03).     	e.g. ZAR; ZMK; NAD; LSL; SZL; BWP   	<br />
        10  FILLER                                PIC  X(50).     		<br />
        10  X30002-HOMING-DETAILS OCCURS 50.                		<br />
            15  X30002-HOM-CO-ID                  PIC  9(05).     	A Valid FNB company identifier.<br />
            15  X30002-HOM-PROD-CODE              PIC  X(03).     	Valid values <br />
																	TPT – must supply account type and branch when using this option.  The product code is then derived <br />
																	DDA<br />
																	TOA <br />
																	TCS; or Valid Card Product eg ZFN<br />
																	ILP; MLS; PLS;<br />
																	NPS<br />
																	REC<br />
																	OBP<br />
																	WES<br />
																	EWA<br />
																	GLB<br />
            15  X30002-HOM-ACCT-TYPE              PIC  9(01).     	1 = Current Acc <br />
																	2 = Savings Acc <br />
																	3 = Transmission Acc <br />
																	4 = Bond Acc        <br />
																	6 = Subscription Share<br />
            15  X30002-HOM-BRANCH                 PIC  9(06).     	Sort Code<br />
            15  X30002-HOM-ACCOUNT-NO             PIC  X(23).     	Right justified, zero filled<br />
            15  X30002-HOM-BUDGET-NO              PIC  9(05).     	Only used for Cardes products  	<br />
            15  X30002-HOM-ACCOUNT-NAME           PIC  X(30).     	Account Name. Not validated<br />
            15  X30002-HOM-AMOUNT                 PIC  9(15)V99.  	Right justified, zero filled, 2 decimal places<br />
            15  X30002-HOM-REF                    PIC  X(20).     	Reference<br />
            15  X30002-HOM-DESCR                  PIC  X(47).     	If not supplied the default description for the transaction code will be applied	<br />
            15  X30002-HOM-TRAN-CODE              PIC  X(05).     	If not supplied the transaction code will be resolved by the PE	<br />
            15  X30002-HOM-REVERSAL-DESCR         PIC  X(47).    	Supply if a different description is required other than the homing description	<br />
            15  X30002-NOM-REFERENCE              PIC  X(20).    	Mandatory when batch is Itemised	<br />
            15  X30002-NOM-DESCRIPTION            PIC  X(47).    	Used when batch is Itemised. If not supplied the default description for the transaction code will be applied 	<br />

This is huge and writing my own class to parse the message to is probably a waist of time. I'll see if I can come up with a better solution or find something and post back.

Thanks Luc

Excellence is doing ordinary things extraordinarily well.

I'll wait.thanks

Every new thing you learn,Gives you a new personality.

Can you explain more about the second one?(Maybe an article or something?)

Every new thing you learn,Gives you a new personality.

something along these lines would show your HTML editor inside your panel "myPanel":

Process p=Process.Start("myHtmlEditor", "someFile.html");
p.WaitForInputIdle();
IntPtr hWndChild=p.MainWindowHandle();
IntPtr hWndParent=myPanel.Handle;
SetParent(hWndChild, hWndParent);

Adapt where necessary!

And the prototype for the native Win32 function would be:

[DllImport("user32.dll", SetLastError = true)]
static extern IntPtr SetParent(IntPtr hWndChild, IntPtr hWndNewParent);

Luc Pattyn [My Articles] Nil Volentibus Arduum

The quality and detail of your question reflects on the effectiveness of the help you are likely to get.
Please use <PRE> tags for code snippets, they improve readability.
CP Vanity has been updated to V2.4

You have significantly less control over starting another app than you do over a web browser OCX, which the OP says isn't enough, so I doubt this solution will fulfil his needs. The best third party app for rendering HTML – a browser – is already available as a control in .Net.

I'm working on a WPF app using MVVM with Telerik controls. I have a Telerik RadTreeView loaded with clients and projects:

<telerik:RadTreeView Width="200" 
                     HorizontalAlignment="Left"
                     ItemsSource="{Binding Path=Clients}"
                     SelectedItem="{Binding SelectedItem, Mode=TwoWay}"/>

Here are the 2 data models:
http://www.maroisconsulting.com/stuff/models.png[^]

When a node is clicked, the SelectedItem's Get calls nodeSelected. Here's where the fun begins.

The nodeSelected methods is responsible for
1) Extracting the DataModel (either the ClientModel or ProjectModel) from the selected node
2) Creating the view (ClientView or ProjectView)
3) Creating the viewmodel (ClientViewModel or ProjectViewModel)
4) Adding a tab to a RadTabControl and assigning the view to it.

While this all works fine, the code doesn't feel right. In the code below, you'll see that both the Client and Project sections have alot in common. I'd like to get some suggestions on refactoring it (Sorry it's a bit long):

private void nodeSelected()
{
    if (SelectedItem == null)
        return;

    string selectedObjectName = SelectedItem.GetType().Name.Trim().ToLower();

    switch (selectedObjectName)
    {
        #region Client
        case "clientmodel":

            // Get the client model from the selected tree node
            ClientModel clientModel = SelectedItem as ClientModel;

            // Check to see if a view with the client's Id is already open in a tab
            TabInfo clientTabInfo = getTabInfo(AppViews.Client, clientModel.ClientId);

            if (clientTabInfo == null)
            {
                string clientName = "Client - " + clientModel.ClientName;
                string clientImage = "/Falcon;component/Media/Graphics/customer_48x48.png";
                string clientIcon = "/Falcon;component/Media/Graphics/customer_16x16.png";

                // Create the client view model, passing in the client model
                ClientViewModel clientViewModel = new ClientViewModel(clientModel)
                {
                    HeaderTitle = clientName,
                    HeaderImage = clientImage
                };

                // Create the client view
                ClientView clientView = new ClientView
                { 
                    /* Assign the client view model to its view */
                    DataContext = clientViewModel };

                // Create the tab info object and add it to the list
                TabInfo info = new TabInfo
                {
                    ItemId = clientModel.ClientId,
                    ViewType = AppViews.Client
                };

                // Assign the Client's view and the tab info object to the tab
                // Create the new tab
                RadTabItem tab = addTab(clientName, clientIcon);
                tab.Content = clientView;
                tab.Tag = info;

                // Add the tab to the Tabs collection and select it
                Tabs.Add(tab);
                SelectedTab = tab;
            }
            else
            {
                // Actvate the tab
                activateTab(AppViews.Client, clientModel.ClientId);
            }
            break;

        #endregion

        #region Project
        case "projectmodel":

            // Get the client model from the selected tree node
            ProjectModel ProjectModel = SelectedItem as ProjectModel;

            // Check to see if a view with the Project's Id is already open in a tab
            TabInfo projectTabInfo = getTabInfo(AppViews.Project, ProjectModel.ProjectId);

            if (projectTabInfo == null)
            {
                string projectName = "Project - " + ProjectModel.Caption;
                string projectImage = "/Falcon;component/Media/Graphics/project_48x48.png";
                string projectIcon = "/Falcon;component/Media/Graphics/project_16x16.png";

                // Create the Project view model, passing in the Project model
                ProjectViewModel ProjectViewModel = new ProjectViewModel(ProjectModel)
                {
                    HeaderTitle = projectName,
                    HeaderImage = projectImage
                };

                // Create the Project view
                ProjectView ProjectView = new ProjectView();

                // Assign the Project view model to its view
                ProjectView.DataContext = ProjectViewModel;

                // Create the tab info object and add it to the list
                TabInfo info = new TabInfo
                {
                    ItemId = ProjectModel.ProjectId,
                    ViewType = AppViews.Project
                };

                // Create the new tab
                RadTabItem tab = addTab(projectName, projectIcon);
                tab.Content = ProjectView;
                tab.Tag = info;

                // Add the tab to the Tabs collection and select it
                Tabs.Add(tab);
                //SelectedTabIndex = Tabs.Count - 1;
                SelectedTab = tab;
            }
            else
            {
                // Actvate the tab
                activateTab(AppViews.Project, ProjectModel.ProjectId);
            }
            break;
        #endregion
    }

}

Everything makes sense in someone's mind

That's not too bad. It looks similar but actually most of it is not identical and therefore isn't an immediate candidate for refactoring. The only thing that is is creating or activating a tab, which is about 10 lines, which isn't critical.

However, the type switching by name is ugly. You should have ClientModel and ProjectModel both implement an interface (let's call it IModelInfoProvider) that exposes what you need to determine tab identity (an AppViews enum, ModelType, and an ID). You could argue that they should go in the view model, so if you have two way linkage you could put them there and expose a base view model through the interface, or make the UI control have ViewModels not Models as its item list. Then you can do:

IModelInfoProvider item = (IModelInfoProvider)SelectedItem;

TabInfo tabInfo = getTabInfo(item.ViewType, item.ID);
if(tabInfo == null){
 TabInfo info = new TabInfo
 {
  ItemId = item.ID,
  ViewType = item.ViewType
 };

 IView view = ...;
 
 RadTabItem tab = addTab(view.DisplayName, view.IconPath);
 tab.Content = view;
 tab.Tag = info;

 SelectedTab = tab;
} else activateTab(tabInfo.Tab); // Why look it up twice? you already assigned it

Now, what goes in ... is essentially 'make your MV and View'. Depending on how strict on layering you're feeling you can either expose that as an interface method as well, or create a factory to which you can pass a model and it will construct views of the appropriate type. In the latter case you will still have to type switch, inside the factory code – please use is if you have to type switch, not string type names!

I have a C# 2010 Windows Forms project that accesses (primarily) two tables in a SQL Server 2008 database. Each table is accessed in two modes: active accounts and cancelled accounts, and both have to be visible and editable at the same time (user requirement). The way I have it set up now is to have one DataSet, one TableAdapterManager, two TableAdapters (one for each table) and four BindingSources (two for each table adapter; one for active, one for cancelled). I have FillBy/GetDataBy methods setup for each TableAdapter that fill my BindingSources correctly. This is the code I’m using for filling them:

htcActiveBindingSource.DataSource = this.htcTableAdapter.ActiveGetDataBy(gpn);
htcCancelledBindingSource.DataSource = this.htcTableAdapter.CancelledGetDataBy(gpn);
vetActiveBindingSource.DataSource = this.vetTableAdapter.ActiveGetDataBy(gpn);
vetCancelledBindingSource.DataSource = this.vetTableAdapter.CancelledGetDataBy(gpn);

Everything works exactly as I was hoping, except for saving. The code I was using for saving is this:

this.Validate();
this.htcActiveBindingSource.EndEdit();
this.htcCancelledBindingSource.EndEdit();
this.vetActiveBindingSource.EndEdit();
this.vetCancelledBindingSource.EndEdit();
this.tableAdapterManager.UpdateAll(this.cramdDataSet);

When I step through in debug mode, the current row in the BindingSource knows that it has been modified after the EndEdits run. However, when it executes the UpdateAll, nothing happens in the database. Playing around with it, I figured out that if I fill the TableAdapter directly, like this:

this.htcTableAdapter.ActiveFillBy(this.cramdDataSet.htc, gpn);

then it works, but only for just active or just cancelled, not both at the same time (it sets both binding sources to the same data). Is there some way to get the data from the independent BindingSources back into the TableAdapter? Or am I going to be forced to create a second table adapter for the cancelled versions of each table? Am I going about this in a completely retarded way? If so, please enlighten me! (As much as I’d like to make my code work, I’m not opposed to rewriting big chunks to do it the “right” way.) I’m also going to need to insert new records created from the form, but I haven’t gotten to that point yet since my basic save is not even working.

Thanks in advance!

Can you show ActiveFillBy and ActiveGetDataBy? I suspect that you are returning a binding source on an in-memory copy of the data and the link back to the database is being lost.

I'm not sure what you mean by "show ActiveFillBy and ActiveGetDataBy". They are queries attached to the htcTableAdapter in the dataset designer. However, I stumbled upon a way to make it work, but I'm not sure exactly why it works. If I alter the code like this before the UpdateAll, then it works:

this.htcActiveBindingSource.EndEdit();
this.htcTableAdapter.Update((cramdDataSet.htcRow)((DataRowView)this.htcActiveBindingSource.Current).Row);

I get what the update is doing: taking the current row from the BindingSource and applying it back to the TableAdapter. What I don't get is the parameter value (starting w/ the first double parentheses). I found that online somewhere a while ago, but I don't understand what it is doing. Is there a better way to pass the current row into the update? More importantly, is there a way to pass the entire set of rows from the binding source? The vetActiveBindingSource and vetCancelledBindingSource can have multiple records in them that can all be edited. Since one overload of the BindingSource.Update method can take a collection of rows, I'd rather do it like that, rather than looping thru all the records and passing them in one at a time. I looked thru the BindingSource properties and methods but didn't see anything that looked promising.

Ah, I thought those were methods you'd written. I'm afraid I'm not familiar enough with this piece of Microsoft magic to offer detailed advice. (Anything with auto-generated code I try to avoid using; I like to know what my program is doing! The form designer I will accept as that code is just layout and not function, but data binding templates and the like are too magic for my taste.)

What that statement seems to be doing is casting the binding source to a view and its current row back to the original schema. I would expect you to be able to update through the view, so perhaps there is some option you missed when creating the binding?

I'm beginning to think the "non-magic" method is the way to go. Even though I got the save working, I keep running into other obstacles trying to make this do what I want. Can you give me some advice on rewriting this manually? Some sample code doing something similar would be great, or a site with a tutorial on setting it up. As long as it doesn't involve a datagridview... I've read several tutorial sites and they seem to only deal with that and it won't work for what I'm trying to do (and I haven't figured out how to convert what they say for the DGV into separate controls).

You can use the DGV without needing to do so much magic. For example you can bind to a List<T>, and catch the OnCellValueChanged to manually send an update to your data source. If you're using WPF, you can bind to an ObservableList<T>, and use the list notifications to update a data source.

If you're trying to view tabular data, the DGV is a good control to use. It's the binding magic that confuses me.

(edit - html fail)

modified on Wednesday, June 15, 2011 7:28 PM

Hi experts,

I am developing an application for a customer. The application uses a USB device for which there is a driver. The customer would like the application setup to include the driver installation.

AFAIK the way to go is
1. Copy driver files to some directory I know.
2. Tell windows to look in that directory for driver files.
3. Plug device in.
Windows will ask for driver files.
4. Choose to install automatically.
Windows will use stored directory (2) to install drivers.

My Problem is step (1). What I tried was:
- include the driver files in a subdirectory of my application project (Right click on project, "Add", "New folder")
- add "Content files" to setup's file system view (Or whatever "Inhaltsdateien" reads in english).
- compile and execute setup on target machine.

Some of the files were copied, but others were not. Here are the files I had added to the application project:

- USBDriver
  - amd64
  | \ ftbusui.dll
  | \ ftcserco.dll
  | \ ftd2xx.lib
  | \ ftd2xx64.dll
  | \ ftdibus.sys
  | \ ftlang.dll
  | \ ftser2k.sys
  | \ ftserui2.dll
  - i386
  | \ ftbusui.dll
  | \ ftcserco.dll
  | \ ftd2xx.dll
  | \ ftd2xx.lib
  | \ ftdibus.sys
  | \ ftlang.dll
  | \ ftser2k.sys
  | \ ftserui2.dll
  - Static
  | - amd64
  | | \ ftd2xx.lib
  | - i386
  |   \ ftd2xx.lib
  \ DPInst.exe
  \ dpinst.xml
  \ ftd2xx.h
  \ ftdibus.cat
  \ ftdibus.inf
  \ ftdiport.cat
  \ ftdiport.inf

And these are the files that got copied:

-USBDriver
  - amd64
  | \ ftbusui.dll
  | \ ftcserco.dll
  | \ ftd2xx64.dll
  | \ ftlang.dll
  | \ ftserui2.dll
  - i386
  | \ ftbusui.dll
  | \ ftcserco.dll
  | \ ftd2xx.dll
  | \ ftlang.dll
  | \ ftserui2.dll
  \ DPInst.exe
  \ dpinst.xml
  \ ftd2xx.h
  \ ftdibus.inf
  \ ftdiport.inf

It seems that files with extensions .sys, .cat and .lib did not get copied.

Why is that?

How can I transfer all of the driver files to their destination?

Ciao,

luker

Everything you need should be in this[^].

A guide to posting questions on CodeProject[^]

Dave Kreskowiak

Unfortunately, this doesn't help me.

The paper describes how to create a driver package. I already have a ready-to-install driver package and am trying to include it in a setup project.

The setup project doesn't include all files that I told its application to include. What to do about that?

Ciao,

luker

Actually, you'd be repackaging your drivers to put the files in the appropriate locations.

A guide to posting questions on CodeProject[^]

Dave Kreskowiak

Hi every body
I have Q
how read/write List OF DataRaw to network Stream
please I want answer Soonly
And thanks

Please read the "How to get an answer to your question" topic on top.
-> Elaborate on what you're trying to achieve
-> Take time to formulate your question and write full sentences.
-> Indicate what the (isolated) problem is and what you have done to solve the problem yourself.

Example:
If you seek a starting point for information on a project you can always ask the question like - I want to do a project on topic X and have search google/Codeproject/ site whatever on these keyword, but results are insufficient. Could you please point me to some good resources on where to start.

hope this helps in getting the answer you seek.

V.

DataTables (and I think DataViews) can be serialised to XML, which can obviously be sent down a stream. Package your DataRows up in one of them and get the XML serialisation.

om_metab wrote:
please I want answer Soonly

You get answers when people are available to give them. You've asked enough questions by now to know that adding comments like this does nothing to increase the urgency with which people reply to you.

As for your question, what have you tried? What technology are you using? WCF? Custom written TCP?

Forgive your enemies - it messes with their heads

My blog | My articles | MoXAML PowerToys | Mole 2010 - debugging made easier - my favourite utility

I will used TCP

Hello,

Most code used to detect USB relies on a Windows form. The program must register its interest in the USB insertion/removal events by sending its main window handle to RegisterDeviceNotification. What I'm trying to do is make a class library which exposes DeviceIsConnected property without having to use any forms. Any idea if this can be done, and if so how?

Thanks!

Mart