看了v2ex的源码,发现一个问题 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
evenkevin
V2EX    Project Babel

看了v2ex的源码,发现一个问题

  •  
  •   evenkevin 2010-12-27 11:33:18 +08:00 7071 次点击
    这是一个创建于 5409 天前的主题,其中的信息可能已经有所发展或是发生改变。
    在filers.py里
    def clly(value):
    imgs = re.findall('(http://cl.ly/[a-zA-Z0-9]+)\s?', value)
    if (len(imgs) > 0):
    for img in imgs:
    img_id = re.findall('http://cl.ly/([a-zA-Z0-9]+)', img)
    if (img_id[0] != 'demo' and img_id[0] != 'whatever'):
    value = value.replace('http://cl.ly/' + img_id[0], '<a href="http://cl.ly/' + img_id[0] + '" target="_blank"><img src="http://cl.ly/' + img_id[0] + '/content" class="imgly" border="0" /></a>')
    return value
    else:
    return value
    register.filter(clly)
    对图片的解析是替换的,如果同时放两张同样的图片地址,估计会有问题。
    用了一张坛子里的图片试一下
    " target="_blank">/content" class="imgly" border="0" />
    " target="_blank">/content" class="imgly" border="0" />
    12 条回复    1970-01-01 08:00:00 +08:00
    evenkevin
        1
    evenkevin  
    OP
       2010-12-27 11:34:27 +08:00
    这里因为是replace,所以生成的img html里的链接也会被循环调用了.
    reorx
        2
    reorx  
       2010-12-27 11:49:22 +08:00
    每一个链接都是在for循环中单独处理的,个人认为不会出问题。
    evenkevin
        3
    evenkevin  
    OP
       2010-12-27 12:29:28 +08:00
    但事实上我只在帖子里放了两个链接,但是出来四张图片,而生成的html里有6个img的tag
    vayn
        4
    vayn  
       2010-12-27 14:06:12 +08:00
    出现两张同样的图片会反复替换,将替换过的链接里的 Url 再次替换才会变成这个样子。确实是个 bug。
    Livid
        5
    Livid  
    MOD
    PRO
       2010-12-27 14:19:43 +08:00 via iPhone
    用户在什么情况下会需要贴两个一模一样的 URL?

    我觉得这是贴的人的问题,没有必要为了这种人为制造出来的“bug”而增加实现的复杂度。
    evenkevin
        6
    evenkevin  
    OP
    &nbp;  2010-12-27 16:02:17 +08:00
    你的逻辑也没问题,只是在无意中发现的问题,告知一下。
    xinzhi
        7
    xinzhi  
       2010-12-27 17:16:12 +08:00
    有一次发很多图片的时候不小心弄重复了,结果帖子一塌糊涂。
    reorx
        8
    reorx  
       2010-12-27 17:22:03 +08:00
    @evenkevin 似乎是这样的,学习了。
    lepture
        9
    lepture  
       2010-12-27 17:46:49 +08:00
    re.sub is better.
    evenkevin
        10
    evenkevin  
    OP
       2010-12-27 19:39:46 +08:00
    这个也挺好解决,遍历时对于重复的照片只循环一次,下面是个解决方案,可能是个笨办法...
    if (len(imgs) > 0):
    for img in imgs:
    repeat_num = imgs.count(img)
    if repeat_num > 1:
    if img_repeat.has_key(img):
    continue
    else:
    img_repeat[img] = repeat_num
    value = value.replace('http://cl.ly/' + img_id[0], '<a href="http://cl.ly/' + img_id[0] + '" target="_blank"><img src="http://cl.ly/' + img_id[0] + '/content" class="imgly" border="0" /></a>')
    return value
    ....
    darcy
        11
    darcy  
       2010-12-27 20:45:13 +08:00
    在某些文章中两个地方引用同一张图片是正常的用途,这个不是bug,是由使用者决定的。
    c
        12
    c  
       2010-12-27 21:47:39 +08:00
    http://v2ex.appspot.com/t/6401

    这里写了个demo,也许可以解决楼主的问题。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     2556 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 28ms UTC 02:29 PVG 10:29 LAX 19:29 JFK 22:29
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86