FrolDy / sinus

0 stars 0 forks source link

А для чего вообще нужен этот буфер? #1

Open alekseevx opened 3 years ago

alekseevx commented 3 years ago

https://github.com/FrolDy/sinus/blob/a790f5c21456cd0a64e63300b10e2db0afd538e3/%D0%A1makeProject1.cpp#L14

FrolDy commented 3 years ago

№1 Здравствуйте, я в этот массив записал значения сами. Массив 16-битных данных вроде как остроконечный

вс, 14 февр. 2021 г., 17:04 Alekseev Alexey notifications@github.com:

https://github.com/FrolDy/sinus/blob/a790f5c21456cd0a64e63300b10e2db0afd538e3/%D0%A1makeProject1.cpp#L14

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/FrolDy/sinus/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASZR5AURJAMETIJP3KICCXTS67J53ANCNFSM4XTIHGYQ .

FrolDy commented 3 years ago

№2 Просто так учили в вузе в примерах, буду использовать константы после вашего замечания

вс, 14 февр. 2021 г. в 17:11, danila frolov ahjkjd2015@gmail.com:

№1 Здравствуйте, я в этот массив записал значения сами. Массив 16-битных данных вроде как остроконечный

вс, 14 февр. 2021 г., 17:04 Alekseev Alexey notifications@github.com:

https://github.com/FrolDy/sinus/blob/a790f5c21456cd0a64e63300b10e2db0afd538e3/%D0%A1makeProject1.cpp#L14

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/FrolDy/sinus/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASZR5AURJAMETIJP3KICCXTS67J53ANCNFSM4XTIHGYQ .

alekseevx commented 3 years ago

№1 Здравствуйте, я в этот массив записал значения сами. Массив 16-битных данных вроде как остроконечный вс, 14 февр. 2021 г., 17:04 Alekseev Alexey notifications@github.com: https://github.com/FrolDy/sinus/blob/a790f5c21456cd0a64e63300b10e2db0afd538e3/%D0%A1makeProject1.cpp#L14 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASZR5AURJAMETIJP3KICCXTS67J53ANCNFSM4XTIHGYQ .

Но зачем вам целый массив? Вы же работаете только с одним элементом. test << buffer[i]; out << buffer[i] << endl;

alekseevx commented 3 years ago

№2 Просто так учили в вузе в примерах, буду использовать константы после вашего замечания вс, 14 февр. 2021 г. в

Это хорошо, но чем плохи макросы?

FrolDy commented 3 years ago

Подождите, я записал в pcm файл и там даже звук был. Я проверял, этот файл тоже нужно на гит хаб залить? И макросы много чем вредны. От появления багов разных до усложнения отладки. Конкретно у меня в программе я не знаю чем они плохи, честно.

вс, 14 февр. 2021 г. в 17:20, Alekseev Alexey notifications@github.com:

№2 Просто так учили в вузе в примерах, буду использовать константы после вашего замечания вс, 14 февр. 2021 г. в

Это хорошо, но чем плохи макросы?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/FrolDy/sinus/issues/1#issuecomment-778784465, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASZR5AS3Z2IXCKY3NWWNCN3S67LZ3ANCNFSM4XTIHGYQ .

alekseevx commented 3 years ago

Проверяете лучше. Вот начало файла, что сгенерировала ваша программа 045418991132581725720907241342687229065306713165631999316953074929181270222431621117174911351192584817278

На pcm совсем не похоже

FrolDy commented 3 years ago

Я создал файл test.pcm и записал туда данные из массива. Скачал аудио редактор. Ну чисел там точно нет. Я сделал вывод в консоль массива чтобы просто увидеть что вообще происходит в моей программе

FrolDy commented 3 years ago

И у меня вот строчка, test.open("D:\test.pcm"); я же открываю для записи pcm файл.

alekseevx commented 3 years ago

И у меня вот строчка, test.open("D:\test.pcm"); я же открываю для записи pcm файл.

И записываете туда текст, а нужно 16 битные отчеты.

FrolDy commented 3 years ago

Если я правильно понял, то я это исправил и изменил код уже на сайте. А насчет того зачем я использую массив, я понял что мог его не использовать. Просто мне так для понимания изначально было удобнее.

alekseevx commented 3 years ago

Ничего вы не исправили. Вы десятичный текстовый вывод заменили на шестнадцатеричный. Пожалуйста, проверяйте лучше.

FrolDy commented 3 years ago

Исправил, посмотрите пожалуйста.

FrolDy commented 3 years ago

Получилось подобное. image

alekseevx commented 3 years ago

https://www.cplusplus.com/reference/ostream/ostream/operator%3C%3C/ https://www.cplusplus.com/reference/ostream/ostream/write/

alekseevx commented 3 years ago

И было бы неплохо, если бы вы указали источник. А еще лучше, если в следующий вы напишете сами.

FrolDy commented 3 years ago

Извините, я не понял ошибки. А делал я по вот этому примеру.
https://habr.com/ru/post/219337/