Po co używać `return x == 5? true: false; `?

Nie widzę korzyści z tej praktyki kodowania. Projekt, nad którym teraz pracuję, jest pełen tych stwierdzeń, więc nie jest to jeden błąd.Po co używać `return x == 5? true: false; `?

Inny przykład:

return getNumberOfBooks() > 5 ? true : false;

I jeszcze:

return isRed() ? true : false;

Źródło

2012-06-08 Christian Nilsson

+10

:-D To bardzo muc h przypomina to, co zrobili moi koledzy z klasy programistycznej w liceum. Jak na przykład 'for (i = 0; i <3; i ++) {if (i == 1) puts (" 1 "); else if (i == 2) puts ("2"); else stawia ("3"); } '. Ergo, wydaje mi się, że to nieporozumienie, że wartość boolowska, która wychodzi z porównania, jest tak dobra, jak wartość określona przez stałe. – Imp

Ktoś miał fascynację potrójnymi wyrażeniami. –

@Imp: Jak na ironię, wynik będzie wynosił "3", "1", "2". – Makoto

Nie ma absolutnie żadnego powodu, aby to zrobić.

Jest zbędny i utrudnia odczyt kodu.

Poniżej są znacznie łatwiejsze do odczytania:

return (getNumberOfBooks() > 5); 

return isRed();

Źródło

2012-06-08 18:55:35 jahroy

Nawet nawiasy klamrowe '(...)' w pierwszym powrocie nie są potrzebne. – Nawaz

Nie uważam, że * więcej * czytelne. Jestem pewien, że wielu też by tego nie zrobiło. – Nawaz

Oczywiście nie jest _ededed_, ale uważam, że sprawia, że kod jest bardziej czytelny. – jahroy

operator Ternary czyni kod prawie nieczytelny, myślał Jego elegancki sposób kodowania, ale nie powinno się zachęcać chyba naprawdę potrzebne

Źródło

2012-06-08 18:59:06

Uzgodniono ...Nadużycie operatora trójskładnikowego doprowadza mnie do szału! – jahroy

yup ..... masz rację ... Każdy głupek może napisać kod, który komputer może przeczytać, ale tylko programiści piszą kody, które człowiek może przeczytać –

... A z jakiegoś powodu tony programistów są dumne pisanie wymyślnego, trudnego do odczytania kodu. Czytelność jest najważniejszą rzeczą przy pisaniu kodu (gdy tylko zacznie działać poprawnie). – jahroy

Wydaje że pracujesz nad projektem z programistami początkującymi. Zacznij refaktoryzacji, gdzie zawsze można zobaczyć:

return getNumberOfBooks() > 5 ? true : false; 
return isRed() ? true : false;

ją zmienić:

return getNumberOfBooks() > 5; 
return isRed();

Źródło

2012-06-08 19:03:20

Jeszcze lepiej: 'return (getNumberOfBooks()> 5);' To jest jeszcze bardziej czytelne (według mnie i zgodnie z konwencjami kodowania). – jahroy

@jahroy masz rację, ale to tylko szczegół: P –

Oczywiście jest to zbędne redundancji.

Proponuję robić to w Eclipse:

znaleźć wszystko, co pasuje do wyrażenia regularnego "return (coś), prawda? False" i zastąpić go tylko something
i wszystko, co pasuje do wyrażenia regularnego „return (coś) fałsz: true”z (coś)

można delegować pisania tego wyrażenia regularnego do tych, który napisał ten projekt: D

Źródło

2012-06-08 19:23:59 Xeon

Po co używać `return x == 5? true: false; `?

Odpowiedz

Powiązane problemy