Форум по Delphi программированию

Delphi Sources



Вернуться   Форум по Delphi программированию > Все о Delphi > Разное
Ник
Пароль
Регистрация <<         Правила форума         >> FAQ Пользователи Календарь Поиск Сообщения за сегодня Все разделы прочитаны

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
  #1  
Старый 08.10.2011, 03:32
Ostrong Ostrong вне форума
Прохожий
 
Регистрация: 08.10.2011
Сообщения: 4
Репутация: 10
Сообщение Проффесионалы зацените!

Извините если я пишу не в тот раздел или мой пост выглядит слишком нубярским но может кто - нибудь оценить мою программу? Рассказать об ошибках и т.д.?
Вот архив с исходником: http://ifolder.ru/26212871
Ответить с цитированием
  #2  
Старый 08.10.2011, 04:11
Аватар для angvelem
angvelem angvelem вне форума
.
 
Регистрация: 18.05.2011
Адрес: Омск
Сообщения: 3,970
Версия Delphi: 3,5,7,10,12,XE2
Репутация: выкл
По умолчанию

Я зверею от этого чуда , давно я не видел такого страшного форматирования кода, точнее полного его отсутствия. Мне хватило мимолётного взгляда на этот шедевр, после чего удалил без сожаления.
P.S. Даже запускать не стал.
__________________
Je venus de nulle part
55.026263 с.ш., 73.397636 в.д.
Ответить с цитированием
  #3  
Старый 08.10.2011, 06:24
Аватар для PhoeniX
PhoeniX PhoeniX вне форума
Always hardcore!
 
Регистрация: 04.03.2009
Адрес: СПб
Сообщения: 3,239
Версия Delphi: GCC/FPC/FASM
Репутация: 62149
По умолчанию

Несколько моментов с утечкой памяти, работа с неинициализированными переменными и пара моментов для потенциальных A/V - только при беглом осмотре. Теме место в "Моих программах", а оценка - на троечку с большим натягом. Даже для начинающего.
__________________
Оставайтесь хорошими людьми...
VK id2634397, ds [at] phoenix [dot] dj
Ответить с цитированием
  #4  
Старый 08.10.2011, 10:04
WinDriver WinDriver вне форума
Прохожий
 
Регистрация: 06.05.2009
Сообщения: 8
Репутация: 10
По умолчанию

Господа, не сочтите за трусость, но таких "программистов" я боюсь. Если вы смотрели индусские коды - они и то в разы лучше.
Ответить с цитированием
  #5  
Старый 08.10.2011, 12:10
Аватар для NIch
NIch NIch вне форума
Продвинутый
 
Регистрация: 02.06.2008
Адрес: Бендеры ПМР
Сообщения: 754
Репутация: 2446
По умолчанию

2 Ostrong. Скажи, полностью сам писал?
Если да, то не обращай внимания на отзывы, пусть сами вспомнят первые свои проекты.
В программу не полез у меня не установлен компонент Gaude. Delphi выдал предупреждение я дальше не полез. Лишь чуть "причесал" текст.
Итак:
1. Форматирование кода ужасное внимательно читай
http://androschuk.blogspot.com/2010/03/delphi_23.html
2. Называй компоненты номальными именами, а не Батоны и Таймеры от 1 до 5
3. Есть множество мягко говоря "нелепых" решений, но верные решения прийдут с опытом.
Вложения
Тип файла: rar TimePRO.rar (182.3 Кбайт, 4 просмотров)
__________________
В начале был Бит, потом Байт и только потом появилось Слово...

Последний раз редактировалось NIch, 08.10.2011 в 13:32.
Ответить с цитированием
  #6  
Старый 08.10.2011, 13:28
Аватар для Pilot_Red
Pilot_Red Pilot_Red вне форума
Продвинутый
 
Регистрация: 01.11.2006
Адрес: Карелия
Сообщения: 702
Версия Delphi: D7
Репутация: 11581
По умолчанию

Цитата:
Сообщение от NIch
2 WinDriver. Скажи, полностью сам писал?
Если да, то не обращай внимания на отзывы, пусть сами вспомнят первые свои проекты.
В программу не полез у меня не установлен компонент Gaude. Delphi выдал предупреждение я дальше не полез. Лишь чуть "причесал" текст.
Итак:
1. Форматирование кода ужасное внимательно читай
http://androschuk.blogspot.com/2010/03/delphi_23.html
2. Называй компоненты номальными именами, а не Батоны и Таймеры от 1 до 5
3. Есть множество мягко говоря "нелепых" решений, но верные решения прийдут с опытом.
NIch причем тут WinDriver стесняюсь спросить?
Ответить с цитированием
  #7  
Старый 08.10.2011, 13:31
Аватар для NIch
NIch NIch вне форума
Продвинутый
 
Регистрация: 02.06.2008
Адрес: Бендеры ПМР
Сообщения: 754
Репутация: 2446
По умолчанию

Прошу прощения, перепутал.
Обращено к Ostrong.
__________________
В начале был Бит, потом Байт и только потом появилось Слово...
Ответить с цитированием
  #8  
Старый 09.10.2011, 08:45
Ostrong Ostrong вне форума
Прохожий
 
Регистрация: 08.10.2011
Сообщения: 4
Репутация: 10
По умолчанию

Спасибо за ответы! Да это моя первая более менее серьезная программа. И я как-бы это сказать "слишком" новичок. Учту все советы при написание новой версии!
Ответить с цитированием
Ответ


Delphi Sources

Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск
Опции просмотра

Ваши права в разделе
Вы не можете создавать темы
Вы не можете отвечать на сообщения
Вы не можете прикреплять файлы
Вы не можете редактировать сообщения

BB-коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.
Быстрый переход


Часовой пояс GMT +3, время: 13:05.


 

Сайт

Форум

FAQ

RSS лента

Прочее

 

Copyright © Форум "Delphi Sources" by BrokenByte Software, 2004-2023

ВКонтакте   Facebook   Twitter