Is this pcb layout good practice?
Starting with what's probably here:

It's possible that the trace between holes is a guard ring for the entire board, meant to be a barrier between the stuff on the board and the stuff around the board.  But that all depends on what the screws are connected to.  Get it wrong, and it becomes an antenna, like The Photon said.  I've never seen it like your example, but it's done quite commonly between sensitive analog stuff and the rest of a board.

More sensible to me would be a ground pour connected to the screws, but you said it's a trace, so we can drop that possibility.

The ferrite to ground is likely a ground loop breaker or something like that, where you want your circuit to be ground-referenced, but a direct connection picks up noise. * It's effectively a "soft" lowpass filter: close to zero ohms at DC (wire resistance) up to a wide peak of few hundred ohms at a few MHz.

As the bead's impedance goes up with frequency, we have another problem: the stray capacitance between the ring and the circuit reduces impedance with frequency.  Depending on how close things are to the ring (capacitance follows area/distance^2), this can take over from the bead so you end up with noise anyway.

* I have a laptop that made what sounded like hard drive noises on both the headphone output and a USB sound card, but only when plugged into power.  It turned out to be a ground loop problem picking up noise from its own power supply.  A USB isolator fixed it.  So ground loop symptoms aren't limited to AC power hum.



As for good practice:

Possibly yes to use the ferrite, but probably not for a whole-board guard ring.  It might be redeemed by connecting the ring to ground AFTER the bead, but only if you have traces for ground instead of a pour.