Generally, they are good. Grains and cereals are not a mainstay of their diet the way they are for mice, and they can indeed keep the mouse population under control. If a person comes across a dead mouse with its skull cracked open and its brains are eaten, that was probably the work of a shrew. They also eat insect pests and are prey for larger animals.