Comments on devBlogi: Najlepszy kod to brak kodu w ogóle

Dokładnie. Aż dziw, że dopiero tak późno ktoś wspo...

2010-07-05T00:50:23.438+02:00

Dokładnie. Aż dziw, że dopiero tak późno ktoś wspomniał o:

if(String.IsNullOrEmpty(s)) ,ewentualnie
if(!String.IsNullOrEmpty(s))

Dla mnie, podczas pracy z .NET jest to podstawa podstaw, bez której nie da się miarodajnie żyć i pracować.

Wiele razy zdarzało mi się sprawdzać podobny input, a że nie zawsze wiemy skąd pochodzi i jaką drogę przebywa (np. fragment

Podany przykład if (s == "") jest fataln...

2010-05-10T11:30:05.389+02:00

Podany przykład if (s == "") jest fatalny. Aby wykonać takie porównanie kompilator musi zrobić następujące rzeczy:
1. Utworzyć nowy obiekt typu String o pustej zawartości
2. Prawdopodobnie w pętli porównywać znak po znaku - w szczególności jeśli s jest puste liczba iteracji będzie równa 1.
Dodatkowo jeśli s = NULL porównanie nie zwróci oczekiwanej wartości (string

Co do wspaniałych wyrażeń, to idealny przykład z C...

2009-12-10T15:15:45.796+01:00

Co do wspaniałych wyrażeń, to idealny przykład z C#:
obrazek.Clone(new Rectangle(x, y, w, h), System.Drawing.Imaging.PixelFormat.DontCare);

tak, to dobry sposób na powiedzenie, że nie zależy mi na formacie :)

@koziolek Zgadzam się co do stosowania bibliotek. ...

2009-12-02T21:29:18.306+01:00

@koziolek
Zgadzam się co do stosowania bibliotek.
Kwoli ścisłości if(s == string.Empty) też może wykorzystywać jakąś zewnętrzną bibliotekę i jej metodę. Konkretnie: == może być przeciążonym operatorem ;)

PS. Pamiętajcie, że ten przykład z if(s == "") to tylko przykład, a przesłanie jest szersze ;) Na deser: tutaj są rozważania nad wyższością jednej opcji nad drugą:

2009-12-02T21:25:39.863+01:00

Ten komentarz został usunięty przez autora.

@Immortal, @rafek, daliście mi do myślenia. Właśni...

2009-12-02T21:25:31.917+01:00

@Immortal, @rafek, daliście mi do myślenia. Właśnie czy minimalizacja i ograniczanie zawsze są dobre.
Kod silnie skompresowany "w edytorze" zazwyczaj jest dobry. Jednak są miejsca, w których taka kompresja nie jest dobra z punktu widzenia utrzymania.
Najlepszymi przykładami są globalne zmienne domyślne w perlu i rubym (w obu jest to bodajże $_). Mechanizm jest super, bo

@koziolek: Jasne, wiem, co chciałeś przekazać i oc...

2009-12-02T21:16:42.764+01:00

@koziolek: Jasne, wiem, co chciałeś przekazać i oczywiście zgadzam się z tym. Ale akurat tak mi się skojarzyło, że czasami jakieś drobne błędy w bibliotekach czy frameworkach potrafią irytować i ograniczać sposób, w jaki piszemy kod :]

Jako dodatek do tego wpisu, proponuję jeszcze lekt...

2009-12-02T21:10:23.659+01:00

Jako dodatek do tego wpisu, proponuję jeszcze lekturę: Spartańskie programowanie

@Immortal, by nie było niedopowiedzeń. Ogranicza i...

2009-12-02T21:03:32.194+01:00

@Immortal, by nie było niedopowiedzeń. Ogranicza ilościowo nasz kod. Zazwyczaj wzbogaca go też jakościowo zdejmując z nas odpowiedzialność za testowanie czarnej skrzynki. Jeżeli coś pójdzie nie tak w kodzie "obcym" to zawsze można go naprawić jeżeli jest na GPLopodobnej licencji lub zgłosić błąd do producenta.

@koziolek Dobrze napisane: "Każda biblioteka...

2009-12-02T20:39:58.580+01:00

@koziolek

Dobrze napisane: "Każda biblioteka zewnętrzna ... ograniczna nasz kod..." ;)

Autor słusznie zauważa, że prostota jest najlepszy...

2009-12-02T20:38:04.392+01:00

Autor słusznie zauważa, że prostota jest najlepszym rozwiązaniem. Jednocześnie prostotę można osiągnąć za pomocą odpowiednio przemyślanego kodu:
if(s == "")
if(s == string.Empty)
to kod zwięzły, ale nieprzemyślany. Odpowiednio przemyślany kod wyglądał by tak:

if(String.isEmpty(s))

W ten sposób przenosimy meritum do zewnętrznej biblioteki, która może

Dla mnie niekoniecznie "" jest bardziej ...

2009-12-02T09:52:35.178+01:00

Dla mnie niekoniecznie "" jest bardziej przejrzyste niż string.Empty. Widząc string.Empty wiem, że jest to string.Empty, widząc "" muszę się przyjrzeć przez kilka milisekund, czy jest to "", czy " ", więc czas percepcji jest dłuższy, choć szybciej czyta się "" :)

@batman Podany przez ciebie przykład ciężko nazwać...

2009-12-02T09:45:16.181+01:00

@batman
Podany przez ciebie przykład ciężko nazwać nadmiarowością. To raczej samodokumentujący się kod. Pozbywamy się nadmiarowości w postaci zewnętrznej dokumentacji, której nikt nie czyta i która trzeba zarządzać, więc PLUS.

Czasami nadmiarowość jest dobra: if(jakas_zmienna ...

2009-12-02T09:03:35.349+01:00

Czasami nadmiarowość jest dobra:
if(jakas_zmienna == Klasa.JakasStala) vs if(jakas_zmienna == 123)
Skąd nowa osoba ma widzieć co oznacza 123? Oczywiście można przeorać dokumentację projektu, ale jak wiadomo czas to pieniądz i szukanie w dokumentacji opisu jakiejś liczby zajmie więcej czasu, niż napisanie kilku liter i kropki.

"najlepszy kod to brak kodu w ogóle" -

Kto z was miał smiałosć spojrzec w lustro/ Co do a...

2009-12-02T08:05:11.620+01:00

Kto z was miał smiałosć spojrzec w lustro/ Co do artykulu to idac za glosem autora proponuje uciac sobie palce ku chwale niewidzialnego kodu.
9tekst napisalem nosem0

Chyba chodziło o to że if (s == "") jest...

2009-12-01T20:54:31.194+01:00

Chyba chodziło o to że if (s == "") jest lepsze, chciaż fakt, przydało by się to nazwać 'login' ;-)

@copernic777: Gdyby zmienna tekstowa nazywała się ...

2009-12-01T20:39:52.444+01:00

@copernic777: Gdyby zmienna tekstowa nazywała się "login" zamiast "s" to nie trzeba by bylo wprowadzac dodatkowej zmiennej typu bool, której to nazwa dopiero sugeruje co robi kod...

Moje rozwiązanie:
if login.empty?

A co jeśli wprowadzenie nowego kodu poprawia jego ...

2009-12-01T20:19:57.205+01:00

A co jeśli wprowadzenie nowego kodu poprawia jego czytelność:

if (s == "")

vs

boolean nieWpisanoLoginu = s == "";
if (nieWpisanoLoginu)

;]

"Nowe funkcjonalności" :(

2009-12-01T20:16:05.906+01:00

"Nowe funkcjonalności" :(