Aktywne Wpisy
Viesti +634
Rozwala mnie ta ostatnia nagonka na RODy – nic a nic nie wygląda na zorganizowaną ustawkę. Na co dzień deweloper to synonim najgorszego zła a tu nagle „oddajmy najlepsze tereny w mieście deweloperom, tylko oni tworzą przestrzeń dodaną w mieście”.
Też zwróćcie uwagę na to, że nie pada w takich tezach słowo sprzedajmy tylko oddajmy, przekażmy itp. Biednych deweloperów z pewnością nie stać na ziemię a oni tam by w czynie
Też zwróćcie uwagę na to, że nie pada w takich tezach słowo sprzedajmy tylko oddajmy, przekażmy itp. Biednych deweloperów z pewnością nie stać na ziemię a oni tam by w czynie
![Viesti - Rozwala mnie ta ostatnia nagonka na RODy – nic a nic nie wygląda na zorganiz...](https://wykop.pl/cdn/c3201142/a62cea8fab14505185f4b18cfb59b6ab08943e325d93485f0698c5833e14e205,w150.jpg?author=Viesti&auth=1a18292e3c5d9e0dc9594b69652b23b0)
źródło: 123
PobierzZawiera treści 18+
Ta treść została oznaczona jako materiał kontrowersyjny lub dla dorosłych.
Mój przykładowy kod, za którego brak komentarzy mają do mnie pretensje: http://pastebin.com/JCQfZNRq
Co sądzicie o nim? Przy okazji jak zobaczycie tam jakąś głupotę to dzięki za info ;)
#programowanie #java
customersWithDetailsInformationList
Co do tego, co wkleiłeś: sent, nie sended ( ͡° ͜ʖ ͡°) Łapanie "Exception" to zwykle zła praktyka, ale nie wiem jak się u Was to rozwiązuje. Metoda "isIdReapeted" to coś a la kod z memów CS Graduate, IMO lepiej tak:
private boolean isIdReapeted(int idWithNoResponse) {
for (int idAlreadySended: displayIdsAlreadySended) {
if (idWithNoResponse == idAlreadySended) {
return false;
}
}
Co do nich samych - javadoc na publicznym API, i komentarze na fragmentach, które do zrozumienia wymgają poznania pewnych szczegółów. Ogólnie, komentarze mają opisywać DLACZEGO coś jest zrobione, a nie JAK jest zrobione.
@Gwozdziuuu: Mądrze powiedziane. Z wyjątkami na opis dlaczego kod jest w jakiś sposób napisany a nie jak.
@karpadoor: Funkcja, która ma 200 linii to patologia.
private boolean isIdReapeted(int idWithNoResponse)
jest do grubej refaktoryzacji - nazwa sugeruje coś innego niż się dzieje w środku, toSend wygląda na pojęcie w ogóle z innego kontekstu... gdyby tam dodać komentarz to obawiam się, że metoda zyskała by jeszcze jedno znaczenie, zupełnie niż wynikające z nazwy metody, jej ciała, bądź kontekstu jej rzeczywistego użycia. Ps. Tu trochę ortodoksyjnie. W erze podejścia funkcjonalnego chętnie wydzieliłbymdisplayIdsAlreadySended
@Gwozdziuuu: Testy są integralną, nierozerwalną częścią procesu deweloperskiego, zatem nie można uznać, że nie ma na nie czasu.
Nie wiem, jakiego podejścia do estymat pracy używacie, ale moim zdaniem jakie by ono nie było, powinno brać testy pod uwagę.
@npsr: do więzienia
jakieś powody, dla których nie możemy użyć if (!displayIdsAlreadySended.contains(idWithNoResponse));
?
Ale komentarze nie tylko są potrzebne, żeby przyśpieszyć komuś innemu pracę (bo nawet jeśli kod jest dobry i czytelny, nie zawsze znaczy, że powinienem go w ogóle czytać), to również porządkują całość wizualnie.
A już nie wspomnę o przypadkach,
@npsr: większość cywilizowanych języków taką informację posiada już w nagłówku funkcji, fakt nie pomyślałem o tych mniej cywilizowanych ( ͡° ͜ʖ ͡°)