[C#] Рефспамер LiveInternet (первый опыт)

Рефспамер LiveInternet



Это мой первый опыт написания программы на C#.
Для этой программы мной был написан класс LiveInternet.Rating и
в нем были использованы такие приемы как многопоточность, синхронизация потоков, события и т.д.

Что такое рефспам?

Просьба знатоков C# прокритиковать код, т.к. писал с помощью гугла и мсдн.


Класс LiveInternet - http://pastebin.com/hrEBycYm
Исходники и бинарники

 

интересный пример )

Если на делфи смог бы тебе помочь... делал много потомность для отправки сообщений на форуме...

 

На делфи то я знаю как делать.
Сам мигрирую из delphi в c# =)

 

первое чо пришло в голову юзай foreach
а и вмест классов с 2мя полями может лучше юзать структуры

 

1. оператор ?:
2. using
3. Convert.ToString(i); -> i.ToString()
4. lock (this), нельзя так лочить, создаешь отдельный Object и его лочишь.

это так, проскроллив сорс от LI

 

1. Мне кажется это снижает читабельность кода =)
2. Что не так с using?
3. Спасибо!
4. в первом варианте так и было. Приму на заметку.

Вроде бы foreach использовал где мог.
Про структуры пока не знаю. Почитаю, спс.

 

он наверн имел ввиду
using(переменная )
{
код
}
ну вмест try catch
но хз

 

Интересный сорец, поковырял =)
Правда не помешало-бы использование соксов, но в стандартном HttpWebRequest этого нету =( Сам вчера мучался... пришлось использовать другую его реализацию =(

 

А, понятно.
Имелось ввиду
using( создание экземпляра)
{
код
}
Вместо Close =)

 

Для генерирования событий, можно сделать отдельные методы.

Пример:
protected virtual void OnCategoryReceived(param1, param2)
{
if (CategoryReceived != null)
{
CategoryReceived(param1, new CategoryEventArgs(param2));
}
}

К именам делегатов, можно добавлять приставку - EventHandler, а к событиям - Event.

Ещё, очень бы приветствовались XML комментарии и использование регионов (#region).

P.S. Здесь можешь почитать, почему лучше не делать lock (this).

 

Ясно. За совет про заворачивания событий в методы, отдельное спасибо)