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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 2922 天前的主题，其中的信息可能已经有所发展或是发生改变。

最近在修图，听说 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;
};

………………

……………

…………

………

……

…

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

第 1 条附言 · 2016-12-01 20:58:52 +08:00

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中的代码好像毫无意义，整个动作似乎也并没有真的执行…

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

75 条回复 • 2016-12-02 20:18:38 +08:00

msg7086

2016-12-01 03:00:45 +08:00

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

半个小时，还好吧？

hjc4869

2016-12-01 03:04:04 +08:00

这就算恶心了？…

wdhwg001

2016-12-01 03:08:41 +08:00

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

ynyounuo

2016-12-01 03:17:01 +08:00 via iPhone

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

wdhwg001

2016-12-01 03:37:40 +08:00

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

lslqtz

2016-12-01 04:00:35 +08:00

@wdhwg001 archive.org

hehuozhao

2016-12-01 04:09:42 +08:00

可能是自动生成的代码？

q397064399

2016-12-01 06:38:32 +08:00

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

sagaxu

2016-12-01 06:45:38 +08:00 via Android

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

ryanzyy

2016-12-01 07:18:02 +08:00

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

pimin

2016-12-01 07:24:28 +08:00 via Android

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

valkyrja

2016-12-01 08:03:46 +08:00 via Android

不如做个动作简单粗暴

happypy1

2016-12-01 08:24:37 +08:00

@sagaxu 我懂你的苦

k9982874

2016-12-01 08:39:24 +08:00 via iPhone

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

harry890829

2016-12-01 08:42:21 +08:00

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

everyx

2016-12-01 08:47:32 +08:00 via Android

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

bigbyto

2016-12-01 08:55:31 +08:00

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

wyntergreg

2016-12-01 08:57:20 +08:00

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

aristotll

2016-12-01 08:59:29 +08:00

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

Ouyangan

2016-12-01 09:03:42 +08:00

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

zwik

2016-12-01 09:28:33 +08:00

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

liuxu

2016-12-01 09:29:25 +08:00

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

t2doo

2016-12-01 09:30:01 +08:00

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

zwik

2016-12-01 09:30:08 +08:00

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

yoke123

2016-12-01 09:43:11 +08:00

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

songz

2016-12-01 09:45:42 +08:00 via Android

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

itqls

2016-12-01 09:50:28 +08:00

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

xiaolai123

2016-12-01 09:53:08 +08:00

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

cnlongxi

2016-12-01 09:53:56 +08:00

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

blackboom

2016-12-01 09:54:38 +08:00

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

cppgohan

2016-12-01 09:54:48 +08:00

太年轻:)

weberCd

2016-12-01 09:55:03 +08:00

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

GOOD21

2016-12-01 09:55:47 +08:00

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

ihuzhou

2016-12-01 10:00:22 +08:00

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

wjh3936

2016-12-01 10:04:12 +08:00

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

yangxiongguo

2016-12-01 10:04:23 +08:00

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

Garnett0328

2016-12-01 10:04:33 +08:00 via iPad

楼主太年轻

jarlyyn

2016-12-01 10:04:50 +08:00

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

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

jukka

2016-12-01 10:07:41 +08:00

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

smallpath

2016-12-01 10:19:52 +08:00

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

mhycy

2016-12-01 10:28:37 +08:00

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

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

yanzixuan

2016-12-01 10:30:25 +08:00

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

jyf

2016-12-01 10:40:45 +08:00

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

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

shellcodecow

2016-12-01 10:40:55 +08:00

还好吧除了命名以外..

hoythan

2016-12-01 10:42:45 +08:00 via iPhone

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

loryyang

2016-12-01 10:53:41 +08:00

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

BigDipper7

2016-12-01 11:09:13 +08:00

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

zongwan

2016-12-01 11:19:20 +08:00

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

hector

2016-12-01 11:21:27 +08:00

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

zongwan

2016-12-01 11:23:36 +08:00

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

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

sup

2016-12-01 11:25:47 +08:00

我懂你的苦

zongwan

2016-12-01 11:26:25 +08:00

都发出来算了
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

2016-12-01 11:35:56 +08:00

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

j4fun

2016-12-01 11:40:54 +08:00

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

mazyi

2016-12-01 11:59:56 +08:00

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

zhuangzhuang1988

2016-12-01 12:19:05 +08:00

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

jason19659

2016-12-01 14:12:16 +08:00

```
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

2016-12-01 14:20:14 +08:00

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

Caratpine

2016-12-01 15:03:14 +08:00

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

Thoxvi

2016-12-01 15:12:56 +08:00 via Android

PS 不是有个录制功能吗…

soland

2016-12-01 15:19:14 +08:00

@jason19659 哈哈哈哈

NCE

2016-12-01 15:33:02 +08:00

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

wdhwg001

2016-12-01 16:35:52 +08:00

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

@mazyi 这文档我看过了…

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