Комментарии на: .NET-сборки для AutoCAD разных версий, часть 2

От: Андрей

Андрей — Mon, 13 Apr 2015 13:34:43 +0000

Так, в поисках нужного pdf-файла наткнулся на эту заметку. Сразу, пока помню - вот мои мультики по обозначенной теме:

первый: https://www.youtube.com/watch?v=Xm3vgnsju0s
второй: https://www.youtube.com/watch?v=Iey1REsvYrw
третий: https://www.youtube.com/watch?v=4zjJXHpDYe4

От: Андрей

Андрей — Mon, 09 Jun 2014 17:13:05 +0000

>Решение с директивами, как мне кажется, поднимает другой вопрос – понадобится а) менять версию .NET и б) менять выходной каталог. Лично я практически гарантирую, что рано или поздно что-нибудь да будет забыто. Честно говоря, я не понял каким образом к директивам препроцессора относятся пункты "а)" и "б)"... Директивы препроцессора предназначены для выполнения условной компиляции. Они никак не связаны ни с "а)", ни с "б)". Забыто ничего не будет, т.к. если что-то будет забыто, то тот проект, в котором ты что-то забыл, попросту не скомпилируется. При этом компилятор тебе точно покажет проблемное место. Многократно проверено на практике.

От: Дима_

Дима_ — Tue, 24 Dec 2013 17:12:10 +0000

Насчет GetAcadVersion... Это Вы ребята "круто" загнули, ну вариант Крыса с учетом, что версии они могут переименовывать черт знает как - чтоб потом не вспоминать где тут, что - это еще куда не шло (не думая добавил проверку версии) - не красиво, но надежно, но у Андрея со словарями - это по моему перебор. Получить из одного другое - 1 формула на уровне 5-го класса - 1 строка (F# - на С# все то-же самое - просто под рукой нет, чтоб не напутать с операторами), еще строка перевода из строка-число-строка (а можно и в той-же написать).
let Test x=(x/10-17)*3+x%10+2007 - все

От: Andrey

Andrey — Sun, 01 Dec 2013 18:26:49 +0000

Выкладываю обещанные мною замечания по программному коду:
1. На мой взгляд, нет особой необходимости в функции getAcadProfileName(), т. к. не составляет особого труда в коде напрямую проверять значение системной переменной. Хотя, пожалуй, это дело вкуса...
2. Функцию getAcadVersion я бы переписал так:

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

public static String getAcadVersion(String coreVersion) {
Int32 minMajor = 17;
Int32 maxMajor = 19;
Int32 year = 2007; // The start value is the year of the [minMajor].0 core version.
Dictionary<String, String> dict = new Dictionary<String, String>();
for (Int32 major = minMajor; major <= maxMajor; ++major) {
for (Int32 minor = 0; minor < 3; ++minor) {
dict.Add(String.Format("{0}.{1}", major, minor), year.ToString());
++year;
}
}
String result = dict.ContainsKey(coreVersion) ? dict[coreVersion] : String.Empty;
return result;
}

В коде я исхожу из предположения, что Autodesk не станет нарушать своё правило, согласно которому назначает нумерацию версий на протяжении уже многих лет. Если в будущем правило будет изменено, то в этот код потребуется внести учитывающие этот момент изменения.
Твой вариант с использованием case гораздо менее гибок, т. к. ты в нём жёстко зашил конкретные значения. Мой вариант вычисляет все значения автоматически. Позднее ты можешь изменить значение maxMajor на другое число, тем самым автоматически расширив диапазон вычисляемых значений.
3. Необходимости в функции getAcadVersionWithBit(), так же как и в функции getAcadProfileName() я не вижу, поскольку разрядность AutoCAD всегда та же, что и разрядность операционной системы. В виду этого разрядность можно определять так:

1	String platform = IntPtr.Size == 4 ? "x86" : "x64";

Объединять год с разрядностью, можно либо просто конкатенацией строк (что ты и делал в своём коде), либо следующим образом (предпочитаемый мною вариант):

1	String.Format("{0}{1}", year, platform); // the 'year' is String

4. Нет никакой необходимости в наличии функций getPathAppData() и getPathTemp(). Искомую информацию можно получить например так:

1 2	String appdata = Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData); String temp = Environment.ExpandEnvironmentVariables("%temp%");

Посмотри перечисление Environment.SpecialFolder - много интересных вариантов найдёшь, а с помощью Environment.ExpandEnvironmentVariables всегда можно развернуть строку, содержащую в своём составе имя системной переменной.
Примечание: Кстати, эти способы повсеместно используются в моём коде на старом сайте, странно, что ты этого не видел, если конечно вообще смотрел мой код...
5. Ты создал два CS-файла: TextObjects2009.cs и TextObjects2013.cs содержимое которых отличается всего лишь несколькими строчками. Т. о. налицо дублирование существенного объема кода и в случае, если тебе потребуется вносить изменения в код - это придётся делать два раза (или столько, сколько ты создашь почти идентичных CS-файлов), о чём я тебе уже писал ранее по Skype.

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

Именно поэтому следует использовать директивы препроцессора. Покажу это на переделанном мною примере одного твоего метода (остальные делаются по аналогии):

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

#define newer_than_2009 // Comment this row for AutoCAD 2009 and older
using Db = Autodesk.AutoCAD.DatabaseServices;
...
public static Db.ObjectId setTextStyle(Db.MText ent, String StyleName) {
Db.ObjectId result = Db.ObjectId.Null;
if (ent == null || StyleName.Trim() == String.Empty) return result;
Db.Database db = ent.Database;
using (Db.Transaction tr = db.TransactionManager.StartTransaction()) {
Db.TextStyleTable txtStyleTable = (Db.TextStyleTable)tr.GetObject(
db.TextStyleTableId, Db.OpenMode.ForRead);
if (txtStyleTable.Has(StyleName)) {
Db.TextStyleTableRecord txtStyleTableRec = (Db.TextStyleTableRecord)tr
.GetObject(txtStyleTable[StyleName], Db.OpenMode.ForRead);
ent.UpgradeOpen();
result = txtStyleTableRec.ObjectId;
#if newer_than_2009
ent.TextStyleId = result;
#else
ent.TextStyle = result;
#endif
ent.DowngradeOpen();
}
tr.Commit();
}
return result;
}
...

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

ВНИМАНИЕ! В данном случае я объявил символ посредством строки:

1	#define newer_than_2009 // Comment this row for AutoCAD 2009 and older

но в данном коде я это делаю лишь для наглядности! Грамотным решением является определение символа (в нашем случае это newer_than_2009) не в коде, а в настройках проекта: поле "Conditional Compilation Symbols" на вкладке "Build". Если потребуется, то там можно объявлять более одного символа, отделяя их друг от друга пробелами.
Т. о. каждый твой проект, настройки которого ориентированы на конкретную версию AutoCAD, по этим символам сам определит какие именно строки кода ему следует компилировать, а какие нет.
В составе одного решения (solution) иметь несколько проектов (project), каждый из которых имеет свои специфичные для конкретной версии AutoCAD настройки (символы, версия .NET, целевой каталог построения, разрядность, подключаемые библиотеки), но при этом общие для всех проектов (project) исходники - это, очень хорошая идея. Она проста в реализации, легка в использовании и достаточно надёжна.

От: Кулик Алексей aka kpblc

Кулик Алексей aka kpblc — Fri, 29 Nov 2013 18:50:49 +0000

Удалил. Решение с директивами, как мне кажется, поднимает другой вопрос - понадобится а) менять версию .NET и б) менять выходной каталог. Лично я практически гарантирую, что рано или поздно что-нибудь да будет забыто. Именно поэтому я сделал так, как сделал
Насчет всего остального - спасибо, возьму на вооружение

От: Андрей

Андрей — Fri, 29 Nov 2013 16:47:40 +0000

Удали моё предыдушее сообщение - там сбилось форматирование кода. Исправил это и дублирую заново:

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

2. Функцию getAcadVersion я бы переписал так:

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

3. Необходимости в функции getAcadVersionWithBit(), так же как и в функции getAcadProfileName() не вижу. Разрядность AutoCAD всегда та же, что и разрядность операционной системы. Поэтому разрядность можно определять так:

1	String platform = IntPtr.Size == 4 ? "x86" : "x64";

Объединять год с разрядностью, можно либо просто конкатенацией строк(что ты и делал), либо так (предпочитаемый мною вариант):

1	String.Format("{0}{1}", year, platform); // year is String

1 2	String appdata = Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData); String temp = Environment.ExpandEnvironmentVariables("%temp%");

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

5. Ты создал два CS-файла: TextObjects2009.cs и TextObjects2013.cs содержимое которых отличается всего лишь несколькими строчками. Т. о. в случае, если тебе потребуется вносить изменения в код - это придётся делать два раза, о чём я уже писал выше. При этом нужно будет стараться не забыть внести все изменения, не пропустив ничего, что порой может быть затруднительно, если изменений было много (есть шанс что-то упустить). Именно поэтому следует использовать директивы препроцессора. Покажу это на примере одного твоего метода (остальные делаются по аналогии):

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

От: Кулик Алексей aka kpblc

Кулик Алексей aka kpblc — Fri, 29 Nov 2013 13:10:41 +0000

Пиши, я ж не спец по .NET. Считай это чуть ли не первым моим экспериментом
P.S. Я повторяю - у меня не получилось влегкую поменять ссылки на *mgd.dll, а также версию .NET. В результате что есть, то есть.

От: Андрей

Андрей — Fri, 29 Nov 2013 12:55:15 +0000

>Переименовываем проект (не решение!)
А почему бы сразу не задать решению одно имя, а проекту другое?

Не всегда понятно, какой код у тебя к какому файлу относится... Например первый код, как я понимаю, относится к файлу kpblcCommon.cs. Неплохо было бы первой строкой кода, в исходном файле, обозначать имя этого файла, как это зачастую делают C++ программисты. Тогда читая код однозначно понимаешь, что это за файл.

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

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

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