It is best practice because instead of implementing your own counting function, you use the standard library one count_if. I do agreed it is hard to read if you have no idea what you are looking at; perhaps comments would've made this easier to understand

While it is clever, it is not the best practice if we're going to be adding more types of valid faces. Well, unless you plan on having a giant list of every combination of possible face!