@qwelukasz: kilka lat temu to by było niezłe, teraz chyba mało kto korzysta z noip.

#codereview

@qwelukasz:

1. Composer i PSR-4 do autoloada klas i zaciągania zależności
2.Pokaż całość

@krasnoludkolo:
Tak z grubsza na szybko:
Nie nazywaj klasy aktywności MainApp, bo to sugeruje rozszerzanie klasy Application, nazywaj je po prostu z sufixem Activity. Poza tym poczytaj o appcompacie i rozszerzaj AppCompatActivity. Jako kontekst w wielu miejscach możesz po prostu podawać kontekst aktywności, nie ma potrzeby przekazywania kontekstu aplikacji. Unikaj jednoliterowych nazw zmiennych typu "i". Postaraj się też bardziej precyzować nazwy metod, configure() nic nie mówi o tym co konfigurujesz.Pokaż całość

@szyna55 Zawsze kiedy moj 1,5 roczny syn ucina sobie popołudniową drzemkę na moim łóżku to mnie zachciewa się srać. I nie mogę iść, bo oczywiście jak tylko wyjdę to mały skula się, p--------e o podłogę i będzie ałka. Więc zaciskam poślady, stękam, chrzakam, ale nie idę. Z Twoją aplikacja koniec z tym! Będę mógł iść na tron, patrzeć czy dzieciak leży spokojnie a w razie czego lecieć na ratunek! Dzięki!

@ppawel: Zrobiłem sobie boilerplate do projektów na webpacku + es6, możesz sobie zerknąć tutaj, może coś wyniesiesz: https://github.com/rbielawski/the-template

@ppawel: Polecam swój config, sporo projektów na tym powstało.
https://github.com/ald890/webpack-starter-light

Pokaż spoiler

@MrGreeneye: Łał. Ta konfiguracja gulpa jest większa niż niejedna prosta apka

λ szuja angular-restapi-app → λ git master → find gulp | xargs wc -l > 460
λ szuja angular-restapi-app → λ git master → find src | xargs wc -l > 374
:D

@matcheek: zmień jeszcze pasek na różowy, bo trochę niepoważnie to brzmi.

Review jest od pilnowania jakości kodu, w tym formatu. Jeśli uzgodniony format używa spacji, trzeba się tego trzymać. Jakość jest bardzo łatwo zepsuć - a pierwszy wyłom szybko pociąga za sobą następne...

Pokaż spoiler

@matcheek: format kodu ma ogromne znaczenie podczas pracy z gitem czy jakimkolwiek innym vcs. Wyobraź sobie ,że robisz reformat codu przy okazji jakiejś zmiany i nagle w commicie ląduje informacja o dziesiątkach/setkach linii które nie dotyczą faktycznej zmianie w kodzie. Takie commity są bezwartościowe, bo utrudniają code review i ustalenie co faktycznie się zmieniło w kodzie.

Pakuj wszystko w jeden plik *.js ewentualnie kilka i doklej do html.

@elmccd: to powinien robić skrypt za nas, na etapie dev powinniśmy mieć raczej wszystko porozbijane względnie

@PsichiX: https://pl.wikipedia.org/wiki/Sp%C3%B3jno%C5%9B%C4%87_oprogramowania

@PsichiX: Hejo, też chwilę przeglądałem kod i kontynuując wątek dodam parę rzeczy od siebie. No offense.

1. Nazewnictwo zmiennych - trzeba się na co zdecydować. Najlepiej jeśli nie używasz swojego stylu tylko potrafisz dostosować się do jakiejś normy. (ms_regexEscapes - to nie c++ :))

2. Brak jednolitej architektury. Widać, że pewne klasy powstawały spontanicznie i bez większego celu (TemplateFilesManager - zaglądam do środka i poza paroma metodami statycznymi nie widzę żadnegoPokaż całość

if (field3.GetState() != State.state.Occupied) ;

    else pawn_alone = false;

@Golomp: lol co

@jaggi:

for (int i = 0; i < chars.Length; i++)
{
if (i % 1 != 0)
chars[i]Pokaż całość

@Hertesus: a nie masz debuggra? Weź krokowo przesledz co się dzieje. Co jest wpisywane do tablicy i co jest odczytywane.

@Hertesus:
Polecam lekturę:
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html

@krasnoludkolo: Ogólnie skup się na trzech zadaniach:

1. Podmioty, obiekty którymi zarządzasz (Task),
2. Zestawy, listy, kolekcje tych podmiotów (TaskCollection, ArrayCollection, Array, TaskArray),
3. Dostarczyciela/zarządcę powyższych:Pokaż całość

@Hertesus: nie korzystaj z getchar - gdy potwierdzasz znak enterem wprowadzasz do bufora 2 znaki i oba znaki są wykonywane w pętli.

@Hertesus: wypisuj sobie kod ASCII wprowadzanego klawisza - to dostaniesz odpowiedź - to co pisałem wyżej :)

@david_luiz: tab nie jest zainicjalizowane

@david_luiz: poszukałbym na stackoverflow, a następnym razem wrzucaj też stacktrace

@Sarpens: ten sam żart po raz drugi już nie śmieszy. Teraz po prostu próbujesz wyżebrać plusy na sprawdzonym pomyśle :)

@Sarpens:

@x1mrdonut1x: Czy udostępniasz swój kod na licencji MIT i będę go mógł wykorzystać w swoich projektach za darmo?

BTW - nazwy zmiennych pisz po angielsku.

@x1mrdonut1x: undefined variable `a'

@Hertesus: pro tip nigdy nie rób zmiennych globalnych jeśli to nie jest CHOLERNIE potrzebne

@Hertesus: Po co skracać? Lepiej poprawić styl, pozbyć się zmiennej globalnej, sprawdzić wartość zwracaną przez scanf.