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

    +160

    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
    #include <iostream>
#include <string>
using namespace std;
int main()
{
	const int hz_limit = 20000;
	string str;
	string str_mass[] = {"a", "b", "ais", "h", "ces", "c", "his", "cis", "des",
		"d", "dis", "es", "e", "fes", "f", "eis", "fis", "ges", "g", "gis", "as"};
	begin:
	while (cin>>str)
	{
		for (size_t ix = 0; ix != str.size(); ++ix)
		{
			str[ix] = tolower(str[ix]);
		}
		size_t f = 0;
		for (size_t ix = 0; ix != 21; ++ix)
		{
			if(str == str_mass[ix])
			{
				f = ix;
			}
		}
		if (str != str_mass[f])
		{
			goto begin;
		}
		double mass[] = {27.500, 29.135, 29.135, 30.868, 30.868, 32.703, 32.703, 34.648, 34.648,
		36.708, 38.891, 38.891, 41.203, 41.203, 43.654, 43.654, 46.249, 46.249, 48.999, 51.913, 51.913};
		for (; mass[f] < hz_limit; mass[f] *= 2)
			{
				cout<<mass[f]<<" Hz"<<endl;
			}
	}
	return 0;
}

    Моя первая прога. Выводит частоты на которых находится введенная нота...

    Запостил: Extrawelt, 21 Марта 2011

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

    • ava TarasB 21.03.2011 16:59 # +1

      Что такое 21? В языке нет конструкции, чтобы узнать размер массива?
      Ответить

      • ava Extrawelt 21.03.2011 17:09 # 0

        Наверно так надо было: sizeof(str_mass)/sizeof(str_mass[0])
        Ответить

        • ava TarasB 21.03.2011 17:21 # 0

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

          • ava YuraTim 21.03.2011 17:28 # 0

            Будет размер массива, т.к он указан явно.
            Ответить

        • ava TheCalligrapher 21.03.2011 17:22 # 0

          sizeof str_mass / sizeof *str_mass
          Ответить

          • ava Extrawelt 21.03.2011 17:32 # 0

            А чем отличается str_mass[0] от *str_mass ?
            Ответить

            • ava YuraTim 21.03.2011 17:34 # 0

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

            • ava TheCalligrapher 21.03.2011 17:39 # +5

              Ничем. Просто '*' - более фундаментальная операция. И индексно-независимая, т.е. не вызывает вопросов о том, почему именно '[0]', а не, скажем, '[5]'.

              В конце концов это просто короче.
              Ответить

      • ava TheCalligrapher 21.03.2011 17:21 # 0

        В смысле: никаких других проблем вы в коде не заметили? Я бы не сказал, что это '21' как-то выделяется на общем фоне.

        Код очень ровный в плане пионерского программирования. Просто аж придраться не к чему :)
        Ответить

      • ava nil 25.03.2011 00:57 # +2

        21 — это 42 пополам. Сакральное число!
        Ответить

    • ava bugmenot 21.03.2011 17:15 # −1

      массив массивен
      Ответить

      • ava absolut 21.03.2011 17:46 # +1

        еще и инициализируется в цикличном цикле
        Ответить

        • ava TheCalligrapher 21.03.2011 19:21 # 0

          Он инициализируется в цикле по вполне понятной причние: значение одного из элементов будет портиться в процессе генерации результата. Поэтому хочешь-не хочешь, а надо переинициализировать. Так что тут придраться не к чему :)
          Ответить

          • ava absolut 21.03.2011 22:23 # 0

            Модифицируется - да. Но логичнее было бы вынести массив из цикла, а в цикле использовать временную переменную.
            Плюс - нелепая проверка на отсутствие совпадений в 25 строке и выход из цикла чтения. А ещё, мне кажется, что автору повезло использовать string и проверку на равенство строк, с char* такое бы не прокатило. Новички на этом часто попадаются.
            Ответить

            • ava TheCalligrapher 22.03.2011 01:04 # 0

              Разумеется, логичнее бы было вынести массив из цикла. Но это бы сущеcтвенно уменьшило своеобразную прелесть данного кода.
              Ответить

    • ava J0hnny 27.03.2011 23:50 # +3

      Для равномерно темперированного строя, массив вообще не нужен.
      Отношение между частотами соседних нот(пол-тона) константа - exp( (1./12.) * ln(2.) ) .
      Ответить

      • ava absolut 28.03.2011 13:13 # 0

        что-то не видно этого по коду
        Ответить

        • ava J0hnny 28.03.2011 13:41 # 0

          Что именно не видно?

          Нет необходимости хранить все частоты, тем более повторяющиеся, достаточно хранить только одну(обычно это нота ля - 440hz).
          Ответить

        • ava J0hnny 28.03.2011 13:54 # 0

          Если же вопрос в том, что конкретно данный массив нельзя просто сходу заменить формулой(из-за повторяющихся частот), то тут можно немного ухитриться: легко заметить патерн - четыре пары(внутри пары одинаковые частоты), потом одиночная частота, и т.д. формула просто будет немного сложней :P
          Ответить

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