Compare Object array in c#

Question

I want to compare elements in a object type array with a user input. To do that I wrote the following code. There is no error in the code but I found out that object comparison does not work as I expected. How can I edit my code to compare user inputs with predefined array?

class Program
{
    static void Main(string[] args)
    {
        object[] Mathfunction = new object[] { '+','-','*','/'};
        Console.WriteLine("Enter");
        object input = Console.ReadLine();
        for(int i=0;i<4;i++)
        {
            if (Mathfunction[i] == input)
            {
                Console.WriteLine("done");
                Console.ReadLine();
            }
        }
    }

@Okuma.Scott, having if's would be harder to maintain and to understand. This is well explained here. stackoverflow.com/questions/1554180/… — codea
– codea, Commented Sep 14, 2014 at 14:02

SLaks · Accepted Answer · 2014-09-14 13:56:05Z

3

You're comparing strings to boxed chars.
Those are not going to be equal.

You should declare your variables as the actual datatypes you're using instead of object; then, the compiler will tell you what you're doing wrong so you can fix it.

answered Sep 14, 2014 at 13:56

SLaks

891k182 gold badges1.9k silver badges2k bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Harry Ninh · Accepted Answer · 2014-09-14 14:01:34Z

1

Is there any reason that you extremely need to convert char to object? If no, I would suggest you to convert it back to char[] and the comparison is fixed. Otherwise, when you compare two objects, it actually is comparing two references, not the (char) values. And since one object was created when you initiated the array, one is input by user, two of them will never be the same, and the comparison will never return true.

answered Sep 14, 2014 at 14:01

Harry Ninh

16.8k5 gold badges64 silver badges56 bronze badges

Comments

KMA · Accepted Answer · 2014-09-14 14:17:15Z

1

I have change my program as followa now it works as I expected.

class Program
{
    static void Main(string[] args)
    {
        object[] Mathfunction = new object[] { '+','-','*','/'};
        Console.WriteLine("Enter");
        String input = Console.ReadLine();
        for(int i=0;i<4;i++)
        {
            String str = (Mathfunction[i].ToString());
            if (String.Equals(str,input))
            {
                Console.WriteLine("done");
                Console.ReadLine();
            }
        }
    }

answered Sep 14, 2014 at 14:17

KMA

2113 silver badges18 bronze badges

Comments

Ateeq · Accepted Answer · 2014-09-14 14:05:04Z

0

try something like this , in your code the input will be in one object while you are using an array of objects to store the characters , you can do as the following :

object[] Mathfunction = new object[] { '+', '-', '*', '/' };
Console.WriteLine("Enter");
object input = Console.ReadLine();
string[] inputString = input.ToString().Split(' ');
bool isEqual = true;
for (int i = 0; i < 4; i++)
{
    if (Mathfunction[i].ToString() != inputString[i])
    {
        isEqual = false;
    }            
}
if (isEqual)
{
    Console.WriteLine("done");
    Console.ReadLine();
}

answered Sep 14, 2014 at 14:05

Ateeq

8232 gold badges10 silver badges30 bronze badges

Collectives™ on Stack Overflow

Compare Object array in c#

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related