microsoft / microsoft/sqlmanagementobjects

Replace SmoSortedList usage of SortedList with SortedList<ObjectKeyBase, T>

Abierto
#140 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
C#
Estrellas
143
Forks
28
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

After completing #130, push the type parameters further down into the implementation so we can remove code like this and just return the Values from the list.

        // nested enumerator class
        // we need that to override the behaviour of SortedList
        // that exposes an IDictionaryEnumerator interface
        internal sealed class SmoSortedListEnumerator : IEnumerator 
        {
            private IEnumerator baseEnumerator;
            
            internal SmoSortedListEnumerator(IEnumerator enumerator) 
            {
                this.baseEnumerator = enumerator;
            }

            public object Current 
            {
                get 
                {
                    return ((DictionaryEntry)baseEnumerator.Current).Value;
                }
            }

            public bool MoveNext() 
            {
                return baseEnumerator.MoveNext();
            }
            
            public void Reset() 
            {
                baseEnumerator.Reset();
            }
        }

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza revisando el trabajo de #130 y localizando el uso de SortedList por parte de SmoSortedList. Sigue cómo su enumerador expone actualmente los valores y, a continuación, verifica que la implementación genérica pueda devolver directamente los Values de la lista sin el código del enumerador anidado.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
csharp
Área
databases
Tipo de issue
Refactorización
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
42/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.