@GlenPL: Też zasugerowałem jak można rozwiązać tłumaczenia

@GlenPL: Tak, ale:

Jeśli chodzi o prędkość - a jak wiadomo programiści c++ mają na nim bzika - idealne rozwiązanie (w twoim przypadku) to utworzenie dwuwymiarowej tablicy

enumPokaż całość

@GlenPL: Wrzucaj na GitHuba – będziesz mógł wprowadzać poprawki, inni będą mogli komentować każdą linię zmian, a przede wszytskim GIT to podstawowe narzędzie pracy, więc warto się za niego wziąć.

@GlenPL: Co do samego kodu:

1. Niby cpp, a kod proceduralny… (jak zaczynasz to OK, ogólnie jak będziesz się uczyć OOP to zrób dokładnie tę samą appkę, ale na klasach – będziesz miał porównanie)
2. Zbyt długie funkcje – jak zaczyna Ci przekraczać 20 linii to bezwzględnie szukaj jak to rozbić na mniejsze funkcje.
3. Max 2 poziomy „wcięcia” w prawo – znów, jak zaczynasz zagnieżdżać IFy czy pętle: czasPokaż całość

@Ginden:
1. #pragma once jest zepsute, uzyj starego #ifdef #endif
2. uzyj std::function zamiast pointerow na funkcje
3. Zreszta jak uzywasz 3 roznych sposobow liczenia pola, to mozesz zrobic klase bazowa i 3 klasy dziedziczace z odpowiednia implementacja tych sposobow
4. don't use fucking pointers main.cpp#L47 memleak
5. main.cpp#L82 rzuca wyjatek, obsluż go, albo uzyj [i]

@Ginden: nie musisz dokonywać dereferencji wskaźników na funkcje. (btw: relevant). Nie uƶywaj std::function nadaremno, szczególnie w ciasnych pętlach takich jak Twoje. Type erasure jest praktycznie nieoptymalizowalne bez jita. Juƶ prędzej zrób klasę szablonem po typie funktora.

Zamiast bawić się w reverse, ustaw odpowiednio min i max w konstruktorze.

W konstruktorze uƶywaj listy inicjalizacyjnej zamiast wrzucać wszystko do ciała funkcji.

Jak na 1. program w C++: zajebiście.

@Jonarz:
Zastrzeżenie - nie znam Javy na poziomie wyższym niż składnia.

LastfmFavToSpotify/src/lastfm/MultiplePagesHandler.java
Co to za nazwaPokaż całość

@Jonarz: Synchroniczne IO - nie możesz pobierać dwóch plików za pomocą jednego wątku, musisz poczekać aż jeden skończy się pobierać.
Asynchroniczne IO - plik pobiera się w tle (można to traktować jako stworzenie nowego wątku, który po zakończeniu pobierania znika) i wykonuje callback (w Javie to rozwiązuje się chyba przez klasę anonimową).

#codereview bardzo mile widziane.

@noisy:

Parę uwag do Dockerfile'a, tak naPokaż całość

@haussbrandt: do tego do powtórzenia podstawy programowania - jedna funkcja powinna wykonywać jedno zadanie. Wrzuciłeś wszystko do jednej, kod nieczytelny, nie do rozbudowania, nietrzymanie się standardu PEP-8.

Bardziej rozsądne podejście niż klepanie małpiegoPokaż całość

@pierre-van-hooijdonk: Python ogarnia ujemne indeksowanie:

http://pl.wikibooks.org/wiki/Zanurkuj_w_Pythonie/Listy

[-1] zwraca ostatni element, w tym wypadku "Lose".

@matiit: Serio klasy tutaj ( ͡° ͜ʖ ͡°)? Papier, kamień, nożyce - Enterprise Edition. Dorobiłbyś kilka dobrych komentarzy i by było Startup Edition.

@noisy: nic nie rozumiem, ale pozdrawiam ( ͡° ͜ʖ ͡°)

@noisy: chodziło, mi o to, że nie analizowałem tego w jakiej kolejności Ty to chciałeś mieć posortowane. wiedziałem, że coś takiego jest możliwe, ale dopiero po rozpisaniu w shellu zorientowałem się, że ma być na odwrót. nieważne, zanim zaczniesz pisać kolejne nowe filtry to sprawdź dokumentację :P

@matiit: no mięty podoba mi się, ale ja się na tym nie znam.

@matiit:

1. Po co zwracasz treść błędu jeśli obsługujesz go przezPokaż całość

@highlander: ani jednego cebuliona nie dajesz i jeszcze masz wymagania XD

Pokaż spoiler

@highlander: Ja też byłem leniwy, też nie chciało mi się tych napisów ustawiać...

Pokaż spoiler

@Ginden: https://github.com/Ginden/wykop-abw-script/commit/ce70cb98af4547592ea3bfcf5c9ae2aef1deaba9

Jak raz wspomniałem o czymś, to się nie powtarzałem (czyli np jeśli gdzieś wspomniałem, żeby nie używać polskich nazw zmiennych, to w innym miejscach jest to równie równie prawdziwe).

No i nie wnikałem w „głębszą” logikę kodu – ogólnie czepiam się niekeszowanie przeszukiwań drzewa DOM, standardy kodowania i dobre praktyki. IMHO jak coś ma więcej niż 3–4 zagłębienia to robi się strasznie nieczytelne.

@dzien_dobry: Mac mini w pracy :(

@noisy: eh programisty, wam to zawsze trzeba wszystko tlumaczyc :)

@npsr: Ja osobiście olał bym jednak event dla capslock i poddał analizie wpisywany tekst tak jak wspomniałes pod względem statystyki wystepowania wielkich liter

@Hauleth: Widzę, że nikt się nie odezwał, więc jeśli chcesz i będe mieć chwilkę wieczorem to chętnie Ci to review zrobię.

Rozszerzanie natywnych obiektów - spoko. Rozszerzanie obiektów hosta - zuo. Wytłumaczenie.

@Hauleth: Poczytam, thx.

@japer:

GroovyShell shell = new GroovyShell();Pokaż całość

tyktak

@japer: Powinien być dodatkowy interfejs Karta, który miał by 2 implementacje KartaDebetowa i KartaKredytowa i to one by służyły za nośniki sesji.