Andrew N. Shalaev isqad88

isqad88 commented on pull request VadimGorbachevsky/ruby_pract#1
@isqad88

ок, попробуй без промежуточного массива. Подсказываю глядеть в сторону модуля Enumerable. Тут ведь фильтрация происходит, примени правильные "инстр…

isqad88 commented on pull request VadimGorbachevsky/ruby_pract#1
@isqad88

вот теперь тут становится неясно: arrays - массив, берешь каждый его массив запихываешь в пустой массив получается следующее [[...], [...], [..]] п…

@isqad88

Не, так не пойдет, задание не засчитаю, пока нормально не напишешь))

@isqad88

что-то тут не то... метод в методе и методом погоняет))))

@isqad88

зачем от Array наследоваться?) классы называем в CamelCase

@isqad88

просто на будущее, если в массиве только строки без пробелов, то можно писать вот так: %w(Banner1 Banner2 Banner3 Banner4) и да, это внутрь класса …

@isqad88

e[0] - e.first circle_center[0] - cricle_center.first

@isqad88

input_array - слишком абстрактно. называй всегда так, чтобы ясно было о чем речь. например: def dots_in_circle(dots, center, radius) # .... end

@isqad88

тут оформлен if плохо

@isqad88

За это задание :+1: только от инстанс переменных избавиться

@isqad88

опять же return не нужен в конце методов

@isqad88

надо найти и применить метод для массива, который убирает nil ы. Просто лучше использовать встроенные методы руби, так как они написаны на C и соот…

@isqad88

внимательно посмотреть в документации, что делает метод map )) и поправить тут. map используется не по назначению

@isqad88

тут можно так написать ( на будущее): if array.first(2).map(&:to_i).include? ip

@isqad88

Не ясно, что делает метод. аргумент array надо переименовать, и всегда помнить о тех, кто будет читать твой код. Код должен быть написан как книга…

@isqad88

return не нужен в конце метода

@isqad88

The comment has since been removed.

@isqad88

Почему используются instance переменные вместо простых локальных?)

@isqad88
  • @isqad88 85d79db
    fix(engine): Фикс инициализации гема в рельсовом окружении
@isqad88
  • @mender cf11c28
    Merge pull request #4 from isqad88/except_formats_support
  • @isqad88 865ef86
    fix(rspec matcher warnings): Правки сообщений identify: no decode del…
  • 6 more commits »
isqad88 commented on pull request VadimGorbachevsky/ruby_pract#1
@isqad88

Давай так, переоформи пожалуйста по вот этому стилю https://github.com/bbatsov/ruby-style-guide и вот, хороший инструмент для проверки себя rubocop

@isqad88
  • @isqad88 ba35db6
    feature(dictonary): Additional frases for Tytsky
@isqad88
  • @isqad88 3330158
    feature(dictonary): Additional frases for Tytsky
isqad88 commented on pull request dkron/tytsky#1
isqad88 opened pull request dkron/tytsky#1
@isqad88
feature(dictonary): Additional frases for Tytsky
1 commit with 3 additions and 1 deletion
@isqad88
  • @isqad88 4b099eb
    feature(dictonary): Additional frases for Tytsky