Last active
August 29, 2015 14:24
-
-
Save codephi/998a8a39097f05c8aa9b to your computer and use it in GitHub Desktop.
Vaga Para Desenvolvedor Web com foco em Frontend ... Detalhes da vaga no script....
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| /** ********************** | |
| Talent wanted! | |
| Web developer. | |
| Focus on Frontend. | |
| ##Salary and assistance## | |
| Salary --- R$ 2.500,00 CLT | |
| Assistance --- R$ 400,00 | |
| ##Local## | |
| Itajaí - SC | |
| More information: | |
| Contact: [email protected] or (47) 3517-1407 | |
| Company Name: Ppiixx Internet & Software | |
| Local: Rua Samuel Heusi, 178 sl 402 | |
| LGN Business - Centro | |
| CEP: 88301-320 Itajaí - SC | |
| ********************** **/ | |
| function headhunter(candidate){ | |
| requirements = ["HTML5", "CSS3", "JavaScript", "Jquery", "PHP", "MySql"] | |
| desirableExperience = ["$.ajax()","window.openDatabase","jsonp"] | |
| desirableExperience.concat(['Restfull','Python','Personal portfolio']) | |
| //... | |
| if(["junior","pleno","senior"].indexOf(candidate.level) && candidate.usesLinuxDistros.lenght >= 1) | |
| return true | |
| return false; | |
| } | |
| candidate = { | |
| 'requirements' : [],//... | |
| 'experiences' : [],//... | |
| 'level' : [],//... | |
| 'usesLinuxDistros' : []//... | |
| } | |
| if(headhunter(candidate)) | |
| $.getMail({ | |
| 'title' : 'Head Hunter | Frontend', | |
| 'to' : '[email protected]' | |
| //With your resume and portfolio, please. | |
| }) |
CasperBraske, obrigado pelo comentário.
Nossa intenção aqui era justamente essa, passar uma informação concreta sobre a vaga.
No indexOf, deveríamos ter comparado com >= 0, pois quando não existe ponteiro setado na matriz o retorna -1.
usesLinuxDistros realmente era bool, porém no meio do caminho o cliente mudou de ideia e pediu para contar as distros, como o prazo era curto, só mudamos o valor.
Porém temos outro erro do nosso desenvolvedor, ele não realizou os teste unitários, pondo a aplicação em produção sem se quer rodar uma só vez.
Mais uma vez obrigado.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Se vocês só procuram "pleno" e "senior", acho melhor tirar o "junior" pra não confundir e/ou usar
já que, se o candidato for "junior", o index será 0, o que dará falso.
Fiddlezinho
Além de que eu faria um oneliner no retorno:
A propriedade "length" tá escrita errado e acho melhor mudar "usesLinuxDistros" (o verbo faz parecer um booleano) pra "linuxDistrosUsed".
Abraço.