So far as it goes, your calculation is correct, although nobody who knew what they were doing would set the calculation up like this.
Not mine. Credit goes to the link which is posted with it.
However, you can see immediately that the problem has been set up wrongly. A bright A-level pupil would realise that since stars are born in clusters, rather than as single stars, one should start with cloud masses of hundreds or thousands of solar masses, not the 1 solar mass used by Dolphin. A more sensible way of setting the problem up is to use the temperatures and the particle densities of observed interstellar clouds, and calculate their Jeans masses, that is, the masses of the objects that will contract under their own gravitation. For the observed parameters of cold dense clouds (T ~ 10-30 K; n ~ 100 to one million particles/cm³), the Jeans masses are those of stars or star clusters.
To revert to my example of the Coalsack, the Jeans length (for T = 30 K) is about 1 parsec (3.2 light-years). Since this is less than the radius of the Coalsack (about 9 parsecs or 30 light-years), the cloud will contract.
These results tend to confirm that star clusters and individual stars are born from the gravitational collapse of interstellar clouds.