Отправка отладочной информации саппорту? / Разработка информационных систем

Мобильная версия Контакт Правила FAQ Помощь

Гость

Войти | Регистрация | Профиль | Очистить

Форумы | Пользователи | Статистика | Мод. лог | Поиск

Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр

Форумы / Разработка информационных систем [игнор отключен] [закрыт для гостей] / Отправка отладочной информации саппорту?

54 сообщений из 54, показаны все 3 страниц

все

Отправка отладочной информации саппорту?

#35237132

Automater

Участник

Сообщения: 30

Рейтинг: 0 / 0

Когда в Windows падает какая-то майкрософтовская софтина, то появляется окно "Отправить отчёт об ошибке Биллу?". Всегда интересовало - что ж там отправляется?

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

Так вот,
1. я как-то написал процедуру, которая в случае ошибки пробегалась с помощью классов Reflection по цепочки вложенных друг в друга функций и собирала список что чего вызывало пока не рухнуло. Каково же было моё удивление, когда оказалось что на продакшн-серваке эта процедура просто не стала работать! Трэйс видать отрублен...

2. тогда я добавил в софтину (интранет портал, ASP.Net 2.0), в базовый класс, от которого наследуются все страницы, глобальный список - типа стек. Во всех функциях изменил возвращаемый тип на boolean - флаг успешности или не успешности выполнения функции, а все возвращаемые значения запихнул в параметры передаваемые по ссылке (о, сколько времени занял этот мега-рефакторинг! сразу надо было так делать, да вот только умнеешь всегда после такого как уже наступил на грабли). Соответственно в каждой функции при вызове дочерней проверяется код, который она возвращает, и если он false, то дальше ничего не выполняется, в стек записывается [название класса+название функции+точка на которой всё споткнулось] и идёт выход из функции с возвратом false. В parent-функции ловится этот false, добавляется информация в стек и снова идёт выход с false. И так далее. В самом же низу - везде поставлены try-catch, которые ловят собственно все исключения.
Вобщем в итоге в трэйс попадает вся цепочка функций, которые вызывались до ошибки. Это здорово помогает при отладке.

3. однако бы хотелось вместе с названиями функций добавлять в трэйс также и список входящих переменных этих функций и их значения. С помощью классов Reflection нельзя выцепить значения переменных - это я уже узнавал, выцепить их список можно - но вдруг тоже не станет работать из-за настроек сервера? Единственным решением, как мне кажется, является добавление первой строкой в каждую функцию вызова специальной процедуры, передавая ей в качестве параметров всё что получает текущая функция, которая бы сохраняла значения входящих переменных на тот случай если трэйс информация понадобится. Однако это опять сложный рефакторинг (хотя если удачно подобрать рекурсивное выражение, то дело на пару часов).

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

...

Рейтинг:

0 / 0

04.04.2008, 22:49

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35237335

905

Участник

Сообщения: 161

Рейтинг: 0 / 0

почему не сделать обычный вывод в лог файл?
поменять выходные параметры методов это ты долго думал?

...

Рейтинг:

0 / 0

05.04.2008, 11:25

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35237563

Николай1

Участник

Откуда: Москва

Сообщения: 345

Рейтинг: 0 / 0

905почему не сделать обычный вывод в лог файл?
поменять выходные параметры методов это ты долго думал?

Да уж...

...

Рейтинг:

0 / 0

05.04.2008, 17:14

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35237678

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

AutomaterВо всех функциях изменил возвращаемый тип на boolean - флаг успешности или не успешности выполнения функции,
О, это мой любимый подход. Уже пятнадцать лет как изобретены исключения - но нет, программисты под разными соусами снова и снова возвращаются к идее кодов завершения. Воистину, это лучше всех учителей доказывает тезис "не знающий ошибок истории будет повторять их снова и снова".

Automaterсразу надо было так делать,

AutomaterСоответственно в каждой функции при вызове дочерней проверяется код, который она возвращает, и если он false,
Утираю слезы умиления. И небось еще в каждой функции стоит catch, чтобы ловить исключения и преобразовывать их в эту суперпуперофигенную систему образца 1960-го года.

AutomaterВ самом же низу - везде поставлены try-catch, которые ловят собственно все исключения.
О! Я знал, я знал.

AutomaterВобщем, прежде чем пускаться во все тяжкие, я хотел сначала узнать мож кто как уже организовывал передачу отладочной информации в саппорт?
В общем, это все делается достаточно просто. Тем более в .NET, в котором, если мне не изменяет память, класс Exception заточен как носитель дополнительной информации. Простой пример:

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.

 procedure  ReadCfgFile;
 begin 
   try 
    ReadTextFile ('c:\config.ini');
    ......
   except 
    on E: Exception  do 
      raise  AddExceptionHeader (E, 'Ошибка при чтении конфигурационного файла');
   end ;
 end ;

 procedure  ReadTextFile ( const  FileName:  string );
 begin 
   try 
    .....
   except 
    on E: Exception  do 
       raise  AddExceptionInfo (E, 'Ошибка при чтении файла', 'Имя файла: %s', [FileName]);
   end ;
 end ;

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

Ошибка при чтении конфигурационного файла
Ошибка при чтении файла

Файл не найден (os error code = 2)

Имя файла: c:\config.ini

Это сообщение можно показать грамотному пользователю, можно записать в лог, отправить по почте итп. Оно дает вполне адекватную информацию, позволяющую исправить ошибку. Разумеется, не обязательно все пихать именно в текст сообщения - как я говорил, сколь мне помнится, Exception в .NET умеет тащить с собой список пар имя-значение, что позволяет запихнуть туда тексты sql-запросов, значения переменных и все прочее, что захочется знать.

...

Рейтинг:

0 / 0

05.04.2008, 21:34

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239420

KGP

Участник

Откуда: Москва

Сообщения: 2 385

Рейтинг: 0 / 0

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

просто реализовали свой call stack? ok, а остальное, имхо от лукавого.

softwarer
AutomaterВобщем, прежде чем пускаться во все тяжкие, я хотел сначала узнать мож кто как уже организовывал передачу отладочной информации в саппорт?
В общем, это все делается достаточно просто. Тем более в .NET, в котором, если мне не изменяет память, класс Exception заточен как носитель дополнительной информации.

заточен не более, чем наличие текстового поля describe (а там уж хоть супер-пупер-свой xml формат используйте).

...

Рейтинг:

0 / 0

07.04.2008, 13:11

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239521

thejediknight

Гость

AutomaterКогда в Windows падает какая-то майкрософтовская софтина, то появляется окно "Отправить отчёт об ошибке Биллу?". Всегда интересовало - что ж там отправляется?
StackTrace

Automater
Так вот,
1. я как-то написал процедуру, которая в случае ошибки пробегалась с помощью классов Reflection по цепочки вложенных друг в друга функций и собирала список что чего вызывало пока не рухнуло. Каково же было моё удивление, когда оказалось что на продакшн-серваке эта процедура просто не стала работать! Трэйс видать отрублен...

Мня бы за такой подход уволили :-)

Automater
2. тогда я добавил в софтину (интранет портал, ASP.Net 2.0), в базовый класс, от которого наследуются все страницы, глобальный список - типа стек. Во всех функциях изменил возвращаемый тип на boolean - флаг успешности или не успешности выполнения функции, а все возвращаемые значения запихнул в параметры передаваемые по ссылке (о, сколько времени занял этот мега-рефакторинг! сразу надо было так делать, да вот только умнеешь всегда после такого как уже наступил на грабли). Соответственно в каждой функции при вызове дочерней проверяется код, который она возвращает, и если он false, то дальше ничего не выполняется, в стек записывается [название класса+название функции+точка на которой всё споткнулось] и идёт выход из функции с возвратом false. В parent-функции ловится этот false, добавляется информация в стек и снова идёт выход с false. И так далее. В самом же низу - везде поставлены try-catch, которые ловят собственно все исключения.
Вобщем в итоге в трэйс попадает вся цепочка функций, которые вызывались до ошибки. Это здорово помогает при отладке.

Это именно то, чего не нужно было делать, ибо все ужа ДАВНО сделано отлажено и работает!

Automater
3. однако бы хотелось вместе с названиями функций добавлять в трэйс также и список входящих переменных этих функций и их значения. С помощью классов Reflection нельзя выцепить значения переменных - это я уже узнавал, выцепить их список можно - но вдруг тоже не станет работать из-за настроек сервера? Единственным решением, как мне кажется, является добавление первой строкой в каждую функцию вызова специальной процедуры, передавая ей в качестве параметров всё что получает текущая функция, которая бы сохраняла значения входящих переменных на тот случай если трэйс информация понадобится. Однако это опять сложный рефакторинг (хотя если удачно подобрать рекурсивное выражение, то дело на пару часов).

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

Да, microsoft уже такое сделал. 5 лет назад. называеться Exception.
try
{
do_some_work();
}
catch (Exception ex)
{
SendToSupport(ex.ToString());
}

...

Рейтинг:

0 / 0

07.04.2008, 13:36

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239639

shelsoft

Участник

Откуда: Питер FM

Сообщения: 408

Рейтинг: 0 / 0

softwarerУже пятнадцать лет как изобретены исключения
1) Для обработки ошибок в коде согласен
2) Для обработки данных ... вот тут уж извольте использование данного механизма не всегда есть хорошо

^{______________________________________________________

Задолбали вихри яростных атак ...}

...

Рейтинг:

0 / 0

07.04.2008, 14:05

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239691

thejediknight

Гость

shelsoft softwarerУже пятнадцать лет как изобретены исключения
1) Для обработки ошибок в коде согласен
2) Для обработки данных ... вот тут уж извольте использование данного механизма не всегда есть Что имеется ввиду под обработкой данных? Анализ ошибок?

...

Рейтинг:

0 / 0

07.04.2008, 14:14

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239717

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

shelsoft1) Для обработки ошибок в коде согласен
2) Для обработки данных ... вот тут уж извольте использование данного механизма не всегда есть хорошо
Соглашусь, но к проблемам автора это явно отношения не имеет.

...

Рейтинг:

0 / 0

07.04.2008, 14:19

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239756

shelsoft

Участник

Откуда: Питер FM

Сообщения: 408

Рейтинг: 0 / 0

авторЧто имеется ввиду под обработкой данных? Анализ ошибок?

Неоднократно сталкивался (в т.ч. не только при внедрении) с такой проблемой когда
1) Алгоритм обработки данных "верный" - все тестовые примеры "шелкаются" на ура.
2) Данные, взятые из реальных источников тоже "идеальные" - выборочный контроль отдельной информации из массива данных показывает 100% достоверность
3) А на выходе, извините "г"

В таком случае в т.ч. спасают коды завершения. Однако обычно такой добавочный проверочный код маскируется в
а) #IF MYDEBUG ... #ENDIF
б) Application.ini (mydebug=enable)

Есть и другие примеры :)

^{______________________________________________________

Задолбали вихри яростных атак ...}

...

Рейтинг:

0 / 0

07.04.2008, 14:25

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239798

thejediknight

Гость

...

Рейтинг:

0 / 0

07.04.2008, 14:34

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239819

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

shelsoftВ таком случае в т.ч. спасают коды завершения.
Вот тут стопроцентно несогласен. Это ситуация как раз для Exception, в дополнительной информации которого должно идти "какие именно данные левые и чем". Скажем, id обрабатываемой записи и/или значения всех полей из нее.

...

Рейтинг:

0 / 0

07.04.2008, 14:37

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239863

thejediknight

Гость

softwarer shelsoftВ таком случае в т.ч. спасают коды завершения.
Вот тут стопроцентно несогласен. Это ситуация как раз для Exception, в дополнительной информации которого должно идти "какие именно данные левые и чем". Скажем, id обрабатываемой записи и/или значения всех полей из нее.

а если что то типа

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.

public void go(int id, string userName)
        {
            try
            {
                .... do some work
            }
            catch (MyCoolExeption ex)
            {
                ex.addParam("id", id.ToString());
                ex.addParam("userName", userName);
                SendToSupport(ex);
            }
        }

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.

[Serializable]
    public class MyCoolExeption : ApplicationException
    {
        Dictionary<String, String> l = new Dictionary<string, string>();
        public void addParam(String variableName, string valiableValue)
        {
            l.Add(variableName, valiableValue);
        }
    }

...

Рейтинг:

0 / 0

07.04.2008, 14:46

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35239968

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

thejediknight
Об этом я и говорю. С теми замечаниями, что SendToSupport в данном конкретном коде прилеплен совершенно не на месте, а такой Dictionary давно уже есть и нафиг не надо придумывать его еще раз.

...

Рейтинг:

0 / 0

07.04.2008, 15:04

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240009

shelsoft

Участник

Откуда: Питер FM

Сообщения: 408

Рейтинг: 0 / 0

softwarerВот тут стопроцентно несогласен ...
Ну в принципе пожалуй вы правы если если это дешевле .
Главное чтобы используемое средство реализации позволяло пропустить через такой механизм " верные " данные отобрав " неверные " допустим в пакетном режиме, дабы разобраться с ними позднее не прерывая технологический процесс.

^{______________________________________________________

Задолбали вихри яростных атак ...}

...

Рейтинг:

0 / 0

07.04.2008, 15:12

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240024

thejediknight

Гость

softwarer thejediknight
Об этом я и говорю. С теми замечаниями, что SendToSupport в данном конкретном коде прилеплен совершенно не на месте, а такой Dictionary давно уже есть и нафиг не надо придумывать его еще раз. Я знаю, что есть, написал для наглядности.

...

Рейтинг:

0 / 0

07.04.2008, 15:15

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240045

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

thejediknightЯ знаю, что есть, написал для наглядности.
Для наглядности стоит как раз делать явные оговорки.... иначе обязательно многие воспримут как прямое руководство к действию :)

...

Рейтинг:

0 / 0

07.04.2008, 15:19

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240058

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

shelsoftГлавное чтобы используемое средство реализации позволяло пропустить через такой механизм " верные " данные отобрав " неверные " допустим в пакетном режиме,
Для этого я поступаю примерно так: в основном цикле "пакетной обработки" коплю вылетающие исключения в список, а дальше, например, выплевываю их как одно суммарное (ну или иначе, по обстановке - например, если речь идет о загрузке плагинов, то логирую список проблематичных, но не торможу дальнейшую работу).

...

Рейтинг:

0 / 0

07.04.2008, 15:23

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240778

Automater

Участник

Сообщения: 30

Рейтинг: 0 / 0

softwarer

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.

 procedure  ReadCfgFile;
 begin 
   try 
    ReadTextFile ('c:\config.ini');
    ......
   except 
    on E: Exception  do 
      raise  AddExceptionHeader (E, 'Ошибка при чтении конфигурационного файла');
   end ;
 end ;

 procedure  ReadTextFile ( const  FileName:  string );
 begin 
   try 
    .....
   except 
    on E: Exception  do 
       raise  AddExceptionInfo (E, 'Ошибка при чтении файла', 'Имя файла: %s', [FileName]);
   end ;
 end ;

Трудно тут что-либо возразить опираясь на теорию.

Но могу оперевшись на практику:
Вы при таком подходе забьёте весь код try-catch конструкциями. У меня же они только на нижнем уровне.

При вот таком вот коде:

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.

 procedure  ReadTextFile ( const  FileName:  string );
 begin 
   try 
    a = Convert.ToInt32(" 1 ");
    b = Convert.ToInt32(" 2 ");
    c = Convert.ToInt32("Авотфиг!");
    readFromFile();
   except 
    on E: Exception  do 
       raise  AddExceptionInfo (E, 'Ошибка при чтении файла', 'Имя файла: %s', [FileName]);
   end ;
 end ;

...

Рейтинг:

0 / 0

07.04.2008, 18:18

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240807

Automater

Участник

Сообщения: 30

Рейтинг: 0 / 0

shelsoft авторЧто имеется ввиду под обработкой данных? Анализ ошибок?
Неоднократно сталкивался (в т.ч. не только при внедрении) с такой проблемой когда
1) Алгоритм обработки данных "верный" - все тестовые примеры "шелкаются" на ура.
2) Данные, взятые из реальных источников тоже "идеальные" - выборочный контроль отдельной информации из массива данных показывает 100% достоверность
3) А на выходе, извините "г"

В таком случае в т.ч. спасают коды завершения. Однако обычно такой добавочный проверочный код маскируется в
а) #IF MYDEBUG ... #ENDIF
б) Application.ini (mydebug=enable)
Трщи, не уходим от темы. Отладка логики - другое дело. Все "врайт-элэны" пишут... Софтварер пишет что не пишет, но скорее всего лукавит. Как чего по-быстрому надо выяснить наверняка по-быстрому вставляет вывод в файл. Ну да не об этом речь.

Вобщем, я так понимаю никто список входящих параметров в каждой процедуре на тех же catch'ах в дебаг-лог не выводит? Т.е. все знают только место в котором софтина упала, а почему она упала на этих данных, хотя на тестах работала - это выясняется уже вручную, путём повторения ситуации созданной юзерами и путём вывода "врайт-элэнами" промежуточных данных в тестовый файл?

...

Рейтинг:

0 / 0

07.04.2008, 18:27

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240812

Automater

Участник

Сообщения: 30

Рейтинг: 0 / 0

softwarer thejediknightЯ знаю, что есть, написал для наглядности.
Для наглядности стоит как раз делать явные оговорки.... иначе обязательно многие воспримут как прямое руководство к действию :)Тогда [от меня] ещё одна оговорка - УДОБОЧИТАЕМОСТЬ кода.
И, как следствие, снижение:
стоимости замены специалиста;

стоимости поддержки кода.

...

Рейтинг:

0 / 0

07.04.2008, 18:30

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240864

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

AutomaterТрудно тут что-либо возразить опираясь на теорию. Но могу оперевшись на практику:
Позволю себе замечание "вообще", не имея в виду использовать его как аргумент в конкретном случае: попытка противопоставить теорию практике.. редко является чертой компетентного специалиста. Не стоит двигаться в эту сторону.

AutomaterВы при таком подходе забьёте весь код try-catch конструкциями. У меня же они только на нижнем уровне.
:) А какая у нас конечная цель - написать короткую хорошо работающую программу? Или освободить часть кода от гнета страшных трайкэтчей, попутно раздув ее втрое в размерах и забив жутким нечитаемым кодом проверки статусов?

AutomaterВы получите ошибку 'Ошибка при чтении файла'. Т.е., в общем случае, трудно установить на какой строчке произошла ошибка.
Чтобы установить строчку, я использую стектрейс, но только потому, что он есть и практически бесплатен. На самом деле в хорошо написанном коде "устанавливать строчку" не требуется, по описанной информации проблема и без того видна практически сходу - это я говорю, опираясь на опыт тех времен, когда стектрейса не было.

AutomaterА если каждую строчку "окальцовывать" try-catch'ами, то код будеть выглядеть пугающе.
Точнее, так же, как он выглядит окольцованный if-ами.

AutomaterВсе "врайт-элэны" пишут...
Я бы не стал априори обзывать ламерами всех коллег.

AutomaterСофтварер пишет что не пишет, но скорее всего лукавит.
Софтварер за возникновение в коде "врайт-элэнов" увольняет. Вернее, предупреждает, что первый же найденный dbms_output.put_line, System.out.println, MessageBox итп станет последним, и отмазки типа "да я сейчас уберу, в релизе не будет" не прокатят. Желающих проверить это на практике он пока не встречал.

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

AutomaterВобщем, я так понимаю никто список входящих параметров в каждой процедуре на тех же catch'ах в дебаг-лог не выводит?
В общем ты еще и читать не умеешь.

AutomaterТогда [от меня] ещё одна оговорка - УДОБОЧИТАЕМОСТЬ кода.
Да, код с обработкой исключений действительно куда удобочитаемее. Но это не главное его достоинство, всего лишь третье по значимости. Первое - он куда надежнее, второе - легче в модификации.

...

Рейтинг:

0 / 0

07.04.2008, 18:53

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35240978

Automater

Участник

Сообщения: 30

Рейтинг: 0 / 0

softwarerредко является чертой компетентного специалиста. По моему глубокому убеждению, программист должен быть "толковым", а не заумным. Это означает что он как не должен почитать Страуструпа как священную корову.
Стремление писать простой код я нахожу уместным в бизнесе.
softwarerЧтобы установить строчку, я использую стектрейсА у меня удалённый сервак где любой трэйс запрещён.
softwarerТочнее, так же, как он выглядит окольцованный if-ами.Не хочу приводить сравнительные примеры - лень. У меня - плюс одна строчка, у него - плюс пять. Не пойму о чём разговор.
softwarer AutomaterВсе "врайт-элэны" пишут...
Я бы не стал априори обзывать ламерами всех коллег.Я бы не стал скрывать очевидное.
softwarerКак чего по-быстрому надо выяснить наверняка по-быстрому вставляет вывод в файл. Чудно: вывод в сообщение и вывод в файл - разные методы отладки.
softwarerПервое - он куда надежнее, второе - легче в модификации.Добавление if'а совершенно не уменьшает надёжность. Все критические ошибки всё равно ловятся catch'ами.
При модификации легче изменить то что написано рядом с потенциально опасной строчкой, а не то что находится catch'е на несколько экранов ниже.

...

Рейтинг:

0 / 0

07.04.2008, 20:16

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35241552

Petro123

Участник

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)

Сообщения: 39 476

Рейтинг: 0 / 0

Automater
часто пишу 2 варианта:

1 вариант

Код: plaintext

1.
2.

  try  a = Convert.ToInt32("1");             except    on E: Exception do      raise 'xvdcd' + fvfv;  end;
  try  b = Convert.ToInt32("2");             except    on E: Exception do      raise 'xvdcd' + fvfv;  end;
  try  c = Convert.ToInt32("Авотфиг!");  except    on E: Exception do      raise 'xvdcd' + fvfv;  end;

2 вариант - в соответствии с ООП оборачиваем Convert.ToInt32("1"); более умной процедурой, которая наверх возвращает райзе. Куча глупых строк никому ненужна.

______________________________________________
Вы имеете право хранить молчание! Всё что Вы скажете может быть использовано против Вас в суде!

...

Рейтинг:

0 / 0

08.04.2008, 09:49

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35241903

thejediknight

Гость

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.

using System;
using System.Collections;
namespace Example
{
    public class Example
    {
        public static void Main()
        {
            try
            {
                new Example().run( 102 ,  104 ,  109 );
            }
            catch (Exception ex)
            {
                Console.WriteLine(ex.ToString());
                Console.WriteLine("---------------");
                foreach (DictionaryEntry d in ex.Data)
                {
                    Console.WriteLine(String.Format("{0}={1}", d.Key, d.Value));
                }
            }
            Console.ReadKey();
        }
        public void run(int z, int y, int x)
        {
            try
            {
                do_usefull_work( 0 , "zero");
                do_usefull_work( 1 , "one");
                do_usefull_work( 2 , "two");
                do_usefull_work( 3 , "three");
            }
            catch (Exception ex)
            {
                ex.Data.Add("z", z);
                ex.Data.Add("y", y);
                ex.Data.Add("x", z);
                throw;
            }
        }
        int do_usefull_work(int a, string b)
        {
            try
            {
                if (a ==  2 ) throw new Exception("so bad :-)");
            }
            catch (Exception ex)
            {
                ex.Data.Add("a", a);
                ex.Data.Add("b", b);
                throw;
            }
            return ( 2  +  2 );
        }
    }
}

OUTPUT:

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.

System.Exception: so bad :-)
   at Example.Example.do_usefull_work(Int32 a, String b) in C:\projects\Composite\Composite\Example.cs:line  51 
   at Example.Example.run(Int32 z, Int32 y, Int32 x) in C:\projects\Composite\Composite\Example.cs:line  38 
   at Example.Example.Main() in C:\projects\Composite\Composite\Example.cs:line
 11 
---------------
a= 2 
b=two
z= 102 
y= 104 
x= 102

any questions?

...

Рейтинг:

0 / 0

08.04.2008, 11:27

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35241976

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

Automater
При вот таком вот коде:

Код: plaintext

1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.

 procedure  ReadTextFile ( const  FileName:  string );
 begin 
   try 
    a = Convert.ToInt32(" 1 ");
    b = Convert.ToInt32(" 2 ");
    c = Convert.ToInt32("Авотфиг!");
    readFromFile();
   except 
    on E: Exception  do 
       raise  AddExceptionInfo (E, 'Ошибка при чтении файла', 'Имя файла: %s', [FileName]);
   end ;
 end ;

Вы получите ошибку 'Ошибка при чтении файла'. Т.е., в общем случае, трудно установить на какой строчке произошла ошибка. А если каждую строчку "окальцовывать" try-catch'ами, то код будеть выглядеть пугающе.

Пример странный, никакого смысла такой код не несет. Ну да ладно... :)

Convert.ToInt32() и File.ReadAllText() (к примеру) будут выбрасывать совершенно разные типы исключений. Вот их и ловите по очереди, а в конце уже ловите общий Exception.

...

Рейтинг:

0 / 0

08.04.2008, 11:45

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35242105

Petro123

Участник

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)

Сообщения: 39 476

Рейтинг: 0 / 0

Диез
будут выбрасывать совершенно разные типы исключений. Вот их и ловите
+1
да, это 3-ий вариант.
Раньше были go to всякие
Есть библиотеки, которые не только стек отправляют, но и скриншот и т.д.

...

Рейтинг:

0 / 0

08.04.2008, 12:18

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35244072

Automater

Участник

Сообщения: 30

Рейтинг: 0 / 0

Диез Automater
При вот таком вот коде:
Вы получите ошибку 'Ошибка при чтении файла'. Т.е., в общем случае, трудно установить на какой строчке произошла ошибка. А если каждую строчку "окальцовывать" try-catch'ами, то код будеть выглядеть пугающе.

Пример странный, никакого смысла такой код не несет. Ну да ладно... :)

Convert.ToInt32() и File.ReadAllText() (к примеру) будут выбрасывать совершенно разные типы исключений. Вот их и ловите по очереди, а в конце уже ловите общий Exception.Мой шеф, любитель как раз таки этого подхода, именно этим и занимается. Прелесно смотреть на его список "потенциальных ошибок", который, естественно, не включает в себя весь спектр возможных. Конечно в итоге он ловит "общую" ошибку, если остальные не подошли, но смшно смотреть на то как человек мучается. Потом он ещё начинает изобретать свои классы исключений. Ведь надо же "вывалиться" из программы аварийным способом не только когда строчка не конвертится в число, а когда и процедура обращающаяся к базе вернула пустое множество, т.е. в случае логической ошибки. Ох, как жешь Билли заставляет людей раскорячиваться!

Кроме того это не решает проблемы удобочитаемости кода. Обработки всевозможных ситуаций находятся на несколько экранов ниже, а не рядом с потенциально опасными строками.

...

Рейтинг:

0 / 0

08.04.2008, 23:40

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35244396

Petro123

Участник

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)

Сообщения: 39 476

Рейтинг: 0 / 0

Automater
Ведь надо же "вывалиться" из программы аварийным способом не только когда строчка не конвертится в число, а когда и процедура обращающаяся к базе вернула пустое множество, т.е. в случае логической ошибки
:) у кого какой бизнес. У некоторых он вываливается когда "база вернула пустое множество"
:)

...

Рейтинг:

0 / 0

09.04.2008, 09:29

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35244798

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

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

Ну помучается, зато результат лучше будет. :) .

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

Automater
Потом он ещё начинает изобретать свои классы исключений. Ведь надо же "вывалиться" из программы аварийным способом не только когда строчка не конвертится в число, а когда и процедура обращающаяся к базе вернула пустое множество, т.е. в случае логической ошибки.

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

Automater
Ох, как жешь Билли заставляет людей раскорячиваться!

Похоже, вы на Java не писали никогда. Вот где заставляют, так заставляют

...

Рейтинг:

0 / 0

09.04.2008, 11:38

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35272584

Automater

Участник

Сообщения: 30

Рейтинг: 0 / 0

Диез
Ну и что? Вы "оборачиваете" внешние исключения плюс логические ошибки - в коды возврата; он - их же оборачивает в бизнес-исключения, которые точно так же обрабатывает вызывающий код (совершенно необязательно вываливаться из программы при этом).
В чем разница-то?В том что у меня код короче! Вопрос в стоимости поддержки софта, т.е. о количестве времени требуемом новому (или старому, но забывшему) программеру на то чтобы разобраться с логикой процедуры. Если обработка события стоит в catch'е на несколько экранов ниже, то проггеру придётся не сладко. Например: видит он багу что программа запихала в лог, лезет в код, ищет catch, находит, над catch'ем - пара экранов кода, и поди пойми какая строчка вывалила ошибку! Допустим пришла "ошибка конветации в число", а команд ToInt32() несколько - которая из них сбойнула?
ДиезПохоже, вы на Java не писали никогда. Вот где заставляют, так заставляют Потому некогда и не пис а л

Вот сегодня опять напоролся на необходимость реализации упомянутой мною системы (вывод в лог не только места где рухнуло, но и всех входящих параметров в ту процедуру, где произошёл эксепшн). Вижу у себя в логе, что пользователь Х. получил ошибку "clsDALBase.SafeExecuteStoredProcedure: The error description is 'An invalid character was found in text content.'. Could not find prepared statement with handle 3. The statement has been terminated." Явно при запаковке введённых текстовых данных в XML-строку туда проскочил какой-то нехороший символ, при распаковке посчитавшийся за специальный (управляющий). Но КАКОЙ ИМЕННО?!

И вот мне, сегодня, пришлось напрягать главного инженера, чтобы он выяснил у инженера Х. что тот тогда вводил. Фантастика! Я задерживаю багафикс, трачу время на описание проблемы на английском, отвлекаю двух инженеров и всё только потому что у меня нет полной информации об ошибке.

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

...

Рейтинг:

0 / 0

22.04.2008, 21:53

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35272632

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

AutomaterВопрос в стоимости поддержки софта,
Какой же ты смешной....

AutomaterВобщем, как я понял, несмотря на достаточно большой разброс в опыте среди присутствующих на этом форуме, никто и никогда ...
Читать ты за эти две недели так и не научился.

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

...

Рейтинг:

0 / 0

22.04.2008, 22:52

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273022

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

Automater
...В том что у меня код короче!

Нашли, чем хвастаться ;)) Шучу...

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

Automater
Вот сегодня опять напоролся на необходимость реализации упомянутой мною системы (вывод в лог не только места где рухнуло, но и всех входящих параметров в ту процедуру, где произошёл эксепшн). Вижу у себя в логе, что пользователь Х. получил ошибку "clsDALBase.SafeExecuteStoredProcedure: The error description is 'An invalid character was found in text content.'. Could not find prepared statement with handle 3. The statement has been terminated." Явно при запаковке введённых текстовых данных в XML-строку туда проскочил какой-то нехороший символ, при распаковке посчитавшийся за специальный (управляющий). Но КАКОЙ ИМЕННО?!

И вот мне, сегодня, пришлось напрягать главного инженера, чтобы он выяснил у инженера Х. что тот тогда вводил. Фантастика! Я задерживаю багафикс, трачу время на описание проблемы на английском, отвлекаю двух инженеров и всё только потому что у меня нет полной информации об ошибке.

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

А вот что, имхо, вам предстоит в ближйшем будущем:

- сначала вам перестанет хватать Boolean на все случаи, вы переделаете коды возврата на Shortint
- потом захотите возвращать еще и некую текстовую информацию, придется возвращать Record.
- с увеличением уровней вложенности методов захотите разработать универсальный обработчик цепочки вызовов.
- с удивлением обнаружите, что повторили модель исключений...

Повторюсь, это имхо, просто сам через это прошел когда-то.

softwarerВидать, это карма. Причем того, кому потом достанется возиться с этим, когдла ты свалишь.
Резковато сказано, но +1

...

Рейтинг:

0 / 0

23.04.2008, 09:38

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273046

KGP

Участник

Откуда: Москва

Сообщения: 2 385

Рейтинг: 0 / 0

Диез
Существует хорошая практика - делать код метода высотой не более полутора экранов.

имхо это применялось для 'обзорности кода', когда студии имеют 'сворачивание кода' это менее актуально (например регионы, скобки ... в ms vs)

...

Рейтинг:

0 / 0

23.04.2008, 09:44

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273244

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

KGP Диез
Существует хорошая практика - делать код метода высотой не более полутора экранов.

имхо это применялось для 'обзорности кода', когда студии имеют 'сворачивание кода' это менее актуально (например регионы, скобки ... в ms vs)

Code folding - штука удобная, не спорю. Но, думаю, дело тут не только в обзорности, но и в структурировании кода.

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

- самодокументируемость кода, при наличии адекватных имен :)
- удобство отладки
- при возникновении эксепшена пишется Stack Trace в лог, а потом мы однозначно устанавливаем место возникновения ошибки по именам методов

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

...

Рейтинг:

0 / 0

23.04.2008, 10:44

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273309

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

ДиезCode folding - штука удобная, не спорю.
Я, кстати, поспорю. Эта штука удобна имхо в двух случаях:

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

2. Плохой, нечитаемый язык, который не отделяет декларации методов от их реализации, а следовательно не дает возможность увидеть интерфейс класса, забивая экран текстом методов. Ну а с code folding-ом видно хоть как-то, просто дополнительный мусор на экране.

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

ДиезТо есть идея такова, что если тело метода больше полутора страниц, то стоит задуматься о рефакторинге.
Только стоит помнить, что это именно "идея", а не безусловное правило.

...

Рейтинг:

0 / 0

23.04.2008, 11:01

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273408

KGP

Участник

Откуда: Москва

Сообщения: 2 385

Рейтинг: 0 / 0

Диез
Логично будет вынести методы работы с БД и с вебсервисом в отдельные классы, тогда тело моего метода будет состоять просто из вызова трех внешних методов и обработки ошибок.

1. а если и это будет 'более полутора экранов'?
2. а если завтра ваш экран будет 22', а не 17'?
3. как связано 'в отдельные классы', 'трех внешних методов' с 'более полутора экранов', почему не просто 3 отдельные функции того же класса?

ps: мне не понятна роль аргумента 'более полутора экранов'

...

Рейтинг:

0 / 0

23.04.2008, 11:23

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273584

Petro123

Участник

Откуда: Загрузочный сектор Москвы (AutoPOI.ru)

Сообщения: 39 476

Рейтинг: 0 / 0

KGP
ps: мне не понятна роль аргумента 'более полутора экранов'
на камне написано: "Направо пойдешь - коня потеряешь. ".
Нет, найдётся человек который неповерит :))

...

Рейтинг:

0 / 0

23.04.2008, 12:01

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273640

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

KGP Диез
Логично будет вынести методы работы с БД и с вебсервисом в отдельные классы, тогда тело моего метода будет состоять просто из вызова трех внешних методов и обработки ошибок.

1. а если и это будет 'более полутора экранов'?
2. а если завтра ваш экран будет 22', а не 17'?
3. как связано 'в отдельные классы', 'трех внешних методов' с 'более полутора экранов', почему не просто 3 отдельные функции того же класса?

ps: мне не понятна роль аргумента 'более полутора экранов'

1, 2. Естественно, полтора - это величина условная. Просто большая длина обычно требует более одного движения для полного обзора :)

3. Никто не мешает сделать методы того же класса, но часто удобнее и логичнее разнести код на уровни, т.е. в отдельные классы (а то и в отдельные библиотеки).

Вообще, все это есть у Фаулера :)

...

Рейтинг:

0 / 0

23.04.2008, 12:11

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273651

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

Petro123 KGP
ps: мне не понятна роль аргумента 'более полутора экранов'
на камне написано: "Направо пойдешь - коня потеряешь. ".
Нет, найдётся человек который неповерит :))

...

Рейтинг:

0 / 0

23.04.2008, 12:14

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273743

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

KGP2. а если завтра ваш экран будет 22', а не 17'?
Насколько мне известно, споры "подпрограмма должна быть не больше сорока строк или все же можно до пятидесяти" шли еще в шестидесятые года прошлого века. Думаю, что и в шестидесятом веке будут ломать копья об аналогичном в целом вопросе

...

Рейтинг:

0 / 0

23.04.2008, 12:34

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273847

KGP

Участник

Откуда: Москва

Сообщения: 2 385

Рейтинг: 0 / 0

softwarer
Насколько мне известно, споры "подпрограмма должна быть не больше сорока строк или все же можно до пятидесяти" шли еще в шестидесятые года прошлого века. Думаю, что и в шестидесятом веке будут ломать копья об аналогичном в целом вопросе

если нет ООП-предпосылки к разбиению, а лишь проблема в не_возможности_одним_вглядом_охватить_нечто, то почему и не ограничится применением Code folding?

...

Рейтинг:

0 / 0

23.04.2008, 12:52

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273939

egorych

Участник

Откуда: и зачем;

Сообщения: 4 817

Рейтинг: 0 / 0

что есть "ООП-предпосылка" и чем она важнее, нежели желание видеть весь код метода без всяких разных свёртываний?

...

Рейтинг:

0 / 0

23.04.2008, 13:10

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35273943

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

softwarer ДиезCode folding - штука удобная, не спорю.
Я, кстати, поспорю. Эта штука удобна имхо в двух случаях:

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

2. Плохой, нечитаемый язык, который не отделяет декларации методов от их реализации, а следовательно не дает возможность увидеть интерфейс класса, забивая экран текстом методов. Ну а с code folding-ом видно хоть как-то, просто дополнительный мусор на экране.

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

С первым пунктом согласен на 100. А по второму все-таки выскажу соображения (не холивара ради :)) )

Если мы пользуемся текстовым редактором, то да - паскаль и си дадут фору в плане читабельности интерфейсов жаве и шарпу.

Но в современных IDE эта разница нивелируется.
В конце концов, это дело привычки: нажимать Ctrl-Shift-Up/Ctrl-Shift-Down в Delphi IDE, или Ctrl-M,Ctrl-O/Ctrl-M,Ctrl-P в Visual Studio.

...

Рейтинг:

0 / 0

23.04.2008, 13:11

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35274684

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

ДиезНо в современных IDE эта разница нивелируется.
Бесспорно, нивелируется. Тем не менее я вижу разницу между "полезной фичой IDE, реально делающей жизнь лучше" и "костылем, который IDE подставляет там, где кто-то схалтурил". По мне code folding куда ближе ко второму варианту. А такие фичи мне не очень нравятся, так как их наличие провоцирует дальнейшее нарастание кривых решений: мол, сделаем плохо, а IDE потом будет нивелировать.

Я вовсе не призываю программировать в notepad-е, но полагаю, что уход от него должен осуществляться там, где оно стоит и так, чтобы стоило. Простейшим примером назову visual diff: делая check in / check out, я заинтересован в том числе компактно и четко увидеть "что поменялось в интерфейсе класса", и code folding мне здесь не сказать, что поможет.

...

Рейтинг:

0 / 0

23.04.2008, 15:44

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35276859

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

softwarer
Я вовсе не призываю программировать в notepad-е, но полагаю, что уход от него должен осуществляться там, где оно стоит и так, чтобы стоило. Простейшим примером назову visual diff: делая check in / check out, я заинтересован в том числе компактно и четко увидеть "что поменялось в интерфейсе класса", и code folding мне здесь не сказать, что поможет.
Да, в простых случаях такой подход сработает. Но в общем случае diff сравнивает текст внутри файла, но отнюдь не языковые элементы.

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

Так что, ИМХО, в вашем примере с diff, отделение интерфейса от реализации - скорее недостаток языка, позволяющий незаметно выстрелить себе в ногу.

...

Рейтинг:

0 / 0

24.04.2008, 12:36

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35276860

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

...

Рейтинг:

0 / 0

24.04.2008, 12:37

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35277729

softwarer

Участник

Откуда: 127.0.0.1

Сообщения: 51 965

Рейтинг: 0 / 0

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

ДиезНо в общем случае diff сравнивает текст внутри файла, но отнюдь не языковые элементы.
Есть такое. Поэтому я с большим скепсисом отношусь к различного рода "автоматическому слиянию" итп. Но информацию для глаз он дает вполне адекватную. Не идеальную, но - о чем Вы, похоже, забыли - кардинально лучшую, нежели полнейшее ее отсутствие .

ДиезВ приложении небольшой пример, который покажет совсем не то, что вы ожидаете.
Любопытно, откуда Вы знаете, чего я ожидаю? :)

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

ДиезА если методов штук тридцать, да еще меняются модификаторы видимости?..
И что страшного? Хоть триста.

ДиезТак что, ИМХО, в вашем примере с diff, отделение интерфейса от реализации - скорее недостаток языка, позволяющий незаметно выстрелить себе в ногу.
Простите, но пока как минимум эта фраза, сказанная оторванно от какого бы то ни было обоснования. "diff в некоторых случаях неидеален" - принимается. "Поэтому не лучше" - не соглашусь, но логику пойму. А вот "выстрелить себе в ногу" Вы взяли просто с потолка.

Фразу "отделение интерфейса от реализации - скорее недостаток", как Вы понимаете, есть желание вырезать, повесить на стенку и цитировать :) На самом деле, я бы сказал, недостатком языка (дельфы) является как раз недостаточное их отделение, в частности, потребность описывать private часть публичного класса в interface секции модуля.

...

Рейтинг:

0 / 0

24.04.2008, 15:56

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35290167

Николай1

Участник

Откуда: Москва

Сообщения: 345

Рейтинг: 0 / 0

[quot Диез
А вот что, имхо, вам предстоит в ближйшем будущем:

- сначала вам перестанет хватать Boolean на все случаи, вы переделаете коды возврата на Shortint
- потом захотите возвращать еще и некую текстовую информацию, придется возвращать Record.
- с увеличением уровней вложенности методов захотите разработать универсальный обработчик цепочки вызовов.
- с удивлением обнаружите, что повторили модель исключений...

Повторюсь, это имхо, просто сам через это прошел когда-то.

[/quot]

Boolean сразу не хватает на все случаи жизни. За код возврата BOOLEAN сразу бы руки отрубал.
У меня вот нету эксепшенов (точне есть, но такие, скромные), так что приходится строку вверх гнать. По пути она обрастает описанием с каждой кокретной точки и, в итоге, видно, путь на котором "сломалось". Удобно. А проверять надо _все_ ибо сломаться может в любом месте, даже там, где, казалось бы, ничего не грозило...

...

Рейтинг:

0 / 0

01.05.2008, 21:10

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35291072

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

Николай1

А на чём пишете? Чем не устраивают исключения?
Кстати, что значит "скромные" эксепшены?

...

Рейтинг:

0 / 0

03.05.2008, 15:52

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35297786

Lepsik

Участник

Откуда: glubinka

Сообщения: 4 167

Рейтинг: 0 / 0

Диез1, 2. Естественно, полтора - это величина условная. Просто большая длина обычно требует более одного движения для полного обзора :)

3. Никто не мешает сделать методы того же класса, но часто удобнее и логичнее разнести код на уровни, т.е. в отдельные классы (а то и в отдельные библиотеки).

Вообще, все это есть у Фаулера :)

это просто у вас программы маленькие. :)

в больших компаниях Microsoft/IBM/SONY, .... таких правил нет.
У нас есть методы с телом в сотню экранов.
А файл с методом тела процесса больше мегобайта.

...

Рейтинг:

0 / 0

06.05.2008, 17:52

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35299091

Диез

Участник

Откуда: Столица Попозже.

Сообщения: 868

Рейтинг: 0 / 0

Lepsik Диез1, 2. Естественно, полтора - это величина условная. Просто большая длина обычно требует более одного движения для полного обзора :)

3. Никто не мешает сделать методы того же класса, но часто удобнее и логичнее разнести код на уровни, т.е. в отдельные классы (а то и в отдельные библиотеки).

Вообще, все это есть у Фаулера :)

это просто у вас программы маленькие. :)

в больших компаниях Microsoft/IBM/SONY, .... таких правил нет.
У нас есть методы с телом в сотню экранов.
А файл с методом тела процесса больше мегобайта.

Ну, вообще-то это не абсолютное правило, а рекомендация (на мой взгляд, вполне разумная).

С другой стороны, код пишут не крупные компании, а конкретные люди в этих компаниях. В том числе всякие "индусы", которым, по легендам, платят за количество строк кода :))

Кстати, можете привести примеры от Microsoft/Sony/IBM ? А то вы так уверенно утверждаете, как будто сами работаете во всех трех... ;)

...

Рейтинг:

0 / 0

07.05.2008, 11:37

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35304415

stoune

Гость

Диез

Ну, вообще-то это не абсолютное правило, а рекомендация (на мой взгляд, вполне разумная).

С другой стороны, код пишут не крупные компании, а конкретные люди в этих компаниях. В том числе всякие "индусы", которым, по легендам, платят за количество строк кода :))

Кстати, можете привести примеры от Microsoft/Sony/IBM ? А то вы так уверенно утверждаете, как будто сами работаете во всех трех... ;)

Всё очень просто. Психологи давно выяснили что в кратковременной (оперативной памяти) может держатся около 4-5 вещей одновременно. Видя весь код на одном экране легче представить себе всю картину. Когда нужна прокрутка вероятность упустить что-то важное существенно увеличивается. Поэтому при code review своих колег, а они при проверке моего требуем без оправданной необходимости(большие switch, генератор кода etc...) укладываться в те пресловутые 1-2 экрана. Как по мне это азы и слышать возражения от профессиональных программистов мне как-то не по себе.

P. S. Вообще рекомендую курить Совершенный код Маккoннелл С. . Там очень большая коллекция здравых рецептов как облегчить себе и колегам жизнь.

...

Рейтинг:

0 / 0

10.05.2008, 15:45

| Ответить | Цитировать | Написать

Отправка отладочной информации саппорту?

#35305749

Гость

http://blogs.technet.com/not-a-kernel-guy/archive/2008/05/10/3053255.aspx
Алексей Пахунов (c)

...

Рейтинг:

0 / 0

12.05.2008, 10:37

| Ответить | Цитировать | Написать

54 сообщений из 54, показаны все 3 страниц

все

Цитировать

Написать

Автор*:

Ввести пароль для входа

Тема*:

Сообщение

Данное сообщение тематическое

Сообщение содержит картинки или видео 18+

Автор:

ВНИМАНИЕ! На данном подфоруме действуют строгие правила. Удостоверьтесь, что ваше сообщение соответствует им!

Форум или тема закрыты для гостей. Необходима авторизация!

Загрузить последнюю сохраненную версию

Вложение:

Вставить как галерею

Максимальный размер вложений: 4,0 МБ, аудио/видео: 8,0 МБ. Картинки большего размера ужимаются, если возможно.

Введите код, изображенный на картинке. Если код нечитаемый, кликните картинку, чтобы загрузить другой вариант.

Отправляя сообщение, я выражаю свое согласие с правилами форума и принимаю пользовательское соглашение.

Читали тему (0):

Читали форум (0):

Пользователи онлайн (0):

start [/forum/moderation_log.php?user_name=%D0%BC%D1%83%D0%BC%D1%83%D0%BF%D0%BB%D0%B8%D0%BA%D0%B0%D1%82%D0%BE%D1%80]:	0ms
get settings:	8ms
get forum list:	12ms
get settings:	9ms
get forum list:	11ms
check forum access:	4ms
check topic access:	4ms
track hit:	49ms
get topic data:	11ms
get forum data:	2ms
get page messages:	68ms
get tp. blocked users:	1ms
others:	441ms

total:	620ms