Сучасна диджитал-освіта для дітей — безоплатне заняття в GoITeens ×
Mazda CX 30
×
Do what you love, use what you love
  • Помогите с code review (Go)

    1. Это минимально рабочая версия. Как видите далеко не идеальная как оказалось :)
    2. Спешил немного...
    3. Приму к сведению, спасибо. Хотя тогда все же проще иметь две ветки я так понимаю.
    4. В Go непринято давать длинные названия переменным.
    5. Ну чтобы каждый раз ФС не дергать.

    было бы это для Вас легче и яснее?

    Думаю да. Неужели сложно указать причину отказа ?

  • Помогите с code review (Go)

    Нет тз, писал для себя, чисто в учебных целях. А до тестового задания дело недошло, мне просто отказали без указания на то причины. Хотя теперь понял почему :(

  • Помогите с code review (Go)

    Я придерживался принципа KISS. Причем стараюсь придерживаться его не только при написании кода, но и в целом по жизни :)
    Есть куча навороченных плееров, а хотелось чего-то простого.

    а може у них тестер лінивий і прочитав тільки 3 пункт)

    :)

  • Помогите с code review (Go)

    Хорошо, признаюсь, я немного поспешил, т.к. вакансии на Go джуниоров разметаются как горячие пирожки. Я мониторю вакансии больше месяца везде ниже сеньеров никто не нужен особо. Мидл минимум :(

    github.com/...​blob/master/player.go#L17 — зачем, если дальше не юзается

    На вырост же :)

    github.com/...​blob/master/player.go#L93 — пишем результат в файл до проверки возможной ошибки?

    Да, тут согласен.

    github.com/...​blob/master/player.go#L69 — а если музычка разложена по сабфолдерам?

    Это не баг, это фича :) Я просто немного упростил себе задачу...

    github.com/...​blob/master/player.go#L79 — а что, «играбельный» mp3 файл не может в реале не иметь метаданных?

    Ну у меня валяется несколько таких на компьютере :)

    зачем его писать в файл

    Хотел в slice или map запихнуть, но тогда при создании плейлистов(подразумевалось что он будет не один)

    1. после закрытия все данные будут утеряны
    2. будет использоваться много RAM
    3. каждый раз нужно будет пересоздавать все списки
    возможно ТЗ было так написано

    Учебный проект, Just for Fun :)

  • Помогите с code review (Go)

    Спасибо. Если ткнете носом в небрежность/неидеальность буду благодарен. Это мои первые проекты на Go, очень интересно мнение более опытных (как я понимаю) коллег. Опыт написал общий включая первые шаги в html. На Go пишу месяца 2-3. Т.е. по сути на гитхабе все что я делал на нем. Сложнее плеера ничего придумать не смог. Хотел сделать проект с распознаванием речи, но такой уже есть, так что на гитхабе только черновик его :)

  • Топік для пошуку роботи

    Всем привет. Ищу (удаленную?) работу/стажировку на Go. Опыт в веб-разработке 5+ лет. Я обожаю веб, но рассмотрю другие сферы. В приоритете именно Go, другие языки пока не рассматриваю.

← Сtrl 1... 67891011 Ctrl →