GitHub pod kątem zatrudnienia

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

0

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

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ż

2
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

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.

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!

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".

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ł?

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.

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