CODEWITHSUNDEEP
c
user379888
user379888

Reputation:

What will the following code evaluate?

I had the following code in a test.I am confused about what (i,j) evaluates,while reading about the "," operator i found that it just evaluates the variables or functions but what does it do here?

main()
{
    int i = 10, j = 20;

    j = i ? (i, j) ? i : j : j;
    printf("%d %d", i, j);
}

Upvotes: 1

Views: 284

Answers (8)

user191776
user191776

Reputation:

The comma operator can be used to link the related expressions together. A comma-linked list of expressions is evaluated left-to-right and the value of the rightmost expression is the value of the combined expression. It acts as a sequence point.

A sequence point guarantees that all side effects of previous evaluations will have been performed, and no side effects from subsequent evaluations have yet been performed.

So, any expression/assignment will be completed & only then will the next expression to the right be evaluated.

For example,

b=(a=4,a=5);

gives,

a=5
b=5

Also, note that the comma operator ranks last in the precedence list of operators in C.

Upvotes: 2

Alam
Alam

Reputation: 1596

Looks like typical software written test question. It is used to confuse candidates. As suggested by sepp2k above it is same as j. One more interview question

i = 10; j = 20;
1) k = i,j;
2) k = (i,j);

Answer for 1 will be 10 and answer for 2 will be 20. As coma operator doesnt do anything. I hope this will clear more.

Upvotes: 1

Cedric H.
Cedric H.

Reputation: 8298

main()
{
    int i = 10, j = 20; // i=10, j=20

    j = i ? /*if i */ ( (i, j) ? /* if j (1) */ i : /* else */ j ) : /* else */ j; // Added parenthesis and see (2)

    printf("%d %d", i, j); // Will therefore print 10 10
}
  1. it is equivallent to if(j) because (i,j) evaluate to j because i has no side effect.
  2. all ifs evaluate to true because i and j are ints and are non zero

Upvotes: 0

Jon Purdy
Jon Purdy

Reputation: 54971

This is equivalent to:

int main() {

    int i = 10, j = 20;

    if (i != 0) {

        (void)i; // Do nothing with i.

        if (j != 0) {

            j = i;

        } else {

            j = j; // Assign j to itself.

        }

    } else {

        j = j; // Assign j to itself.

    }

    printf("%d %d", j);

}

Upvotes: 1

Guffa
Guffa

Reputation: 700212

It's not really doing anyting. It's evaluating the expression i, discards the result, evaluates the expression j and returns the result.

As evaluating the expression i has no side effects, (i,j) has the same meaning as just j.

Upvotes: 0

Pascal Cuoq
Pascal Cuoq

Reputation: 80266

in (i,j), the , operator does nothing because the left-hand side expression does not have side-effects.

The assignment is thus equivalent to:

j = i? (j? i : j) : j;

And since i and j are non-zero, to j = i;

Upvotes: 3

sepp2k
sepp2k

Reputation: 370112

(i,j) is exactly the same as just j, because i is just a variable and evaluating it doesn't cause any side effect.

So basically it's just obfuscation here.

Upvotes: 4

sje397
sje397

Reputation: 41812

It will print:

10 10

This is because you can break the expression down like:

j = i ? ((i, j) ? i : j) : j;

The comma operator evaluates to the last expression - so (i, j) == j. That is non-zero, so the center expression evaluates to i. 'i' being non-zero, the outer expression evaluates to i, so j is assigned to the value of i.

Upvotes: 1

Related Questions