Proyecto de Yoan y Carla

[YoanRene]

Modifique las líneas siguientes con los detalles relevantes:

Equipo

• C212, Yoan René Ramos Corrales (@YoanRene)
• C212, Carla Sunami Perez Valera (@carlapvalera)

Proyecto

• Repositorio: https://github.com/YoanRene/Domino-Project
• Proyecto: https://github.com/YoanRene/Domino-Project/blob/master/Report.pdf

Checklist

(Esta lista es para el mentor. Hasta que no estén chequeados estos elementos no se procederá a la evaluación en persona.)

Básicos

• El repositorio indicado existe y tiene un Readme.md no vacío.
• Los miembros del equipo tienen un usuario de Github válido.
• El proyecto puede ser ejecutado siguiendo las instrucciones del Readme.md.
• El reporte existe y tiene la longitud adecuada.

Funcionalidades

• El proyecto permite configurar al menos 5 elementos relevantes de la mecánica del dominó.
• El proyecto contiene 2 implementaciones diferentes de cada elemento configurable.
• El proyecto permite personalizar las estrategias de los jugadores virtuales.
• El proyecto contiene al menos 3 implementaciones diferentes de jugadores virtuales.

Ingeniería de Software

• Las estrategias de jugadores virtuales pueden combinarse con cualquier configuración del juego de forma transparente.
• La interfaz gráfica permite configurar el juego en toda la extensión que brinda la lógica.
• Las jerarquías de clases y demás abstracciones existentes son adecuadas para modelar el dominio.
• El código muestra una organización y estructura mantenible y extensible, respetando las buenas prácticas de la ingeniería de software.

Documentación

• El reporte técnico explica de forma suficientemente detallada los pormenores del proyecto.
• Los nombres de clases, métodos y variables son descriptivos.
• Los comentarios son legibles y útiles para entender el funcionamiento del código.

Revisión

Fecha prevista: 2022/09/08

Comentarios

(Para rellenar por el mentor)

[e1Ru1o]

Los pasos de ejecucion no son suficientemente detallados. Se brindas dos opciones:

1. Ejecutar un fichero en la carpeta bin la cual no se encuentra en el repo (lo cual esta correcto!!), por tanto para iniciar el proyecto no puedo ir a ese fichero. Ademas de que es un .exe por tanto en otros sistemas no se podria encontrar, como es el caso de Linux, el cual se menciona en el readme.
2. "ejecutarlo por los métodos tradicionales ... se debe establecer como Startup Project a Visual Interface". Esto no es correcto, se debe dar detalles especificos. Cualquier persona leyendo el readme debe ser capaz de ejecutar el proyecto. Yo asumo se refieran a dotnet run --project Visual\ Interface/ pero no deberia tener que asumir nada.

Como dato adicional, el comando que menciono en el punto 2 me da el siguiente error

/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018: The "ResolvePackageAssets" task failed unexpectedly. [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018: System.IO.InvalidDataException: Found invalid data while decoding. [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.ValidateHeader(BinaryReader reader, Byte[] settingsHash) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.OpenCacheStream(Stream stream, Byte[] settingsHash) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.CreateReaderFromDisk(ResolvePackageAssets task, Byte[] settingsHash) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader..ctor(ResolvePackageAssets task) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ReadItemGroups() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ExecuteCore() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.TaskBase.Execute() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]

The build failed. Fix the build errors and run again.```

Mejorar la seccion sobre la ejecucion del proyecto es necesario

[YoanRene]

Los pasos de ejecucion no son suficientemente detallados. Se brindas dos opciones:

1. Ejecutar un fichero en la carpeta bin la cual no se encuentra en el repo (lo cual esta correcto!!), por tanto para iniciar el proyecto no puedo ir a ese fichero. Ademas de que es un .exe por tanto en otros sistemas no se podria encontrar, como es el caso de Linux, el cual se menciona en el readme.
2. "ejecutarlo por los métodos tradicionales ... se debe establecer como Startup Project a Visual Interface". Esto no es correcto, se debe dar detalles especificos. Cualquier persona leyendo el readme debe ser capaz de ejecutar el proyecto. Yo asumo se refieran a dotnet run --project Visual\ Interface/ pero no deberia tener que asumir nada.

Como dato adicional, el comando que menciono en el punto 2 me da el siguiente error

/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018: The "ResolvePackageAssets" task failed unexpectedly. [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018: System.IO.InvalidDataException: Found invalid data while decoding. [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.ValidateHeader(BinaryReader reader, Byte[] settingsHash) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.OpenCacheStream(Stream stream, Byte[] settingsHash) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader.CreateReaderFromDisk(ResolvePackageAssets task, Byte[] settingsHash) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.CacheReader..ctor(ResolvePackageAssets task) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ReadItemGroups() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.ResolvePackageAssets.ExecuteCore() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.NET.Build.Tasks.TaskBase.Execute() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute() [/some-path/Issue22/Visual Interface/Visual Interface.csproj]
/usr/share/dotnet/sdk/6.0.300/Sdks/Microsoft.NET.Sdk/targets/Microsoft.PackageDependencyResolution.targets(267,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask) [/some-path/Issue22/Visual Interface/Visual Interface.csproj]

The build failed. Fix the build errors and run again.```

Mejorar la seccion sobre la ejecucion del proyecto es necesario

Corregido