刚入职,遇到这样的代码,怎么办...

[qinxi]

两周前来的初创公司,当时来面试的时候说之前的 代码是外包写的,我已经有心理准备了.

但是却没想到,居然这么渣.

这只是冰山一脚,for 循环用的出神入化,sql 拼的清新脱俗,

没有用户登陆认证的说法,只需要传 userId 来识别用户身份,

没有任何安全意识,用户列表直接返回了用户的所有信息,包括手机号 密码(明文).

没有订单概念,微信支付只负责生成订单 id,不保存,支付宝由客户端自己生成订单(私钥当然也在 app 啦~)支付成功后由客户端直接提交 userid,money 直接存放数据库..

数据库有一个查询要 14 秒 ,居然还不建索引,不优化.不过想想 之前的 sql,也就明白了

这仍然只是冰山一角.

负责人不懂技术,现在技术只剩下两个 ios 和我了(说是目前不打算做 android)

现在年底了.工作不太好找.不造该咋办了.

工作正好2年半,不过学历是大专

[Rice]

去年我做的公司就是这样，连我个前端都看出他数据库和后台有问题了，赶紧找了个借口跑了。

[coderluan]

从市场需求角度来说，很正常，先和公司反映一下，把主要问题和风险指明了，保留邮件，防止背锅，剩下的看公司决定。

[dreamwar]

@lijsh 心疼你

[Mogugugugu]

外包都是这种代码....开发速度至上，老板等不及，客户也等不及。这很正常的，很多都是代码生成器直接生成的。。。

[Email]

```
for (int i = 0; i < 1000; i++) {
save(); //存一千次 保证存成功
}
```

我又想起了这一段代码. for 循环用的出神入化

[chuanqirenwu]

以前做过一个小外包，原则是只保证程序正常情况下不出问题，至于异常处理，安全性处理什么的不在考虑范围内。

[KentY]

不知道是不是我老了, 观念跟不上年轻人了, 我对这个贴有个别的看法.
1, 外包的不一定差, digg 也是外包做的. 不要觉得是外包做的 就"有思想准备"
2. 某种角度说, 解决问题的能力是自己价值的体现的一种方式. 这可以是漂亮地实现一个复杂功能, 也可能是把烂摊子规整好.
3. 就问题本身, 当你在某领域的知识比别人深厚了, 看别人的工作, 就会发现非常多的问题, 首先一个锻炼自己的事就是, 不抱怨,不笑话别人. 要么指导别人, 要么改正 /进别人的工作. 这个说来容易, 其实很难做到, 包括我自己. 你可以注意观察下, 公司里的高深人士, 很少抱怨, 笑话别人的工作, 反而都是那些还不错 ,或者半瓶醋的水平人, 才做这样的事. 你接了一些不好的代码, 这的确头疼, 你可以大概归纳一下, 代码里你能发现的问题, 问题可能导致的后果以及你能想到的改进方案, 并且做个评估, 时间啊, 精力等. 然后跟你的领导谈一下, 说明如果投入这个会对将来有什么好处, 让领导决定是否要改进. 自己态度上是一个积极态度, 这并不是说要让领导看到你积极, 而是自己真要积极. 不要怕担责任, 你想, 你心里各行各业的牛人, 没有怕担责任的, 都是勇于担当. 主要审视自己能不能做好, 尽力去做到最好.
当然, 如果对现在公司, 职位实在不满意, 可以考虑换工作.

my 2 cents.

[wemore]

话说网站安全处理一般要干啥，用户输入的非法字符转码还有啥别的吗

[unknownservice]

给跪了，赶紧走，要不你要被毁了。。。

[tabris17]

传说中的 1 分钱下订单吗？

[Sendya]

@sagaxu 5K 工资看哭了

[uxstone]

已经不错了，还好是 Java 写的，
要是 PHP 写的就。。。。。

[775229836]

@sagaxu 不要用工资衡量啊，我 6K 也知道要预防这些常识问题

[flyingfz]

必须要和 老板讲清楚现在的系统的风险，以及你的意见。

这是对你后面的工作负责（不论 是否 继续做下去）， 同时也是对老板给你的工资负责。

如果老板不重视你的意见，那只能说： 赶紧弃坑，否则最后会不欢而散。这样对谁都不好。

[sorra]

@qinxi 一看代码就是做 SNS 的。
如果工资满意，可以向老板说明现状，让他支持和认可你接下来的工作。虽然既有的代码不好，但毕竟后端架构归你管了，是个练手机会。一切都做好了还有你什么事呢？
如果你费力不讨好，那就不值得。

[amery2010]

年末其实工作并没有想象的那样难找，你如果真心不想留在这公司的话，可以试着先更新网上的简历，边上班边找工作。

[lijsh]

@yunying 其实公司不差，主要是新来的 CTO 有问题。

[guyskk]

如果公司业务上确实有前景， boss 也愿意听你的建议，这就是你提高的机会，代码烂不烂都是次要的。
否则，现在就开始找下家。

[sagaxu]

@Sendya 创业元老和体制内的别闹

[heeexy]

讲真 要从外包公司离职. 这半年写的代码都是楼主讲得这种#24 怎么办...