CODEWITHSUNDEEP
cassignment-operator
Asmodiel
Asmodiel

Reputation: 1062

c: assignment used in calculation

I have a question that I'd love to google but I have no idea on what to ask her/she/it (google :) )

Having found assignments like this in several example codes (this particular one originated here where they copied the code from the Numerical Recipes in C p. 508) on the net.

I tried to understand the usage of the assignment in the following line:

wr = (wtemp = wr) * wpr - wi * wpi + wr;

as far as my own understanding of programming logic goes, I would translate it to 

wtemp = wr;
wr = wtemp * wpr - wi * wpi + wr;

is that correct? or should it be wpr * wpr - wi [...]? or something completely different :D?

(according to wikipedia the assignment operator returns a reference to the assignee, which according to the free dictionary - i checked to be sure - is the value which something is being assigned to)

Upvotes: 4

Views: 180

Answers (2)

Speed8ump
Speed8ump

Reputation: 1317

@md5 has the correct answer, but I've seen this misused so many times I felt the need to comment further. In almost all circumstances doing assignments like this is wrong. Not "won't compile" wrong, but "I can't tell if that's what you really meant to do" wrong. It immediately requires the reader to stop and think hard about what you are doing. If there's a practical way to avoid doing assignments mid-process use it. Create a temporary variable, break the evaluation into stages, whatever.

Now, there are times when this is actually helpful. As an example, here is some code which calls a very slow function (canFreelyStop), but only if it actually must. At the same time it is doing some evaulation of system state:

        //determine whether transport motor should be running and make it happen
        bool canStop = true;
        if (    m_requestErrorStop
             || (    !m_receiving                       // not receiving new items from upstream.
                  && ((canStop = canFreelyStop()) != 0) // can stop without damaging anything (!=0 to avoid compiler warning)
                )
           )
        {
           //ready to turn it off
           ...
        }
        else if (!canStop)
        {
           //only failed to stop due to documents in transit.  Wait a "cycle" and check again
           ...
        }

It's not trivial to rework this code to eliminate the internal assignment (it's also relying on short circuit evaluation). This is one of the only conditionals I've ever written where internal assignment was actually better than the alternative. If anyone comes along wanting to change/debug this code I want them to have to stop and think hard about it.

Upvotes: 2

md5
md5

Reputation: 23699

The value of the expression a = b is the value of a after assignment, so:

wr = (wtemp = wr) * wpr - wi * wpi + wr;

is equivalent to:

wtemp = wr;
wr = wtemp * wpr - wi * wpi + wr;

or:

wtemp = wr;
wr = wr * wpr - wi * wpi + wr;

Reference:

6.5.16 Assignment operators

An assignment expression has the value of the left operand after the assignment (...).

Upvotes: 6

Related Questions