C# Original orderd list and Original reverse list help

Question

2.33/5 (3 votes)

See more:

Hello

I am trying to write this program that will list user input from 5 textbox and display them in a orderd and reverse order list in two lables

the way I got it now shows it but I like to have it accending like textbox1 textboxt2 for the orderd and reverse textbox2 , textbox1
And have it allign to left in that order in the lable

Hear is the code I have now with two I don't know error I like to fix also

C#

public form1()
{
    InitializeComponent();
}


public partial class Form1 : Form
{
    Array arrayList();
    //Error	1	'Form1.arrayList()' must declare a body because it is not marked abstract, extern, or partial	
}      

private void btndisplay_Click(object sender, EventArgs e)
{
    arrayList list = new arrayList();
    //Error	2	The type or namespace name 'arrayList' could not be found (are you missing a using directive or an assembly reference?)	


    list.Add = txtitem1.Text;
    list.Add = txtitem2.Text;
    list.Add = txtitem3.Text;
    list.Add = txtitem4.Text;
    list.add = txtitem5.Text;
    //
    // Sort the ArrayList.
    //
    list.Sort.accend ();
    //
    // Display the ArrayList elements.
    //
    lblOrder.Text = list();

        {

        }
    //
    // Reverse the ArrayList.
    //
    list.Reverse.decend();
    //
    // Display the ArrayList elements again.
    //
    foreach (string data in list)
        list.reverse = lblReverse.Text();

    return (list);

}

private void btnClear_Click(object sender, EventArgs e)
{
    lblOrder.Text = ("");
    lblReverse.Text = ("");
    txtitem1.Text = ("");
    txtitem2.Text = ("");
    txtitem3.Text = ("");
    txtitem4.Text = ("");
    txtitem5.Text = ("");
}

private void btnExit_Click(object sender, EventArgs e)
{
    this.Close();
}

Posted 1-Apr-12 12:05pm

leetank

Updated 1-Apr-12 12:12pm

Shahin Khorshidnia

v3

Add a Solution

2 solutions

Solution 2

The reasons for errors were nicely explained by Wes Aday in Solution 1.
The following code can be used to display the text of TextBoxes in Order and in Reverse order.

C#

private void btndisplay_Click(object sender, EventArgs e)
{
    List<string> textBoxesContent =new List<string>{
        txtitem1.Text,txtitem2.Text,txtitem3.Text,txtitem4.Text,txtitem5.Text};
    textBoxesContent.Sort();
    //Using LINQ
    lblOrder.Text = textBoxesContent.Aggregate( (s1,s2) => s1 + ", " + s2 );
    //Using ForEach
    lblOrder.Text = ConcatStrings(textBoxesContent,", ");
    textBoxesContent.Sort((s1,s2)=>s2.CompareTo(s1));
    //Using LINQ
    lblReverse.Text = textBoxesContent.Aggregate ((s1,s2) => s1 + ", " + s2);
    //Using ForEach
    lblReverse.Text = ConcatStrings(textBoxesContent,", ");
}
//To concatenate strings of the List using ForEach loop
public static string ConcatStrings(IEnumerable<string> stringList, string separator){
    string concatenatedString = string.Empty;
    foreach (string element in stringList)
    {
    	if(!string.IsNullOrEmpty(concatenatedString))
    		concatenatedString += separator;
    	concatenatedString += element;
    }
    return concatenatedString;
}

Posted 4-Apr-12 19:33pm

ProEnggSoft

Updated 4-Apr-12 19:35pm

v2

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Wes Aday · Accepted Answer · 2012-04-01T13:40:00

C#

public partial class Form1 : Form
{
    ArrayList arrayList;

}      
 
private void btndisplay_Click(object sender, EventArgs e)
{
    ArrayList list = new ArrayList();

Although arrayList is never used according to your code.

I take it from your comment that you want to know why as well at 'what'.

Your first error:

C#

Array arrayList();

The () means that you are trying to call the constructor for a variable declaration. No such thing exists. Variable declarations do not use (). What you really wanted is:

C#

Array arrayList;

Which would declare a variable arrayList of type Array. But since you did not assign the variable any value it will have its default value of null. AND since arrayList is declared within the scope of the form constructor, that is the only place were you could use it.

Your second error comes at the line:

C#

arrayList list = new arrayList();

Which means that you are declaring a variable list of type arrayList and instantiating an instance of that class. Well the problem is that you have not defined what class 'arrayList' is. So you get the type or namespace error. 'arrayList' in your original code would be a variable and could not be instantiated as a new instance of a class. And would not be visible to this method since that variable is scoped to the form constructor.

To actually fix your code depends on what it is that you really want to do. Do you want an Array or an ArrayList? Do you really want a class level or a function level variable? I do not see where you are using the list outside of the function so I would not imagine that you need a class level variable.

The third problem that has not been mentioned is:

C#

return (list);

as your function is declared to return void then trying to return a list from it, won't work.