大部分的Action方法都有明确的返回类型
这里的返回类型相当于是OK里面套了一个companyDtos的List
争对返回类型比较明确的，实际上IActionResult还有一个实现类ActionResult<T>的形式，这里T就是List<CompanyDto>
也可以改成IEnmerable，因为List也实现了IEnumerable。这样做的好处相当于把返回的类型给具体明确了。这样其他的代码就可以推断出你这个Action返回的类型，到底是个什么类型了。
优势可以体现在Swagger UI上，如果你返回的ActionResult里面有具体的T这样在文档上就会体现出来。知道你返回的具体有哪些属性。
使用ActionResult<T>的形式，还能让你语法更加的简化一些
最后可以允许你返回OK
还可以返回companyDtos的List
所以说能尽量使用ActionResult<T>的时候就尽量使用。
优化
循环的这块代码不是很好。这种写法挨个属性的赋值，如果属性比较少还可以。如果属性很多写起来就很麻烦了。而且还容易出错。
解决方法就是使用对象映射器。AutoMapper