Fix defects detected by static analysis tool 48/318548/1
authorCORP\mobaswirul.i <mobaswirul.i@samsung.com>
Wed, 2 Oct 2024 05:29:02 +0000 (11:29 +0600)
committerCORP\mobaswirul.i <mobaswirul.i@samsung.com>
Wed, 2 Oct 2024 05:29:02 +0000 (11:29 +0600)
[Problem] Issues detected by Static Analysis Tool
[Cause & Measure]
 Cause   : No Default Case in switch statements
 Measure : Added Default Case

Change-Id: Ide1d85dd859765a0db5351f886c00a4814c965fa

music-player/Core/MusicPlayer.cs
music-player/ViewModels/PlayerViewModel.cs
music-player/Views/Utils/ContentViewFactory.cs
music-player/Views/Utils/OperationViewFactory.cs
music-player/Views/Utils/SubContentViewFactory.cs

index 7d4c5a43e1f4a85f7765bb8e49fa8bf1ea524cdc..b9c0484690c21e64bc9c8abf1e41e3954fadc4e9 100755 (executable)
@@ -219,6 +219,9 @@ namespace MusicPlayer.Core
                 case PlayerError.AudioCodecNotSupported:
                     errorDesc = "Audio codec not supported";
                     break;
+                default:
+                    errorDesc = "Unknown Error";
+                    break;
             }
             return errorDesc;
         }
index 6ad737ccf8912da7106e36aece5c907c948ee24d..ea6df6e620c9153f999b2aa14dfa7f922fe0279f 100755 (executable)
@@ -355,6 +355,8 @@ namespace MusicPlayer.ViewModels
                     CurrentPlayingTrack.Instance.SetPlayingStatus(false);
                     lyricsViewModel.UpdateTrackThumbState(AnimationState.AnimationPause);
                     break;
+                default:
+                    break;
             }
         }
 
index 437b22f9687fd84c05d68985bf4b217ba906f686..aff3b406291104933bda6c4b3df0392d619a9a76 100755 (executable)
@@ -50,6 +50,9 @@ namespace MusicPlayer.Views.Utils
                     ArtistView artistView = new ArtistView(artistViewModel);
                     contentView = artistView;
                     break;
+                default:
+                    Tizen.Log.Warn(AppConstants.LogTag, "No ContentViewType Matched");
+                    break;
             }
             return contentView;
         }
index f5f5a8ab2b8bd24cc11d9c22649ad73ac6e4a567..f4e937dc003ad3d141eb04aab6c02d09a84648f0 100755 (executable)
@@ -60,6 +60,9 @@ namespace MusicPlayer.Views.Utils
                     SelectorView<Track> removeFromCurrentPlaylistView = new SelectorView<Track>(OperationViewType.RemoveFromCurrentPlaylist, removeFromCurrentPlaylistViewModel);
                     view = removeFromCurrentPlaylistView;
                     break;
+                default:
+                    Tizen.Log.Warn(AppConstants.LogTag, "No OperationViewType Matched");
+                    break;
             }
             return view;
         }
@@ -85,6 +88,8 @@ namespace MusicPlayer.Views.Utils
                     SelectorViewModel<Artist> artistViewModel = new SelectorViewModel<Artist>((ListViewModel<Artist>)viewModelData);
                     deleteView = new SelectorView<Artist>(OperationViewType.Delete, artistViewModel);
                     break;
+                default:
+                    break;
             }
             return deleteView;
         }
@@ -106,6 +111,8 @@ namespace MusicPlayer.Views.Utils
                     SelectorViewModel<Artist> artistViewModel = new SelectorViewModel<Artist>((ListViewModel<Artist>)viewModelData);
                     addToPlaylistView = new SelectorView<Artist>(OperationViewType.AddToPlaylist, artistViewModel);
                     break;
+                default:
+                    break;
             }
             return addToPlaylistView;
         }
index 6d32004699ff63d28af6ef8e600b300d6cb83d2f..7de39ad249480e048838ce99e861a00c7f635011 100755 (executable)
@@ -44,6 +44,9 @@ namespace MusicPlayer.Views.Utils
                     ArtistDetailView artistDetailView = new ArtistDetailView(artistDetailViewModel);
                     subcontentView = artistDetailView;
                     break;
+                default:
+                    Tizen.Log.Warn(AppConstants.LogTag, "No SubContentViewType Matched");
+                    break;
             }
             return subcontentView;
         }