[micro] Проверка ScriptDev2 скриптов

[LordJZ]

Эта небольшая программка, написанная на C# под .NET Framework 3.5, проверяет исходники ScriptDev2 скриптов на очевидные ошибки.

В данный момент умеет:

• проверять полноту ScriptLoader.cpp, парся каждый файл в директории scripts на наличие функций AddSC_xxx.

Планируется в будущем:

• проверка на валидность перегруженных функций ScriptedAI, ScriptedInstance и др.
• предупреждения об использовании Unit::GetUnit, а так же Map::GetUnit в неподобающих местах
• сверка файлов проектов MSVS и Makefile и всех скриптов из scripts

Скриншот: http://dl.dropbox.com/u/9241118/sd2_scriptcheck_ss.gif

Отзывы, идеи, предложения и критика приветствуются!

____________________________________________

Список изменений:

• Версия 1.0.0: DropBox | Filebeam
  • Реализован парсинг ScriptLoader.cpp и скриптов из ./scripts с очисткой от комментариев, сравнение called, externed и defined загрузчиков скриптов.
• Версия 1.1.0: DropBox | Filebeam
  • Код программы разложен по нескольким файлам, добавлены новые классы и методы для работы с C++ файлами.
  • Реализована запись данных из консоли в лог файл SD2_ScriptCheck.log.
  • Добавлены цвета в консоли: красный для ошибок, желтый для предупреждений.
  • Добавлена новая проверка на наличие PlayerTalkClass->ClearMenus перед вызовами AddMenuItem и SendGossipMenu в функциях GossipSelect.

____________________________________________
Последняя версия на данный момент: 1.1.0
Исходный код: http://github.com/LordJZ/MyWowTools/...2_ScriptCheck/
Последний бинарник: DropBox | Filebeam

[Lordronn]

Вот идеи, в списке "Что нужно сделать" я их не увидел:
1.Проверка на закрыты ли скобки
2.Правильное ли кол-во скобок в условии
3.Стоит ли после функции ";"
4.Верный ли файл подключен
5.Проверка на наличие запятой в enum'e
6.Проверка условий на кол-во знака равенства

[KiriX]

Клёвая штука! Спасибо!

[LordJZ]

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

Вот идеи, в списке "Что нужно сделать" я их не увидел:
1.Проверка на закрыты ли скобки
2.Правильное ли кол-во скобок в условии
3.Стоит ли после функции ";"
4.Верный ли файл подключен
5.Проверка на наличие запятой в enum'e
6.Проверка условий на кол-во знака равенства

Это общие проверки синтаксиса, программа не для этого сделана. Ее назначение — проверять очевидные ошибки при создании скриптов специально для SD2.

[Lordronn]

Сделай проверку на не объявленные идендефикаторы
Проверку на наличие подключенных файлов(хедеров) сделай

а для тринити такого не будет?

[Konctantin]

Сделай проверку на не объявленные идендефикаторы
Проверку на наличие подключенных файлов(хедеров) сделай

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

а для тринити такого не будет?

Ну я бы не делал, ведь мы не тринити...

[LordJZ]

Еще раз, эта программа делает проверку тех важнейших аспектов скриптования в SD2, проверку коих не делает ни компилятор, ни система управления версиями.

К примеру, это — правильность и полнота загрузки существующих в репозитории С++ скриптов. Вы, написав скрипт для SD2, можете забыть добавить его в ScriptLoader.cpp, и эта программа выкинет вам ошибку.
Еще — валидность перегрузчиков функций (планируется). Это, по-большому счету, проверка «на дурака» — то есть, проверка на изменение по незнанию или по невнимательности — предупреждение случаев, когда оверлоад декларирован некорректно, оптимизирован-удален компилятором и не будет вызван никогда. Такие случаи легко дебажатся, но некоторые могут быть не знакомы с такими особенностями компиляторов. Кстати, в C# эта проверка встроена в MS компилятор.

[rsa]

Добавь проверку на наличие playertalkclass->clearmenu() в gossipselect перед выдачей нового меню. хотя это латентная ошибка (и ей страдает почти весь SD2 team), но ей можно ядро положить при умении.

[Konctantin]

А если сделать так:
Это изначально виновс-форм приложение, в котором есть такой контрол как ListView или TextBox (Стек ошибок) в который будут складываться "ошибки".
Стек ошибок представляет собой перечень ошибок с ссылками на указанный файл (по возможности и место ошибки, номер строки...) и ДаблКликом возможность открыть этот файл.
Вот только как сделать еще и установку курсора в предполагаемом месте с ошибкой.

[LordJZ]

Пока это будет просто консольное приложение. В промежуточной версии сделал запись в лог файл и раскраску консоли.

[Lordronn]

Вот то, на что компилятор не ругнется, и то, что положит сервер.Правится элементарно, но уже после краша. Когда используешь таргеты, делаешь по ним проверку, зачастую забываешь написать проверку на то, что это таргет - if(pTarget && pTarget->isAlive), и пишешь без проверки - if(pTarget->isAlive()) что приведет к крашу

что это таргет

Это проверка не "что это таргет", а что указатель не указывает на NULL.

Извиняюсь за занудство.

[Йоха]

используйте так называемые "умные указатели", не будет никогда засады с нулевыми указателями

[Konctantin]

а нельзя ли по подробнее что это за "умные указатели" Впервые слышу о таких.

Гугл все рассказал

[virusav]

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

[Lordronn]

Можно еще проверку на наличие скрипта в файле проэкта сделать.

Компилятор сам сообщит.

[Lordronn]

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

[Йоха]

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

Гугл все рассказал

я в общем-то имел ввиду следующую фишку умных указателей:

PHP код:

template <class Type>
class Pointer
{
private:
    Type* ptr;
    static Type* ptr2;
public:
    Type* operator->()
    {
        if (ptr) return ptr
        else return ptr2;
    } 


то есть когда будет попытка обратиться к указателю, который NULL, то возвратится не NULL, а указатель на некий объект.

Подробнее про это дело можно почитать у Джеффа Элджера "Библиотека программиста с++"

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

[Йоха]

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

[Йоха]

вот например так

PHP код:

#include <iostream>
using namespace std;

template <class Type>
class Pointer
{
private:
    Type* ptr;
    static Type* ptr2;
public:
    Pointer(): ptr(NULL) {}
    Pointer(Type* p): ptr(p) {}
    operator Type*() {return ptr;}
    Type* operator->()
    {
        if (ptr) return ptr;
        else return ptr2;
    }
};

class A
{
public:
    A(){}
    virtual void aaa() { cout << "this is aaa()" << endl;}
};

class B: public A
{
public:
    B() {}
    virtual void aaa() { cerr << "using NULL ptr" << endl;}
};

A* Pointer<A>::ptr2 = new B();

int _tmain(int argc, _TCHAR* argv***91;***93;)
{
    
    Pointer<A> a = new A();
    Pointer<A> b;

    a->aaa();
    b->aaa();

    return 0;
} 


[LordJZ]

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

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

Эту проверку проводит SD2 при загрузки.

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

Можно еще проверку на наличие скрипта в файле проэкта сделать.

Уже есть в «Что надо сделать».