flamencist / flamencist/Core.System.Configuration.Install

Evaluate replacing JsonConvert serialization

Abierto
#1 3 comentarios 1 reacción 0 asignados Ver en GitHub
Lenguaje dominante
C#
Estrellas
26
Forks
12
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

It looks like you're just using JSON.net to serialize `IDictionary` in savedState and this could be accomplished with built-in serializers that would remove the dependency on JSON.net (so that a different version can be used or just not bring that dependency along with someone using System.Configuration.Install).

I looked at the reference source and the built-in classes use `NetDataContractSerializer` which can serialize/deserialize arbitrary types in an `IDictionary`. We don't have that in `netstandard2.0`. Another possibility is `BinaryFormatter`, but that requires `ISerializable` which `IDictionary` is not. The concrete type is `Hashtable` in the underlying implementation for installers.

@flamencist thought [this approach](https://blogs.msdn.microsoft.com/adam/2010/09/10/how-to-serialize-a-dictionary-or-hashtable-in-c/) might work, but it relies on `ToString()` to put it in a key value store. Since `IDictionary` (and `Hashtable`) can hold any object, this doesn't really work. Some objects may not have a sensible `ToString()` which would end up with the type name in the output, and they can't really be deserialized in this manner.

In the `Installer` source I looked at, it appeared that it was putting an integer in there for the count of `Installer`s and then saving all the child `Installer` states (which can be any object since anyone can inherit `Installer`).

The reason `JsonConvert` works is that it serializes the type information too. Is it possible to replace it with something we have in `netstandard2.0` so that we don't have to bring along a JSON.Net dependency?

Guía de contribución

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

Línea de trabajo

Start by tracing how Installer savedState is serialized and deserialized, then compare the netstandard2.0 serializer options discussed in the issue, including NetDataContractSerializer and BinaryFormatter. Verify whether arbitrary IDictionary and child Installer state can round-trip without Json.NET; done means identifying a compatible replacement or documenting why the dependency must remain.

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

Evaluación

Stack tecnológico
csharp
Área
tooling
Tipo de issue
Refactorización
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.