作者carlcarl (carl)
看板Ajax
标题[讨论] js程式稍微一长 就会暴走得很难懂orz
时间Sat Sep 1 05:41:01 2012
https://github.com/carlcarl/CaGo/blob/master/js/cago.js
大家好 最近想改一下之前写的程式 然後发现我很多又都看不懂了囧
总觉得就是乱~.~
这个程式主要是parse围棋棋谱 用canvas画出来这样@@
不知道能不能给我一点意见 关於架构或是style方面的 或是其他任何意见都可以
比如说
1.
有些变数我是为了不想每次都算才设的 像这种变数我就觉得很难命名orz
2.
object的宣告好像也有点乱 有些必须先在prototype里面写好一些function才能宣告
3.
然後有些是object包含多重array 多重array里又包含object
object里面又有一些属性的值
像这种 如果要拿object的值 我不知道是要直接一个function做完
还是前面的部份用function 然後後面自己拿物件的属性这样
一个function做完的话 又会因为整个架构一层包一层 命名又会变得很鳖扭orz
--
http://blog.carlcarl.tw
--
※ 发信站: 批踢踢实业坊(ptt.cc)
◆ From: 1.175.132.171
1F:推 LaPass:物件导向.... 不过我觉得js用物件导向蛮痛苦的.... 09/01 09:06
2F:→ terlin10:用物件去包吧!减少全域变数。 09/01 10:17
3F:→ terlin10:然後…注解应该可以写的更明确一点? 09/01 10:18
4F:→ carlcarl:嗯嗯 注解在改善中 我自己也觉得不太明确~.~ 09/01 16:59
5F:→ carlcarl:有些我已经有用物件去包了 09/01 17:00
6F:→ carlcarl:全域的部份 我觉得应该是还好(?) 毕竟都包在library里 09/01 17:01
7F:→ mrbigmouth:推荐你Backbone 09/01 17:01
8F:推 tomap41017:js拆Code的话就用Backbone吧 09/01 19:05
9F:→ ousapas:canvas的部分可以用KinectJS物件化 09/01 21:33
10F:→ carlcarl:楼上指的应该是kineticjs? 09/02 01:30
11F:→ carlcarl:刚看了一下 感觉不错 optimize的部份都有作@@ 09/02 01:32
12F:→ carlcarl:像是pre rendering, multiple layer, render only diff 09/02 01:33
13F:→ hermitwhite:有用过,真的是不错,只是和excanvas.js不相容所以... 09/02 01:45
14F:→ carlcarl:喔喔 了解 我是不太想管IE就是@@ 09/02 02:36