Merge branch 'main' of https://github.com/poppastring/buggy-demo-code

Author: poppastring

.gitignore

@@ -12,3 +12,4 @@
 /src/BuggyDemoWeb/bin
 /src/BuggyDemoWeb/obj
 /src/BuggyDemoWeb/Properties/
+/src/BuggyDemoWeb/Controllers/UnresponsiveLowCPUController.cs

src/BuggyDemoWeb/Controllers/UnresponsiveLowCPUController.cs

@@ -15,8 +15,6 @@ namespace BuggyDemoWeb.Controllers
     public class UnresponsiveLowCPUController : Controller
     {
         private readonly LegacyService legacyService;
-        private Account _accountChecking;
-        private Account _accountSaving;
 
         public UnresponsiveLowCPUController(LegacyService legacyService)
         {
@@ -155,20 +153,37 @@ public IActionResult CreateDeadlockTaskYieldReturn()
 
             return Ok();
         }
-
-        /// <summary>
-        /// Classic deadlock... Thread A locked waiting on Thread B, and vice versa
-        /// </summary>
-        /// <returns></returns>
+        
         [HttpGet("lowcpu/deadlocked-tasks-v4")]
-        public IActionResult TypicalDeadlockIssue()
+        public IActionResult MoneyTransfer()
         {
-            legacyService.Transfer(_accountChecking, _accountSaving, 20);
-            legacyService.Transfer(_accountSaving, _accountChecking, 50);
+            Transfer(_accountChecking, _accountSaving, 20);
+            Transfer(_accountSaving, _accountChecking, 50);
 
             return Ok();
         }
 
+        private Account _accountChecking;
+        private Account _accountSaving;
+
+        public Task Transfer(Account fromaccount, Account toaccount, int sum)
+        {
+            var task = Task.Run(() =>
+            {
+                lock (fromaccount)
+                {
+                    Thread.Sleep(1000);
+                    lock (toaccount)
+                    {
+                        fromaccount.Balance -= sum;
+                        toaccount.Balance += sum;
+                    }
+                }
+            });
+
+            return task;
+        }
+
         [HttpGet("lowcpu/delayed-data-retrieval/{delay}")]
         public async Task<IActionResult> RetrievingDataWithDelay(int delay = 0)
         {