Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Сделать вывод N самых больших целых чисел из текстовых данных #5

Merged
merged 14 commits into from
Apr 6, 2020
Merged

Сделать вывод N самых больших целых чисел из текстовых данных #5

merged 14 commits into from
Apr 6, 2020

Conversation

inner-whisper
Copy link
Owner

@inner-whisper inner-whisper commented Feb 7, 2020 •
edited
Loading

  • Сделать вывод файла многострочным
  • Сделать вывод всех чисел из файла, которые в нем встречаются
  • Сделать вывод N чисел из тестового файла (каждое число на новой строке)
  • Сделать вывод N самых больших чисел из файла
  • Добавить передачу параметра N через CLI
  • Тест: если не передан параметр N на вход, то возвращаем сигнал ошибки и текст ошибки
  • Тест: если параметр N передан отрицательным или равным нулю, то возвращаем сигнал ошибки и текст ошибки
  • Тест: возвращаем ошибку, если передано больше параметров, чем 1
  • Тест: возвращаем ошибку, если передан не числовой параметр
  • Тест: если нет файла на вход, то возвращаем ошибку
  • Рефакторинг: вынести класс для валидации ARGV и STDIN(?)

@inner-whisper inner-whisper mentioned this pull request Feb 7, 2020
Closed
15 tasks
@inner-whisper
добавил вывод N чисел из текстового файла
886941a
@inner-whisper
починил rubocop, добавив top-level comment
aea5f41
@inner-whisper
реализовал вывод N первых чисел из файла
4a9fdf8
@inner-whisper
добавил вывод N самых больших чисел из входного файла
aed5c85
@inner-whisper
добавил передачу параметра N через CLI
0021952
@inner-whisper
добавил тест ошибки (не передан параметр N в аргументы скрипта)
540117c
@inner-whisper
добавил проверку на отрицательность или равность нулю N
098a216
@inner-whisper
немного почистил код
ce2fc25
@inner-whisper
перенес development-зависимости из Gemfile в gemspec
05197e3
@inner-whisper
добавил тест того, что в скрипт должен быть передан только один атрибут
73d2845
@inner-whisper
добавил в скрипт передачу нечислового аргумента
97310aa
@inner-whisper
добавил валидацию ввода файлового потока
c57ed80
@inner-whisper
вынес валидацию STDIN и ARGV в отдельный класс
44c643e
@inner-whisper inner-whisper merged commit 26bde56 into master Apr 6, 2020
@inner-whisper inner-whisper deleted the 1-script-can-see-numbers branch April 6, 2020 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@inner-whisper