search

s-afchar / Blueflap-Universe

Blueflap as a universal app
13 stars 2 forks source link

Rendre mon code propre #54

Open s-afchar opened 8 years ago

A-312 commented 8 years ago

Utiliser un peu plus With ... End ? ;)

A-312 commented 8 years ago 

            Await newView.Dispatcher.RunAsync(Windows.UI.Core.CoreDispatcherPriority.Normal, Function()
                                                                                                 Dim frame As New Frame()
                                                                                                 frame.Navigate(GetType(MiniPlayer), Nothing)
                                                                                                 Window.Current.Content = frame
                                                                                                 'You have to activate the window in order to show it later.
                                                                                                 Window.Current.Activate()

                                                                                                 newViewId = ApplicationView.GetForCurrentView().Id

                                                                                             End Function)

C'est illisible ce genre de chose. (Une des normes de programmation, conseil de ne pas trop aller à droite, pour ne pas à avoir à scroll à gauche et à droite.

A-312 commented 8 years ago

Et il y a beaucoup de c/c de code, pourquoi ne pas avoir utilisé une fonction ?

elegaanz commented 8 years ago

Sinon utiliser du DataBinding pour des trucs genre le titre de la fenêtre peut faire gagner quelques lignes de code pas très utiles. Et pourquoi passer plus ou moins sur du MVVM.