I have noticed that often sport clubs are tagged as an entire building. In some cases the building should most likely be leisure=sports_hall or leisure=sports_centre instead since it is not actually used by a single club but simply for sport. However, I also see many buildings tagged as club=sport that are in fact used only by one specific sports club. Is that the normal way to tag such clubhouses? I would probably just place a node on the building.
I would always separate non-building tags like club=sport from the building itself, because usually these clubs have names, and the building doesn’t, and setting building=* + club=sport + name=XXX, the name could refer to the building, or the sports club. I consider adding these tags to the building itself a quick hack – nothing more.
I’d map the club=sport separated from the building, either as a node, or as an area that spans the whole area, and also add tags like leisure=sports_centre to that POI, but not to the building.
It is totally ok. And yes, in my mind it would be better to separate the building from the amenity and thus moving the tags associated with the community_centre either to a node or to an area covering all of the club home, including the bbq out back