Просмотрщик фотографий на C# релиз и исходный код

[almostdj]

Здравствуйте, вот написал, на мой взгляд, не плахой, симпатичный просмотарщик фотографий. Выкладываю готовый релиз и код(VS10), вдруг, кто, что доделать захочет.
Также, если у кого появятся конструктивные предложения или замечания с удовольствием их прочту.
Да, кстати если кто – то подскажет, как этот долбанный виндовс заставить отображать иконку моего приложения на файлах, для которых оно установлена как программа по умолчанию. Я буду Вам очень благодарен!

P.S. Для работы приложения нужен FrameWorke 4.0

http://www.programmersclub.ru/просмо...тографий-на-c/

[Casper-SC]

Например, когда я нажал на "Сделать фотографию чёрно-белой", то она сделалась, но потом вылетело исключение:
System.InvalidOperationException не обработано
Message=Недопустимая операция в нескольких потоках: попытка доступа к элементу управления 'pictureBox2' не из того потока, в котором он был создан.
Source=System.Windows.Forms
StackTrace:
в System.Windows.Forms.Control.get_Ha ndle()
в System.Windows.Forms.Control.SetVis ibleCore(Boolean value)
в System.Windows.Forms.Control.set_Vi sible(Boolean value)
в WindowsFormsApplication1.ArtView.Bl ackWhiteImage() в E:\ИТ_1334\C#\Labs\Forms\WindowsFor msApplication1\WindowsFormsApplicat ion1\Form1.cs:строка 1269
в System.Threading.ThreadHelper.Threa dStart_Context(Object state)
в System.Threading.ExecutionContext.R un(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean ignoreSyncCtx)
в System.Threading.ExecutionContext.R un(ExecutionContext executionContext, ContextCallback callback, Object state)
в System.Threading.ThreadHelper.Threa dStart()
InnerException:


Понравилось в программе:
Фон окна самой программы, в общем не плохо выглядит. Хотел сначала придраться к острым углам, но с тенью норм в принципе.

Не понравилось в программе:
Кнопки сначала не видно. Я даже и не понял, что это за окно появилось. Мне кажется кнопки надо как-то сделать по другому и выделить область на которой видно, что она под изображение специально отведена или, что-то наподобие.


Не понравилось в коде:
Названия контролов для телепатов: button1, button2... button5. pictureBox1, pictureBox2.

Я бы назвал как-то так:
btnBack, btnForward, pictureBoxPreview (или pictBoxPreview). Когда (или если) проект разрастётся, то ты уже в нём и сам не разберёшься. Вообще сам теперь стараюсь каждый метод комментировать xml комментариями (или как они там правильно называются). И группировать в коде, что либо по регионам, тоже не плохая идея, облегчает навигацию по коду в разы

Вот как ты получаешь путь к папке для временного хранения там картинки установленой на рабочий стол:

PathForDesck = @"C:\Users\" + System.Environment.UserName + @"\ArtView";//путь к папке с именем пользователя для хранения там картинки установленой на рабочий стол

Советую переделать так:

PathForDesck = Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData) + @"\ArtView";

Конечно дело твоё, но мне кажется так будет правильнее. Да и помоему в Windows XP вообще нету папки Users.


Зацени мою прогу, не верх совершенства, просто зацени, писал в процессе самообучения. Только отвечай там, а не тут, если заценишь.