undefined | Better HN

0 pointsjaredsohn4y ago0 comments

or a model method or private method in the class containing this code

0 comments

But why?!?! There's no way this conditional is a well-defined named concept and just adds indirection for the sake of line length.

>There's no way this conditional is a well-defined named concept

Sure about that? I'm saying if it is then extracting as a method might be clearer. All depends and these are other options.

maybe two well-named concepts then? If you're unable to reduce it at all, I'd consider that a code smell.

j / k navigate · click thread line to collapse

But why?!?! There's no way this conditional is a well-defined named concept and just adds indirection for the sake of line length.

>There's no way this conditional is a well-defined named concept

Sure about that? I'm saying if it is then extracting as a method might be clearer. All depends and these are other options.

maybe two well-named concepts then? If you're unable to reduce it at all, I'd consider that a code smell.

j / k navigate · click thread line to collapse