GitHub pod kątem zatrudnienia

Odpowiedz Nowy wątek
2018-12-03 17:03
0

Hej, proszę o roast mojego GH pod kątem ew. zatrudnienia i jego analizy przez ew. pracodawców.

Czyli co byście zmienili, dodali, odjęli, czego za dużo, za mało, za źle. Konstruktywna krytyka wskazana.

Zaczynam szukać pracy jako C# / ASP.NET / WPF Dev, bez komercyjnego doświadczenia.

https://github.com/przemyslawbak?tab=repositories

Pozostało 580 znaków

2018-12-03 17:09
0

Tutaj nie ma kodu źródłowego https://github.com/przemyslawbak/Dluzyna_Szkola


((0b10*0b11*(0b10**0b101-0b10)**0b10+0b110)**0b10+(100-1)**0b10+0x10-1).toString(0b10**0b101+0b100);
Zgadza się, póki projekt nie jest zamknięty, nie wrzucam, jeszcze czekam na komentarz dyrekcji szkoły, na razie umieściłem jedynie log prac nad aplikacją. - bakunet 2018-12-03 17:11

Pozostało 580 znaków

2018-12-03 17:11
1

Po co te komentarze do oczywistych rzeczy, dodatkowo po polsku?
https://github.com/przemyslaw[...]b/master/Models/AdresModel.cs

Dodatkowo śmiesznie wyglądają nazwy commitow, np:
Create ZalacznikViewModel.cs
Pisz po angielsku, w ostateczności całość po polsku

Co do reszty, nie piszę w C#, więc może ktoś inny się wypowie też

edytowany 3x, ostatnio: JrQ-, 2018-12-03 17:14
Słuszna uwaga, pousuwam oczywiste rzeczy. Choć na samym początku pomagały mi one ogarnąć każdą klasę, jak jeszcze próbowałem ogarnąć MVVM :D - bakunet 2018-12-03 17:30

Pozostało 580 znaków

2018-12-03 23:15
2
edytowany 2x, ostatnio: Ktos, 2018-12-03 23:29

Pozostało 580 znaków

2018-12-03 23:28
1
  if (!string.IsNullOrWhiteSpace(adresSerwera) && !string.IsNullOrWhiteSpace(port)
                && !string.IsNullOrWhiteSpace(imieNazwisko) && !string.IsNullOrWhiteSpace(adresEmail)
&& !string.IsNullOrWhiteSpace(haslo) && (adresEmail).Contains('@') && (adresEmail).Contains('.') && regex.IsMatch(port))

Raczej da się to uprościć :P

Pozostało 580 znaków

2018-12-04 09:36
0

Formatowanie
Mieszasz polski z angielskim
Często się powtarzasz.
Komentowanie co robi np pętla foreach jest bezcelowe.
Trochę mało jednak na tym gicie.

edytowany 1x, ostatnio: kzkzg, 2018-12-04 09:37

Pozostało 580 znaków

2018-12-04 09:47
0

Dzięki za poświęconą chwilę nad moim kodem oraz feedback. Na taką wiadomość po cichu liczyłem! Czasami coś mogłem przeoczyć, bądź zwyczajnie o czymś nie wiedzieć.

Ktos napisał(a):

Słuszna uwaga. W kontrolerach używających Dropdown List, metody zamieniłem na:

var techQuery = repositoryTech.TechnologiesByNames;
            if (techQuery == null)
            {
                Technology newTech = new Technology
                {
                    Name = "change my name",
                    PictureLink = "#"
                };
                repositoryTech.SaveTechnology(newTech);
            }

A w repozytorium wygląda to tak:

public IEnumerable<Technology> TechnologiesByNames
        {
            get
            {
                var technologiesByName = from t in _context.Technologies
                                         orderby t.Name
                                         select t;
                return (technologiesByName);
            }
        }

To bezmyślnie wkleiłem z podręcznika. Poprawione.

Done. Przyznaję, że o tym nie wiedziałem.

  • Tytuły commitów masz zawsze niezbyt wiele mówiące - "Create Something.cs". Ale dlaczego? Co to za zmiana? Co to zmienia w projekcie? Czy jeden commit to pojedyncza funkcja twojego systemu? Jeżeli dobrze rozumiem, to nie korzystasz z gita, ale wrzucasz/edytujesz pliki bezpośrednio na GH... a to nie świadczy o tym, że umiesz w ogóle gita używać - a szkoda, bo to bardzo fajne narzędzie. W pewnym sensie będzie mógł ci zastąpić twoje pliki WORK LOG

I tu, jak się okazuje, do końca (albo wcale) jednak nie wiem jak działają Git i GH. Dzisiaj posiedzę nad tym.

Prawda. Zastanawiam się gdzie ją wrzucić. W Infrastructure?

To akurat był mój roboczy komentarz, odznaczałem sobie metody które chciałem wrzucić w Repo, bazując na książce A. Freemana. Jak się później okazało później, niektóre nie powinny się tam w ogóle znaleźć. Ale to już inna historia.

Z początku wahałem się czy umieścić w GH kontroler logowania, ale z drugiej strony złe doświadczenie to dobre doświadczenie. Szczególnie, że może ktoś zwróci moją uwagę na ew. błędy i może to zaszkodzić jedynie mnie.

  • W ogóle masz gdzieś odpaloną tę swoją stronę-portfolio?

Tak. http://przemyslaw-bak.pl/

  • Chyba w tym projekcie Portfolio_Website nie ma wszystkich plików tego projektu - na przykład packages.json?

Dorzuciłem.

Dzięki raz jeszcze!

Pozostało 580 znaków

2018-12-04 10:10
0
kzkzg napisał(a):

Formatowanie

co konkretnie masz na myśli?

Mieszasz polski z angielskim

W którym projekcie? Czy chodzi może o commity w GH?

Często się powtarzasz.

W jakim przykładzie?

Komentowanie co robi np pętla foreach jest bezcelowe.

O który projekt chodzi?

Trochę mało jednak na tym gicie.

To trochę ciężko mi przeskoczyć, ale pracuję nad tym. Nie chciałem wrzucać na GH "hello worldów", tutoriali, czy testowych kodów. Doszedłem do wniosku, że to by było już "try too hard".

Pozostało 580 znaków

2018-12-04 11:09
0
WeiXiao napisał(a):
  if (!string.IsNullOrWhiteSpace(adresSerwera) && !string.IsNullOrWhiteSpace(port)
                && !string.IsNullOrWhiteSpace(imieNazwisko) && !string.IsNullOrWhiteSpace(adresEmail)
&& !string.IsNullOrWhiteSpace(haslo) && (adresEmail).Contains('@') && (adresEmail).Contains('.') && regex.IsMatch(port))

Raczej da się to uprościć :P

Słuszna uwaga, dzięki. Zrobiłbyś to tak?

if ((new[] { adresSerwera, port, imieNazwisko, adresEmail, haslo }).All(c => !string.IsNullOrWhiteSpace(c))
            && (new[] { "@", "." }).Contains(adresEmail) && regex.IsMatch(port))

Czy masz może lepszy pomysł?

edytowany 1x, ostatnio: bakunet, 2018-12-04 11:09
Ewentualnie może w ogóle wydzielić to gdzieś do jakiejś funkcji :P - WeiXiao 2018-12-04 18:49

Pozostało 580 znaków

2018-12-04 19:47
1

Z poprawką zaproponowaną przez @Ktos to nieźle popłynąłeś - https://github.com/przemyslaw[...]4ccd95782f5469cc002ce761d4a2e
To co zrobiłeś nie zmieniło absolutnie nic.

edytowany 1x, ostatnio: Saalin, 2018-12-04 19:49
Nie. Jeżeli robisz tak jak teraz, to pod Windows i tak dostajesz np. "C:\\Users\\Marcin\\wwwroot/src/img/ciesielstwo", czyli przemieszane separatory katalogów (na szczęście Windows jest bardzo liberalny pod tym względem i to powinno nadal działać). Problem polega na tym, że wejściem do Path.Combine() nie powinny być ścieżki, ale tylko jej elementy. Ale mogło być gorzej, bo masz tylko jeden parametr - a na przykład wynikiem Path.Combine("/var", "/test/test2") nie jest /var/test/test2, ale /test/test2 i aplikacja w ogóle mogła by nie działać :-) - Ktos 2018-12-04 20:53
Poprawne rozwiązanie to coś w stylu: Path.Combine(webRootPath, "src", "img", "ciesielstwo"). - Ktos 2018-12-04 20:54
Ok, teraz doczytałem, że argumentami funkcji są same stringi, albo tablica stringów. Bądź co bądź, aplikacja działa bez zarzutu, dla przykładu Path.Combine() zwraca mi przy debugowaniu C:\\Users\\asus\\Desktop\\Praca IT\\Programowanie\\Projekty\\Stolarstwo_Ciesielstwo 2.0\\BasicConfig\\wwwroot/src/img/ciesielstwo/ - bakunet 2018-12-05 00:13
No, działa bez zarzutu, bo Windowsowi tak naprawdę wszystko jedno, czy masz \ czy /. - Ktos 2018-12-05 12:39

Pozostało 580 znaków

2018-12-05 05:57
._.
2

http://przemyslaw-bak.pl/MyProjects/Details?projectID=5

Main features:
The application was originally built on ASP.NET MVC 5 and then converted to ASP.NET Core 2 MVC.
Architectural pattern of MVC was inspired by Adam Freeman's book along with its advantages and disadvantages.

Sory, miałem sobie dać spokój z flame, ale no ur... nie dałem rady :---|

Widać, po wzorcach, które stosujesz i jak je stosujesz, że jesteś początkującym, widać też, że się starasz. Tylko słabe książki czytasz :-|

Hehe, ale wiesz, osobiście nie żałuję lektury. Pomogła mi poznać Core 2. Teraz mam przewidziane do poczytania o popularnych wzorcach projektowych, żeby trochę wyprostować temat architektury. Ale najpierw chcę ogarnąć temat testów, bo jestem zielony, a chyba warto umieć pisać dobre testy. - bakunet 2018-12-05 15:22
Art of testing. Ci się spodoba. - ._. 2018-12-05 19:15
"The Art of Unit Testing, With Examples in C# 2nd" R.Osherove ? - bakunet 2018-12-06 07:12
yes, Jako uzupełnienie, Kent beck TDD. Może będziesz jednym z tych którzy rozumie ją co to jest mock i jakie jest prawdziwe - główne klu używania TDD. - ._. 2018-12-07 07:39

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0