PacktPublishing / C-Sharp-Data-Structures-and-Algorithms---Second-Edition

C# Data Structures and Algorithms - Second Edition by Packt Publishing
MIT License
21 stars 16 forks source link

Chapter08 broken #7

Open iankode opened 2 months ago

iankode commented 2 months ago

In Chapter08 the dependencies are broken expecting "..\08-Graph\08-Graph.csproj" but actual is "..\Graph\08-Graph.csproj",

Severity    Code    Description Project File    Line    Suppression State
Warning     The referenced project '..\08-Graph\08-Graph.csproj' does not exist.    08-DirectedWeightedEdges    C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets 2153    
Warning     The referenced project '..\08-Graph\08-Graph.csproj' does not exist.    08-PathInGame   C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets 2153    
Warning     The referenced project '..\08-Graph\08-Graph.csproj' does not exist.    08-TelecommunicationCable   C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets 2153    
Warning     The referenced project '..\08-Graph\08-Graph.csproj' does not exist.    08-UndirectedUnweightedEdges    C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets 2153    
Warning     The referenced project '..\08-Graph\08-Graph.csproj' does not exist.    08-VoivodeshipMap   C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets 2153    

failing

F:\MarcinJ>tree

Folder PATH listing for volume Training
Volume serial number is 9CBD-A162
...
├───Chapter08
│   ├───DirectedWeightedEdges
│   ├───Graph
│   ├───PathInGame
│   ├───TelecommunicationCable
│   ├───UndirectedUnweightedEdges
│   └───VoivodeshipMap
...

because the actual independent project saught is (note "Graph" vs "08-Graph") ..

F:\MarcinJ>type Chapter08\DirectedWeightedEdges\08-DirectedWeightedEdges.csproj
<Project Sdk="Microsoft.NET.Sdk">
        <PropertyGroup>
                <OutputType>Exe</OutputType>
                <TargetFramework>net8.0</TargetFramework>
                <RootNamespace>_08_DirectedWeightedEdges</RootNamespace>
                <ImplicitUsings>enable</ImplicitUsings>
                <Nullable>enable</Nullable>
        </PropertyGroup>

        <ItemGroup>
                <ProjectReference Include="..\08-Graph\08-Graph.csproj" />
        </ItemGroup>

</Project>

F:\MarcinJ>

Unclear when this error has arisen (e.g. author code or polishing by Packt), and sad no reviewer caught these errors thus Chapter08 as friday-edit-ship is almost identical to the train-wreck of Chapter07 (Issue #5, PR #6)

I will shortly provide a PR to address this mismatch, and my approach will be to fix those 5 dependent parents I accept that changing that single independent child would be an alternate fix

marcinjamro commented 2 months ago

Hello,

As I mentioned recently, the publisher modified the files I sent to them and uploaded the wrong version to the repository. Immediately after receiving the information from you, I contacted the publisher regarding this problem. It seems that the appropriate version of the files is currently already in the repository and there should be no problems with building the solution. I would be very grateful if you could check if everything is working fine and thanks again for reporting the problem!

If you have any questions or problems, I will be happy to help. The easiest way to contact me directly is via the contact details on my website, i.e. https://marcin.com :-)

Have a nice day! Marcin