javascript écrit par développeur ruby. besoins refactorisation

Question

je suis tombé sur le code suivant

function currentSlideFromParams() { 
    var result; 
    if (result = window.location.hash.match(/#([0-9]+)/)) { 
    return result[result.length - 1] - 1; 
    } 
} 

Il est clair que ce code est écrit par un développeur de rubis. Cependant, ce qui est idiomatique ruby ​​n'est pas nécessairement idiome JavaScript. Deuxièmement, le code se plaint lorsque je cours à travers Jslint parce qu'il avertit d'avoir une affectation à l'intérieur du contrôle conditionnel.

S'il vous plaît suggérer le code refactorisé qui est selon l'idiome JavaScript. Il suffit de retirer l'affectation de variable de l'instruction if() pour la remplacer par

Answer 1

function currentSlideFromParams() { 
    var result = window.location.hash.match(/#([0-9]+)/); 
    if (result) { 
    return result[result.length - 1] - 1; 
    } 
} 

Cela lui attribuer d'abord. Je ne vois pas quel est le problème, cependant ... vous n'avez pas vraiment à obéir aux règles de jslint.

Answer 2

function currentSlideFromParams() { 
    var result = window.location.hash.match(/#([0-9]+)/); 
    if (result) { 
    return result[result.length - 1] - 1; 
    } 
} 

En outre, si vous prévoyez plusieurs correspondances et recherchez le dernier, vous devez ajouter l'identificateur global g à l'expression rationnelle.

window.location.hash.match(/#([0-9]+)/g) 

Answer 3

Il est clair que ce code est écrit par un développeur ruby.

Vraiment? Un rubyist écrirait:

def current_slide_from_params 
    (location.hash =~ /#(\d+)/ && $1).to_i - 1 
end 

qui se traduit:

function currentSlideFromParams() { 
    return (/#(\d+)/.test(location.hash) && RegExp.$1) - 1; 
} 

;)