Покритикуйте код. Интересно мнение. Пойму ошибки. Проект и подробности в теме...

@Casper-SC · Регистрация: 27.03.2010

Студворк — интернет-сервис помощи студентам

Здравствуйте! В одной теме я спрашивал как можно организовать отправку новостей, как сохранить новость с форматированным текстом в файл. Отправку я не делал и врятли (хотя возможно) буду делать, так как есть FileZilla. По сути я написал программу для редактирования новостей (хотя скорее просто для сохранения и организации хранения у себя на компьютере, с Office Word можно форматированный текст копировать). В папке "Временное" всё что угодно может находиться, а в папке "Релиз" должны находиться новости, которые я заливаю на сервер. Вот там совсем немного недоделано то, что должен создаваться xml файл со списком файлов новостей, который будет скачивать уже основная программа и сверять, есть ли у неё уже эта новость. Ну это в принципе не важно. Это не окончательная версия, это делалось для себя. С интерфейсом особо не мудрил. Кстати основной целью было сделать так, чтобы пользователь мог щёлкать в новости по ссылке и открывался браузер установленный в системе в качестве браузера по умолчанию, этого я достиг к счастью.

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

. Хочу увидеть свои ошибки.

И это... Я как бы не профессиональный программист. Так что отнеситесь с пониманием, но код критикуйте

. Кстати под "критикуйте" я имею ввиду не только что плохо, что хорошо тоже хотелось бы услышать.

@NickoTin · 28.01.2011, 01:12

Моё имхо:
1) Удаляй ненужные using'и: ПКМ в коде -> Organize Usings -> Remove And Sort.
2)

C#
1
//НЕДОДЕЛАНО!!! Тестовый класс!

Забудешь ведь в большом проекте, используй

C#
1
#warning НЕДОДЕЛАНО!!! Тестовый класс!

3) На счет меню, да - загнул, используй более короткие имена: mnuOpen, mnuClose, mnuCreate и т.д. Не помню, давно еще читал где-то соглашение об именах для контролов, поищи может найдешь что-нибудь.
4) GO - так и пиши GlobalObjects. А то непонятно, кого и куда ты посылаешь

Лень движитель прогресса! Но всё же лучше не сокращай так )

XAML не смотрел - далёк я от него

@m0nax · 28.01.2011, 02:20

Вот заметил кучи таких конструкций

C#
1
2
            FileInfo fi = (sender as FrameworkElement).DataContext as FileInfo;
            ShowPreviewDocument(fi);

так вот оба as не имеют смысла тут, ибо нет проверки на null
если хочется юзать as тогда пиши

C#
1
2
3
4
5
6
7
8
9
            var fve = sender as FrameworkElement;
           
            if (fve != null)
            {
                FileInfo fi = fve.DataContext as FileInfo;
           
                if (fi != null)
                    Process.Start(fi.Directory.FullName);
            }

или же без лишних букв

C#
1
2
            FileInfo fi = (FileInfo)((FrameworkElement)sender).DataContext;
            ShowPreviewDocument(fi);

Вообще не совсем понял что делает программа )

@Casper-SC · 29.01.2011, 14:01 **[ТС]**

Сообщение от SSTREGG

Забудешь ведь в большом проекте, используй
Код C#1 #warning НЕДОДЕЛАНО!!! Тестовый класс!

Спасибо! Про это не знал.

Сообщение от m0nax

Вообще не совсем понял что делает программа )

Просто сохраняет новости и сохраняет их в папке, чтобы ничего не потерять, не перепутать. Может ещё что нибудь добавлю.

@r0fL · 29.01.2011, 14:30

SSTREGG, а где можно прочитать про подобные инструменты? Про юзинги не знал, удалял всегда вручную, оказалось - вон оно что есть, про #warning - тоже.

@HIMen · 29.01.2011, 14:34

r0fL, Директивы препроцессора C#

@r0fL · 29.01.2011, 16:27

Спасибо, но я имел ввиду вообще инструменты студии для оптимизации кода, а не только директивы.

@nicolaus2 · 29.01.2011, 21:54

Сообщение от r0fL

Спасибо, но я имел ввиду вообще инструменты студии для оптимизации кода, а не только директивы.

Смотрите в сторону Resharper.
Почитайте здесь сначала http://ru.wikipedia.org/wiki/ReSharper
Очень много настроек, здорово помогает в рефакторинге(лично для меня, ускорило и написание кода, так как можно заменить некоторые базовые функции студии, как то автодополнение и т.п.). На трекерах можно скачать и программу, и крек. Если желаете только попробовать - на оф. сайте можете взять полнофункциональную версию на 30 дней.

Да что тут рассказывать, не попробуете - не узнаете.

@Casper-SC · 30.01.2011, 02:42 **[ТС]**

Как мне кажется значительно улучшил взаимодействие с пользователем через меню (все пункты те же) и вообще доработал его функционал. Во многих местах использую MessageBox с одним и тем же текстом, но думаю для данной проги это не критично. Может кому-то будет нужно. По крайней мере новичкам будет мне кажется интересно.

Новые блоги и статьи Все статьи Все блоги /
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .

@r0fL Кодило 251 / 179 / 23 Регистрация: 25.11.2009 Сообщений: 685
	29.01.2011, 14:30
	SSTREGG, а где можно прочитать про подобные инструменты? Про юзинги не знал, удалял всегда вручную, оказалось - вон оно что есть, про #warning - тоже. 0

@HIMen 4340 / 1509 / 101 Регистрация: 12.04.2009 Сообщений: 2,342
	29.01.2011, 14:34
	r0fL, Директивы препроцессора C# 1

@r0fL Кодило 251 / 179 / 23 Регистрация: 25.11.2009 Сообщений: 685
	29.01.2011, 16:27
	Спасибо, но я имел ввиду вообще инструменты студии для оптимизации кода, а не только директивы. 0

Покритикуйте код. Интересно мнение. Пойму ошибки. Проект и подробности в теме...

Решение