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生成了带../的路径目前我也不知道。

巴特西

textureCache中的等价路径问题

最新文章

热门文章