Pour obtenir une clé
d'essai remplissez le formulaire ci-dessous
Demandez des tariffs
Nouvelle licence
Renouvellement de licence
--Sélectionnez la devise--
USD
EUR
RUB
* En cliquant sur ce bouton, vous acceptez notre politique de confidentialité

Free PVS-Studio license for Microsoft MVP specialists
To get the licence for your open-source project, please fill out this form
** En cliquant sur ce bouton, vous acceptez notre politique de confidentialité.

I am interested to try it on the platforms:
** En cliquant sur ce bouton, vous acceptez notre politique de confidentialité.

Votre message a été envoyé.

Nous vous répondrons à


Si vous n'avez toujours pas reçu de réponse, vérifiez votre dossier
Spam/Junk et cliquez sur le bouton "Not Spam".
De cette façon, vous ne manquerez la réponse de notre équipe.

>
>
>
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.

22 Jul 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
Nous utilisons des cookies pour améliorer votre expérience de navigation. En savoir plus
Accepter