Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

C#

Jared Salwei
Jared Salwei
11,974 Points

I have created a new Class and I am calling that class to my main class to write the information to the display.

I have created a new Class and I am calling that class to my main class to write the information to the display. my question is I call the class three times one for each button. I also have to loop it three times in the new class. Can I shorten the code at all?

 public MainPage()
        {

            this.InitializeComponent();
            txtBoxRas.Text = "Rasmussen College";
            txtBoxRas.FontSize = 14;

            txtBoxRas.Foreground = new SolidColorBrush(Windows.UI.Colors.Red);

        }
        private void btnSubmit_Click(object sender, RoutedEventArgs e)
        {
            ProcessCalc();
        }

        private void ProcessCalc()
        {
            Int32 Var1 = Convert.ToInt32(txtBoxInput1.Text) + Convert.ToInt32(txtBoxInput2.Text);
            txtBoxDisplay.Text = Convert.ToString(Var1);
        }

        async private void btnCourse1_Click(object sender, RoutedEventArgs e)
        {
            BL_PageContent.Cource1();
            txtBoxCourse.Text = BL_PageContent.VarOutput;
            var dialog = new MessageDialog(BL_PageContent.VarOutput);
            await dialog.ShowAsync();
        }

        async private void btnCourse2_Click(object sender, RoutedEventArgs e)
        {

            BL_PageContent.Cource2();
            txtBoxCourse.Text = BL_PageContent.VarOutput;
            var dialog = new MessageDialog(BL_PageContent.VarOutput);
            await dialog.ShowAsync();
        }

        async private void btnCourse3_Click(object sender, RoutedEventArgs e)
        {
            BL_PageContent.Cource3();
            txtBoxCourse.Text = BL_PageContent.VarOutput;
            var dialog = new MessageDialog(BL_PageContent.VarOutput);
            await dialog.ShowAsync();
        }







    }

}

++++++++++++++++++++++ The other class++++++++++++++++++++

namespace App1
{
    class BL_PageContent
    {
        public static string VarOutput { get; set; }

        public static void Cource1()
        {
            VarOutput = "";
            string[] names = new string[3] { "COP3488C", "UWP1", "This course is mobile app development." };

            for (int i = 0; i < names.Length; i++)
            {

                VarOutput = VarOutput + names[i] + "  ";

            }


        }
        public static void Cource2()
        {
            VarOutput = "";
            string[] names = new string[3] { "DOP3488B,", "UWC1,", "This course is Cloud Computing." };

            for (int i = 0; i < names.Length; i++)
            {
                VarOutput = VarOutput + names[i] + "  ";
            }

        }

        public static void Cource3()
        {
            VarOutput = "";
            string[] names = new string[3] { "BOP3589,", "UWP2,", "This course Computer Programming Java 1." };

            for (int i = 0; i < names.Length; i++)
            {
                VarOutput = VarOutput + names[i] + "  ";
            }

        }
    }
}

Edited: Made the question more readable - Dane E. Parchment Jr. (Moderator)

Steven Parker
Steven Parker
220,415 Points

Is this part of a course? Could you provide a link to the course page you were working with?

Jared Salwei
Jared Salwei
11,974 Points

This is not part of the course.

1 Answer

I see two parts that can be improved here:

  1. The three buttons could have the same event, lets say the 3 of them call to "btnCourse_Click". Also, you would need to put a name/id to each one of the buttons so you will know through the sender which one of them invoked the method.

    async private void btnCourse_Click(object sender, RoutedEventArgs e)
    {
        Button clickedButton = sender as Button;
        if (clickedButton.Name() == "Button1")
            BL_PageContent.Cource1();
        else if (clickedButton.Name() == "Button2")
            BL_PageContent.Cource2();
        else if (clickedButton.Name() == "Button3")
            BL_PageContent.Cource3();
    
        txtBoxCourse.Text = BL_PageContent.VarOutput;
        var dialog = new MessageDialog(BL_PageContent.VarOutput);
        await dialog.ShowAsync();
    }
    
  2. On the other class, the 3 methods look almost the same. You could keep only one and send an array (that is what changes between them). This array should be send from the "btnCourseClick", so you can know which array iterate, so let's change that on the event. And I would declare the arrays as universal variables at the beggining. The final changes:

    // global variables
    string[] names1 = new string[3] { "COP3488C", "UWP1", "This course is mobile app development." };
    string[] names2 = new string[3] { "DOP3488B,", "UWC1,", "This course is Cloud Computing." };
    string[] names3 = new string[3] { "BOP3589,", "UWP2,", "This course Computer Programming Java 1." };
    
    async private void btnCourse_Click(object sender, RoutedEventArgs e)
    {
        Button clickedButton = sender as Button;
        if (clickedButton.Name() == "Button1")
            BL_PageContent.Cource1(names1); // Pass the corresponding array according to clicked button
        else if (clickedButton.Name() == "Button2")
            BL_PageContent.Cource2(names2);
        else if (clickedButton.Name() == "Button3")
            BL_PageContent.Cource3(names3);
    
        txtBoxCourse.Text = BL_PageContent.VarOutput;
        var dialog = new MessageDialog(BL_PageContent.VarOutput);
        await dialog.ShowAsync();
    }
    
    // The other class, receiving the array to iterate
    public static void Cource1(string[] names)
    {
        VarOutput = "";
        for (int i = 0; i < names.Length; i++)
            VarOutput = VarOutput + names[i] + "  ";
    }