Wprowadzenie do firmy nowych standardów CodeReview / UnitTesty i inne

Wprowadzenie do firmy nowych standardów CodeReview / UnitTesty i inne
OB
  • Rejestracja:prawie 12 lat
  • Ostatnio:26 dni
  • Postów:33
0

Cześć,
Temat może śmieszny, ale pracuje w firmie gdzie do tej pory nie było w sumie żadnych standardów, code review, unit test itp. jeśli chodzi o programowanie.
Jest auto build / deploy, SVN i tyle.
Programistów jest kilkunastu (część ze stażem 10+ lat, część ok 3-4 i paru <1 rok).
Problem w tym, że w sumie nikt nie pracował jako dev nigdzie indziej (są od początku, albo odeszli i już nie wrócili).

Kod każdy pisze sam sobie i do zgłoszonych błędów sam po sobie poprawia (no chyba, że jakiś urlop/l4 to ktos inny musi sie zapoznać).
Ogólnie jak na takie coś aplikacja działa na prawdę dobrze i stabilnie.

Za w sumie moimi namowami doszli w końcu do wniosku, żeby od nowego roku spróbować wprowadzić jakieś standardy i ewentualnie nowsze frameworki
W tym momencie .NET 4.8 + MSSQL + React (aplikacja desktop + web). Żadnego ORM

No i tu zaczyna się problem, może ktoś podpowiedzieć jak to zrealizować w średniej firmie, albo w którą stronę w ogóle szukać?
Nie mamy Jenkinsa/GitLabów/Azure i innych chmurowych rozwiązań, żadnych sformalizowanych scrumów/agileów/sprintów itp. (kilka razy w roku wychodzi wersja i tyle więc powiedzmy że sprint).

Błędy zgłaszane w naszym wewn systemie. Przy commicie wpisuje sie nr zgloszenia i tyle.
Na ten moment znalazłem jedynie takie coś https://www.reviewboard.org/

edytowany 1x, ostatnio: obieq
.andy
  • Rejestracja:ponad 16 lat
  • Ostatnio:około 3 lata
  • Postów:1524
1

Przecież możecie sobie podłączyć GitLaba wewnątrz firmy. On ma wbudowaną funkcjonalność do CR.

Jeżeli chodzi o testy. Jeżeli coś nowego nie ma napisanych testów, to choćby skały srały to nie przechodzi CR. Podobnie kwestia przy zmianach w istniejącym kodzie - konieczność pisania testów dla zmienianych funkcji.

Dodatkowo raz na kwartał np. wydzielenie czasu na otestowanie istniejącego kodu bez testów.


Software is like sex: it's better when it's free.
- Linus Torvalds
UglyMan
  • Rejestracja:około 6 lat
  • Ostatnio:około 3 lata
  • Postów:2206
3

Mam nadzieje, tylko że te zmiany nie zabiją tej firmy. Jest już znane kilka firm w historii, które chciały coś poprawić i już ich nie ma (np netscape). Co do ORM to nie jestem przekonany, że zawsze jest potrzebny - zwłaszcza jak komunikacji z bazą jest nie dużo (zawłaszcza ze jest trochę złych opinii o EF z poprzednich wersji). Co do procesu produkcyjnego: jak obecny działa to, po co go zmieniać?
Testy do "legacy" kody nie są łatwe do zrobienia, a często nieopłacalne. O ile jestem się zgodzić z testami dla nowych funkcjonalności (modułów) to w starym kodzie może się to nie udać - zwłąszcza jak narzędzie jest robione od kilkunastu lat.

Zobacz pozostały 1 komentarz
UglyMan
Jasne, że zmiany (poprawa aktualnego stanu) są dobre, ale czasem mogą spowodować odwrotny skutek - np odejdą ludzie, którym te zmiany się będą się podobać. Dlatego z takimi rzeczami trzeba delikatnie powoli małymi krokami. I raczej nie zaczynaj od narzędzi a od przekonani ludzi, że takie narzędzia są w stanie coś zmienić. W firmie gdzie jestem wdrożenie GIT (zamiast ftp) jako repozytorium kodu zajęło mi 2 lata. A wdrożenie Azure DevOps do zarządzania taskami kolejne 3.
jarekr000000
Jasne, że zmiany (poprawa aktualnego stanu) są dobre, ale czasem mogą spowodować odwrotny skutek - np odejdą ludzie, którym te zmiany się będą się podobać. jak odejdą ludzie, którym nie podoba się robienie testów i jakieś standardy w kodzie to bardzo dobry skutek. To, że niektóre "dobre zmiany" są złe to inna inszość.
UglyMan
Jasne, ale jak masz 10 programistów co robią tak od 10 lat i nagle połowa z nich odejdzie to jesteś tam, gdzie słońce nie dochodzi - widzaiłem kiedyś agonie takiej firmy.
jarekr000000
@UglyMan: you never know. Może jakby zostali też by umarło. Kilka razy byłem w projekcie, gdzie przejęto duży projekt w standardzie kodowania "totalny bajzel", bez przekazania wiedzy i w zasadzie udało się uratować/ustabilizować. Z tym, że firma miała kasę na refaktoring.
UglyMan
Może, ale można było ich eksterminować w bardziej przewidywalny sposób. Małe firmy balansują na krawędzi opłacalności i często żyją z utrzymania obecnych klientów, jak zabraknie tych obecnych klientów to często kończy się biznes.
Kliknij, aby dodać treść...

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)