Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+24
#ifndef JSON_SPIRIT_READER_TEMPLATE
#define JSON_SPIRIT_READER_TEMPLATE
// Copyright ***
// Distributed under the MIT License, see accompanying file LICENSE.txt
// json spirit version 4.05
#if defined(_MSC_VER) && (_MSC_VER >= 1020)
# pragma once
#endif
//some classes
#endifСначала проверим многократное включение через макросы, а потом # pragma once - чтобы наверняка!
Try,
24 Декабря 2012
+16
#include <iostream>
using namespace std;
struct T{int a;T(){cout<<"T"<<this<<endl;}~T(){cout<<"~T"<<this<<endl;}};
struct D:public T{int b;D(){cout<<"D"<<this<<endl;}~D(){cout<<"~D"<<this<<endl;}};
int main() {
T* a = new D[2];
cout<<"mission failure"<<endl;
delete[] a;
return 0;
}
Писал тест специально для Тараса:
Виртуальный деструктор в базовых классах нужен:
*Для вызова деструкторов всех потомков. Иногда это не нужно, тк поля структуры тривиальны. Согласен с Тарасом.
*Для правильного выбора перегруженного оператора delete. Создал одним менеджером памяти, а удалил в другой - это плохо. Но также бывает редко и только в специфичных проектах.
Но вот то, что демонстрируется в этом примере - думаю чего-нибудь такого создатели той библиотеки, что ковырял Тарас, и опасались.
Не просто было вызван деструктор только для предков, забыв о деструкторах потомков, но и:
*Деструктор предков был вызван для this не по тем адресам, где расположены объекты.
Не удивлюсь, если:
*В некоторых кулхацкерских реализациях менеджеров памяти будет удален буфер меньшего размера и куча попортится или оставшаяся не удаленная часть массива в куче просто утекет.
http://liveworkspace.org/code/2X3MR0$8
LispGovno,
22 Декабря 2012
+10
virtual ~T() {}
нахуя?
у всех наследников то же самое и наличие чего-либо внутри не предполагается
Первая ссылка по гуглозапросу "c++ mersenne twister" выдаёт склад оопиозного говнокода:
http://www.bedaux.net/mtrand/
TarasB,
19 Декабря 2012
+13
#include <iostream>
using namespace std;
class Manager
{
private:
int *ar;
int _size;
public:
void CreateArray(int size);
void ResizeArray(int size);
void MakeIntArray(int iter, int number);
int GetIntArray(int iter);
~Manager()
{
delete [] ar;
}
};
void Manager::CreateArray(int size)
{
_size = size;
ar = new int[size];
}
void Manager::ResizeArray(int size)
{
int *nar = new int[size];
for(int i = 0; i < _size; i++)
{
*(nar+i) = *(ar+i);
}
_size = size;
delete [] ar;
*ar = *nar;
}
int Manager::GetIntArray(int iter)
{
return iter < _size ? *(ar+iter) : NULL;
}
void Manager::MakeIntArray(int iter, int number)
{
if(iter < _size)
*(ar+iter) = number;
}
int main()
{
Manager Man;
Man.CreateArray(10);
for(int i = 0; i < 10; i++)
Man.MakeIntArray(i, i);
for(int i = 0; i < 10; i++)
cout << Man.GetIntArray(i) << " ";
Man.ResizeArray(20);
cout << endl;
for(int i = 10; i < 20; i++)
Man.MakeIntArray(i, i);
for(int i = 0; i < 20; i++)
cout << Man.GetIntArray(i) << " ";
return 0;
}
> Собственно написал менеджер для массива, хочется узнать правильно ли все сделано, и нет ли утечек памяти?
Очередной шедевр с самизнаетеоткуда.
LispGovno,
12 Декабря 2012
+79
//Unit1.h
const float RADIAN=0.01744444444;
TForm1 *Form1;
Graphics::TBitmap*ZadniyBuffer;
float ShipPosX,ShipPosY;
float ShipAngle;
float ShipSpeed;
bool LEFT,RIGHT,UP;
TList*Bullets;
TList*Asteroids;
class Bullet
{
public:
float x,y,dx,dy;
void CreateBullet()
{
x=ShipPosX;y=ShipPosY;
dx=10*cos(RADIAN*ShipAngle);
dy=10*sin(RADIAN*ShipAngle);
}
void Draw()
{
ZadniyBuffer->Canvas->Pen->Color=clRed;
ZadniyBuffer->Canvas->MoveTo(x,y);
ZadniyBuffer->Canvas->LineTo(x+dx,y+dy);
}
};
class CRock
{
public:
float x,y;
int Figure[50];
float Angle;
float Spin;
float XSpeed,YSpeed;
int Tip;
void CreateAsteroid(int t)
{
Tip=t;
XSpeed=random(4)+(random(9)/10.0)-2;
YSpeed=random(4)+(random(9)/10.0)-2;
Spin=random(10)-5;
for(int i=0;i<20;i++)
{
if(Tip==1)Figure[i]=random(10)+25;
if(Tip==2)Figure[i]=random(10)+10;
if(Tip==3)Figure[i]=random(5)+5;
}
}
void Move()
{
x+=XSpeed;y+=YSpeed;
Angle+=Spin;
if(x>440)x=-40;if(x<-40)x=440;
if(y>440)y=-40;if(y<-40)y=440;
}
void Draw()
{
ZadniyBuffer->Canvas->Pen->Color=clRed;
float tAng=0;
int tx=cos(RADIAN*(Angle+tAng))*Figure[0]+x;
int ty=sin(RADIAN*(Angle+tAng))*Figure[0]+y;
ZadniyBuffer->Canvas->MoveTo(tx,ty);
for(int i=0;i<20;i++,tAng+=19)
{
int tx=cos(RADIAN*(Angle+tAng))*Figure[i]+x;
int ty=sin(RADIAN*(Angle+tAng))*Figure[i]+y;
ZadniyBuffer->Canvas->LineTo(tx,ty);
}
}
};
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
: TForm(Owner)
{
randomize();
ZadniyBuffer=new Graphics::TBitmap();
ZadniyBuffer->Width =400;
ZadniyBuffer->Height=400;
ZadniyBuffer->Canvas->Pen->Color=clWhite;
Bullets =new TList();
Asteroids=new TList();
LEFT=false;RIGHT=false;
ShipPosX=200;ShipPosY=200;
ShipAngle=0;
ShipSpeed=0;
for(int i=0;i<4;i++)
{
CRock*a=new CRock();
a->x=random(400);
a->y=40;
a->CreateAsteroid(1);
Asteroids->Add((void*)a);
}
}
//---------------------------------------------------------------------------
void TForm1::CLS()
{
ZadniyBuffer->Canvas->Brush->Color=clBlack;
ZadniyBuffer->Canvas->FillRect(Rect(0,0,400,400));Привет. Вот, украл, вернее, скачал с розришения автора код для курсача. На учебу билдера тупо времени нет, ибо есть много хвостов. Можете ли вы мне обьяснить весь код? Буду благодарен.
kolybasov,
12 Декабря 2012
+15
LambdaVar<1> X;
LambdaVar<2> Y;
// The next line prints 10:
cout << lambda(X,Y)[ plus[ multiplies[3,X], Y ] ] (3,1) << endl;
cout << lambda(X,Y)[ (3 %multiplies% X) %plus% Y ] << endl;
//...
lambda(X)[ X %plus% getCurrentTime[_*_] ]
//...
let[ X == someLambdaExp,
Y == someOtherLambdaExpWhichMayInvolveX ]
.in[ someLambdaExpInvolvingXandY ]
//...
lambda(X)[
letrec[ F == lambda(Y)[ if1[ Y %equals% 0,
1,
Y %multiplies% F[Y %minus% 1] ] ] ]
.in[ F[X] ] ]
//...
Maybe<int> mx = just(2);
Maybe<int> my = just(3);
mx = lambda()[ compM<MaybeM>()[ plus[X,Y] | X<=mx, Y<=my, guard[false] ] ]();
cout << mx << endl; // Nothing
//...
compM<ListM>()[ makePair[X,Y] | X<=list_with(1,2), guard[true],
Y<=list_with(3,4), guard[ (Y %divides% X) %equal% 3 ] ] ]
Грибки отсюда:
http://people.cs.umass.edu/~yannis/fc++/
LispGovno,
10 Декабря 2012
+18
/*
* The use of singletons for globals makes globals not
* actually be initialized until it is first needed, this
* makes the library faster to load, and have a smaller
* memory footprint
*/
#define json_global_decl(TYPE, NAME, VALUE) \
class jsonSingleton ## NAME { \
public: \
inline static TYPE & getValue() json_nothrow { \
static jsonSingleton ## NAME single; \
return single.val; \
} \
protected: \
inline jsonSingleton ## NAME() json_nothrow : val(VALUE) {} \
TYPE val; \
}
#define json_global(NAME) jsonSingleton ## NAME::getValue() \
json_global_decl(json_string, CONST_TRUE, JSON_TEXT("true"));
json_global_decl(json_string, CONST_FALSE, JSON_TEXT("false"));
json_global_decl(json_string, CONST_NULL, JSON_TEXT("null"));
/* Использование */
json_global(ERROR_NULL_IN_CHILDREN)
Наткнулся на утечку памяти, отловленную Valgrind'ом
А внутри вот это. И главное, совершенно непонятно, зачем воротить эту свистопляску, если линкеры умеют распознавать "true" по всюду (читаем про ROMability тут: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2002/n1396.pdf )
http://stackoverflow.com/questions/690176/c-c-optimization-of-pointers-to-string-constants
Сам код отсюда: http://code.google.com/p/wot-replay-parser/source/browse/libjson/Source/JSONGlobals.h
myaut,
10 Декабря 2012
+25
public OnPlayerSpawn(playerid) {
IsSpawned[playerid]=1;
if(Team[playerid]==ZOMBIE) { TextDrawSetString(Textdraw9[playerid],"You are a ~r~Zombie~w~. Kill & Infect every Human to complete your mission.");
} else if(Team[playerid]==HUMAN) { TextDrawSetString(Textdraw9[playerid],"You are a ~r~Human~w~. Prevent Zombie attacks and survive till the end."); }
TextDrawHideForAll(Box); TextDrawHideForAll(text_Top5[0]); TextDrawHideForAll(text_Top5[1]);
TextDrawShowForPlayer(playerid, Textdraw0); TextDrawShowForPlayer(playerid, Textdraw1);
TextDrawShowForPlayer(playerid, Textdraw2); TextDrawShowForPlayer(playerid, Textdraw3);
TextDrawShowForPlayer(playerid, Textdraw4); TextDrawShowForPlayer(playerid, Textdraw5);
TextDrawShowForPlayer(playerid, Textdraw6); TextDrawShowForPlayer(playerid, Textdraw7);
TextDrawShowForPlayer(playerid, Textdraw9[playerid]);
TextDrawShowForPlayer(playerid, Textdraw15); TextDrawShowForPlayer(playerid, Textdraw16[playerid]);
SetTeam(playerid); SetClass(playerid);
if(Infection==1) {
Infection=0;
if(Totalplayers>=2) {
SetTimer("InfectionLoad",0,0);
} else if(Totalplayers>=10) {
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
}else if(Totalplayers>=10) {
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
}else if(Totalplayers>=20) {
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
}else if(Totalplayers>=30) {
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
}else if(Totalplayers>=45) {
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
}else if(Totalplayers>=60) {
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
}else if(Totalplayers<=75) {
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
SetTimer("InfectionLoad",0,0);
}
} return 1; }Серьезно?
d3n4,
08 Декабря 2012
+1
#include "stdafx.h"
#include <iostream>
#include <stdio.h>
using namespace std;
class radian
{protected:
double x;
public:
radian();//конструктор без параметров, возвращает дежурное значение
radian(double x1);//конструктор с параметрами
radian sum_rad(radian a );
radian unsum_rad(radian a);//разность 2-х радиан
double get_r(){return x;}//нужный геттр
void set_r(double x1);//бесполезный сеттрер
double radian::quarter(radian a);//определение четверти угла заданного в радианах
double operator *();//перегрузка операции. возводит во вторую степень
double radian::rad_to_grad();//перевод из радиан в градусы
~radian();//деструкторуу
};
class namerad:public radian
{
private:
string name;
public:
namerad::namerad(double corner,string name)
{set_name(name);}
void namerad::set_name(string newname)
{name=newname;}
void namerad::get_name(string currentname)
{currentname=name;}
};
radian::radian()//конструктор по умолчанию
{
x=0;
}
radian::radian(double x1)//конструктор с 1 параметром
{
x=x1;
}
radian radian::sum_rad(radian a)//сумма 2-х радиан
{radian n(x+a.x);
return n;
}
void radian::set_r( double x1)//сеттер
{ x=x1;}
radian::~radian() {x=0;}//деструктор
radian radian::unsum_rad(radian a)//разность 2-х радиан
{radian n(x-a.x);
return n;
}
double radian::rad_to_grad() //перевод из радиан в градусы
{
return(x*(180/3.14));
}
double radian ::operator *()//перегрузка операции
{
return x*x;
}
double radian::quarter(radian a)//определение четверти угла заданного в радианах
{while(a.x>6.28){a.x-=6.28;}
if ((a.x >=0)&&(a.x<=1.57)) return 1;
if ((a.x>=1.57)&&(a.x<=3.14)) return 2;
if ((a.x>=3.14)&&(a.x<=4.71)) return 3;
return 4;
}
int main()
{
radian a,b,c(5.0),d(6.20);
cout<<"summa=";b=c.sum_rad(d);cout<<b.get_r() << endl;//сумма
cout<<"raznost=";b=d.unsum_rad(c);cout<<b.get_r() << endl; //разность
cout<<"chetvert: ";cout<< d.quarter(d) << endl;//четверть
cout<<"iz radian v gradusi: ";cout<<d.rad_to_grad() << endl;//перевод из радиан в градусы
cout<<"peregruz sqr "<<*c;//возведение в квадрат. перегруженная операция
system ("PAUSE");
return 0;
}
vovkalamer,
03 Декабря 2012
+12
#include <iostream>
#include <functional>
using namespace std;
struct T
{
int a;
T(const T&):a(1){cout<<"copy_construct T"<<endl;}
T():a(1){cout<<"construct T"<<endl;}
~T(){a=0;cout<<"destruct T"<<endl;}
const T& operator=(const T&){cout<<"copy_ass T"<<endl;return *this;}
};
struct M: public T{};
void f(const T& fa)
{
cout<<"fa.a= "<<fa.a<<endl;
}
int main() {
f(std::cref(T()));
cout<<endl;
const T& b = T();
cout<<"b.a= "<<b.a<<endl;
cout<<endl;
const T& a = std::cref(T());
cout<<"a.a= "<<a.a<<endl;
return 0;
}
http://ideone.com/BmHo9w
Есть на этом ресурсе великий знаватель крестов и вот он меня уверял, что объект, на который ссылается ссылка - должен дожить до конца выхода ссылки из скоупа. Почему мы этого не наблюдаем? А знаватель? Ты меня прямо даже убедил, и тут такая подстава от тебя. a - не дожил до конца.
LispGovno,
03 Декабря 2012
Follow us!