Code cleanup and conforming to a coding covention

Controllers/v1/SongController.cs

@@ -16,118 +16,101 @@ using Icarus.Database.Repositories;
 
 namespace Icarus.Controllers.V1
 {
-	[Route("api/v1/song")]
-	[ApiController]
-    	public class SongController : ControllerBase
-    	{
-		#region Fields
-		private readonly ILogger<SongController> _logger;
-		private IConfiguration _config;
-		private SongManager _songMgr;
-		#endregion
+    [Route("api/v1/song")]
+    [ApiController]
+    public class SongController : ControllerBase
+    {
+        #region Fields
+        private readonly ILogger<SongController> _logger;
+        private IConfiguration _config;
+        private SongManager _songMgr;
+        #endregion
 
 
-		#region Properties
-		#endregion
+        #region Properties
+        #endregion
 
 
-		#region Constructor
-		public SongController(IConfiguration config, ILogger<SongController> logger)
-		{
-			_config = config;
-			_logger = logger;
-			_songMgr = new SongManager(config);
-		}
-		#endregion
+        #region Constructor
+        public SongController(IConfiguration config, ILogger<SongController> logger)
+        {
+            _config = config;
+            _logger = logger;
+            _songMgr = new SongManager(config);
+        }
+        #endregion
 
 
-        	[HttpGet]
-		[Authorize("read:song_details")]
-        	public IActionResult Get()
-        	{
-			List<Song> songs = new List<Song>();
-			Console.WriteLine("Attemtping to retrieve songs");
-			_logger.LogInformation("Attempting to retrieve songs");
-			
-			SongRepository context = HttpContext
-				.RequestServices
-				.GetService(typeof(SongRepository)) as SongRepository;
+        [HttpGet]
+        [Authorize("read:song_details")]
+        public IActionResult Get()
+        {
+            List<Song> songs = new List<Song>();
+            Console.WriteLine("Attemtping to retrieve songs");
+            _logger.LogInformation("Attempting to retrieve songs");
+            
+            SongRepository context = HttpContext.RequestServices
+                .GetService(typeof(SongRepository)) as SongRepository;
 
-			songs = context.GetAllSongs();
+            songs = context.GetAllSongs();
 
-			if (songs.Count > 0)
-			{
-				return Ok(songs);
-			}
-			else
-			{
-				return NotFound();
-			}
-        	}
+            if (songs.Count > 0)
+                return Ok(songs);
+            else
+                return NotFound();
+        }
 
-		[HttpGet("{id}")]
-		[Authorize("read:song_details")]
-		public IActionResult Get(int id)
-		{
-			SongRepository context = HttpContext
-				.RequestServices
-				.GetService(typeof(SongRepository)) as SongRepository;
-			
-			Song song = new Song { Id = id };
-			song = context.GetSong(song);
+        [HttpGet("{id}")]
+        [Authorize("read:song_details")]
+        public IActionResult Get(int id)
+        {
+            SongRepository context = HttpContext.RequestServices
+                .GetService(typeof(SongRepository)) as SongRepository;
+            
+            Song song = new Song { Id = id };
+            song = context.GetSong(song);
 
-			Console.WriteLine("Here");
+            Console.WriteLine("Here");
 
-			if (song.Id != 0)
-			{
-				return Ok(song);
-			}
-			else
-			{
-				return NotFound();
-			}
-		}
+            if (song.Id != 0)
+                return Ok(song);
+            else
+                return NotFound();
+        }
 
-		[Authorize("update:songs")]
-        	[HttpPut("{id}")]
-		public IActionResult Put(int id, [FromBody] Song song)
-        	{
-			SongRepository context = HttpContext
-				.RequestServices
-				.GetService(typeof(SongRepository)) as SongRepository;
+        [Authorize("update:songs")]
+        [HttpPut("{id}")]
+        public IActionResult Put(int id, [FromBody] Song song)
+        {
+            SongRepository context = HttpContext.RequestServices
+                .GetService(typeof(SongRepository)) as SongRepository;
 
-			ArtistRepository artistStore = HttpContext
-				.RequestServices
-				.GetService(typeof(ArtistRepository)) as ArtistRepository;
+            ArtistRepository artistStore = HttpContext.RequestServices
+                .GetService(typeof(ArtistRepository)) as ArtistRepository;
 
-			AlbumRepository albumStore = HttpContext
-				.RequestServices
-				.GetService(typeof(AlbumRepository)) as AlbumRepository;
-			
-			GenreRepository genreStore = HttpContext
-				.RequestServices
-				.GetService(typeof(GenreRepository)) as GenreRepository;
+            AlbumRepository albumStore = HttpContext.RequestServices
+                .GetService(typeof(AlbumRepository)) as AlbumRepository;
+            
+            GenreRepository genreStore = HttpContext.RequestServices
+                .GetService(typeof(GenreRepository)) as GenreRepository;
 
-			YearRepository yearStore = HttpContext
-				.RequestServices
-				.GetService(typeof(YearRepository)) as YearRepository;
+            YearRepository yearStore = HttpContext.RequestServices
+                .GetService(typeof(YearRepository)) as YearRepository;
 
-			song.Id = id;
-			Console.WriteLine("Retrieving filepath of song");
-			_logger.LogInformation("Retrieving filepath of song");
+            song.Id = id;
+            Console.WriteLine("Retrieving filepath of song");
+            _logger.LogInformation("Retrieving filepath of song");
 
-			if (!context.DoesSongExist(song))
-			{
-				return NotFound(new SongResult
-					{
-						Message = "Song does not exist"
-					});
-			}
+            if (!context.DoesSongExist(song))
+                return NotFound(new SongResult
+                {
+                        Message = "Song does not exist"
+                });
 
-			var songRes = _songMgr.UpdateSong(song, context, albumStore, artistStore, genreStore, 
-					yearStore);
+            var songRes = _songMgr.UpdateSong(song, context, albumStore, artistStore, genreStore, 
+                yearStore);
 
-			return Ok(songRes);
-        	}
-    	}
+            return Ok(songRes);
+        }
+    }
 }