Профессионалы оцените29.04.2014, 20:24. Показов 794. Ответов 4
Метки нет (Все метки)
Уважаемые профессионалы оцените пожалуйста по этому коду мой опыт по 10 бальной шкале, и тянет ли это на уровень стажера, или лучше вообще мне больше не браться за программирование???
Кликните здесь для просмотра всего текста
Программа парсит новости из сайта www.e1.ru нужно вставить адрес сайта в поле рядом с кнопкой "Парсинг" Не по теме: P.S. Вопрос только для профессионалов Добавлено через 2 часа 42 минуты upa
0
|
||||||
| 29.04.2014, 20:24 | |
|
Ответы с готовыми решениями:
4
Профессионалы, ответьте! Подскажите профессионалы. Тут есть профессионалы по JS |
|
2399 / 2224 / 565
Регистрация: 28.12.2010
Сообщений: 8,672
|
||
| 30.04.2014, 02:11 | ||
|
* Все свалено в один класс * Построение GUI не разделено никак * Не использованно никаких design patterns даже там где они напрашиваются * Все ActionListener's реализованны в анонимных классах что приемлимо только в том случае когда реализация сведена к вызову одного метода (не ваш случай) * Программа не выдает никаких сведений об ошибках (ошибку можно увидеть только в консоли IDE) * Нет никакого логера * Ничего не говорящие имена переменных (con, br, pw, a[] fileOne etc) * Применяется ручной парсинг хотя задача парсинга давно уже решена кучей библиотек * Нигде нет никакой инкапсуляции. Все видят всё. * В некоторых исключениях вообще ничего не выводится никуда (Line 96: } catch (IOException e) {} ) * Не используются константы там где они должны быть (вообще их нет по-моему) (например имя файла 1.txt встречается дважды в виде текста) Это только с первого взгляда. Пока писал, подумал что даже 3\10 многовато особенно с учетом что программа выкинула исключение при попытке получить какой-нибудь результат.
1
|
||
|
Антикодер
1888 / 870 / 48
Регистрация: 15.09.2012
Сообщений: 3,088
|
|
| 30.04.2014, 13:42 | |
|
"профессионалы" обычно строят свои программы на основе некоторых математических моделей.
это я считаю большей проблемой по сравнению с вышеперечисленным. Поэтому, в программировании, и ценятся книги Кнута. Ошибка начинается постановки задачи. Для чего вы написали этот код? Кому он будет полезен? Тут целая пропасть выявления требований к ПО. Но если это просто задачка для тестирования, то где лицо заинтересованное в проверке результатов? Со стороны это выглядит так, как будь то вы сами выбрали наиболее простую задачку для себя. В простых тестовых задачках вы скорее должны показать, что знаете все базовые понятия языка(вспомним замечание про инкапсуляцию), а также знакомы с прилегающими к программированию областям. Следовательно, любая оценка такого кода не даёт нам полезной информации. Скажем, тесты на сайте intuit.ru дадут вам больше полезной инфы. На уровень стажера, в не очень известной компании, вы подходите. Но сложности возникнут в быстром изучении предметной области в которой работает компания. При этом я всё же советую между работой и обучением выбирать обучение программированию. Добавлено через 20 минут вот хороший пример на тему воплощения мат модели в жизнь: Создание робота балансера на arduino
0
|
|
| 30.04.2014, 13:42 | |
|
Помогаю со студенческими работами здесь
5
Профессионалы программирования дайте совет Если найдутся профессионалы Матлаба Ищуться профессионалы экстра класса Профессионалы помогите пожалуйста разобраться! Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D
8Observer8 04.03.2026
Содержание блога
Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip
На первой гифке отладочные линии отключены, а на второй включены:. . .
|
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий
8Observer8 02.03.2026
Содержание блога
Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip
https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536
Одним из. . .
|
Реалии
Hrethgir 01.03.2026
Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .
|
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
|
|
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
|
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога
Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
|
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование
. \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json>
Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом.
# Check if. . .
|
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так:
https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347
Основана на STM32F303RBT6.
На борту пять. . .
|