Please tell me what do you think of this netwrok configuration?

ADSL –> Gbit router with 4 ports –> Gbit switch with 24 ports –> clients

Is it ok or am I way off?
If I am, what would you recommend instead?
I need a cost-effective network solution because we use a 10/100 router now(for 8 computers) and the network is already unstable as it is.
I need the router to share the ADSL on the network without the use of a computer that would normally do this.
(ADSL plugged directly in the router without the use of a PC)
But you are right, the ADSL is only 8 Mbit so a 10/100 will be enough.

So far: ADSL –> 10/100 router –> Gbit switch –> clients