妈的，这是我这辈子见过的最恶心的代码。

[wdhwg001]

最近在修图，听说 Photoshop 可以用 JavaScript 写脚本，于是就去看了看怎么写，觉得自己作为一个前端，总不能看不懂代码吧。

目标是对选中的所有图层和组里的所有图层应用一个动作…需求很简单，也很常见吧。

查了一下代码，对特定的一个图层应用一个动作是这样的：

var doc = app.activeDocument;
doc.activeLayer = doc.artLayers.getByName("背景色");
app.doAction("foo", "foo_folder");

感觉已经没什么好害怕的了。但是发现这样只能选一个图层或者一个组，颇为不方便，要知道 PhotoShop 是可以同时选中多个图层的，而文档里也没有说明这个功能…

于是，在继续找下去的过程中，我看到了这玩意。

function getSelectedLayersIdx() {
  var selectedLayers = new Array;
  var ref = new ActionReference();
  ref.putEnumerated(charIDToTypeID("Dcmn"), charIDToTypeID("Ordn"), charIDToTypeID("Trgt"));
  var desc = executeActionGet(ref);
  if (desc.hasKey(stringIDToTypeID('targetLayers'))) {
    desc = desc.getList(stringIDToTypeID('targetLayers'));
    var c = desc.count
    var selectedLayers = new Array();
    for (var i = 0; i < c; i++) {
      try {
        activeDocument.backgroundLayer;
        selectedLayers.push(desc.getReference(i).getIndex());
      } catch (e) {
        selectedLayers.push(desc.getReference(i).getIndex() + 1);
      }
    }
  } else {
    var ref = new ActionReference();
    ref.putProperty(charIDToTypeID("Prpr"), charIDToTypeID("ItmI"));
    ref.putEnumerated(charIDToTypeID("Lyr "), charIDToTypeID("Ordn"), charIDToTypeID("Trgt"));
    try {
      activeDocument.backgroundLayer;
      selectedLayers.push(executeActionGet(ref).getInteger(charIDToTypeID("ItmI")) - 1);
    } catch (e) {
      selectedLayers.push(executeActionGet(ref).getInteger(charIDToTypeID("ItmI")));
    }
    var vis = app.activeDocument.activeLayer.visible;
    if (vis == true) app.activeDocument.activeLayer.visible = false;
    var desc9 = new ActionDescriptor();
    var list9 = new ActionList();
    var ref9 = new ActionReference();
    ref9.putEnumerated(charIDToTypeID('Lyr '), charIDToTypeID('Ordn'), charIDToTypeID('Trgt'));
    list9.putReference(ref9);
    desc9.putList(charIDToTypeID('null'), list9);
    executeAction(charIDToTypeID('Shw '), desc9, DialogModes.NO);
    if (app.activeDocument.activeLayer.visible == false) selectedLayers.shift();
    app.activeDocument.activeLayer.visible = vis;
  }
  return selectedLayers;
};

………………

……………

…………

………

……

…

…那个，我盯着这堆玩意已经看了半个多小时了，有谁能看懂这堆玩意干了啥吗？

var getSelectedLayersIndex = function () {
  var selectedLayers = [];
  var ref = new ActionReference();
  ref.putEnumerated(stringIDToTypeID('document'), stringIDToTypeID('ordinal'), stringIDToTypeID('targetEnum'));
  var desc = executeActionGet(ref);
  if (desc.hasKey(stringIDToTypeID('targetLayers'))) {
    desc = desc.getList(stringIDToTypeID('targetLayers'));
    for (var i = 0, c = desc.count; i < c; i++) {
      selectedLayers.push(desc.getReference(i).getIndex());
    }
  }
  return selectedLayers;
};

进度： “看了几遍和没看一样” → “把不懂的代码删了也能正常用”

基本知道了它准备做一个图层顺序相关的动作，有可能是图层编组，但else中的代码好像毫无意义，整个动作似乎也并没有真的执行…

…虽然还是一脸茫然，但好像这样精简了一下养眼多了，在彻底弄懂之前先凑合用吧。

[msg7086]

我这有不少代码我已经来回看了几个月了，还有很多地方我完全看不懂到底干了啥。

半个小时，还好吧？

[hjc4869]

这就算恶心了？…

[wdhwg001]

@msg7086 没有办法调试， API 不一样，也没有文档说明，而且这代码写的让人根本不知道发生了啥…
反正我一步一步从头到尾看了几遍了，然而依然对这玩意到底怎么实现的毫无头绪。

[ynyounuo]

@wdhwg001
没有文档和任何注释么？

[wdhwg001]

@ynyounuo 没有，只能一次一次猜，之前有人整理过文档但是那是 5 年前的事情了，而且那些文档的网站都挂了…

[lslqtz]

@wdhwg001 archive.org

[hehuozhao]

可能是自动生成的代码？

[q397064399]

其实这种代码的问题显而易见，既没有向读者展示高层次的逻辑意图，又暴露了太多的操作实现细节

[sagaxu]

我这有大量长得很像但又并不完全一样的代码，一个方法写 500 行以上很普遍，几个开发者都离职了，我一边加功能的同时还要一边梳理重构。测试人员都没有，老代码单元测试覆盖率为零。这可是每天 pv 过亿，每天净利润六位数的项目。

[ryanzyy]

你需要 console.log 每个 API 的结果 然后尝试变一些参数 对照 PS 去理解
Good Luck

[pimin]

你只知道 JS ，而不懂 PS API ，如果这都让你看懂了，大概知道怎么搬砖就盖摩天大楼咯？

[valkyrja]

不如做个动作简单粗暴

[happypy1]

@sagaxu 我懂你的苦

[k9982874]

adobe ：我没给你压缩你说偷着乐吧

[harry890829]

习惯就好了，我现在处理的项目，什么利用 if 分支写死的变量，手动拼 json ，手动解析 json 什么的，简直不忍直视

[everyx]

Photoshop 是有 api 文档的，如果需要使用文档中未提及的操作，就需要使用另一个动作到处代码的工具，打开 PS 执行一个动作，他就会生成对应的代码，直接替换参数用，这些代码的确会看不懂

[bigbyto]

见过 switch 套 switch 中间还有 while 循坏和 if 判断的;还见过写 android 定义了 800 多行变量;这种看起来还行。

[wyntergreg]

你这被子还长着呢，以后还有的是更恶心的代码

[aristotll]

关键都是些 API
通用的 JavaScript 知识对阅读代码无效

[Ouyangan]

太年轻了 , 来我这看看一个屏幕看不完的 if,else

[zwik]

上次像写个 ps 的外挂插件，翻了几个文档，完全没有头绪，看来看出都是以 ps 内置扩展文件.8x 的形式出现的插件开发，外挂插件的说明感觉看到关键的地方，然后一下又扯得很宽泛...
最后就写了个简单的 ps js 的脚本 函数的说明能不能详细点，返回值都没有

[liuxu]

我就看到外部有个 var selectedLayers = new Array();，然后 if 里面又有一个。。

[t2doo]

这就最恶心啦，年轻人你还 too young ，看哥给你写一段。。。

[zwik]

r#21 @zwik 网上发帖求助 人家又一个劲地让你去看文档，我勒个去 我能搞掂这个文档，我闲着蛋疼去求助你们

[yoke123]

这点算什么 too young too simple （ doge 脸）

[songz]

这属于 extended script,用了 Adobe 分给 ps 的接囗，写法还是通 js 的

[itqls]

无非是循环找各种图层啥的

[xiaolai123]

还好吧 我看别人的代码感觉都知道它们在干吗

[cnlongxi]

@sagaxu 想知道是哪个项目。。

[blackboom]

@Ouyangan 我懂你，一个函数里 80%的 if else 还巨长

[cppgohan]

太年轻:)

[weberCd]

这就恶心了？那你是没看过我的代码

[GOOD21]

太年轻了 , 来我这看看一个屏幕看不完的 if,else

[ihuzhou]

真实的故事， rightSomeThing = LeftSomeThing()。。前人挖坑，后人掉坑

[wjh3936]

@sagaxu 除了 pv 和利润，我的项目跟你一样……

[yangxiongguo]

ps 不熟看这些代码就是噩梦

[Garnett0328]

楼主太年轻

[jarlyyn]

呵呵，有本事看我当年写的代码。



真的看懂的话别忘记告诉我下这些代码是干啥的-_____-

[jukka]

不算 try exception 圈复杂度才 4 的代码，哪里恶心了。

[smallpath]

ps 的标准 api 文档都找不到， ame 特么还没文档呢，楼上说操作实现细节的， ps 通篇都是类似浏览器的 dom 操作，要不暴露只能自己再造个 adobe 的 jQuery ，哦对了， Object.prototype.watch 可以截取 get set ，要不要再造个 mvvm ？

[mhycy]

http://gist.github.com/anonymous/b59e1de584e8f923a2c4e29eeb6b8143

重新格式化一下
除了命名略坑并不难懂

[yanzixuan]

搞不好是机器自动生成的代码。当年玩 FPGA 的时候看混淆过的 VERILOG 的路过。。。

[jyf]

这代码挺厚道的 已经把意图弄到命名上了 看变量名基本能看出来是在干啥
你要碰到那种 i p 变量满天飞的 那才真是生不如死呢

这个代码最大的问题只是没有把片段分离出来导致一个函数太长了 但是这只是 better 追求

[shellcodecow]

还好吧 除了命名以外..

[hoythan]

慢慢看下去逻辑还是很明显的

[loryyang]

这代码很难懂吗？我觉得这个代码已经质量不错了，函数、变量的命名相当规范，你试想一下里面的变量全部叫做 ab, c, xx, tmp, str 的时候，你是怎样的感觉？

[BigDipper7]

擦，看来我的承受能力还不错，我上次还看到用拼音做 annotation 的，还有什么变量名 abcd 的，你这个算是好的了好么， PS 我的也是没有文档的，啥也没有

[zongwan]

只能用 alert 调试 这点不太方便
api 官网是找的到文档的
可以下载别人写的 plugins 做参考 然后再使用
小动作 使用 fireworks 写脚本会更方便（不过这玩意过时了 被 adobe 抛弃了）

[hector]

我们 20 个程序员两年的项目，到现在没有一行注释，这算个啥

[zongwan]

http://wwwimages.adobe.com/content/dam/Adobe/en/devnet/photoshop/pdfs/photoshop-cc-scripting-guide-2015.pdf

脚本 + 自己写的 plugins 可以自动化很多事情

[sup]

我懂你的苦

[zongwan]

都发出来算了
script:
http://wwwimages.adobe.com/content/dam/Adobe/en/devnet/photoshop/pdfs/photoshop-cc-scripting-guide-2015.pdf
applescript:
http://wwwimages.adobe.com/content/dam/Adobe/en/devnet/photoshop/pdfs/photoshop-cc-applescript-ref-2015.pdf
javascript:
http://wwwimages.adobe.com/content/dam/Adobe/en/devnet/photoshop/pdfs/photoshop-cc-javascript-ref-2015.pdf
vbs:
http://wwwimages.adobe.com/content/dam/Adobe/en/devnet/photoshop/pdfs/photoshop-cc-vbs-ref-2015.pdf

CS 5 之前的版本
http://www.adobe.com/cn/devnet/photoshop/scripting.html

[kaifeii]

见过业务逻辑全写到 sql 里，跑程序就是跑 sql 。……跨语言、跨平台性 MAX 。

[j4fun]

这就恶心啦？。。。还差的远那。。。我们有 C 写的各路回调（ N 层），各种强制类型转换，各种名字一样，其实完全不同含义的形参。。各种一个函数上千行的代码。。你要来试试不 ：）

[mazyi]

51 楼有文档？？？楼主的救星啊哈哈哈。

[zhuangzhuang1988]

这还恶心??
另外 ps 可以调试的

[jason19659]

```
function a(a) {
var f = 1 << a[27] | 1 << a[28] | 1 << a[29] | 1 << a[30] | 1 << a[31] | 1 << a[32] | 1 << a[33];
if (16 <= f) return ! 1;
if (2 == (f & 3) && 2 == a[0] * a[8] * a[9] * a[17] * a[18] * a[26] * a[27] * a[28] * a[29] * a[30] * a[31] * a[32] * a[33] || !(f & 10) && 7 == (2 == a[0]) + (2 == a[1]) + (2 == a[2]) + (2 == a[3]) + (2 == a[4]) + (2 == a[5]) + (2 == a[6]) + (2 == a[7]) + (2 == a[8]) + (2 == a[9]) + (2 == a[10]) + (2 == a[11]) + (2 == a[12]) + (2 == a[13]) + (2 == a[14]) + (2 == a[15]) + (2 == a[16]) + (2 == a[17]) + (2 == a[18]) + (2 == a[19]) + (2 == a[20]) + (2 == a[21]) + (2 == a[22]) + (2 == a[23]) + (2 == a[24]) + (2 == a[25]) + (2 == a[26]) + (2 == a[27]) + (2 == a[28]) + (2 == a[29]) + (2 == a[30]) + (2 == a[31]) + (2 == a[32]) + (2 == a[33])) return ! 0;
if (f & 2) return ! 1;
var q = a[0] + a[3] + a[6],
e = a[1] + a[4] + a[7],
n = a[9] + a[12] + a[15],
d = a[10] + a[13] + a[16],
r = a[18] + a[21] + a[24],
k = a[19] + a[22] + a[25],
p = (q + e + (a[2] + a[5] + a[8])) % 3;
if (1 == p) return ! 1;
var l = (n + d + (a[11] + a[14] + a[17])) % 3;
if (1 == l) return ! 1;
var s = (r + k + (a[20] + a[23] + a[26])) % 3;
if (1 == s || 1 != (2 == p) + (2 == l) + (2 == s) + (2 == a[27]) + (2 == a[28]) + (2 == a[29]) + (2 == a[30]) + (2 == a[31]) + (2 == a[32]) + (2 == a[33])) return ! 1;
q = (1 * q + 2 * e) % 3;
e = g(a, 0);
n = (1 * n + 2 * d) % 3;
d = g(a, 9);
r = (1 * r + 2 * k) % 3;
a = g(a, 18);
var sb = f & 4 ? !(p | q | l | n | s | r) && c(e) && c(d) && c(a) : 2 == p ? !(l | n | s | r) && c(d) && c(a) && b(q, e) : 2 == l ? !(s | r | p | q) && c(a) && c(e) && b(n, d) : 2 == s ? !(p | q | l | n) && c(e) && c(d) && b(r, a) : !1
return sb
}
```

头疼

[mhycy]

@jason19659 谁家的验证代码？看起来还行

[Caratpine]

这段代码至少在风格少还是比较清晰吧。哎，遇到风格千奇百怪的代码才让人头大。。

[Thoxvi]

PS 不是有个录制功能吗…

[soland]

@jason19659 哈哈哈哈

[NCE]

用到递归了吧？如果有子文件夹之类的，很全面了

[wdhwg001]

@zongwan 这个文档里不包括对各种 ID 对应了什么的解读，之前也查过了，找到了 StringID 之后可以猜出来一些…

@mazyi 这文档我看过了…

感谢帮助，已经知道这代码基本是做了什么了…它好像是将选中的所有图层进行分组，选择分组里的所有图层，然后撤销分组…大概这样，虽然依旧搞不懂里面一些地方为毛要 try catch …

[alqaz]

@kaifeii 顺便解决热更问题(O_O)？

[innoink]

那些史前流传下来不知道改了几遍的 C 代码才叫一个恶心

[loveuqian]

谁把 oc 那个 if 判断的图贴一下

[gouchaoer]

缺乏类型的脚本语言就这样咯

[holy_sin]

最起码还有代码缩进啊

[muyege]

这辈子还长,别这么诅咒自己

[znoodl]

见过几百行代码很多 if ，中间是缩进几十行的 tab ，代码像一条龙，那感觉。。。

[yangff]

嘻嘻，比起 jazz 好多了

[zhuangzhuang1988]

去 看下 matlab 代码比这些短
看一星期都不一定看得懂

[huntzhan]

你对恶心一无所知（跑

[thedarkside]

没有那么难懂吧～

[DingSoung]

你都用自动生成代码了 那代码还有可读性么