[d.....s]

#programowanie #java #studbaza Czy ktoś może mi te metodę jakoś spakować, na mniej linijek kodu? Metoda do podawania ilości występowania liter, cyfr, znaków ze Stringa.

http://pastebin.com/0z57BELD

[k.....r]

@devaitis: tak, to bardzo proste - wystarczy usunąć znaki nowej linii

[komeniusz]

@devaitis: http://pastebin.com/Y5Nuw6mf

[echelon_]

@devaitis: A w ogóle po co?

[1.....p]

@devaitis: np.

countDigits += Character.isDigit(text.charAt(i)) ? 1 : 0 

- rozliczaja was z bajtow kodu wtf;D?

[dagon_666]

@1080p: to jest pewnie wg nich "optymalizacja"

[Rincewind]

@devaitis: http://pastebin.com/fhyZBCU3

[Rincewind]

@devaitis: Swoją drogą, w programowaniu, szczególnie w Javie, nie chodzi o mało linii kodu, tylko o przejrzystość.

[A.....k]

@devaitis: Metoda nic nie zwraca.

[d.....s]

@echelon_: @1080p: @dagon_666: A tak mi sie wydawało, że coś pokićkałem, że taki długi i brzydki kod mi wyszedł dla tak trywialnego zadania.

[404notfound]

taka funkcja swoją drogą powinna zwracać coś innego niż void

[d.....s]

@Analityk: Zgadza się.

[d.....s]

@404notfound: Poniżej drukuję te "county" i docieram do nich statycznie w metodzie Main(), nie tworząc obiektu. Robię to, żeby było mniej linijek kodu.

[1.....p]

@devaitis: co ty sie uczepiles tak tych linijek kodu ;p problem jest jak funkcja nie miesci sie na ekranie (uncle bob mowi w wiekszosci przypadkow ~20 loc na metode,) ;p funkcja powinna robic jedna rzecz - jak cos liczy to juz tego nie wypisuje. Nazwa metody: howMany - how many co?;p Zrob sobie jakas klase TextStatistics, metoda: TextStatistics calculateTextStatistics(string text) i na podstawie zwroconego obiektu wypisz wartosci

[d.....s]

@1080p: > funkcja powinna robic jedna rzecz - jak cos liczy to juz tego nie wypisuje.

Dlaczego?

[1.....p]

@devaitis: wyobraz sobie, ze bedziesz chcial skorzystac z takiej funkcjonalnosci gdzies w programie - np. zapisujac statystyki tekstu do pliku - przy kazdym razie bedzie wypisac glupi tekst, ktorego np. nie chcesz. Jezeli funkcja robi kilka rzeczy to przestaje byc "re-używalna" co moze (a raczej na pewno) bedzie powodowac klopoty w przyszlosci. Przeczytaj clean code ;)