To get a trial key
fill out the form below
Team license
Enterprise license
** By clicking this button you agree to our Privacy Policy statement

Request our prices
New License
License Renewal
--Select currency--
USD
EUR
RUB
* By clicking this button you agree to our Privacy Policy statement

Free PVS-Studio license for Microsoft MVP specialists
** By clicking this button you agree to our Privacy Policy statement

To get the licence for your open-source project, please fill out this form
** By clicking this button you agree to our Privacy Policy statement

I am interested to try it on the platforms:
** By clicking this button you agree to our Privacy Policy statement

Message submitted.

Your message has been sent. We will email you at


If you haven't received our response, please do the following:
check your Spam/Junk folder and click the "Not Spam" button for our message.
This way, you won't miss messages from our team in the future.

>
>
>
V3078. Sorting keys priority will be re…
Analyzer diagnostics
General Analysis (C++)
General Analysis (C#)
General Analysis (Java)
Diagnosis of micro-optimizations (C++)
Diagnosis of 64-bit errors (Viva64, C++)
Customer specific requests (C++)
MISRA errors
AUTOSAR errors
OWASP errors (C#)
Problems related to code analyzer
Additional information
Contents

V3078. Sorting keys priority will be reversed relative to the order of 'OrderBy' method calls. Perhaps, 'ThenBy' should be used instead.

Jul 22 2022

The analyzer has detected a potential error: 'OrderBy' or 'OrderByDescending' methods are called twice in a row. The result of such sorting may differ from the expected one.

Consider an example:

var seq = points.OrderBy(item => item.Primary)
                .OrderBy(item => item.Secondary);

Let's say a programmer wants to sort a collection in the following way: the elements must be grouped and sorted by 'Primary', and after that, each group must be sorted by 'Secondary' inside the resulting collection.

In fact, the collection's elements will be grouped and sorted by 'Secondary', and after that, each group will be sorted by 'Primary' inside the resulting collection.

To get the expected behavior, the second 'OrderBy' call should be replaced with the 'ThenBy' call.

var seq = points.OrderBy(item => item.Primary)
                .ThenBy(item => item.Secondary);

To get the intended behavior, it is also possible to use two 'OrderBy' calls. To do this, we need to swap the calls:

var seq = points.OrderBy(item => item.Secondary)
                .OrderBy(item => item.Primary);

A programmer can make a similar error when writing code with the help of query syntax:

var seq = from item in points
          orderby item.Primary
          orderby item.Secondary 
          select item;

We can fix the code as follows:

var seq = from item in points
          orderby item.Primary, item.Secondary
          select item;

You can look at examples of errors detected by the V3078 diagnostic.

Unicorn with delicious cookie
Our website uses cookies to enhance your browsing experience.
Accept