Advertisement
Mikescher

Untitled

Jun 20th, 2019
88
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 1.75 KB | None | 0 0
  1. [[ 7ad7139 ]] Armin:
  2.  
  3. - Irgendwie hast du es geschafft die ProjektID in Android.csproj zu ändern - keine Ahnung wie, aber wär nett wenn du beim comitten schauen könntest was du da pushed
  4.  
  5. - Ich habe keine Ahnung was ich mit Klassen im Testing namespace machen soll, ich würde sagen erstell richtige Objekte oder commite die nicht
  6.  
  7. - Deine sourcedateien sind jetzt ein misch-masch aus tabs und spaces - das ist irgendwie noch schlimmer als vorher :D
  8.  
  9. - Die xaml Datei ist voller Warnings, ich glaube das android projekt hat bewiesen dass niemand im nachhinein durchgeht und die später fixed
  10.  
  11. - ios:Page.UseSafeArea sollte man nur im top level element verwenden und nicht mitten in der hierachie, wie in einer tabpage
  12.  
  13. - bei den anderen seiten haben wir einen expliziten margin, von mir aus können wir auch ios:Page.UseSafeArea benutzen - nur eine einheitliche Lösung wäre schön
  14.  
  15. - die xaml dateien bitte nicht so wie android formatieren und (grundlos) alle attribute in extra zeilen machen
  16.  
  17. - allgemein die xaml datei auch ordentlich formatten, mit einheitlicher enrückung und keinen random leerzeilen (wenn sie zur überischtlichkeit beitragen sind sie ok)
  18.  
  19. - x:Name sollte allgemein fast nie verwendet werden weil der datenfluss über bindings kommen/gehen sollte
  20.  
  21. - Kein Code in die Page Klasse stecken - maximal redirects für die click events (und da schaue ich mal wenn ich zeit habe dass wir das auch binden können)
  22.  
  23. - Der "as" Parameter in C# ist ein conditional-cast wenn du den sowieso nicht überprüfst dann mach ein hard-cast - ist schneller und im problemfall sagt die exception wenigstens was aus
  24.  
  25. - Methoden mit default (internal) Sichtbarkeit sind außer in Ausnahmefällen falsch. (das sind auch alles Warnings)
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement