DEV Community

Discussion on: No love for boolean parameters

View post

Replies for: I find that quite often flags are added to toggle code at the end or at the beginning of a subroutine. For a simple example: function equal_stri...

Collapse
 
patricktingen profile image
Patrick Tingen • • Edited

But now you have the logic in two places, shouldn't you call one function from the other to avoid that?


function equal_strings(a, b)
   return a == b
end
function equal_strings_ignore(a, b)
   a, b, = a:lower(), b:lower()
   return equal_strings(a, b)
end
Enter fullscreen mode Exit fullscreen mode
Collapse
 
darkwiiplayer profile image
𒎏Wii 🏳️‍⚧️ •

Indeed, that was what I intended to do, but apparently I forgot to actually change that line after copy-pasting it. You're totally right in that this only makes sense when you actually use the subroutine with the extracted logic :D