SpellWork C#

[LordJZ]

Мы тут с Konctantin начали переписывать SpellWork на C#.

Репозиторий тут: http://github.com/LordJZ/spellwork_cs/

Пока что реализованы не все функции оригинального spell_work-а, но вроде бы все работает.
Скриншоты: (пока что не все реализовано)

Ссылка на скомпилированный файл во вложении.
Пожалуйста пишите свои пожелания и баг-репорты.

[Konctantin]

Ждем баг-репортов и пожеланий

[xmolex]

Хотел спросить, имена в 3.3.3 в Spell.dbc уже не используются? Использую enGB. Вроде всегда имена были, а тут не видно.

[Konctantin]

у нас тут автодетект, проверялось только на русских, щас проверю не английски

====
Поправлено, спасибо.

[TOM_RUS]

Небольшой патчик http://paste2.org/p/778934, добавлен sln файл для VS2010 (компиляция в VS2008 не проверялась, теоретически должно компилится с варнингами о неподдерживаемой версии .NET Framework), перенесена часть кода в класс Extensions...

[xmolex]

Сообщение от Konctantin

Поправлено, спасибо.

Если не сложно, можно бинарник, а то C# не стоит.

[TOM_RUS]

Кстати, содержимое файла Utility.cs тоже можно переделать в Extensions (хотя эти методы все равно не используются)...

Сообщение от xmolex

Если не сложно, можно бинарник, а то C# не стоит.

А что мешает поставить? Если VS установлена (не Express), то даже места на винте дополнительного не потребуется...

[Konctantin]

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

исходник прилепил

[TOM_RUS]

Сообщение от Konctantin

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

Ну хотя бы для того чтобы компилятор не ругался

[Konctantin]

Есть один баг, не правильно отображаются Stances и Not Stances, пока не знаю как поправить

[TOM_RUS]

В GenerateSpellDesc не лучше ли заменить str += ... на StringBuilder? А то выглядит ужасно

            return String.Format("SpellRange: ID - {0} {1} (unk = {2}) MinRange = {3}, MinRangeFriendly = {4}, MaxRange = {5}, MaxRangeFriendly = {6}\r\n",
                q.Key, q.Value.Description1, q.Value.Description2, q.Value.Field5, q.Value.MinRange,
                q.Value.MinRangeFriendly, q.Value.MaxRange, q.Value.MaxRangeFriendly);

Неверное количество аргументов.

        static String GetFormInfo(ulong val, string name)
        {
            int i = 1;

Fix для Stances

        public ulong Stances;                                     // 12-13    m_shapeshiftMask
        public ulong StancesNot;                                  // 14-15    m_shapeshiftExclude

структура

"\r\n"

можно заменить на Environment.NewLine

public static StringBuilder AppendFormatLine(this StringBuilder sb, string format, params object[] args)
{
    return sb.AppendFormat(format, args).AppendLine();
}

Extension метод, может пригодиться при замене "str += ..." на StringBuilder...

Для

""

есть readonly константа String.Empty либо метод String.IsNullOrEmpty(string str). При желании можно добавить Extension метод чтобы можно было делать
так:

var str = "abc";
if(str.IsNullOrEmpty())
{
   ...
}

вместо

var str = "abc";
if(String.IsNullOrEmpty(str))
{
   ...
}

это как кому больше нравится...

В файле Structure.cs опечатка curent -> current

GetAuraModTypeName()
GenerateSpellDesc()
GetProcInfo()
GetTriggerSpell()
...

и другие подобные методы, почему бы их не перенести в структуру SpellEntry...

var spell = (from s in DBC.Spell where s.Key == spellId select s.Value).First();

зачем тут LINQ вообще? Можно просто var spell = DBC.Spell[spellId] сделать...

sb.Append(Environment.NewLine);

можно просто заменить на sb.AppendLine()

sb.AppendFormat("CasterAuraState 0x{0:X8}, TargetAuraState 0x{1:X8}\r\n", spell.CasterAuraState, spell.TargetAuraState);

и подобные заканчивающиеся на "\r\n" не предлагаемый выше Extension method "public static StringBuilder AppendFormatLine(this StringBuilder sb, string format, params object[] args)"

[Konctantin]

Поправил, правда не все, оставил еще LordJZ, пускай и он поклацает.

[TOM_RUS]

Еще патчик http://paste2.org/p/779518.

[LordJZ]

Сообщение от TOM_RUS

Еще патчик http://paste2.org/p/779518.

Спасибо.

[TOM_RUS]

http://paste2.org/p/779641

[Konctantin]

Может права на репозитарий лучше у LordJZ взять?

[TOM_RUS]

http://paste2.org/p/779792

[Konctantin]

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

Спасибо Kirix за замечание.

Сделал, вот как это будет примерно выглядеть: осталось только расставить стили и выделить важную информацию.

[Konctantin]

Я зашел в тупик, не знаю от куда и как берутся некоторые поля для таблицы spell_proc_event, а именно:
-ProcFlagEx - подозреваю что это spell.ProcFlags - но не уверен
-PPMRate - даже не представляю

с каких полей dbc составляются эти данные?

и чуточку не уверен в Cooldown, так что нужна помощь, или хотя бы намек или пинок в нужном направлении.

Вот пока последняя на данный момент скомпилированая версия SpellWork

[Laise]

Сообщение от Konctantin

Я зашел в тупик, не знаю от куда и как берутся некоторые поля для таблицы spell_proc_event, а именно:
-ProcFlagEx - подозреваю что это spell.ProcFlags - но не уверен
-PPMRate - даже не представляю

с каких полей dbc составляются эти данные?

и чуточку не уверен в Cooldown, так что нужна помощь, или хотя бы намек или пинок в нужном направлении.
SpellWork

procEx - из enum ProcFlagsEx в мангосе, в дбс этого нет
PPMRate - шанс прока в зависимости от скорости оружия, в старом спеллворке было просто поле для ввода
Cooldown - тоже было просто поле для ввода

[Konctantin]

Cooldown - я взял spell.RecoveryTime / 1000f - но если не устраивает можем поставить свое

PPMRate, procEx - это то чего нету в dbc но то что мы хотим видеть.

Спасибо, вот я и получил тот пинок в нужном направлении.

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

Какие есть у кого пожелания, что бы вы еще хотели видеть?

[Laise]

Сообщение от Konctantin

Какие есть у кого пожелания, что бы вы еще хотели видеть?

поиск по spellIconID и по attributes/attributesEx(было в старом)

[Konctantin]

Готово http://github.com/LordJZ/spellwork_c...6ee711b73fdc60

[LordJZ]

Кстати да, поиск было бы классно расширить и спрятать в отдельный блок

[Konctantin]

У меня опять вопрос, как для таблицы spell_proc_event надо брать данные по столбцам SpellFamilyMask0, SpellFamilyMask1, SpellFamilyMask2, они берутся прямо из dbc и не меняются или как-то генерируются?

Просто не нашел в исходниках СпеллВорка как это дело обрабатывается

[Konctantin]

Реализована работа с базой данных для таблицы spell_proc_event (пока немного кривовато, надо проверок натыкать).
Остался открытым вопрос с моего предыдущего поста. Необходимо проверять, но я сейчас не могу, сделал все чисто теоретически.

Прикрепил последнюю скомпилированную версию, там же и MySql.Data.dll для работы с базой данных.
http://filekeeper.org/download/share...3a__11723_.rar

[Konctantin]

+ Добавил фильтр на выборку из базы данных
+ Добавлено несколько проверок для роботы с базой данных
+ Добавил форму для подсчета флагов
+ Добавил бинарные файлы "SpellWork.exe" и "MySql.Data.dll" в репозитарий
скачать можно тут http://github.com/LordJZ/spellwork_c...k/bin/Release/

[Konctantin]

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

[PSZ]

Предлагаю сделать не одну вкладку Spell Info, а две - удобно сравнивать спеллы.

[LordJZ]

А лучше сделать вкладку Compare Spell

[Konctantin]

Я уже работаю над этим, хочу сделать что-то на подобии TortoiseIDiff, с 2 полями, и выделение цветом разбежностей (красным) и совпадений (зеленым)

[Konctantin]

Предлагаю сделать не одну вкладку Spell Info, а две - удобно сравнивать спеллы.

По идее вкладки "Spell Info" и "Spell Proc Event" идентичны, информация что в них выводится одинаковая, а отборы независимые.
Если мешают дополнительные панели во вкладке "Spell Proc Event" - их легко можно сдвинуть, и они не будут мешать.

[Konctantin]

Помогите с задачей, для реализации расширенного фильтра в SpellWork
Есть структура:

public struct SpellEntry
{
      public uint ID;
      ....
};

как мне представить структуру как массив полей (для того чтобы вывести в комбобокс и пользователь мог сам выбирать поле структуры и присваивать ей значение)

Я частично решил проблему, чтобы заполнить ComboBox нужными данными из структуры сделал такой метод расширения:

public static void SetStructFields<T>(this ComboBox cb) where T : struct
{
	cb.Items.Clear();

	DataTable dt = new DataTable();
	dt.Columns.Add("ID");
	dt.Columns.Add("NAME");

	var type = typeof(T).GetMembers();
	int i = 0;
	foreach (var str in type)
	{
		dt.Rows.Add(new object[] 
		{ 
			str, 
			String.Format("({0:000}) {1}", i, str.Name) 
		});
		i++;
	}

	cb.DataSource    = dt;
	cb.DisplayMember = "NAME";
	cb.ValueMember   = "ID";
}

Но появился второй вопрос, как можно теперь применить это в запросе LINQ?
все данные у меня хранятся в Dictionary<uint, SpellEntry> Spell;
Что я имею ввиду: скажем есть такой простенький запрос:

var query = from s in DBC.Spell.Value where s.SpellName == val select s;

как сделать чтобы в запрос вставить переменную из ComboBox
чтобы было что-то на подобии:

var field = ((ComboBox)sender).SelectedValue;
var query = from s in DBC.Spell.Value where s[field] == val select s;

Естественно так как я написал - не работает, по сему и прошу помощи.

И возможно ли вообще присвоение переменной в єтом участке кода:

DBC.Spell.Value where s[field]

Много гуглил но так и не нашел способа реализации данной задачи, возможно ответ очевиден, но ...

[Konctantin]

пока ничего серьезного не делал, но в последней версии ускорил загрузку программы, теперь ДБЦ читаются быстрее.
Так же должна показываться ошибка, если структура не подходит к ДБЦ файлу.
От нечего делать добавил иконки в SpellFamilyTree

[Konctantin]

Сделал сравнение спелов, сравнение идет построчно:

[Konctantin]

Есть просьба: залейте кто нибудь ДБЦ с клиентов 112 и 243

[MangMan]

Вот такая вот ошибка

Установлен .NetFramework 4
(Ранние версии NetFramework отсутствуют)

[Konctantin]

я собираю на 3.5 .NetFramework, попробуйте установить его.
Да и там еще библиотека для работы с базой данных, вы ее скачали?
http://github.com/LordJZ/spellwork_c...MySql.Data.dll

[Konctantin]

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



Спасибо всем за идеи, жду новых пожеланий.

[crAwling]

Можно ли прок перенести в спел прок ивент, как было ранее сделано в старом спелл ворке?
И не могу разобраться, как выбрать маску для прока?