Click here to Skip to main content
11,503,113 members (73,321 online)
Click here to Skip to main content

Breaking Changes in Argument List Evaluation in C# 5.0

, 30 Jul 2012 CPOL 17.9K 11
Rate this:
Please Sign up or sign in to vote.
In C# 4.0 there was an error in the C# compiler in the order of the evaluation of the arguments in argument lists.

The C# Language Specification states on §7.5.1.2 that “(…) the expressions or variable references of an argument list are evaluated in order, from left to right (…)”.free hit counters

So, when this code is compiled with the C# 4.0 compiler:

static void M(
    int x = 10,
    int y = 20,
    int z = 30)
{
    Console.WriteLine(
        "x={0}, y={1}, z={2}", x, y, z);
}

static void Main(string[] args)
{
    int a = 0;

    M(++a, z: ++a);
}

and run, this unexpected output is obtained:

x=2, y=20, z=1

In fact, fixing this compiler flaw was the cause of one of the few breaking changes introduced in C# 5.0.

Using the 5.0 compiler, the expected result is obtained:

x=1, y=20, z=2

To avoid this type of surprises, expression evaluation should be avoided in argument lists.

With this code:

int a = 0;

int i = ++a;
int j = ++a;

M(i, z: j);

the same result is obtained for both C# 4.0 and C# 5.0:

x=1, y=20, z=2

License

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

Share

About the Author

Paulo Morgado
Software Developer (Senior) Paulo Morgado
Portugal Portugal

Comments and Discussions

 
GeneralMy vote of 4 Pin
eferreyra30-Jul-12 3:05
membereferreyra30-Jul-12 3:05 
GeneralExcellent ARTICLE Pin
SSerjo26-Jul-12 8:04
memberSSerjo26-Jul-12 8:04 
GeneralRe: Excellent ARTICLE Pin
Paulo Morgado26-Jul-12 8:24
memberPaulo Morgado26-Jul-12 8:24 
GeneralMy vote of 5 Pin
johannesnestler26-Jul-12 6:05
memberjohannesnestler26-Jul-12 6:05 
GeneralRe: My vote of 5 Pin
Paulo Morgado26-Jul-12 8:23
memberPaulo Morgado26-Jul-12 8:23 
QuestionMessage Automatically Removed Pin
26-Jul-12 3:58
memberfkjgjhjhjnj26-Jul-12 3:58 
QuestionNot an article... Pin
Dave Kreskowiak26-Jul-12 3:38
mvpDave Kreskowiak26-Jul-12 3:38 
AnswerRe: Not an article... Pin
Paulo Morgado26-Jul-12 4:04
memberPaulo Morgado26-Jul-12 4:04 
GeneralNot an article Pin
Richard MacCutchan25-Jul-12 23:09
mvpRichard MacCutchan25-Jul-12 23:09 
GeneralRe: Not an article Pin
Paulo Morgado26-Jul-12 0:34
memberPaulo Morgado26-Jul-12 0:34 
GeneralRe: Not an article Pin
Richard MacCutchan26-Jul-12 1:17
mvpRichard MacCutchan26-Jul-12 1:17 
GeneralRe: Not an article Pin
Paulo Morgado26-Jul-12 3:46
memberPaulo Morgado26-Jul-12 3:46 
GeneralRe: Not an article Pin
Richard MacCutchan26-Jul-12 5:18
mvpRichard MacCutchan26-Jul-12 5:18 
GeneralRe: Not an article Pin
Paulo Morgado26-Jul-12 5:27
memberPaulo Morgado26-Jul-12 5:27 
GeneralRe: Not an article Pin
Richard MacCutchan26-Jul-12 5:56
mvpRichard MacCutchan26-Jul-12 5:56 
GeneralRe: Not an article Pin
Paulo Morgado26-Jul-12 6:14
memberPaulo Morgado26-Jul-12 6:14 
GeneralRe: Not an article Pin
Matt T Heffron26-Jul-12 8:29
memberMatt T Heffron26-Jul-12 8:29 
GeneralRe: Not an article Pin
Paulo Morgado26-Jul-12 8:50
memberPaulo Morgado26-Jul-12 8:50 
GeneralRequires more explanation and clarity Pin
PIEBALDconsult25-Jul-12 7:10
memberPIEBALDconsult25-Jul-12 7:10 
GeneralRe: Requires more explanation and clarity Pin
Paulo Morgado25-Jul-12 13:41
memberPaulo Morgado25-Jul-12 13:41 
GeneralRe: Requires more explanation and clarity Pin
PIEBALDconsult26-Jul-12 3:21
memberPIEBALDconsult26-Jul-12 3:21 
GeneralRe: Requires more explanation and clarity Pin
Paulo Morgado26-Jul-12 3:34
memberPaulo Morgado26-Jul-12 3:34 
GeneralRe: Requires more explanation and clarity Pin
PIEBALDconsult26-Jul-12 3:57
memberPIEBALDconsult26-Jul-12 3:57 
GeneralRe: Requires more explanation and clarity Pin
Paulo Morgado26-Jul-12 4:06
memberPaulo Morgado26-Jul-12 4:06 
GeneralRe: Requires more explanation and clarity Pin
PIEBALDconsult26-Jul-12 5:23
memberPIEBALDconsult26-Jul-12 5:23 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.

| Advertise | Privacy | Terms of Use | Mobile
Web04 | 2.8.150520.1 | Last Updated 30 Jul 2012
Article Copyright 2012 by Paulo Morgado
Everything else Copyright © CodeProject, 1999-2015
Layout: fixed | fluid