Говнокод #12562 — C++ — Говнокод.ру

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

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

C++ / Говнокод #12562

template<typename T>
class Enumerable
{
public:
	Enumerable() : enumerableInProcess(false) { };
	virtual void begin() = 0;
	virtual void end() = 0;
	virtual bool enumeration(T* item) = 0;
protected:
	bool enumerableInProcess;
};
template<typename T>
class List : Enumerable<T*>
{
public:
	class ListItem
	{
	public:
		friend class List;
		T item;
		ListItem(T item) : item(item), next(nullptr), previous(nullptr)
		{
		}
	private:
		class ListItem* next;
		class ListItem* previous;
	};

	/* ... */
	
	void begin()
	{
		if(enumerableInProcess) {
			throw Exception("Error Enumerable!");
		}

		enumerableInProcess = true;
		enumerationItem = first;
	}
	bool enumeration(T** item)
	{
		if(enumerableInProcess) {
			if(enumerationItem != nullptr) {
				(*item) = &(enumerationItem->item);
				enumerationItem = enumerationItem->next;
				return (true);
			} else {
				(*item) = nullptr;
				return (false);
			}
		} else {
			throw Exception("Error Enumerable!");
		}
	}
	bool enumeration(ListItem **listItem)
	{
		if(enumerableInProcess) {
			if(enumerationItem != nullptr) {
				(*listItem) = enumerationItem;
				enumerationItem = enumerationItem->next;
				return (true);
			} else {
				(*listItem) = nullptr;
				return (false);
			}
		} else {
			throw Exception("Error Enumerable!");
		}
	}
	void end()
	{
		if(!enumerableInProcess) {
			throw Exception("Error Enumerable!");
		}
		enumerableInProcess = false;
	}
private:
	const int size;
	int count;
	ListItem *first;
	ListItem *last;
	ListItem *enumerationItem;
};
void list_t_1()
{
	List<int> list(8);
	List<int>::ListItem *item;
	list.add(1);
	list.add(2);
	list.add(4);
	list.add(8);
	list.add(16);

	list.begin();
	while(list.enumeration(&item))
	{
		printf("%i\n", (item->item));
	}
	list.end();
}

Запостил:

dreesto, 09 Февраля 2013

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

defecate-plusplus 09.02.2013 21:03 # +4

гуры гуру джавы идут к успеху на крестах?
Ответить
- bormand 09.02.2013 21:04 # +1
  
  > гуры гуру
  гуро
  
  > джавы
  Скорее шарпика.
  Ответить
absolut 09.02.2013 21:10 # +2

походу и С++11 привлекли (nullptr)
Ответить
- bormand 09.02.2013 21:14 # +2
  
  Тогда надо было и новый for вместо того while и возврата результата через анус.
  Ответить

Xom94ok 09.02.2013 21:20 # 0

dreesto, будь добр, выложи код остальных методов. Очень хочется увидеть картину целиком.

Ответить

dreesto 09.02.2013 21:24 # +1

List(int size) : first(nullptr), last(nullptr), count(0), size(size)
{
}

~List()
{
	ListItem *item, *next;
	item = first;

	while(item != nullptr) {
		next = item->next;
		delete item;
		item = next;
	}
}

inline bool isEmpty()
{
	return (first == nullptr);
}
inline bool isFull()
{
	return (count == size);
}

void add(T item)
{
	ListItem* newItem;

	if(isFull()) {
		throw Exception("List full!");
	} else {
		if(last == nullptr) {
			first = new ListItem(item);
			last = first;
		} else {
			newItem = new ListItem(item);
			newItem->previous = last;
			last->next = newItem;
			last = newItem;
		}

		count += (1);
	}
}
void remove(ListItem* item)
{
	if(isEmpty()) {
		throw Exception("Empty list!");
	} else {
		if(item->previous == nullptr) {
			first = item->next;
		} else {
			item->previous->next = item->next;
		}
		if(item->next == nullptr) {
			last = item->previous;
		} else {
				item->next->previous = item->previous;
		}

		count -= (1);
		delete item;
	}
}

int getCount()
{
	return (count);
}
int getSize()
{
	return (size);
}

Ответить

Xom94ok 09.02.2013 22:00 # +6

О да. remove(ListItem*) так и просит, чтобы его вызвали с указателем на ListItem из другого списка.
А еще интригует класс Exception; его существование как бы намекает на то, что чувак хочет построить свой STL с блекджеком и шлюхами.
Ответить
movaxbx 09.02.2013 22:05 # +1

>return (first == nullptr);
>return (count == size);
>count += (1);
>count -= (1);
>return (count);
>return (size);
Зачем?
Ответить
- absolut 09.02.2013 23:16 # +2
  
  не уверен - ставь скобки
  Ответить

guest 09.02.2013 22:27 # −3

Hello, bros! How I can default construct object "holder" at one expression? "holder()" makes function type and not call constructor.
http://liveworkspace.org/code/2oXK1R$0
Ответить
- bormand 09.02.2013 23:24 # 0
  Just add more parentheses. Like this:
  PRINT_NAME_OF_TYPE((holder()));
  Ответить
  - guest 09.02.2013 23:38 # 0
    
    Thanks, Borman!
    Ответить
Psionic 09.02.2013 22:46 # 0

Свой СТЛ? А где шлюхи и блекджек?
Ответить

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

C++ / Говнокод #12562

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

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