ضمن تشکر از شما بابت آغاز این پروژه،
در نگاه اول کنترلر FileManager کمی شلوغ به نظر می‌رسد. این کارها را می‌شود جهت بالا بردن کیفیت کد انجام داد:
- Iconlist و GetIcon به یک کلاس جدا منتقل شوند. ترجیحا به صورت یک متد الحاقی. ضمنا نیازی نیست تا اصلا آیکون‌ها تعریف شوند. پسوند فایل را پیدا کنید. یک پوشه تصویری بر اساس ext.pngها درست کنید. Ext در اینجا انواع و اقسام پسوندها است. به این ترتیب اگر پسوندی موجود بود از این پوشه بخواند، نبود یک فایل پیش فرض را نمایش دهد.
- region helper به کلاس دیگری که شامل یک سری متد الحاقی برای کار با فایل‌ها است منتقل شود.
- هر کدام از regionهای دیگر هم قابل انتقال هستند به کنترلرهای کوچکتر و با تعداد سطر کمتر. به این ترتیب نگهداری ساده‌تری در طول زمان خواهند داشت.
- baseLocation = HttpRuntime.AppDomainAppPath بهتر است در وب کانفیگ قابل تنظیم باشد. از این جهت که عموما یک سری پوشه و زیر پوشه خاص است که باید قابل مرور باشند نه کل برنامه و نه همه جای سیستم.
- پیاده سازی‌های داخل اکشن متدها را به یک کلاس سرویس منتقل کنید. اکشن متدها باید مصرف کننده باشند با حداقل پیاده سازی در حد فراخوانی متدهای سرویس مثلا کار با فایل‌ها و پوشه‌ها.