Стоит ли серьёзно относится к исходникам, где встречается

← →
Германн © (2008-07-03 01:12) [0]

что-то подобное такому:
strPath:=ExtractFileDir(Application.ExeName)+"\";
?

← →
vuk © (2008-07-03 01:20) [1]

А что здесь не нравится-то?

← →
Германн © (2008-07-03 01:24) [2]

> vuk © (03.07.08 01:20) [1]

Ну есть же функция ExtractFilePath.

← →
McSimm © (2008-07-03 01:28) [3]

А не мелочи ли это ? Все функции знать сложно.
Только по этому фрагменту я бы не стал судить.

← →
sniknik © (2008-07-03 01:28) [4]

т.е. перефразируя вопрос -
стоит ли доверять человеку если он знает одну функцию, а другую (ExtractFilePath) по какойто причине пропустил при изучении. так?

а исходники то рабочие? делают то что от них ожидают? может этому больше внимания уделить?

← →
sniknik © (2008-07-03 01:33) [5]

> Ну есть же функция ExtractFilePath.
ну, к примеру, есть и такая функция inc, и тем не менее я всегда пишу i:= i + 1; что, надо уволить меня за это?

← →
Тын-Дын © (2008-07-03 01:34) [6]

> и тем не менее я всегда пишу i:= i + 1;

А почему? Религия?

← →
vuk © (2008-07-03 01:34) [7]

Ну, если в этом смысле, то я думаю, что относиться можно нормально. Вот то, что использовано Application.ExeName, делает код завязанным на Application... Хотя, если это с VCL-формами использовать, то пофиг.

И вообще, код вида ExtractFileDir(ParamStr(0)) + "\" есть в VCL. :)

← →
Хохол (2008-07-03 01:45) [8]

Если бы он вместо Application.ExeName написал ParamStr(0) вам бы полегчало?

Я тоже часто использую такую конструкцию
WideIncludeTrailingPathDelimiter(WideExtractFilePath(Application.ExeName)) + какой-то там путь

← →
sniknik © (2008-07-03 01:51) [9]

> А почему? Религия?
нет. привычка, и мне это кажется нагляднее. уж не знаю, что из этого причина, а что следствие.

← →
Германн © (2008-07-03 01:57) [10]

> Только по этому фрагменту я бы не стал судить.
>

Это только начало.
Начал сегодня "причёсывать" некий проект удалённого ЖЗЧ.
.

> sniknik © (03.07.08 01:28) [4]

Именно.

← →
Германн © (2008-07-03 02:09) [11]

> sniknik © (03.07.08 01:28) [4]

Моё "именно" относилось к первой части поста [4].
Вторая часть нуждается в серьёзной переработке. Но ЖЗЧ тут ни при чём. Ему не совсем правильно сформулировали задачу.

← →
Германн © (2008-07-03 02:22) [12]

> vuk © (03.07.08 01:34) [7]

> И вообще, код вида ExtractFileDir(ParamStr(0)) + "\" есть
> в VCL. :)
>

Хм. Действительно есть.
Борланд потерял ещё несколько пунктов!
:)

← →
Anatoly Podgoretsky © (2008-07-03 07:18) [13]

> Германн (03.07.2008 1:57:10) [10]

ЖЗЧ = Жизнь Замечательных Частей

← →
Anatoly Podgoretsky © (2008-07-03 08:58) [14]

> Борланд потерял ещё несколько пунктов!

Не вижу оснований, в отличии от
IncludeTrailingPathDelimiter(ExtractFilePath(Application.ExeName))

ExtractFileDir(ParamStr(0)) + "\" это равно ExtractFilePath(ParamStr(0)),
то есть равноценные конструкции, ну может вторая на пару микросекунд быстрее. И еще надо помнить, что Dir без завершающего слеша, а Path с ним.
Есть еще одна тонкость, но обсуждать в отрыве от реального кода не стоит.

← →
Игорь Шевченко © (2008-07-03 09:48) [15]

> я всегда пишу i:= i + 1; что, надо уволить меня за это?

конечно уволить

← →
Sha © (2008-07-03 10:04) [16]

> Anatoly Podgoretsky © (03.07.08 08:58) [14]
> ExtractFileDir(ParamStr(0)) + "\" это равно ExtractFilePath(ParamStr(0)),

Не всегда равно.
Например, для корня диска первое выражение даст два слеша в конце.

> Игорь Шевченко © (03.07.08 09:48) [15]
> конечно уволить

Когда рядом мнгого Inc/Dec смысл кода мне уловить сложнее.
И меня увольняй :)

← →
Дуб © (2008-07-03 10:07) [17]

> конечно уволить

Жалкая попытка казаться суровым. Я бы расстрелял.

← →
clickmaker © (2008-07-03 10:36) [18]

а тому, кто в Дельфи пишет венгерской записью strPath - руки оторвать
o)

← →
Anatoly Podgoretsky © (2008-07-03 10:42) [19]

> Игорь Шевченко (03.07.2008 9:48:15) [15]

Может подождать?
А вот когда начнет писать так, то тогда не раздумывая.

i:= 1 + i;

← →
Anatoly Podgoretsky © (2008-07-03 10:43) [20]

> Sha (03.07.2008 10:04:16) [16]

Я же сделал эту оговорку, что бы меня не укоряли в этом. Цитата:

> Есть еще одна тонкость, но обсуждать в отрыве от реального кода не стоит.

Я вообще то расчитывал на понимание :-)

> Например, для корня диска первое выражение даст два слеша в конце.
это не страшно
var F: TextFile; S: string; begin AssignFile(F, "C:\\boot.ini"); Reset(F); Readln(F, S); Edit1.Text := S; CloseFile(F); end;

Я вообще не знал, что существует свойство Application.ExeName.
Всю жизнь использую ParamStr(0).
Раньше использовал ExtractFileDir (как в сабже, но с if-ом, проверяющим, нет ли уже слеша в конце), потом стал использовать ExtractFilePath, не помню почему, возможно чтобы избежать этот if. Сам if, кажется, связан был с ситуацией в корне диска.

Серьезно относиться вообще ни к чему не стоит.

> я всегда пишу i:= i + 1; что, надо уволить меня за это?

я тоже так пишу, если предполагаю, что возможно и + 2.
для меня inc() прибавление только единицы, и вариант inc(i, 2) меня пугает.

i = ++i++
:)

← →
Гаврик (2008-07-03 13:48) [26]

> KilkennyCat (03.07.2008 13:00:24) [24]

Прими таблетки для храбрости - Озверин

> > sniknik ©
>
>
> > это не страшно
>
>
> От оси зависит. Под 9x - страшно

Как сказать, проводник не согласен с формой C:\\Temp

Серега, ты явно переработал - зайди в гантелик, возьми пивка, расслабься, посмотри на шастающих туда-сюда ногасто-грудастых созданий - и люди потянутся к тебе ;)

Стоит ли серьёзно относится к исходникам, где встречается Найти похожие ветки