大家怎么看code test后连个反馈也没有？

bpbsfsj

kksp 发表于 2017-8-3 13:45
楼主我给点意见吧

首先，你那个form做得太烂了，按回车都提交不了，估计人家这么一按就直接fail你了

好吧，我承认不知道什么叫material design，现在明白了，多谢
回车这个是improvement

bpbsfsj

刚有鞋穿的人 发表于 2017-8-3 13:48
好奇别人的要求是什么

react做个form，
The task is to write a simple React application that works with data from GitHub. It should:
* allow the user to type in a GitHub username and click a button called 'Load Repos'
* show a table of repos returned with columns: id, name, description, created at
* allow user to show the default order, or sort on id or name
* show a button per row called "Hide" which removes it from the list
* show a message under the table which says "Not showing 'x' repos" or another appropriate message
Considerations:

* what validation on the 'username' input is needed
* this would be part of a much larger app and that the repo list will be used by other higher-level components
* how to re-render React when anything changes
* consider any failure case when loading the repo list

Not needed:
* pagination (just the first page of results is fine)
* CSS, though feel free to add anything which helps

吐槽下他们给的范例api fetch居然用的是jquery ajax
fetch是react官方推荐的方法，而不是用其他promise库，jquery ajax大概是最烂的ajax lib，即使是用Q也好那么一点

bpbsfsj

楼上很多同学给了我很多意见，我非常感激，也学到了很多东西
但是从我自己个人角度，我更喜欢看编程能力本身，这个提高不是一朝一夕的
比如bootstrap，material design，甚至flux，redux，react框架这种，以前不知道，给点时间熟悉下文档，很快就可以掌握

编程能力提高就慢了，大家看我编程本身，有重大瑕疵吗？

刚有鞋穿的人

bpbsfsj 发表于 2017-8-3 14:49
楼上很多同学给了我很多意见，我非常感激，也学到了很多东西
但是从我自己个人角度，我更喜欢看编程能力本 ...

bootstrap不知道。有点脱节。傻瓜式的。一看就懂。一天足以。很有用

whocares2014

完全看不懂，果然比自动化测试麻烦多了。

mrface

好多都要code test了, 我遇到2个, 都是大公司, 确实花时间, 没办法, 这是等一关

mzbac

bpbsfsj 发表于 2017-8-3 11:17
有点道理吧，多谢指点
但这次他们叫我写react的东西，我从没用过react
从0开始看文档，用3个晚上搭了个re ...

用create-react-app得 我一般看都懒的看..

bpbsfsj

mzbac 发表于 2017-8-3 16:17
用create-react-app得 我一般看都懒的看..

牛，我以为大哥会说凡是不用机器语言的，我一般看都懒得看呢

mzbac

bpbsfsj 发表于 2017-8-3 16:28
牛，我以为大哥会说凡是不用机器语言的，我一般看都懒得看呢

不服气,，一看用cli的就是新手， 还继续看下去干嘛？
随便看看那 https://github.com/gamesover/loa ... ist/RepoList.js#L19 这是什么鬼？ 这就是所谓的编程能力？ 这是埋坑能力吧..  我们公司intern 也不敢这样写...

另外 proptypes 也没有，defaultProps.

xiejiannan

bpbsfsj 发表于 2017-8-3 14:49
楼上很多同学给了我很多意见，我非常感激，也学到了很多东西
但是从我自己个人角度，我更喜欢看编程能力本 ...

你的这些想法并不是说不对，但是适合大公司，微软谷歌什么的，人家确实是注重算法，重编程能力。
但是土澳多数是做各种app的小公司，人家就是care他需要的那几样技术熟不熟练，你算法编程能力再强，人家也不需要，说白了，多数小公司的职位根本没什么晋升空间。就是招个熟练工而已。

刚有鞋穿的人

mzbac 发表于 2017-8-3 16:31
不服气,，一看用cli的就是新手， 还继续看下去干嘛？

为什么不用cli?

mzbac

刚有鞋穿的人 发表于 2017-8-3 16:38
为什么不用cli?

熟手多少都会改点东西..一个崭新的create react app 基本都是新手..

bpbsfsj

mzbac 发表于 2017-8-3 16:31
不服气,，一看用cli的就是新手， 还继续看下去干嘛？
随便看看那 https://github.com/gamesover/load-git ...

比较object array啊，因为里面没啥特殊的，就可以这样比较
当然严格讲是应该逐个比较，这个是应该改

type审查这个，我们其他spa用ts都是做的，我当时看文档急就没看到了
给我时间的话，我会改成ts based的react

bpbsfsj

mzbac 发表于 2017-8-3 16:43
熟手多少都会改点东西..一个崭新的create react app 基本都是新手..

至少我把css改成scss了吧，虽然官方文档有documentation
我没说不是新手啊，对于react

mzbac

bpbsfsj 发表于 2017-8-3 17:04
至少我把css改成scss了吧，虽然官方文档有documentation
我没说不是新手啊，对于react ...

都没看到sass loader，你改成scss还得意..所以人家招熟手的看一眼就不会继续看了

mzbac

bpbsfsj 发表于 2017-8-3 16:57
比较object array啊，因为里面没啥特殊的，就可以这样比较
当然严格讲是应该逐个比较，这个是应该改

你应该做前端时间不长，为比较两个array相不相同，你serialize array的行为在code test里面应该不可能通过的..
Immutable 概念去看看..另外还有react life cycle的should component update 也去看看吧..你life cycle方法都用不对..

bpbsfsj

mzbac 发表于 2017-8-3 17:06
都没看到sass loader，你改成scss还得意..所以人家招熟手的看一眼就不会继续看了 ...

create-react-app并没有直接支持sass loader，目前官方支持是转成css
我写的angular都是用loader的，因为他们官方cli支持这个

你可以全部用webpack写，这些cli本身就是基于webpack+默认配置
angular2的cli出来之前就是需要自己搭webpack的，全部自己控制，就是花时间

bpbsfsj

mzbac 发表于 2017-8-3 17:11
你应该做前端时间不长，为比较两个array相不相同，你serialize array的行为在code test里面应该不可能通 ...

好吧，component控制child component看起来的确是shouldComponentUpdate合适点，多谢了

mzbac

bpbsfsj 发表于 2017-8-3 17:16
create-react-app并没有直接支持sass loader，目前官方支持是转成css
我写的angular都是用loader的，因为 ...

所以说了人家看一眼你code就放弃了，如果你用过webpack，直接eject 下改配置支持sass loader..

bpbsfsj

mzbac 发表于 2017-8-3 17:28
所以说了人家看一眼你code就放弃了，如果你用过webpack，直接eject 下改配置支持sass loader.. ...

以前都用loader的，这是做react第一次要转css
按照https://medium.com/@Connorelsea/using-sass-with-create-react-app-7125d6913760 应该很容易改的，给我周末点时间，我改改

stacknotoverflo

whocares2014 发表于 2017-8-3 11:25
Fit hit多做几个，楼主就掌握流行技术了。然后放一起做过项目，贴cover Letter 上，顿时体面过人。 ...

请教一下，Fit hit是什么

whocares2014

stacknotoverflo 发表于 2017-8-3 18:15
请教一下，Fit hit是什么

就是为了出效果努力锻炼一下的意思，哈哈哈哈。

gzrain

bpbsfsj 发表于 2017-8-3 11:17
有点道理吧，多谢指点
但这次他们叫我写react的东西，我从没用过react
从0开始看文档，用3个晚上搭了个re ...

参考  reactboilerplate， 默认的example就是这个display github repo的，

目前貌似redux比较流行，投react的job 不会这个的话估计不行， egghead有 redux 的 free video tutorial ； 其实redux的source code很简练， 看完video 再看source code， 印象深刻很多

刚有鞋穿的人

mzbac 发表于 2017-8-3 17:28
所以说了人家看一眼你code就放弃了，如果你用过webpack，直接eject 下改配置支持sass loader.. ...

请教一下。spa上觉得sass优势比直接css不大?因为都是碎片式的代码。不知道这种理解是否恰当？

mzbac

刚有鞋穿的人 发表于 2017-8-4 12:22
请教一下。spa上觉得sass优势比直接css不大?因为都是碎片式的代码。不知道这种理解是否恰当？ ...

因为css module  和 post css 后不需要sass了..

刚有鞋穿的人

mzbac 发表于 2017-8-4 15:08
因为css module  和 post css 后不需要sass了..

谢谢

oim5nu

bpbsfsj 发表于 2017-8-3 11:17
有点道理吧，多谢指点
但这次他们叫我写react的东西，我从没用过react
从0开始看文档，用3个晚上搭了个re ...

我React/Redux玩了1年多才敢申请工作，你就看3天。牛

xxmplus

这个楼充分说明前端是个大坑，新手入坑风险自负

gozh2005

我觉得你的code的问题是没有最大化用组件。而且你用了个react-table的库没必要。

Shark-AU

有相似经验，花了几个小时写的code 结果一点反馈都没有，发邮件去问也没回复
HR 这样做太不厚道了，好歹给点反馈，最少回复下邮件