code-review

Użycie PDO try-catch w funkcjach

Myślę o użyciu PDO w całej mojej przyszłej webapp. Obecnie (korzystając z tego, czego się do tej pory nauczyłem), to, co mam ... żadnych wad ani rzeczy, które można tam poprawić. Przepraszam, jeśli wydaje się to niejasne lub zbyt długie. Z góry dzięki.

Używanie git do przeglądania kodu?

Chciałbym opracować lekki proces przeglądu kodu, który jest podobny do tego, którego używałem, gdy pracowałem w MSFT. Oto og ... prawdopodobne, że recenzje kodu zostaną zrobione, i jestem przekonany, że prosty proces przeglądu kodu zwiększy jakość kodu.

Typy Nullable: najlepszy sposób na sprawdzenie null lub zero W c#

Pracuję nad projektem, w którym w wielu, wielu miejscach sprawdzam: if(item.Rate == 0 || item.Rate == null) { } Bardziej ... rse(obj.ToString(), out parsedInt); return IsNull(obj) || (parsed && parsedInt == 0); } Czy jest lepszy sposób?

C # XNA: Optymalizacja wykrywania kolizji?

Pracuję nad prostym demo do wykrywania kolizji, które zawiera tylko kilka obiektów odbijających się w oknie. (Celem jest spra ... uzyskać sąsiadów dla obiektu jest trywialnie krótki. Ta metoda się nie zmieniła, więc może nazywam ją bardziej niż kiedyś...

ResultSet nie jest zamknięty po zamknięciu połączenia?

Robiłem przegląd kodu (głównie za pomocą narzędzi takich jak FindBugs) jednego z naszych projektów pet i FindBugs oznaczał na ... ków przy zamykaniu połączeń (używamy Oracle db, stąd moje poprawki). java.sql.api nie mówi nic w połączeniu.close () javadoc.

Gerrit: połącz wiele commitów w jedną " zmianę"

Jako najlepsza praktyka git, należy commit często, ale aby przejrzeć kod, może być konieczne sprawdzenie poprawki składającej się z wielu commitów naraz. Czy istnieje sposób na sprawdzenie wielu commitów i ich scalenie lub odrzucenie jednocześnie?

Jak sprawić, by zły kod wyglądał źle? Jakich szablonów używasz, aby uniknąć błędów semantycznych? [wstrzymane]

Odkąd po raz pierwszy popełniłem błąd robiąc zadanie w if zawsze pisałem moje ifs tak: if (CONST == variable) { Aby unikn ... jakich innych wzorców używasz, aby zły kod wyglądał źle, lub aby wymusić Błędy składniowe, jeśli popełnisz błąd semantyczny?

Nie można połączyć w Gerrit

Ilekroć wysyłam recenzję do Gerrit i jeśli recenzja jest w oczekiwaniu na jakiś czas, otrzymuję cannot merge wiadomość w Gerr ... rudno jest mojemu recenzentowi sprawdzić to ponownie. Szukam sposobu na usunięcie cannot merge z bieżącej recenzji. Dzięki!

Edycja projektu.config w projekcie Gerrit

Chcę dodać etykietę "verified" do mojego projektu Gerrit, aby Jenkins mógł zweryfikować, czy kod buduje się i przechodzi test ... alue = -1 Fails value = 0 No score value = +1 Verified Jak jednak dostać się do tego pliku, aby go edytować?

Jak uniknąć polityki w code reviews? [wstrzymane]

Więc staram się wprowadzić przegląd kodu do moich zespołów. Ale znaczna liczba rówieśników obawia się, że aspekt polityczny m ... lądania kodu? Jeśli tak, to jak tego uniknąć? A dokładniej, jak zrobić porządny przegląd kodu, żeby nie było w nim polityki?

Generowanie odrębnych liczb losowych w C#

Zastanawiam się, czy mógłbyś potwierdzić poprawność poniższego kodu i doradzić, czy istnieją dla niego lepsze alternatywy? P ... ult.Exists(value => value == curValue)) { curValue = rand.Next(1, 100000); } result.Add(curValue); }

Jak zabezpieczyć bagażnik przed nieszczęśliwymi nowicjuszami?

Współpracownik przedstawił następujący problem, powiedzmy jest fikcyjny, aby chronić winnych: Zespół 5-10 pracuje nad pro ... oma poprawkami, nie sądzę, aby coś tak sztywnego lub drakońskiego, jak to, co zaproponowałem powyżej, było konieczne. Dzięki!

Czy generator haseł Bookmarklet z SuperGenPass.com bezpieczny w użyciu?

Przejrzałem 981 linijek kodu dla bieżącej wersji http://supergenpass.com i nic nie znalazłem, jednak mogłem coś przeoczyć. ... pt: http://supergenpass.com.nyud.net:8090/SGPv4.js Czy generator haseł Bookmarklet z SuperGenPass.com bezpieczny w użyciu?

Lepszy sposób wyzwalania OnPropertyChanged

Mamy projekt WPF, który podąża za wzorcem MVVM. W modelu widoku jest dużo kodu, który wygląda tak: private string m_Fi ... sposób, który wymagałby mniej kodu? Byłoby miło z czymś takim: [NotifyWhenChanged] public string Fieldname { get; set ; }

Abstrakcja kontenera za Singletonem - czy to źle?

Ogólnie rzecz biorąc, lubię trzymać aplikację całkowicie nieświadomą kontenera IoC. Jednak napotkałem problemy, w których mu ... wstrzyknąć wszystko, co chcę: Zamek, Stub, podróbki, itp. Czy to śliska droga? Czy po drodze napotkam potencjalne problemy?