Говнокод #16103 — Си — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

Си / Говнокод #16103

+137

void getCoeffFromString()
{
	if (sscanf(input.c_str(),"%i*x^2%i*x%i=%i",&a ,&b ,&c,&d)==4)   {}                                                    //ax^2+bx+c=d
	else 
		{
			a,b,c,d=0;
			if (sscanf(input.c_str(),"%i*x^2%i%i*x=%l",&a ,&c ,&b,&d)==4)  {}
			 else
			  {
			  a,b,c,d=0;
			  if (sscanf(input.c_str(),"%i*x%i%i*x^2=%i",&b ,&c ,&a,&d)==4)        {}
		else	
			{
				a,b,c,d=0;
				if (sscanf(input.c_str(),"%i*x%i*x^2%i=%i",&b ,&a ,&c,&d)==4)        {}
		else
		{
			a,b,c,d=0;
			if (sscanf(input.c_str(),"%i%i*x^2%i*x=%i",&c ,&a ,&b,&d)==4)         {}
		else	
			{
				a,b,c,d=0;
				if (sscanf(input.c_str(),"%i%i*x%i*x^2=%i",&c ,&b ,&a,&d)==4)            {}
		else		
			{
				a,b,c,d=0;
				if (sscanf(input.c_str(),"%i=%i*x^2%i*x%i",&d ,&a ,&b,&c)==4)         {}
		else 
			{
				a,b,c,d=0;
				if (sscanf(input.c_str(),"%i=%i*x%i*x^2%i",&d ,&b ,&a,&c)==4)     {}
			else		
				{
					a,b,c,d=0;
					if (sscanf(input.c_str(),"%i=%i%i*x^2%i*x",&d ,&c ,&a,&b)==4)   {} 
			else
				{
					a,b,c,d=0;
					if (sscanf(input.c_str(),"%i=%i%i*x%i*x^2",&d ,&c ,&b,&a)==4)     {}
		    else  
			    { 
					a,b,c,d=0;
					if (sscanf(input.c_str(),"%i=%i*x^2%i%i*x",&d ,&a ,&c,&b)==4)   {}
		    else
                  {
					  a,b,c,d=0;
					  if (sscanf(input.c_str(),"%i=%i*x%i%i*x^2",&d ,&b ,&c,&a)==4)    {}
					else	
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x^2%i*x=%i",&a ,&b ,&c)==3)    {}
					else
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x%i*x^2=%i",&b ,&a ,&c)==3)    {}
				    else
					    {
						a,b,c,d=0;
						if (sscanf(input.c_str(),"%i*x=%i%i*x^2",&b ,&c ,&a)==3)    {}
				     else	
					      {
                            a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x=%i*x^2%i",&b ,&a ,&c)==3)    {}
					else				
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x^2=%i%i*x",&a ,&c ,&b)==3)    {}
					else		
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x^2=%i*x%i",&a ,&b ,&c)==3)    {} 
					else
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i=%i*x%i*x^2",&c ,&b ,&a)==3)    {}
			        else	
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i=%i*x^2%i*x",&c ,&a ,&b)==3)    {}
					else
						{
							a,b,c,d=0;
					      	if (sscanf(input.c_str(),"%i*x^2=%i",&a ,&c)==2)    {}
					else
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x^2=%i*x",&a ,&b)==2)  {}
					else
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x=%i",&b ,&c)==2)    {cout<<a<<"vda"<<b<<"v"<<c<<"v"<<d;}
					else
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i*x=%i*x^2",&b ,&a)==2)    {cout<<a<<"vda"<<b<<"v"<<c<<"v"<<d;}
					else
						{
							a,b,c,d=0;
							if (sscanf(input.c_str(),"%i=%i*x",&c ,&b)==2)    {}
					else

Как я парсил квадратное уравнение. "У меня 27 сканэфов и ни один не работает"

Запостил:

betkaaa2, 03 Июня 2014

Комментарии (16) RSS

Abbath 03.06.2014 20:13 # 0

Регекспом было бы проще, или рекурсивно распарсить, если нужен велосипед.
Ответить
- bormand 03.06.2014 20:19 # +3
  
  Только boost::spirit, только хардкор.
  Ответить
  - defecate-plusplus 03.06.2014 20:48 # +4
    
    как ни смешно, но это был бы самый наглядный и быстродейственный путь
    а т.к. мы в разделе сишки, то надо брать заплесневелый хардкор - як/бизон
    Ответить
    - bormand 03.06.2014 22:11 # +1
      
      Эх, давненько не брал я в руки шашек...
      template <typename Iterator> struct equation_grammar : qi::grammar<Iterator, equation(), ascii::space_type> { equation_grammar() : equation_grammar::base_type(eq) { eq = (side >> char_('=') >> side) [ _val = _1 - _3 ]; side = (part [ _val = _1 ] | (char_('-') >> part) [ _val = - _2 ]) >> *( char_('+') >> part [ _val += _1 ] | char_('-') >> part [ _val -= _1 ] ); part = (x2 | x) [ _val = _1 ] | (double_ >> -char_('*') >> (x2|x)) [ _val = _3 * _1] | double_ [ _val = construct<equation>(0.0, 0.0, _1) ]; x = char_('x') [ _val = construct<equation>(0.0, 1.0, 0.0) ]; x2 = (char_('x') >> char_('^') >> char_('2')) [ _val = construct<equation>(1.0, 0.0, 0.0) ]; } qi::rule<Iterator, equation(), ascii::space_type> eq, side, part, x2, x; };
      http://pastebin.com/2wnuEJP5
      Ответить
      - bormand 03.06.2014 22:14 # +2
        
        P.S. Прекомпиленный хедер весит 480 мег. Пиздец.
        Ответить
        
        laMer007 04.06.2014 00:16 # 0
        
        Вот этого кода? Да беда у крестов. Вычисления производит и на каждую операцию выделяет память, а сборку мусора не производят (по стандарту, инстанцирование, constexpr должен сильно все разгрузить).
        У нас на предкомпилированный хедер и гига не хватало. Но там то большой проект, запачканый бустом по самое немогу (но без спирта). Год назад измерял, когда временные файлы сборки на винт в памяти сувал. Раз в два ускорило сборку кстати. Только браузер тормозить стал. Надо ещё хотя бы 4 гб
        Ответить
        
        wvxvw 04.06.2014 01:15 # 0
        
        Немножко другое, и писалось на скорую руку, только чтобы задачки решать, но тем не менее: https://github.com/wvxvw/coursera-logic/blob/master/formula.lisp
        Ответить
        
        laMer007 04.06.2014 01:21 # 0
        
        wolfram mathematica
        велосипедисты как всегда рядом.
        Ответить
        
        wvxvw 04.06.2014 01:33 # +1
        
        История неприятия лисперами Вольфрама - это история обид, разочарований, недопониманий, драматических и эксцентрических эпизодов и откровенного высмеивания. Особенно, ввиду печальноизвестной книжки (зато в ней куча картинок). Вольфрам - это протоип царя, у меня даже где-то цитата завалялась. http://www.ymeme.com/why-wolfram-%28mathematica%29-did-not-use-lisp.html
        Плюс, мне было самому интересно, и я готовился к специальной олимпиаде, где темой было синтетическое программирование, и задание было как раз втему.
        Ответить
        
        bormand 04.06.2014 05:27 # +1
        
        > Вот этого кода?
        Нет. Бустоблядских инклудов (вон тот список на pastebin, который перечислен над pch.h).
        Ответить
FireBurd 16.06.2014 17:18 # +1

У sscanf есть прекрасные средства для разбора таких строк.
Это элементарно делается при помощи %[^] и %[] и %n для определения длинны.
Ответить
- laMer007 16.06.2014 17:39 # 0
  
  Вот никогда не смотрел в его сторону после того как понял, что sprintf - главный друг атак на переполнение буферов. Сканф кажется сильным, но насколько он помогает проводить атаки, да и вообще по ошибке портить память?
  Ответить
  - eth0 16.06.2014 19:18 # 0
    
    Какие-то безопасные процедуры его использования возможны?
    Ответить
    - laMer007 17.06.2014 11:34 # +1
      
      Вы про sprintf?
      
      Не использовать sprintf
      Никогда не использовать sprintf
      Если уж приспичило, то использовать хотя бы snprintf
      В качестве строки форматирования никогда не передавать изменяемую строку. Только "%s", если нужно, а дальше саму строку.
      Никогда не передавать ссылки
      Никогда не ошибаться с количеством и типом форматирующих параметров
      Включать варнинги проверки строки форматирования, а лучше проверять анализатором кода
      Для size_t и результатов sizeof использовать %zu
      Проверяете результат snprintf, чтобы узнать о проблемах переполнения, если это необходимо
      Ответить
kegdan 16.06.2014 17:34 # +2

АААААААААААААААААААААААААААААААААААААААА АААААААААААААААААААААААААА!!!!
Ответить
- kegdan 16.06.2014 17:34 # +2
  
  Простите, я сам не ожидал
  Ответить

Говнокод: по колено в коде.

Си / Говнокод #16103

Комментарии (16) RSS

Добавить комментарий