当前位置:  开发笔记 > 程序员 > 正文

重构这个嵌套在try/catch中的嵌套IF函数

如何解决《重构这个嵌套在try/catch中的嵌套IF函数》经验,为你挑选了1个好方法。



1> Andrew Kenna..:

我很想去:

    private static bool CanMoveToSafeFolder(string filename)
    {
        return IsValidFileFormat(filename)
            && GetFolderIDFromFilename(filename) > 0
            && HasNoViruses(filename)
            && VerifyFileSize(filename);
    }

    public static void HandleUploadedFile(string filename)
    {

        try
        {

            if (CanMoveToSafeFolder(filename))
            {
                // file is OK
                MoveToSafeFolder(filename);
            }
            else
            {
                DeleteFile(filename);
            }
        }
        catch (Exception ex)
        {

        }
        finally
        {
            // do some things
        }

    }


我认为原始问题更多的是关于构造嵌套条件而不是实际决定上传文件是否可以移动到安全文件夹.我的观点是,将检查移动到另一种方法可以保留逻辑,同时提高可读性.
推荐阅读
U友50081205_653
这个屌丝很懒,什么也没留下!
DevBox开发工具箱 | 专业的在线开发工具网站    京公网安备 11010802040832号  |  京ICP备19059560号-6
Copyright © 1998 - 2020 DevBox.CN. All Rights Reserved devBox.cn 开发工具箱 版权所有