- 1
- 2
- 3
- 4
- 5
- 6
- 7
template<class Exception>
bool GenerateException(const Exception* const e)
{
throw e;
delete e;
return true;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+175
template<class Exception>
bool GenerateException(const Exception* const e)
{
throw e;
delete e;
return true;
}
Продолжение цикла говнокодов про исключения из того же самого проекта.
Проблем от неё почти нет, тк в основном исключения генерятся так:
ExceptionТакойто e;
GenerateException(&e);
+169
class SeismicDriver {
void set2DVolume (GlData2DVol* vol = 0, bool reserv = false);
GlData2DVol* _glData2DVol; /**< Seismic 2D Line */
}
void SeismicDriver::set2DVolume (GlData2DVol* vol, bool reserv)
{
_glData2DVol = vol;
}
Наш подопытный китаец очень предусмотрительный. Если в будущем вдруг случится, что функции понадобится второй параметр типа bool - опа! - он уже есть :)
+162
class GlData2DLine
{
virtual std::vector<Vec2d> getDecimatedPoints();
protected:
std::vector<Vec2d> c_decimated_points;
}
std::vector<Vec2d> GlData2DLine::getDecimatedPoints()
{
return c_decimated_points;
}
class GlData2DLineOD : public GlData2DLine
{
virtual std::vector<Vec2d> getDecimatedPoints();
}
std::vector<Vec2d> GlData2DLineOD::getDecimatedPoints()
{
return c_decimated_points;
}
Наш любимый китаец увлекся ООП и полиморфизмом что забыл очевидное.
+161
/**
* Standard QWidget Constructor
*/
SeismicSelectionDG::SeismicSelectionDG(QWidget *parent)
: QDialog(parent), ui(new Ui::SeismicSelectionDG)
{
...
}
/**
* Destructor
*/
SeismicSelectionDG::~SeismicSelectionDG()
{
if(ui) delete ui;
}
/**
* Set title to dialog box
* @param title The title for dialog box
*/
void SeismicSelectionDG::setDialogTitle(const QString& title)
{
QDialog::setWindowTitle(title);
}
Обратим внимание как мой коллега китаец тщательно документирует код. метод setDialogTitle - в особенности. А учитывая что QDialog::setWindowTitle - библиотечный метод, необходимость оборачивать его в свой метод полностью отсутствует. Также глупость в деструкторе - для delete необязательно проверять указатель на 0.
+161
// из объявления класса
...
float _x_3DMin; /**< Min X value */
float _x_3DMax; /**< Max X value */
float _z_3DMin; /**< Min Z depth value */
float _z_3DMax; /**< Max Z depth value */
/* 2D seismic variables */
float _x_2DMin; /**< Min X value */
float _x_2DMax; /**< Max X value */
float _z_2DMin; /**< Min Z depth value */
float _z_2DMax; /**< Max Z depth value */
/* No seismic selection variables */
float _xMin; /**< Min X value */
float _xMax; /**< Max X value */
float _zMin; /**< Min Z depth value */
float _zMax; /**< Max Z depth value */
...
void SeismicSelectionDG::initMembers()
{
// initialise values
_x_3DMin = 0.0F;
_x_3DMax = 0.0F;
_z_3DMin = 0.0F;
_z_3DMax = 0.0F;
_x_2DMin = 0.0F;
_x_2DMax = 0.0F;
_z_2DMin = 0.0F;
_z_2DMax = 0.0F;
_xMin = 0.0F;
_xMax = 0.0F;
_zMin = 0.0F;
_zMax = 0.0F;
...
}
Ну и весь остальной код состоит из этих говно-паттернов. Об абстракциях товарищ понятия не имеет.
+162
SeismicDriver* SeismicSelectionDG::seismicDriver()
{
Exf2dMW* mw = Exf2dMW::getMW(this);
_seismicDriver = mw->seismicDriver();
if ( !mw || !_seismicDriver)
return 0;
return _seismicDriver;
}
Все тот же коллега китаец. Exf2dMW - класс главного окна приложения. Во-первых бесмыссленно проверять его на 0, потому что без него вообще бы ничего не работало, но и как видно проверяет его он все равно не там. Ну и проверка на 0 _seismicDrvier тоже лишняя. Итог никакие проверки не нужны да и вообще первую строку в методе можно перенести в конструктор а возвращать член класса _seismicDriver.
+162
//...
CString signature(pMsg->clipEvent.signature);
int indx = signature.Find('-');
if(!indx)
signature.Delete(indx);
indx = signature.Find('-');
pEvent->eventData.signature.doorNum = _ttoi(signature.Left(indx));
signature.Delete(0, indx+1);
indx = signature.Find('-');
pEvent->eventData.signature.eventType = _ttoi(signature.Left(indx));
signature.Delete(0, indx+1);
indx = signature.Find('-');
pEvent->eventData.signature.eventID = _ttoi(signature.Left(indx));
signature.Delete(0, indx+1);
indx = signature.Find('-');
pEvent->eventData.signature.itemNumber = _ttoi(signature.Left(indx));
signature.Delete(0, indx+1);
indx = signature.Find('-');
CString dateTimeStr;
dateTimeStr = signature.Mid(5, 2);
dateTimeStr += '/';
dateTimeStr += signature.Mid(8, 2);
dateTimeStr += '/';
dateTimeStr += signature.Left(4);
dateTimeStr += ' ';
dateTimeStr += signature.Mid(11, 2);
dateTimeStr += ':';
dateTimeStr += signature.Mid(14, 2);
dateTimeStr += ':';
dateTimeStr += signature.Mid(17, 2);
pEvent->eventData.signature.eventDateTime = DEM_getTimeDateStampFromTimeDateString(dateTimeStr);
signature.Delete(0, 20);
pEvent->eventData.signature.cameraIndex = _ttoi(signature);
//...
инициализация числовых переменных из строки с заранее известным форматом...
так как автор "кончал на MFC", с функцией sscanf западло было работать
+161
// Часть первая. Объявления. Exceptions.hpp
template <typename E>
struct ExeptionKeeper
{
E * exptn;
ExeptionKeeper(E * ex)
{
exptn = ex;
}
~ExeptionKeeper()
{
if(exptn != NULL )
{
delete exptn;
}
}
E * operator -> ()
{
return exptn;
}
};
#define EX_TRY(XX) try { XX }
#define EX_CATCH(TT, NAME, XX) catch (TT *_ex) { ExeptionKeeper<TT> NAME(_ex); XX }
/// Часть вторая. Использование. ModelsHelper.cpp
.... // код
EX_TRY
(
............
TestParams(args); // кидает new ParamsException в случае неудачи
............
TestFunctions(funclist); // Кидает new FuncsException ...
............
//// тут еще есть вызовы, кидающие исключения
............
)
EX_CATCH
( ParamsException, exp,
GLogger(levelerror).Log("Invalid params for model ", exp->model(), " [", exp->paramlist().str(), "]");
)
EX_CATCH
( FuncsException, exp,
GLogger(levelerror).Log("Invalid functions for model ", exp->model(), " [", exp->funclist().str(), "]");
)
EX_CATCH
( LevelsException, exp,
GLogger(levelerror).Log("Invalid levels for model ", exp->model(), " [", exp->levelslist().str(), "]");
)
EX_CATCH
( .... , exp,
........
)
.... // код
И снова на тему исключений.
Прислал приятель. Какой-то софт для аэропортов.
Комменты на русском мои.
+154
//построение суффиксного массива
vector <int> getarr(string s)
{
//s - исходная строка
//суффиксный массив
vector <int> arr;
arr.resize(s.size());
//массив цветов
vector <int> col;
col.resize(s.size());
//массив для временных данных
vector <int> buf;
buf.resize(s.size());
//массив для карманов сортировки
vector <int> buck;
buck.resize(max(L, (int) s.size()));
//Шаг первый - начальная сортировка
//мы хотим отсортировать буквы строки
//посчитаем количество всех букв
for (int i = 0; i < (int) s.size(); i++)
buck[s[i]]++;
//преобразуем массив так, чтобы каждый элемент указывал на положение в массиве первой данной буквы
int sum = 0;
for (int i = 0; i < L; i++)
{
sum += buck[i];
buck[i] = sum - buck[i];
}
//теперь заполним массив arr: Теперь в нем суффиксы отсортированы по первой букве
for (int i = 0; i < (int) s.size(); i++)
arr[buck[s[i]]++] = i;
//теперь проставляем цвета: цвет увеличивается на 1 если следующая буква - другая
col[arr[0]] = 0;
for (int i = 1; i < (int) s.size(); i++)
col[arr[i]] = col[arr[i-1]] + (s[arr[i]] != s[arr[i-1]]);
int cn = col[arr[s.size() - 1]] + 1;
//Шаг второй - постепенное расширение подстрок
//в начале цикла отсортированы подстроки длины l, а в конце - длины 2l
for (int l = 1; l < (int) s.size(); l *= 2)
{
//обнуляем массив buck и заполняем для сортировки по col
for (int i = 0; i < (int) s.size(); i++)
buck[i] = 0;
for (int i = 0; i < (int) s.size(); i++)
buck[col[i]]++; sum = 0;
for (int i = 0; i < cn; i++)
sum += buck[i], buck[i] = sum - buck[i];
//строим новый массив в buf (не забываем сдвинуть указатель по модулю на l влево), затем копируем его в arr
for (int i = 0; i < (int) s.size(); i++)
buf[buck[col[(arr[i] - l + s.size()) % s.size()]]++]=(arr[i] - l + s.size()) % s.size();
arr = buf;
//теперь перекрашиваем массив col: заполняем массив buf, увеличиваем цвет на единицу если один из цветов отличается, затем копируем
buf[arr[0]] = 0;
for (int i = 1; i < (int) s.size(); i++)
buf[arr[i]] = buf[arr[i - 1]] + (col[arr[i]] != col[arr[i - 1]] || col[(arr[i] + l) % s.size()] != col[(arr[i - 1] + l) % s.size()]);
cn = buf[arr[s.size() - 1]] + 1;
col = buf;
}
//возвращаем результат
return arr;
}
Это просто жуть
+170
double* (*(*(*fp)(int*,int*,int*))[])(float*,float*,float*);
уже третий семпл из жгучей темы на форуме сайта wasm.ru... я цитирую: "Объявление указателя на функцию, принимающую 3 аргумента-указателя на int, а возвращающая указатель на массив указателей на функцию принимающую 3 аргумента-указателя на float и возвращающую указатель на double"))) http://www.wasm.ru/forum/viewtopic.php?pid=419403#p419403