[VernonRoche]

Mireczki mam problem z kodem. Jako argument funkcja dostaje string i ma za zadanie zwrócić index wszystkich samogłosek w postaci tablicy. Funkcja działa jeśli samogłoski się nie powtarzają np. "abcde" => [1, 5] (index ma zaczynać się od 1), ale gdy samogłoski się powtarzają, dostaje coś takiego np. "robot" => [2, 2], zamiast [2, 4]. Gdzie popełniłem błąd?
http://pastebin.com/8uD6xXZK


#naukaprogramowania #ruby #roastmycode

[f.....1]

@VernonRoche index zwraca pierwszy index, nie obecny

[Ragnarokk]

@VernonRoche:
Ja bym to zrobił w drugą stronę - nie iterował po stringu wejściowym, a po nowym arrayu intów

W zasdzie cała logika tego zadania w jednej dość niejasnej linii (str to wejściowy string) :)

[*0..str.length-1].keep_if{|v| vowels.include? str[v]}.to_a

W skrócie - tworzę tablicę indexów i opróżniam je z tych indexów, gdzie w stringu nie ma samogłoski.

[VernonRoche]

@flyer01: @Ragnarokk: Dzięki za pomoc. Ta linia nie jest wcale taka niejasna. Nie rozumiem tylko tej gwiazdki, przed 0, ale jak dobrze kojarzę, to służy ona do przyjmowania nieokreślonej ilości argumentów?

[Ragnarokk]

@VernonRoche: To tzw splash, w tym wypadku robi z Range'a Array :) Pewnie daloby sie ladniej napisac, tak na szybko to wklepalem :)

[godotow]

@VernonRoche:

lubie jednolinijkowce ;)

vowels = %w[a e i o u y]

"Tralala".downcase.each_char.with_index.inject([]){|acc,pair| vowels.include?(pair[0]) && acc.push(pair[1]) || acc}