[Feature]: Keep dedicated .Sort(), instead of deprecating

Ouverte
#963 8 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
35/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
csharp

Piste de recherche

Start by reviewing the deprecation of .Sort() and the SortAndBind() alternative, then compare them with the example chain using DistinctValues and Bind. Determine whether keeping an immediate sort supports a case that SortAndBind() cannot handle; done means the issue’s requested behavior and its relationship to the deprecation are resolved.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Describe the functionality desired 🐞

With the .Sort() method being depreciated there are a couple of rare cases where applying a sort immediately is desired because the properties used in the comparison do not exist downstream.

example:

    changeSet
            .Sort(complexSortComparer) //Sorts on input data type
            .DistinctValues(x => x.Name) //DistinctValues is transforming to Name data type 
            .Bind(out names); //List of names is sorted 

Changing the call chain to use SortAndBind() is not possible in this scenario.

I had a conversation on Slack about this topic:
https://reactivex.slack.com/archives/C4LF8S19N/p1733511277945719

The steps the functionality will provide
  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error
Considerations

SortAndBind

Langage dominant
C#
Étoiles
1.9k
Forks
194
Merge moyen
12 min
PR mergées (30 j)
1

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de reactivemarbles/DynamicData

Toutes les issues de reactivemarbles/DynamicData

Issues similaires

Plus d'issues C#

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.