в чём логика?

ну производятся одни операции если even иначе иные

прямо так и нечитаемь?))

по стилю какой код лучше

ок

является ли хорошим стилем такой код: true if index.even? false if odd.even? или лучше будет считаться код: if index.even? true else false end

я бы просто написал: index.even? безо всяких тру/фолс

вот вам пример оригинальный @format_string.each_with_index do |element, index| @result_string << (element.to_i * 2) if index.even? @result_string << element.to_i if index.odd? end @result_string = @result_string.reverse!.join("").split("").map{|x| x.to_i }.inject(0, :+)

реально не читаемый код так то гыгы

Гист пожалуйста

подозреваю что ты прав

Да тут у тебя подход сам пахнет

@result_string << index.even? && element.to_i * 2 || element.to_i

а вообще да, надо все там ещё улучшать))

@result_string << index.even? && element.to_i * 2 || element.to_i

Что это

а вообще да, надо все там ещё улучшать))

согласен на все 146%

Что это

это то, что у него внутри блока))

each_with_index - и для каждого второго?

хех)

я понял что главная проблема это последняя строчка

а each_slice(2) нельзя?)

а each_slice(2) нельзя?)

там ведь нужно собрать массив из каждого элемента просто для каждого второго элемента преобразование в инт

поправил

each_with_index.map не радует душу?)

end.compact :D

там каждый елемент преабразуется в инт но каждый четный дополнительно умножается на 2

да и вообще там строку потом реверсят, склеивают и суммируют можно за один проход все говно сделать

ладно, "перефразирую" код: 2 + 2 if index.even? 2 + 4 if index.odd? гыг

все ок кроме odd.even? я вот сломался тут

все ок кроме odd.even? я вот сломался тут

поправил

а вообще конечно, надо офункционалить надл малость, типа того: @format_string.map {|x| x.to_i }.with_index do |element, index| index.even? && element * 2 || element end.reverse.join("").split("").map{|x| x.to_i }.reduce(0, :+)

а вообще конечно, надо офункционалить надл малость, типа того: @format_string.map {|x| x.to_i }.with_index do |element, index| index.even? && element * 2 || element end.reverse.join("").split("").map{|x| x.to_i }.reduce(0, :+)

ну может хватит уже издеваться? я же прошу помочь код сделать читаемым и правильным по стилю

А что надо сделать?

index.even? ? element * 2 : element помоему кто-то опечатался

ок но вопрос тогда такой еще - что делать с этой строкой? @result_string = @result_string.reverse!.join("").split("").map{|x| x.to_i }.inject(0, :+)

index.even? ? element * 2 : element помоему кто-то опечатался

точно, круто, спасибо. просто в реальном случае код длинне и запутанее

ну не нужно более читаемой сделать и разбить на методы?

точно, круто, спасибо. просто в реальном случае код длинне и запутанее

ну он будет работать, но логика очень не явная

а это задачка такая

a == 2 && 'двойка' || 'не двойка' я бы не разрешил

задачка проверки кода определенным алгоритмом - алгоритм как бы такой

2.3.0 :006 > %w(a b c).join("") => "abc" 2.3.0 :007 > %w(ab c).join("") => "abc"

.map{|x| x.to_i } эквивалентно map(&:to_i)

гыгы

а в джойне аргумент не нужен

без .join("").split("") не работает =)

а тесты написаны? а то щас зарефакторим а результатом будет говно

я имею в виду .join.split("")

а тесты написаны? а то щас зарефакторим а результатом будет говно

еще нет я не дошел до такого уровня но писать тесты для этого кода намереваюсь

split("") медленней chars, кстати

ERROR: S client not available

еще нет я не дошел до такого уровня но писать тесты для этого кода намереваюсь

а ты тестировать будешь в браузере или консольке?

да и chars как-то понятней:)

а ты тестировать будешь в браузере или консольке?

rspec

ну вот щас ты метод переписал на 90% как его тестировать?

у тебя есть ручная методика хотябы?

пограничные условия еще было бы хорошо знать

у тебя есть ручная методика хотябы?

что?

ну вот ты как щас оперделяешь что все работает?

ааа

так ну да в консоле

и на каких наборах данных ты проверяешь?

так, ох, ладно, показать весь код?

зачем?

как ты проверяешь в консольке что работает?

работа такая

зачем?

format_string = ARGV.first.delete(" ").split("").reverse

он жив

format_string = ARGV.first.delete(" ").split("").reverse

ну это как-бы самостоятельный код, он вроде как и должен работать

вы про амазоновского сотрудника или тут тоже такой кадр был?

ну мне не показалось что Антон издевается - вроде как помогает

николай рип

:(