Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

C# conditional operator error Only assignment, call, increment, decrement, await, and new object expressions can be used as a statement

Hi I was writing a basic program to find if the input number is prime or not. I have a checkPrime(num) function that checks for prime number and returns true if num is prime else returns false. Now in my main() function I used conditional operator to shorten the code but I am getting an error which I am not sure about. Below is my code :

static void Main(String[] args) {
    int n = Int32.Parse(Console.ReadLine());
    while (n-- > 0) {
        int num = Int32.Parse(Console.ReadLine());
        (checkPrime(num) == true) ? Console.WriteLine("Prime") : Console.WriteLine("Not Prime");
    }
}

When I compile, I get the error as Only assignment, call, increment, decrement, await, and new object expressions can be used as a statement in my while loop at conditional statement line. I am not sure what is it that I am missing. There is similar question here and people have answered that the conditional operator line is an expression and not a statement so there has to be some sort or assignment for the value of the expression. Same kind of example is given in MSDN reference where the explanation does something like this

// ?: conditional operator.
classify = (input > 0) ? "positive" : "negative";

But what I am not able to understand is in my function all I am trying to do is check the return value of the function and then print the output. So where does this expression thing comes in my case.

like image 420
Naphstor Avatar asked Jun 01 '16 15:06

Naphstor


People also ask

What is the full name of C?

In the real sense it has no meaning or full form. It was developed by Dennis Ritchie and Ken Thompson at AT&T bell Lab. First, they used to call it as B language then later they made some improvement into it and renamed it as C and its superscript as C++ which was invented by Dr.

What C is used for?

C programming language is a machine-independent programming language that is mainly used to create many types of applications and operating systems such as Windows, and other complicated programs such as the Oracle database, Git, Python interpreter, and games and is considered a programming foundation in the process of ...

Is C language easy?

C is a general-purpose language that most programmers learn before moving on to more complex languages. From Unix and Windows to Tic Tac Toe and Photoshop, several of the most commonly used applications today have been built on C. It is easy to learn because: A simple syntax with only 32 keywords.


1 Answers

The conditional operator is an expression, not a statement, that means it cannot stand alone, as the result has to be used somehow. In your code, you don't use the result, but try to produce side effects instead.

Depending on the condition before the ? the operator returns the result of either the first or the second expression. But Console.WriteLine()'s return type is void. So the operator has nothing to return. void is not a valid return type for the ?: operator. So a void-method cannot be used here.

So you could do something like that:

while (n-- > 0) {
    int num = Int32.Parse(Console.ReadLine());
    string result = checkPrime(num) ? "Prime" : "Not Prime";
    Console.WriteLine(result);
}

or you use the operator inside the Console.WriteLine() call:

while (n-- > 0) {
    int num = Int32.Parse(Console.ReadLine());
    Console.WriteLine(checkPrime(num) ? "Prime" : "Not Prime");
}

In both examples, the operator returns one of the two strings depending on the condition. That's what this operator is for.


Note that you do not need to compare the result of checkPrime() to true. The result already is a bool.

like image 144
René Vogt Avatar answered Nov 08 '22 00:11

René Vogt