textureCache中的等价路径问题

自己的引擎里做了个简单的TextueCache,每次新创建一个纹理,先到TextureCache里查找有没有路径相同的,如果有就直接返回纹理,如果没有加载图片创建纹理并将图片路径缓存起来。另外为了标准统一,我们可以规定路径都转化成全路径(full path)再缓存。

不过发现对于使用了返回父级符号 ../ 的路径,这样简单处理是有问题的,比如 a/b/../x.png 和 a/c/../x.png 这两个路径,形式上不同,实际上却是等价。为了解决这个问题,在做两个全路径比较时要先将两个全路径都转成不带 ../ 符号的形式再比较。下面代码可用于临时解决问题但未必完善:

   bool isPathEqual(const string&fullPath1,const string&fullPath2){//fullPath1 and fullPath2 must be full path

        string path1=convertToDirectPath(fullPath1);

        string path2=convertToDirectPath(fullPath2);

        const int path1Len=(int)path1.size();

        const int path2Len=(int)path2.size();

        if(path1Len!=path2Len)return false;

        for(int i=0;i<path1Len;i++){

            if(path1[i]=='/'||path1[i]=='\\'){

                if(path2[i]=='/'||path2[i]=='\\'){

                    //ok

                }else{

                    return false;

                }

            }else{

                if(path1[i]!=path2[i])return false;

            }

        }

        return true;

    }

    string convertToDirectPath(const string&path){//convert path to equivalent form without ../

        string pathDirect;

        for(int i=0;i<(int)path.size();i++){

            if(i+3<(int)path.size()

               &&(path[i]=='/'||path[i]=='\\')

               &&path[i+1]=='.'

               &&path[i+2]=='.'

               &&(path[i+3]=='/'||path[i+3]=='\\'))

            {

                for(int j=(int)pathDirect.size()-1;j>=0;j--){

                    if(pathDirect[j]=='/'||pathDirect[j]=='\\'){

                        pathDirect.resize(j+1);

                        break;

                    }

                }

                i+=3;

                assert(path[i]=='/'||path[i]=='\\');

            }else{

                pathDirect=pathDirect+path[i];

            }

        }//got pathDirect

        return pathDirect;

    }

当然,支持带../路径是一种选择,另一种选择是引擎直接规定根本不支持带../的路径,但若是如此则一定要对于用户传进来的路径进行检查,如果发现其中带有/../或者开头是../,则给出一个assert fail中断和错误提示,否则既接受带../的输入,又暗自里将纹理加载N次,就坑了。

不知道cocos2dx里的TextureCache有没有考虑这种情况,等有时间看下。

更新(2015-4-9):

刚才测试了一下,cocos2dx不会识别等价路径:

TextureCache::getInstance()->addImage("res/a.png");
Texture2D*tex=TextureCache::getInstance()->getTextureForKey("res/b/../a.png");
cout<<"tex:"<<tex<<endl;
输出结果:
tex:0x0

因此,如果在cocos2dx里对纹理图片使用带../的路径是会悲剧的。

补充(2015-4-9):

还有一种藏得更深的悲剧情况,即可能你自己写的路径都不含../,但你用tiledmap生成的.tmx中却含有../,这一点极易忽视。我把公司的cocos2dx项目中的.tmx文件查看了一遍,发现里面还真有带../的路径,非常奇怪,多数路径都是不带../的,但却有个别带,是什么原因导致tiledmap生成了带../的路径目前我也不知道。

posted on 2015-04-08 20:42  wantnon  阅读(751)  评论(0编辑  收藏  举报

导航