Ruby: Comment trouver l'indice de la durée minimale de l'élément de tableau?

Est-il possible de réécrire cette plus élégant? Je pense que c'est un mauvais morceau de code et doit être refait.

>> a = [2, 4, 10, 1, 13]
=> [2, 4, 10, 1, 13]
>> index_of_minimal_value_in_array = a.index(a.min)
=> 3
Sérieusement? a.index(a.min)?
Je ne suis pas sûr à ce sujet. Peut-être que c'est mon anxiété excessive.
Je dirais que c'est à peu près aussi propre qu'il l'obtient, donc pas besoin de "refactoring" ici.
Que faire si il n'y a plus d'une valeur minimale de la matrice? Voulez-vous le premier, le dernier, ou l'ensemble d'entre eux? BTW, je pense que cela vaut la peine d'être question.
Pouvez-vous mentionner les raisons pour lesquelles vous êtes à la recherche pour l'index en premier lieu? Si vous pouvez décrire le problème plus vaste, peut-être il ya une autre approche.

OriginalL'auteur kyrylo | 2011-02-11

  1. 5

    Il serait intéressant de lire sur d'autres situations (la recherche de tous et ne dure minimale de l'élément). 

    ary = [1, 2, 1]

# find all matching elements' indexes
ary.each.with_index.find_all{ |a,i| a == ary.min }.map{ |a,b| b } # => [0, 2]
ary.each.with_index.map{ |a, i| (a == ary.min) ? i : nil }.compact # => [0, 2]

# find last matching element's index
ary.rindex(ary.min) # => 2
    Est-il une différence notable entre: ary.chaque.with_index... et ary.each_with_index...? J'ai trouvé, que each_with_index n'est pas documenté. Mais ary.des méthodes.grep(/each_with_index/); est vrai.
    Ils sont à la fois documenté dans Énumérateur, le Tableau qui en hérite. each.with_index prend le tableau et ajoute un index pour elle de faire un tableau de tableaux, à l'intérieur des tableaux contenant les éléments d'origine, plus les indices. Ensuite, vous pouvez passer à d'autres transformateurs comme map. each_with_index veut effectuer une itération sur les tableaux de tableaux. C'est une différence subtile, mais je ne veux pas d'un each boucle, j'ai voulu transformer.
    Cet algorithme rendement sera faible ectremely
    La question n'était pas sur la vitesse, c'était à propos d'autres façons d'accomplir quelque chose. Pourquoi en était-il sélectionné? Je ne sais pas.

    OriginalL'auteur the Tin Man

  2. 38

    Je crois que ça va parcourir le tableau qu'une seule fois et il est encore facile à lire:

    ary = [2,3,4,5,1]        # => [2,3,4,5,1]
ary.each_with_index.min  # => [1, 4]
                         # where 1 is the element and 4 is the index
    Solution élégante. Malheureusement Array#last rend laid (ary.each_with_index.min.last).
    Peut-être ary.each_with_index.min.second est plus joli.

    OriginalL'auteur andersonvom

  3. 7

    Ce parcourt le tableau qu'une seule fois alors que ary.index(ary.min) traverserait deux fois:

    ary.each_with_index.inject(0){ |minidx, (v,i)| v < a[minidx] ? i : minidx }
    ary.index(ary.min) est beaucoup plus simple à lire.

    OriginalL'auteur x17y19

  4. 3

    En fait, j'aime @andersonvom 's réponse, il suffit de boucler le tableau une fois et toujours obtenir l'indice.

    Et dans le cas où vous ne souhaitez pas utiliser ary.each_with_index.min, voici ce que vous pouvez faire:

    ary = [2,3,4,5,1]                                             # => [2,3,4,5,1]
_, index_of_minimal_value_in_array = ary.each_with_index.min  # => [1, 4]
index_of_minimal_value_in_array                               # => 4

    OriginalL'auteur Leo Liang