1. C++ / Говнокод #2008

    +63.6

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    class WordCount{

private:
	string _t;
	vector <int> _w;

	int wc(){
		string tw;
		int s = 0;
		for(int i = 0; i < _t.size(); ++i){
			if(_t[i] == ' '){
				tw.clear();
				for(int j = s;j < i;++j){
					tw += _t[j];
				}
				
				size_t z = 0;
				for(int k = 0; k < i - s; ++k){
					int jj = 1;
					for(int d = 0; d < k;  ++d)
						jj *= 10;

					z += jj * static_cast<int>(_t[k]);

				}

				bool sa = true;
				for(int az = 0; az < _w.size(); ++az)
					if(_w[az] == z) sa = false;
				if(sa)_w.push_back (z);

				s = i+1;
			}
		}
		if(s < _t.size() ){
			size_t z = 0;
			for(int k = s;k < _t.size();++k){
					
					int jj = 1;
					for(int d = 0; d < k;  ++d)
						jj *= 10;

					z += jj * static_cast<int>(_t[k]);
			}

			bool sa = true;
				for(int az = 0; az < _w.size(); ++az)
					if(_w[az] == z) sa = false;
				if(sa)_w.push_back (z);
		}
		return _w.size();
	}

public:

	int WordCount(string text){

		_t = text;

		return wc();

	}
};

    Клас, который должен был подсчитывать кол-во уникальных слов в строке.
    Наслаждайтесь, господа!

    Запостил: hacker-l0maker, 21 Октября 2009

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

    • ava guest 21.10.2009 08:09 # 0

      Выглядит логично. Найди комбинацию символов до пробела -- это слово. Проверь: есть ли слово в словаре? Если есть, ничего не делай, если нет, то добавь слово в словарь, уникальных на одно больше.
      Правда кодирование слова смущает... А так... Для C программиста всё так и было бы. Весьма вероятно, что в C++ есть более продвинутые методы или функции-нечлены для работы со строками.

      Ну да... И название метода wc (ватер клазет) сразу сообщает нам, что необходимо запостить это на говнокод =]
      Ответить

      • ava hacker-l0maker 21.10.2009 11:37 # +1

        что меня смущает:
        1) код почти нечитабельный. например, здесь
        [code language="c++"]for(int j = s;j < i;++j){
        tw += _t[j];
        }
        [/code]
        можно было использовать strcpy. Плюс зачем строка tw, если она нигде не используется потом? Очевидно, программируя в таком стиле, легко потерять логику
        2) вообще утерян смысл программирования класов
        3) самое забавное: повторения кода.
        [code language="c++"]
        bool sa = true;
        for(int az = 0; az < _w.size(); ++az)
        if(_w[az] == z) sa = false;
        if(sa)_w.push_back (z);
        [/code]
        Почему не отдельные метод?
        Ответить

        • ava interested 21.10.2009 11:54 # +2

          С точки зрения С++ здесь много чего "не так". К вашим замечаниям можно ещё добавить, что не следует искать что-то в векторе, есть алгоритмы группы find. Функции неявно встроенные. Копируется строка в закрытую переменную. Как я и написал выше, похоже, что это код С программиста на С++. Мы все такие коды писали курсе на втором. На первом нас активно учили структурному программированию, а на втором "новым эффективным методам".

          Ясно же, что ни вектор не используется, ни строка, ни сама суть класса. Зато C виден насквозь.

          Я потому этот код и не минусую и не плюсую.
          Ответить

        • ava interested 21.10.2009 12:00 # 0

          Да. И как обычно.
          Хорошо бы всё, что вы считаете говнокодом, сразу писать в комментарии к коду. Иначе можно с разных сторон на код смотреть и обнаружить, что не такое уж и говно. Или на вменяемый код говном обзываться.
          Ответить

    • ava guest 21.10.2009 10:06 # +1

      Использовать static_cast<int> для преобразования от char к int это как-то неоправданно. Для встроенных типов и неуказателей лучше C-style cast.
      Ответить

      • ava guest 21.10.2009 15:38 # 0

        Ничего проти static_cast не имею, но само суммирование просто маразм (и по стилю и по эффективности)

        [code]
        for(int k = 0; k < i - s; ++k){
        int jj = 1;
        for(int d = 0; d < k; ++d)
        jj *= 10;

        z += jj * static_cast<int>(_t[k]);

        }
        Ответить

    • ava Lexey 21.10.2009 12:34 # +1

      Нафиг нужен весь этот изврат, если есть strtok (или boost::tokenizer) и std::set (или tr1::unordered_set)?
      Ответить

    • ava guest 21.10.2009 13:47 # 0

      wc - программа в *nix осях,
      Выдержка из мана:
      wc - print newline, word, and byte counts for each file
      Ответить

    • ava guest 21.10.2009 15:05 # 0

      А в самом деле, как лучше подсчитать количество слов в строке (не применяя внешние библиотеки)?
      Ответить

    • ava guest 21.10.2009 15:20 # 0

      не знаю как сам алгоритм, но стиль написания кода в данном случае -- говно редкое.
      Ответить

    • ava guest 21.10.2009 17:25 # 0

      очередной ОО-идиот. Зачем здесь создавать класс?
      Ответить

    • ava guest 30.10.2009 01:17 # 0

      вообще нада юзать std::string::find_first_not_of(" \n\r\t", currentPos)
      А так видно, что код начинающим написан
      Ответить

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