Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+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.
kitaec,
09 Февраля 2011
+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;
...
}Ну и весь остальной код состоит из этих говно-паттернов. Об абстракциях товарищ понятия не имеет.
kitaec,
09 Февраля 2011
+162
SeismicDriver* SeismicSelectionDG::seismicDriver()
{
Exf2dMW* mw = Exf2dMW::getMW(this);
_seismicDriver = mw->seismicDriver();
if ( !mw || !_seismicDriver)
return 0;
return _seismicDriver;
}Все тот же коллега китаец. Exf2dMW - класс главного окна приложения. Во-первых бесмыссленно проверять его на 0, потому что без него вообще бы ничего не работало, но и как видно проверяет его он все равно не там. Ну и проверка на 0 _seismicDrvier тоже лишняя. Итог никакие проверки не нужны да и вообще первую строку в методе можно перенести в конструктор а возвращать член класса _seismicDriver.
kitaec,
09 Февраля 2011
+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 западло было работать
ShuraKotov,
09 Февраля 2011
+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,
........
)
.... // код
И снова на тему исключений.
Прислал приятель. Какой-то софт для аэропортов.
Комменты на русском мои.
ossa,
09 Февраля 2011
+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;
}Это просто жуть
chexov,
09 Февраля 2011
+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
ReL,
08 Февраля 2011
+165
if (seismicDriver()->has3DSeismic() && !seismicDriver()->has2DSeismic() ) {
enableSeismicTab(0, true);
enableSeismicTab(1, false);
}
else if (!seismicDriver()->has3DSeismic() && seismicDriver()->has2DSeismic() ) {
enableSeismicTab(0, false);
enableSeismicTab(1, true);
}
else if (seismicDriver()->has3DSeismic() && seismicDriver()->has2DSeismic() ) {
enableSeismicTab(0, true);
enableSeismicTab(1, true);
}
else if (!seismicDriver()->has3DSeismic() && !seismicDriver()->has2DSeismic() ) {
enableSeismicTab(0, false);
enableSeismicTab(1, false);
}
Этим своим первым постом предворяю серию говнокода, который пишет мой коллега, китаец, имеющий PhD по физике.
В этом сниппете советую обратить внимание на два компонента в условиях if и на расстановку true и false в соответствии с ними.
kitaec,
08 Февраля 2011
+161
throw &std::exception();
Из того же проекта, что и http://www.govnokod.ru/5581 .
Естественно это потом иногда удаляется по delete...
Говногость,
08 Февраля 2011
+161
// говно-код - вступление
KEY_ACCELERATOR_DATA acceleratorData;
WORD id; int i, j, k;
acceleratorData.line = 255;
acceleratorData.address = 255;
acceleratorData.sensors = 255;
acceleratorData.action = 255;
KeyAcceleratorDataList tempList;
// говно-код - квинтэссенция
for(i = 1; i < listWholeString.ItemsCount(); i++)
{
k = 0;
strCurrSensor = listWholeString[i];
if((k = strCurrSensor.Find(_T('-'), k)) <= 0)
break;
acceleratorData.line = (BYTE)_ttoi(strCurrSensor.Left(k));
// ну кто блядь это поймет??? Кто???!!!
j = k;
if((k = strCurrSensor.Find(_T('-'), k + 1)) <= 0)
break;
acceleratorData.address = (BYTE)_ttoi(strCurrSensor.Mid(j + 1, k - (j + 1)));
j = k;
if((k = strCurrSensor.Find(_T('-'), k + 1)) <= 0)
break;
acceleratorData.sensors = (BYTE)_ttoi(strCurrSensor.Mid(j + 1, k - (j + 1)));
if((k = strCurrSensor.Find(_T('-'), k)) <= 0)
break;
acceleratorData.action = (BYTE)_ttoi(strCurrSensor.Right(strCurrSensor.GetLength() - (k + 1)));
if((acceleratorData.line == 255) || (acceleratorData.address == 255) ||
(acceleratorData.sensors == 255) || (acceleratorData.action == 255))
break;
tempList.push_back(acceleratorData);
}
mapKeyAccelerators.operator[](id) = tempList;
}
// Аут… Тушите свет и сидите, где сидели… А я нахуй сам пойду…
парень "кончал на компьютер" и даже получил сертификат Microsoft по MFC
лучше бы он кончал на дорожного рабочего...
ShuraKotov,
08 Февраля 2011
Follow us!