C ++这是正统的使用这样的开关案例吗？

Question

假设我有一个变量int x，决定switch/case，我想要两个可能的操作：一个用于x=1, x=2, x=5，另一个用于x=3, x=4。以下代码在codiquette方面是否可以接受？

switch (x)
{
case 1:
case 2:
default:
    // do something
case 3:
case 4:
    // do something else
}

我意识到它也适用于x>5，但在我的情况下这不是问题，它似乎有效，gcc 6.1.1，archlinux 64.原因是do something涉及调用a不那么小的功能，我想避免写两次（代码膨胀？），即使它只是复制粘贴的问题。