	[x] Posted via EE Mobile
	Search, ask, and monitor your questions on the go with EE Mobile. Visit Experts Exchange from your mobile device and never be out of touch again.

Question

	[x] Attachment Details

[x]

The Solution Rating System

With so many solutions, how can you tell which solutions are most likely to help you and which ones are not? To provide you with a tool to use, we rate our solutions based on various elements that most accurately determine if a solution is a quality solution. To explain what factors affect the solution rating, here are the elements we take into consideration when formulating our solution rating.

The Grade of the Solution
The Zone Rank of the Expert Providing the Solution
The Number of Author and Expert Comments
The Number of Experts Contributing
The Feedback of the Community

Your Input Matters
Because of the way the system is set up, the most important variable in this equation is you. As a member of Experts Exchange, you are able to cast your vote on the quality of the solutions in regard to how complete, accurate, helpful and easy to understand each solution is. When you provide your feedback, each rating is adjusted accordingly. So, if you see a solution that has a poor rating that you think is a good solution, let us know by rating it. As you do, the rating will be adjusted and will become more accurate for other members of our site.

If you have any suggestions that you would like to make for our rating system, please ask a question in the Suggestions Zone of Community Support.

Thank you!

7.4

Looking for Cleaner Code structure, and little xml help.

Asked by Smyken in Microsoft Visual C#.Net

I'm pretty new to C# and need advise on how to Structure up code in a good way, making classes and methods.

Here is a small example code I wrote that I would like to structure better.
For example the xml node fetching is declared 2 times, when thay load on a button and when they are saved by another button.

How could I for example write a class or method that is located in a different .cs file and just call the method or class instead of writing the code several times ?

I also wonder how I can create the LocalTitle node directly under Root as the first child ?
As it is now I create LocalTitle node by saying that it should be added before OriginalTitle which is the firstchild, but if the OriginalTitle is missing then this wont work I guess.

Any examples ?

View the Solution FREE for 30 Days

         
           
             1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
53:
54:
55:
56:
57:
58:
59:
60:
61:
62:
63:
64:
65:
66:
67:
68:
69:
70:
71:
72:
73:
74:
75:
76:
77:
78:
79:
80:
81:
82:
83:
84:
85:
86:
87:
88:
89:
90:
91:
92:
93:
94:
95:
96:
97:
98:
99:
100:
101:
102:
103:
104:
105:
106:
107:
108:
109:
110:

           
           
             using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Xml;
 
 
namespace XML_Update_sample
{
    public partial class Form1 : Form
    {
        XmlDocument xmlDoc = new XmlDocument();
        public String Xml_File;
 
        public Form1()
        {
            InitializeComponent();
        }
    
        private void Load_xml_button_Click(object sender, EventArgs e)
        {
            if (open_xml_dialog.ShowDialog() == DialogResult.OK)
            {
                
                Xml_File = open_xml_dialog.FileName;
                xmlDoc.Load(Xml_File);
 
                XmlNode root = xmlDoc.DocumentElement;
                XmlNode LocalTitle = xmlDoc.SelectSingleNode("/Title/LocalTitle");
                XmlNode OriginalTitle = xmlDoc.SelectSingleNode("/Title/OriginalTitle");
                XmlNode SortTitle = xmlDoc.SelectSingleNode("/Title/SortTitle");
 
                
                if (LocalTitle == null)
                {
                    //MessageBox.Show("Node Missing Creating New Element");
                    XmlElement New_LocalTitle = xmlDoc.CreateElement("LocalTitle");
                    root.AppendChild(New_LocalTitle);
                    root.InsertBefore(New_LocalTitle, xmlDoc.SelectSingleNode("/Title/OriginalTitle"));
                }
                if (LocalTitle != null)
                {
                    LocalTitle_textbox.Text = LocalTitle.InnerText;
                }
 
                
                if (OriginalTitle == null)
                {
                    //MessageBox.Show("Node Missing Creating New Element");
                    XmlElement New_OriginalTitle = xmlDoc.CreateElement("OriginalTitle");
                    root.AppendChild(New_OriginalTitle);
                    root.InsertAfter(New_OriginalTitle, xmlDoc.SelectSingleNode("/Title/LocalTitle"));
                }
                if (OriginalTitle != null)
                {
                    OriginalTitle_textbox.Text = OriginalTitle.InnerText;
                }
 
 
 
                if (SortTitle == null)
                {
                    //MessageBox.Show("Node Missing Creating New Element");
                    XmlElement New_SortTitle = xmlDoc.CreateElement("SortTitle");
                    root.AppendChild(New_SortTitle);
                    root.InsertAfter(New_SortTitle, xmlDoc.SelectSingleNode("/Title/OriginalTitle"));
                }
                if (SortTitle != null)
                {
                    SortTitle_textbox.Text = SortTitle.InnerText;
                }
 
 
            }
 
        
        }
 
        private void Load_trailer_button_Click(object sender, EventArgs e)
        {
            if (open_trailer_dialog.ShowDialog() == DialogResult.OK)
            {
                string Trailer_File = open_trailer_dialog.FileName;
                Trailer_textbox.Text = Trailer_File;
                MessageBox.Show(Trailer_File);
            }  
        }
 
 
 
 
        private void Save_xml_button_Click(object sender, EventArgs e)
        {
            XmlNode LocalTitle = xmlDoc.SelectSingleNode("/Title/LocalTitle");
            XmlNode OriginalTitle = xmlDoc.SelectSingleNode("/Title/OriginalTitle");
            XmlNode SortTitle = xmlDoc.SelectSingleNode("/Title/SortTitle");
 
            LocalTitle.InnerText = LocalTitle_textbox.Text;
            OriginalTitle.InnerText = OriginalTitle_textbox.Text;
            SortTitle.InnerText = SortTitle_textbox.Text;
 
            xmlDoc.Save(Xml_File);
        }
 
    }
}

           
         

20091111-EE-VQP-92 - Hierarchy / EE_QW_3_20080625

Hall of Fame

1. abel303,979
2. TheLearn...282,629
3. Idle_Mind249,839
4. Dhaest196,022
5. Fernando...185,894
6. jaime_oliv...137,634
7. ragi0017122,712
8. CodeCruiser117,823
9. emoreau110,866
10. daveamour101,502
11. kaufmed100,249
12. guru_sami98,322
13. nmarun95,810
14. carl_tawn85,152
15. jinal79,675
16. NNirmalan71,162
17. pivar70,080
18. djjaries68,743
19. CuteBug67,855
20. ToddBeau...65,180
21. angelIII63,038
22. Omego2K62,179
23. copyPast...58,048
24. gauthampj55,925
25. DanielWilson55,725

1. TheLearn...947,463
2. jaime_oliv...692,406
3. Fernando...587,102
4. Idle_Mind443,846
5. Dhaest408,820
6. abel323,059
7. emoreau280,250
8. ragi0017261,795
9. p_davis258,560
10. gregoryyo...250,365
11. angelIII245,318
12. philipjonat...184,763
13. gauthampj183,730
14. graye176,130
15. naspinski147,750
16. CodeCruiser134,301
17. CuteBug130,752
18. GreenGho...130,180
19. jpaulino128,849
20. anarki_ji...128,524
21. prairiedog121,213
22. kaufmed120,089
23. ShazbotOK119,352
24. JimBrandley118,431
25. ororiole115,714

Looking for Cleaner Code structure, and little xml help.

Asked by Smyken in Microsoft Visual C#.Net

About this solution