Apollon_Diamed Apr 21 2016 at 12:31

Пишем простой парсер файлов (для начинающих)

4 min

48K

.NET*C#*

From sandbox

-6

Comments 31

UFO just landed and posted this here

forcewake Apr 21 2016 at 12:59

Зачем это здесь? Никакой полезной нагрузки данная статья не несёт.
Автор, мы очень рады, что вы решили примкнуть к стройному ряду .net разработчкиков, но, поверьте, этот опус не нужен.

Apollon_Diamed Apr 21 2016 at 13:40

Цель данной статьи была в том, чтобы начинающие разработчики тратили меньше времени для поиска подобной информации, ибо я при написании такого простейшего парсера столкнулся с некоторыми проблемами. Но я Вас понял, что нужно побольше опыта, дабы писать статьи на Хабр.

MeGaPk Apr 21 2016 at 13:48

А почему нельзя было теми же средствами, скачать файл через код? Зачем еще браузер подключать?
Кодом будет работать гораздо быстрее.

Apollon_Diamed Apr 21 2016 at 13:49

Пытался, потратил пару часов, но в итоге все равно пришел к этому варианту. Буду благодарен, если напишите ссылки, где об этом написано.

forcewake Apr 21 2016 at 13:56

Можно посмотреть здесь http://codereview.stackexchange.com/a/18679

Apollon_Diamed Apr 21 2016 at 13:59

Спасибо. Надеюсь когда-нибудь наберусь опыта и опубликую что-нибудь действительно полезное.

KvanTTT Apr 21 2016 at 14:14

Это вредные советы, не надо таких советов новичкам давать. Если у вас проблемы с UserAgent, то браузер не обязательно запускать. Вот, например, одно из решений.

Apollon_Diamed Apr 21 2016 at 14:16

Благодарю. Учту.

forcewake Apr 21 2016 at 13:51

Дело не только в опыте. Не поймите меня привратно, но так лучше не писать. Мне кажется, что все когда-то начинали с такого кода и это прекрасно, что вы не боитесь пробовать. Но выставлять что-то подобное как «введение в C#» мне кажется неправильным.

Apollon_Diamed Apr 21 2016 at 13:55

Благодарю, но все же надеюсь, что эта статья кому-нибудь окажется полезной.

arcenciel Apr 21 2016 at 13:38

При всем моем уважении к автору, было бы неплохо, если бы пособия для новичков писали не новички, а более опытные участники сообщества.
Потому что из данного моструозного решения непросто выделить какие-то полезные практики, равно как и невозможно научиться чему-либо полезному.

Apollon_Diamed Apr 21 2016 at 13:40

Благодарю, учту.

Eugeny1987 Apr 21 2016 at 13:54

Я то думал, здесь будет ПАРСЕР, а тут… (

UFO just landed and posted this here

Apollon_Diamed Apr 21 2016 at 14:34

Плохо искал. Спасибо.

MooNDeaR Apr 21 2016 at 18:09

Вот и я как-то надеялся наконец-то познать немного теоретической составляющей анализа текстов. Но когда увидел запуск браузера из кода мягко говоря прибалдел :)

Apollon_Diamed Apr 21 2016 at 19:00

Ну извините, что не оправдал Ваши ожидания. Надеюсь в следующий раз исправлюсь, если он предоставится. В любом случае спасибо за внимание)

codecity Apr 21 2016 at 15:40

Откройте для себя Selenium.

Apollon_Diamed Apr 21 2016 at 17:18

Спасибо.

ashumkin Apr 21 2016 at 18:23

Т.к. до недавнего времени я знал только 1С и немного командную строку, то недолго думая я решил написать его в командной строке

можно было подумать подольше, и вот что могло бы/должно получиться:

for i in $(seq 790 795); do wget "http://aimp.ru/index.php?do=download&sub=catalog&id=$i"; done

ах, ну да, Windows же!

FOR /L %i IN (790,1,795) DO wget "http://aimp.ru/index.php?do=download&sub=catalog&id=%i"

есть инструменты, которые уже давно придуманы для таких вещей

З.Ы. если бы Вы реально сделали парсер, я бы понял — мол, в образовательных целях… но таак извратиться!..

Apollon_Diamed Apr 21 2016 at 18:59

Именно такое у меня и получалось, но он не ожидал между запусками, в итоге практически одновременно открывалась куча вкладок в браузере, что в итоге могло привести к переполнению оперативной памяти. Но да, если немного подумать можно было бы доработать чтоб этого не возникало.

ashumkin Apr 21 2016 at 19:03

Причём тут браузер?
Вы невнимательно читаете. Срочно гуглить

wget

Apollon_Diamed Apr 21 2016 at 19:08

А, прошу прощения, не заметил, да, я гуглил по поводу wget, но не хотелось ставить сторонние утилиты. Да и развиваться же нужно в плане C#.

ashumkin Apr 21 2016 at 19:09

Oook
*facepalm*

KvanTTT Apr 22 2016 at 00:28

А браузер это не "сторонняя утилита"?

Apollon_Diamed Apr 22 2016 at 05:29

Сторонняя, но он у меня уже стоял и в принципе всегда стоит по умолчанию, а wget для одного раза скачивать не хотелось. Хотя наверное следовало, дабы упростить задачу.

yarosroman Apr 21 2016 at 19:24

А для парсинга HTML есть HtmlAgilityPack.

Apollon_Diamed Apr 22 2016 at 05:31

Да, я находил про него, но не стал вдаваться в подробности, раз смог сделать все необходимое через DownloadString. В следующий раз не буду извращаться, а воспользуюсь уже готовыми библиотеками.

ForNeVeR Apr 22 2016 at 18:42

Дорогой автор, не расстраивайтесь по поводу отрицательных оценок вашей работы, просто она действительно реализована не совсем лучшим или рекомендуемым образом. Вам есть ещё куда расти, и я надеюсь, что теперь вы примерно видите направления для дальнейшего развития навыков. Не опускайте рук!

Apollon_Diamed Apr 22 2016 at 21:32

Я и не расстраивался, я понимаю, что мне ещё долго придется расти. Я ценю и уважаю конструктивную критику, а здесь мне её дали сполна. Спасибо за поддержку!)