ReSQL.ru
     
powered by simpleCommunicator - 2.0.59     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Delphi [игнор отключен] [закрыт для гостей] / Флейм про оформление и begin-end
25 сообщений из 261, страница 6 из 11
  6  все
Флейм про оформление и begin-end
    #40063351
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
fraks
не всегда есть смысл связываться с множеством

Наличие

Код: plaintext
1.
2.
3.
4.
5.
CFG.LT_GoodsMark_ID_LTA1
CFG.LT_GoodsMark_ID_LTA2
CFG.LT_GoodsMark_ID_LTA3
CFG.LT_GoodsMark_ID_LTA4
CFG.LT_GoodsMark_ID_LTA5
CFG.LT_GoodsMark_ID_LTA6

означает, что любой работающий с ними код будет иметь тенденцию к шестикратному копированию. А это - очень существенный аргумент в пользу того, чтобы сразу от такого уйти. Потому что если я, допустим, в случае чего придумаю, как без этого обойтись, то добрая половина разработчиков на голубом глазу возьмёт фрагмент строк в пятьдесят и откопирует его пять раз, меняя в нём CFG.LT_GoodsMark_ID_LTA1 на CFG.LT_GoodsMark_ID_LTA2...
...
Рейтинг: 0 / 0
17.04.2021, 16:57
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063352
fraks
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
fraks
Участник
softwarer
fraks
не всегда есть смысл связываться с множеством

Наличие

Код: plaintext
1.
2.
3.
4.
5.
CFG.LT_GoodsMark_ID_LTA1
CFG.LT_GoodsMark_ID_LTA2
CFG.LT_GoodsMark_ID_LTA3
CFG.LT_GoodsMark_ID_LTA4
CFG.LT_GoodsMark_ID_LTA5
CFG.LT_GoodsMark_ID_LTA6

означает, что любой работающий с ними код будет иметь тенденцию к шестикратному копированию. А это - очень существенный аргумент в пользу того, чтобы сразу от такого уйти. Потому что если я, допустим, в случае чего придумаю, как без этого обойтись, то добрая половина разработчиков на голубом глазу возьмёт фрагмент строк в пятьдесят и откопирует его пять раз, меняя в нём CFG.LT_GoodsMark_ID_LTA1 на CFG.LT_GoodsMark_ID_LTA2...

В данном месте 50 элементов невозможно принципиально, я уже писАл, это цветА.
Перебрать контролы в виде множества - отдельная заморочка.
В данном случае проще делать именно так, поименные переменные в конфиге.
Я единственный разработчик этого продукта.
...
Рейтинг: 0 / 0
17.04.2021, 17:04
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063353
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
fraks
В данном месте 50 элементов невозможно принципиально, я уже писАл, это цветА

Шестикратное копирование - уже достаточно плохо.

fraks
Я единственный разработчик этого продукта.

Не факт, что так сохранится навечно.
...
Рейтинг: 0 / 0
17.04.2021, 17:07
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063354
fraks
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
fraks
Участник
softwarer
означает, что любой работающий с ними код будет иметь тенденцию к шестикратному копированию.

У нас на работе есть питонист, самостоятельно переучившийся из 1С-ника.
Он любитель хардкодить частные случаи, из чего проистекает то что любой код у него в единственном экземпляре.
Одинаковых баз у нас штук 10, но справочники в каждой - свои.
Бэкапов не любит, системы контроля версий не использует вообще.
Вот это - проблема.
...
Рейтинг: 0 / 0
17.04.2021, 17:10
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063355
fraks
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
fraks
Участник
softwarer
fraks
В данном месте 50 элементов невозможно принципиально, я уже писАл, это цветА

Шестикратное копирование - уже достаточно плохо.

Переделки этого кода не сулят никаких выгод.

softwarer
fraks
Я единственный разработчик этого продукта.

Не факт, что так сохранится навечно.

Я не в ответе за чужие ошибки в будущем.
...
Рейтинг: 0 / 0
17.04.2021, 17:13
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063357
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
fraks
Я не в ответе за чужие ошибки в будущем.

Не в ответе. Но код, который уберегает от ошибок, качественнее, чем код, который к ним подталкивает.
...
Рейтинг: 0 / 0
17.04.2021, 17:21
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063359
Гаджимурадов Рустам
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гаджимурадов Рустам
Участник
softwarer> Это был бы такой же шаг в прошлое с его end if, end loop итп.

Всему надо знать меру, необязательно от
EndClass-a прыгать на EndIf и EndLoop. :)

Понятно, что это малоактуально, но если
бы был "код методов прямо в декларации" -
было бы гораздо более актуально. :)
Posted via ActualForum NNTP Server 1.5
...
Рейтинг: 0 / 0
17.04.2021, 17:29
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063360
fraks
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
fraks
Участник
softwarer
fraks
Я не в ответе за чужие ошибки в будущем.

Не в ответе. Но код, который уберегает от ошибок, качественнее, чем код, который к ним подталкивает.

Если приведешь пример кода, может быть я бы понял о чем речь. Пока что мне видится что любой перевод в N-мерность только усложнит и увеличит количество кода, и приведет к необходимости контролировать это самое N (границы).
...
Рейтинг: 0 / 0
17.04.2021, 17:30
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063362
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
Гаджимурадов Рустам
Всему надо знать меру, необязательно от EndClass-a прыгать на EndIf и EndLoop. :)

Но их история побуждает не прыгать и к end class.

Гаджимурадов Рустам
Понятно, что это малоактуально, но если бы был "код методов прямо в декларации" -
было бы гораздо более актуально.

Да. И по этой причине мне не нравится тенденция напихивать в декларации классов всякие хрени в java-стиле. Лучше сохранять компактные декларации классов, при которых достаточно end.

Вот, кстати, чего мне действительно не хватает, так это возможности объявить в interface секции только часть класса, а кучу технических деталей убрать в implementation. Что-то вроде partial классов в .net.
...
Рейтинг: 0 / 0
17.04.2021, 17:38
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063364
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
fraks
Если приведешь пример кода, может быть я бы понял о чем речь.

Имхо, в тот момент, когда ты писал эти строки в CFG, стоило остановиться и сделать их, например, массивом.
...
Рейтинг: 0 / 0
17.04.2021, 17:42
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063369
fraks
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
fraks
Участник
softwarer
fraks
Если приведешь пример кода, может быть я бы понял о чем речь.

Имхо, в тот момент, когда ты писал эти строки в CFG, стоило остановиться и сделать их, например, массивом.

И как потом к ним обращаться?
Вот конкретно этот код как будет выглядеть если параметры будут массивом?

Код: pascal
1.
2.
3.
4.
5.
6.
7.
  // обработаем ситуацию когда набор товаров выбранный для выделения - пропал. Скинем в 0
  if Item_GoodMark1.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA1 := 0; Item_GoodMark1.Caption := nosel; end;
  if Item_GoodMark2.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA2 := 0; Item_GoodMark2.Caption := nosel; end;
  if Item_GoodMark3.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA3 := 0; Item_GoodMark3.Caption := nosel; end;
  if Item_GoodMark4.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA4 := 0; Item_GoodMark4.Caption := nosel; end;
  if Item_GoodMark5.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA5 := 0; Item_GoodMark5.Caption := nosel; end;
  if Item_GoodMark6.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA6 := 0; Item_GoodMark6.Caption := nosel; end;
...
Рейтинг: 0 / 0
17.04.2021, 17:50
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063374
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
fraks
И как потом к ним обращаться?

Как к элементам массива.

fraks
Вот конкретно этот код как будет выглядеть если параметры будут массивом?

Код: pascal
1.
2.
3.
4.
5.
6.
7.
  // обработаем ситуацию когда набор товаров выбранный для выделения - пропал. Скинем в 0
  if Item_GoodMark1.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA1 := 0; Item_GoodMark1.Caption := nosel; end;
  if Item_GoodMark2.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA2 := 0; Item_GoodMark2.Caption := nosel; end;
  if Item_GoodMark3.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA3 := 0; Item_GoodMark3.Caption := nosel; end;
  if Item_GoodMark4.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA4 := 0; Item_GoodMark4.Caption := nosel; end;
  if Item_GoodMark5.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA5 := 0; Item_GoodMark5.Caption := nosel; end;
  if Item_GoodMark6.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA6 := 0; Item_GoodMark6.Caption := nosel; end;


Конкретно этот?

Код: pascal
1.
2.
3.
4.
5.
6.
7.
  // обработаем ситуацию когда набор товаров выбранный для выделения - пропал. Скинем в 0
  if Item_GoodMark1.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[1] := 0; Item_GoodMark1.Caption := nosel; end;
  if Item_GoodMark2.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[2] := 0; Item_GoodMark2.Caption := nosel; end;
  if Item_GoodMark3.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[3] := 0; Item_GoodMark3.Caption := nosel; end;
  if Item_GoodMark4.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[4] := 0; Item_GoodMark4.Caption := nosel; end;
  if Item_GoodMark5.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[5] := 0; Item_GoodMark5.Caption := nosel; end;
  if Item_GoodMark6.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[6] := 0; Item_GoodMark6.Caption := nosel; end;


Неужели без меня было сложно ответить на этот вопрос?
...
Рейтинг: 0 / 0
17.04.2021, 18:12
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063377
fraks
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
fraks
Участник
softwarer
fraks
И как потом к ним обращаться?

Как к элементам массива.

fraks
Вот конкретно этот код как будет выглядеть если параметры будут массивом?

Код: pascal
1.
2.
3.
4.
5.
6.
7.
  // обработаем ситуацию когда набор товаров выбранный для выделения - пропал. Скинем в 0
  if Item_GoodMark1.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA1 := 0; Item_GoodMark1.Caption := nosel; end;
  if Item_GoodMark2.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA2 := 0; Item_GoodMark2.Caption := nosel; end;
  if Item_GoodMark3.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA3 := 0; Item_GoodMark3.Caption := nosel; end;
  if Item_GoodMark4.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA4 := 0; Item_GoodMark4.Caption := nosel; end;
  if Item_GoodMark5.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA5 := 0; Item_GoodMark5.Caption := nosel; end;
  if Item_GoodMark6.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA6 := 0; Item_GoodMark6.Caption := nosel; end;


Конкретно этот?

Код: pascal
1.
2.
3.
4.
5.
6.
7.
  // обработаем ситуацию когда набор товаров выбранный для выделения - пропал. Скинем в 0
  if Item_GoodMark1.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[1] := 0; Item_GoodMark1.Caption := nosel; end;
  if Item_GoodMark2.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[2] := 0; Item_GoodMark2.Caption := nosel; end;
  if Item_GoodMark3.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[3] := 0; Item_GoodMark3.Caption := nosel; end;
  if Item_GoodMark4.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[4] := 0; Item_GoodMark4.Caption := nosel; end;
  if Item_GoodMark5.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[5] := 0; Item_GoodMark5.Caption := nosel; end;
  if Item_GoodMark6.Caption = '' then begin CFG.LT_GoodsMark_ID_LTA[6] := 0; Item_GoodMark6.Caption := nosel; end;


Неужели без меня было сложно ответить на этот вопрос?

Я думал что массив должен принести какую-то пользу, а принес один вред.
Где контроль границ массива?
Где инициализация этого массива?
Где сокращение кода, хотя бы в перспективе?
Количество строк такое же.
В чем смысл массива?

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

Код с конкретными переменными в большинстве случаев проверяется при компиляции. С массивом - неа, в рантайме извольте словить.
...
Рейтинг: 0 / 0
17.04.2021, 18:21
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063379
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
fraks
Я думал что массив должен принести какую-то пользу, а принес один вред.

И ты придумал эту фразу ещё до того, как задал вопрос

fraks
Где инициализация этого массива?

А где инициализация твоих переменных? Ты хоть какую-то логику пытаешься соблюсти?

fraks
Где сокращение кода, хотя бы в перпективе?

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

fraks
Количество строк такое же.
В чем смысл массива?

Смысл массива в том, что разработчик, когда начнёт писать подобные шесть строк - задумается и напишет получше.

fraks
Я бы еще понял если бы у меня контролы в виде массива были

И в чём проблема сделать их в виде массива? И, например, вместо шести строк ResetCodes, о которых шла речь выше, остаётся одна:

Код: pascal
1.
ResetCodes([Item_GoodMark1, Item_GoodMark2, Item_GoodMark3, Item_GoodMark4, Item_GoodMark5, Item_GoodMark6]);


fraks
Код с конкретными переменными в большинстве случаев проверяется при компиляции. С массивом - неа, в рантайме извольте словить.

Ты сам не первый час бьёшь себя пяткой в грудь о том, что их шесть, ровно шесть и никогда не будет иначе. Так какой тебе контроль нужен? Придумываешь "аргументы" без всякой логики, лишь бы звучали "весомо".
...
Рейтинг: 0 / 0
17.04.2021, 18:32
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063395
fraks
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
fraks
Участник
softwarer
fraks
Я думал что массив должен принести какую-то пользу, а принес один вред.

И ты придумал эту фразу ещё до того, как задал вопрос

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

От массива вижу только лишние ненужные заморочки.

С массивами фиксированного размера работал последний раз наверное в еще в фортране.
...
Рейтинг: 0 / 0
17.04.2021, 19:03
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063400
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
fraks
У меня первым пунктом идет понимание что любое действие должно приносить пользу.

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

fraks
С массивами фиксированного размера работал последний раз наверное в еще в фортране.

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

Код: pascal
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.
  
  ...
  protected
    function GetCell(ALine, ACol, AIndex: integer): integer;
    procedure SetCell(ALine, ACol, AIndex, AValue: integer);
    property Width[ALine, ACol: integer]: integer index 0 read GetCell write SetCell;
    property Carry[ALine, ACol: integer]: integer index 1 read GetCell write SetCell;
  ...

function TRulers.GetCell(ALine, ACol, AIndex: integer): integer;
begin
  Dec(ALine, MinLine);
  if (AIndex < Length(Cells)) and (ALine < Length(Cells[AIndex])) and (ACol < Length(Cells[AIndex][ALine]))
    then Result := Cells[AIndex][ALine][ACol]
    else Result := 0;
end;

procedure TRulers.SetCell(ALine, ACol, AIndex, AValue: integer);
begin
  Dec(ALine, MinLine);
  if Length(Cells) <= AIndex then SetLength(Cells, AIndex + 1);
  if Length(Cells[AIndex]) <= ALine then SetLength(Cells[AIndex], ALine + 1);
  if Length(Cells[AIndex][ALine]) <= ACol then SetLength(Cells[AIndex][ALine], ACol + 1);
  Cells[AIndex][ALine][ACol] := AValue;
end;
...
Рейтинг: 0 / 0
17.04.2021, 19:21
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063401
Фотография defecator
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
defecatorПривилегированный пользователь
Модератор форума
если в правой части ещё и пустой строкой отделить,
то правая часть будет просто замечательной
а левая часть - УГ
DimaBr
К сожалению весь код Delphi без использования Exit и с лишними begin/end
DelphiМогло бы быть
Код: pascal
1.
2.
3.
4.
5.
6.
7.
8.
9.
procedure TCustomForm.SetWindowMenu(Value: TMenuItem);
begin
  if FWindowMenu <> Value then
  begin
    FWindowMenu := Value;
    if Value <> nil then Value.FreeNotification(Self);
    RefreshMDIMenu;
  end;
end;

Код: pascal
1.
2.
3.
4.
5.
6.
7.
8.
procedure TCustomForm.SetWindowMenu(Value: TMenuItem);
begin
  if FWindowMenu = Value then Exit;

  FWindowMenu := Value;
  if Value <> nil then Value.FreeNotification(Self);
  RefreshMDIMenu;
end;

Код: pascal
1.
2.
3.
4.
5.
6.
7.
8.
procedure TCustomForm.SetPosition(Value: TPosition);
begin
  if FPosition <> Value then
  begin
    FPosition := Value;
    if not (csDesigning in ComponentState) then RecreateWnd;
  end;
end;

Код: pascal
1.
2.
3.
4.
5.
6.
7.
procedure TCustomForm.SetPosition(Value: TPosition);
begin
  if FPosition = Value then Exit;

  FPosition := Value;
  if not (csDesigning in ComponentState) then RecreateWnd;
end;

Код: pascal
1.
2.
3.
4.
5.
6.
7.
8.
9.
procedure TCustomForm.CMColorChanged(var Message: TMessage);
begin
  inherited;
  if FCanvas <> nil then
  begin
    FCanvas.Brush.Color := Color;
    Perform(CM_BORDERCHANGED, 0, 0);
  end;
end;

Код: pascal
1.
2.
3.
4.
5.
6.
7.
8.
procedure TCustomForm.CMColorChanged(var Message: TMessage);
begin
  inherited;
  if FCanvas = nil then Exit;

  FCanvas.Brush.Color := Color;
  Perform(CM_BORDERCHANGED, 0, 0);
end;

...
Рейтинг: 0 / 0
17.04.2021, 19:22
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063402
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
defecator
если в правой части ещё и пустой строкой отделить, то правая часть будет просто замечательной

Вот с этим не соглашусь. Я считаю, что каждый раз, когда внутри подпрограммы хочется написать пустую строку - есть решение лучше. Чаще всего - это комментарий, но в данном случае они излишни. Приведённые Димой варианты лучше, и незачем портить их пустыми строками.
...
Рейтинг: 0 / 0
17.04.2021, 19:26
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063431
Гаджимурадов Рустам
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гаджимурадов Рустам
Участник
softwarer> Да. И по этой причине мне не нравится тенденция
softwarer> напихивать в декларации классов всякие хрени в java-стиле.
softwarer> Лучше сохранять компактные декларации классов

Иногда удобно всё вместе, без разделения на декларацию и реализацию.
В любом случае, лучше иметь возможность, чем не иметь её (в т.ч. ту, о
которой говоришь ты).
Posted via ActualForum NNTP Server 1.5
...
Рейтинг: 0 / 0
17.04.2021, 20:43
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063440
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
Гаджимурадов Рустам
Иногда удобно всё вместе, без разделения на декларацию и реализацию.
В любом случае, лучше иметь возможность, чем не иметь её

К сожалению, не в любом.

Я был бы согласен с этим утверждением при условии, что с инструментом работают достаточно компетентные люди, которые не творят совсем уж глупостей. Но в ситуации, когда фича, с одной стороны, не даёт заметных преимуществ при грамотном использовании, а с другой - "заставь дурака богу молиться"... Лучше не давать фичи, главным и единственным результатом которой будут новые тонны говнокода.
...
Рейтинг: 0 / 0
17.04.2021, 21:13
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063460
white_nigger
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
white_nigger
Участник
Гаджимурадов Рустам
Иногда удобно всё вместе, без разделения на декларацию и реализацию.
В любом случае, лучше иметь возможность, чем не иметь её (в т.ч. ту, о
которой говоришь ты).
+1
...
Рейтинг: 0 / 0
17.04.2021, 21:54
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063464
Фотография defecator
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
defecatorПривилегированный пользователь
Модератор форума
softwarer
defecator
если в правой части ещё и пустой строкой отделить, то правая часть будет просто замечательной

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


ты что, экономишь строки ?
может, ты ещё и комментарии экономишь ????
...
Рейтинг: 0 / 0
17.04.2021, 22:14
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063479
ъъъъъ
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
ъъъъъ
Гость
fraks
под WnXP где этого PowerShell нету.

https://www.microsoft.com/ru-ru/download/details.aspx?id=16818
...
Рейтинг: 0 / 0
17.04.2021, 22:57
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063483
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
defecator
ты что, экономишь строки ?

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

На всякий случай оговорюсь, что здесь я не имею в виду ассемблер, там зачастую фрагменты стоило разделять пустыми строками парой пустая строка - комментарий. Ну и пустые строки следует использовать в секции деклараций подпрограммы, если там есть вложенные подпрограммы.
...
Рейтинг: 0 / 0
17.04.2021, 23:02
| Ответить | Цитировать | Написать  
Флейм про оформление и begin-end
    #40063490
rgreat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rgreat
Участник
У меня пустая строка разделитель логических блоков внутри процедуры.
И наличие комментария не отменяет пользы пустой строки перед ним.

Между процедурами - 3 пустых строки.
...
Рейтинг: 0 / 0
17.04.2021, 23:47
| Ответить | Цитировать | Написать  
25 сообщений из 261, страница 6 из 11
  6  все
Форумы / Delphi [игнор отключен] [закрыт для гостей] / Флейм про оформление и begin-end
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]