Refactor DebugLayer

Author: deccer

src/Cpp/1-getting-started/1-2-2-DebugLayer/1-2-2-DebugLayer.vcxproj

@@ -112,17 +112,13 @@ xcopy /Y $(ProjectDir)Assets\Textures\*.* $(OutDir)Assets\Textures\</Command>
   </ItemDefinitionGroup>
   <ItemGroup>
     <ClCompile Include="DebugLayerApplication.cpp" />
-    <ClCompile Include="DeviceContext.cpp" />
     <ClCompile Include="Main.cpp" />
-    <ClCompile Include="Pipeline.cpp" />
-    <ClCompile Include="PipelineFactory.cpp" />
+    <ClCompile Include="ShaderCollection.cpp" />
   </ItemGroup>
   <ItemGroup>
     <ClInclude Include="DebugLayerApplication.hpp" />
     <ClInclude Include="Definitions.hpp" />
-    <ClInclude Include="DeviceContext.hpp" />
-    <ClInclude Include="Pipeline.hpp" />
-    <ClInclude Include="PipelineFactory.hpp" />
+    <ClInclude Include="ShaderCollection.hpp" />
     <ClInclude Include="VertexType.hpp" />
   </ItemGroup>
   <ItemGroup>

src/Cpp/1-getting-started/1-2-2-DebugLayer/1-2-2-DebugLayer.vcxproj.filters

@@ -21,13 +21,7 @@
     <ClCompile Include="DebugLayerApplication.cpp">
       <Filter>Source Files</Filter>
     </ClCompile>
-    <ClCompile Include="PipelineFactory.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="DeviceContext.cpp">
-      <Filter>Source Files</Filter>
-    </ClCompile>
-    <ClCompile Include="Pipeline.cpp">
+    <ClCompile Include="ShaderCollection.cpp">
       <Filter>Source Files</Filter>
     </ClCompile>
   </ItemGroup>
@@ -38,16 +32,10 @@
     <ClInclude Include="Definitions.hpp">
       <Filter>Header Files</Filter>
     </ClInclude>
-    <ClInclude Include="Pipeline.hpp">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="PipelineFactory.hpp">
-      <Filter>Header Files</Filter>
-    </ClInclude>
-    <ClInclude Include="DeviceContext.hpp">
+    <ClInclude Include="VertexType.hpp">
       <Filter>Header Files</Filter>
     </ClInclude>
-    <ClInclude Include="VertexType.hpp">
+    <ClInclude Include="ShaderCollection.hpp">
       <Filter>Header Files</Filter>
     </ClInclude>
   </ItemGroup>

src/Cpp/1-getting-started/1-2-2-DebugLayer/DebugLayerApplication.cpp

@@ -1,7 +1,4 @@
 #include "DebugLayerApplication.hpp"
-#include "DeviceContext.hpp"
-#include "Pipeline.hpp"
-#include "PipelineFactory.hpp"
 #include "VertexType.hpp"
 
 #include <GLFW/glfw3.h>
@@ -29,12 +26,11 @@ DebugLayerApplication::~DebugLayerApplication()
 {
     _deviceContext->Flush();
     _triangleVertices.Reset();
-    _pipeline.reset();
-    _pipelineFactory.reset();
     DestroySwapchainResources();
     _swapChain.Reset();
     _dxgiFactory.Reset();
-    _deviceContext.reset();
+    _shaderCollection.Destroy();
+    _deviceContext.Reset();
 #if !defined(NDEBUG)
     _debug->ReportLiveDeviceObjects(D3D11_RLDO_FLAGS::D3D11_RLDO_DETAIL);
     _debug.Reset();
@@ -85,7 +81,7 @@ bool DebugLayerApplication::Initialize()
         return false;
     }
 
-    _deviceContext = std::make_unique<DeviceContext>(std::move(deviceContext));
+    _deviceContext = deviceContext;
 
     DXGI_SWAP_CHAIN_DESC1 swapChainDescriptor = {};
     swapChainDescriptor.Width = GetWindowWidth();
@@ -116,28 +112,17 @@ bool DebugLayerApplication::Initialize()
 
     CreateSwapchainResources();
 
-    _pipelineFactory = std::make_unique<PipelineFactory>(_device);
-
     return true;
 }
 
 bool DebugLayerApplication::Load()
 {
-    PipelineDescriptor pipelineDescriptor = {};
-    pipelineDescriptor.VertexFilePath = L"Assets/Shaders/Main.vs.hlsl";
-    pipelineDescriptor.PixelFilePath = L"Assets/Shaders/Main.ps.hlsl";
-    pipelineDescriptor.VertexType = VertexType::PositionColor;
-    if (!_pipelineFactory->CreatePipeline(pipelineDescriptor, _pipeline))
-    {
-        std::cout << "PipelineFactory: Failed to create pipeline\n";
-        return false;
-    }
+    ShaderCollectionDescriptor shaderDescriptor = {};
+    shaderDescriptor.VertexShaderFilePath = L"Assets/Shaders/Main.vs.hlsl";
+    shaderDescriptor.PixelShaderFilePath = L"Assets/Shaders/Main.ps.hlsl";
+    shaderDescriptor.VertexType = VertexType::PositionColor;
 
-    _pipeline->SetViewport(
-        0.0f,
-        0.0f,
-        static_cast<float>(GetWindowWidth()),
-        static_cast<float>(GetWindowHeight()));
+    _shaderCollection = ShaderCollection::CreateShaderCollection(shaderDescriptor, _device.Get());
 
     constexpr VertexPositionColor vertices[] = {
         {  Position{ 0.0f, 0.5f, 0.0f }, Color{ 0.25f, 0.39f, 0.19f }},
@@ -223,11 +208,37 @@ void DebugLayerApplication::Update()
 void DebugLayerApplication::Render()
 {
     float clearColor[] = { 0.1f, 0.1f, 0.1f, 1.0f };
+    ID3D11RenderTargetView* nullRTV = nullptr;
     constexpr uint32_t vertexOffset = 0;
 
-    _deviceContext->Clear(_renderTarget.Get(), clearColor);
-    _deviceContext->SetPipeline(_pipeline.get());
-    _deviceContext->SetVertexBuffer(_triangleVertices.Get(), vertexOffset);
-    _deviceContext->Draw();
+    D3D11_VIEWPORT viewport = {
+        0.0f,
+        0.0f,
+        static_cast<float>(GetWindowWidth()),
+        static_cast<float>(GetWindowHeight()),
+        0.0f,
+        1.0f
+    };
+
+    _deviceContext->RSSetViewports(1, &viewport);
+    _deviceContext->IASetPrimitiveTopology(D3D11_PRIMITIVE_TOPOLOGY::D3D11_PRIMITIVE_TOPOLOGY_TRIANGLELIST);
+
+    _shaderCollection.ApplyToContext(_deviceContext.Get());
+
+    _deviceContext->OMSetRenderTargets(1, &nullRTV, nullptr);
+
+    _deviceContext->ClearRenderTargetView(_renderTarget.Get(), clearColor);
+    _deviceContext->OMSetRenderTargets(1, _renderTarget.GetAddressOf(), nullptr);
+
+    UINT stride = _shaderCollection.GetLayoutByteSize(VertexType::PositionColor);
+    _deviceContext->IASetVertexBuffers(
+        0,
+        1,
+        _triangleVertices.GetAddressOf(),
+        &stride,
+        &vertexOffset);
+
+    _deviceContext->Draw(3, 0);
+
     _swapChain->Present(1, 0);
 }

src/Cpp/1-getting-started/1-2-2-DebugLayer/DebugLayerApplication.hpp

@@ -2,15 +2,12 @@
 
 #include "Definitions.hpp"
 #include <Application.hpp>
+#include "ShaderCollection.hpp"
 
 #include <d3d11_2.h>
 
 #include <memory>
 
-class DeviceContext;
-class Pipeline;
-class PipelineFactory;
-
 class DebugLayerApplication final : public Application
 {
 public:
@@ -30,11 +27,10 @@ class DebugLayerApplication final : public Application
     bool CreateSwapchainResources();
     void DestroySwapchainResources();
 
-    std::unique_ptr<DeviceContext> _deviceContext = nullptr;
-    std::unique_ptr<Pipeline> _pipeline = nullptr;
-    std::unique_ptr<PipelineFactory> _pipelineFactory = nullptr;
+    ShaderCollection _shaderCollection;
 
     WRL::ComPtr<ID3D11Device> _device = nullptr;
+    WRL::ComPtr<ID3D11DeviceContext> _deviceContext = nullptr;
     WRL::ComPtr<IDXGIFactory2> _dxgiFactory = nullptr;
     WRL::ComPtr<IDXGISwapChain1> _swapChain = nullptr;
     WRL::ComPtr<ID3D11RenderTargetView> _renderTarget = nullptr;