|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Есть класс, который хранит размер и буфер описывающий в общем случае строку данных. После выделения буфера, сразу прописываю размер буфера, однако vs2017 выдает warning. Я так полагаю из за того, что конкретно для *m_buffer не выделяется память. Как правильно обойти этот варнинг? Код: plaintext 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29.
... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:27 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor Код: plaintext 1.
Ты разыменовываешь указатель на char и записываешь в него значение типа int. Это очень опасная дорожка. Попробуй сначала преобразовать его к указателю на int и только потом разыменовывать. ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:43 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
NekZПопробуй сначала преобразовать его к указателю на int и только потом разыменовывать. Не стоит. Ибо можно нарваться на забавные грабли с выравниванием и оптимизацией. Если раскладка байт в буфере совпадает с платформой, то лучше пользоваться memcpy. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:50 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dimitry SibiryakovНе стоит. Ибо можно нарваться на забавные грабли с выравниванием и оптимизацией. Если раскладка байт в буфере совпадает с платформой, то лучше пользоваться memcpy. Насчёт оптимизации согласен, а вот по поводу выравнивания не понял. Там же нет структуры. Или POD-типов это тоже касается? ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:53 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
NekZпо поводу выравнивания не понял На высоких уровнях оптимизации компилятор может предполагать, что указатель на целое выровнен правильно и превратить присваивание в один movd или аналогичную команду процессора. Если это предположение неверно и команда не приспособлена работать с невыровненной памятью - будет БУМ! В данном коде такого быть не может, но лучше не привыкать к плохому. И, кстати, я только сейчас обратил внимание, что аффтар-то пишет в буфер всего один байт, а не всё целое. Хотя остаток кода предполагает иначе. И warning компилятора в данном случае это указание на жосткого бага. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 17:01 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorЕсть класс, который хранит размер и буфер описывающий в общем случае строку данных. После выделения буфера, сразу прописываю размер буфера, однако vs2017 выдает warning. Я так полагаю из за того, что конкретно для *m_buffer не выделяется память. Как правильно обойти этот варнинг? и зачем делать через одно место когда можно сделать правильно? Достаточно сказать компилятору что ты хочешь. Код: plaintext 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40.
... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 20:45 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
OoCcи зачем делать через одно место когда можно сделать правильно? Достаточно сказать компилятору что ты хочешь.а зачем извращаться с вложенной структурой, если в класс просто нужно добавить поле для длины?)) sizt_t m_Length, например )) ну, и раз уж тут у нас C++, то фтопку malloc, нужно new[] ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 01:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
OoCcи зачем делать через одно место когда можно сделать правильно? Как вы можете судить о правильности, если не знаете предназначения? Моя строка - подготавливается для дальнейшей передачи через сокеты и в отличии от вашего случая - не выполняет лишних выделений памяти. Но это не суть важно. авторИ, кстати, я только сейчас обратил внимание, что аффтар-то пишет в буфер всего один байт, а не всё целоеХм, проверю, спасибо ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 06:52 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Доработал по комментариям. Вышло обыденно и неинтересно. Но главное что работает и без варнингов, спасибо, не зря решил таки с ним разобраться :) Код: plaintext 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25.
... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 07:11 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
egorychа зачем извращаться с вложенной структурой, если в класс просто нужно добавить поле для длины?)) sizt_t m_Length, например )) ну, и раз уж тут у нас C++, то фтопку malloc, нужно new[] egorych ты неправ. ИМХО ТС хотел держать строку сериализованной с длинной. Вложенная структура это не изврещение - это описание сериализованного хранилища. ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 09:06 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
[quot Ciplusor] .... Моя строка - подготавливается для дальнейшей передачи через сокеты и в отличии от вашего случая - не выполняет лишних выделений памяти. Но это не суть важно. ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 09:30 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor Код: plaintext 1. 2.
В первой строке прописывается значение типа char приведенное из int. Это нулевой байт. 1 штука. Во второй строке что-то копируется после первых 4-х штук байт. Тут как-бы предупреждение должно быть о неявном усечении int в char с возможной потерей информации и как-бы что-то мусорное в первом, втором и третьем байтах с адреса m_Buffer. Если по первому байту программа сможет определить что дальше 3 байта мусор - то как-бы но проблем. А если нет... ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 13:58 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor, Вот так надо! Код: plaintext 1.
... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 17:19 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor, структуры для описания буфера я так понимаю лучше не предлагать? только хардкор? тынц Код: plaintext 1. 2. 3. 4.
... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 17:47 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)структуры для описания буфера я так понимаю лучше не предлагать? только хардкор? Структуры это хорошо до тех пор пока не начинается обмен между платформами с разной раскладкой байт. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 18:14 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan), В структуре как раз и может случится выравнивание. ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 19:02 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
А еще malloc() может вернуть NULL ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 21:08 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dimitry Sibiryakovkealon(Ruslan)структуры для описания буфера я так понимаю лучше не предлагать? только хардкор? Структуры это хорошо до тех пор пока не начинается обмен между платформами с разной раскладкой байт. там всё равно придётся переворачивать что пришло, и смысл париться с высчитыванием адресов? с выравниванием тоже бороться можно но создавать себе гемор на ровном месте... ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 00:33 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorВышло обыденно и неинтересно. А какова была цель хранения размера и контента в одном сплошном массиве? ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 05:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CEMbCiplusorВышло обыденно и неинтересно. А какова была цель хранения размера и контента в одном сплошном массиве?обычная практика для базовых классов, зачем два раза память выделять ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 09:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)обычная практика для базовых классов, зачем два раза память выделятьНууу... а почему два раза-то? Вот: kealon(Ruslan) Код: plaintext 1. 2. 3. 4.
выделяешь один раз для data и тут же пишешь размер в len? ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 11:53 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CEMbвыделяешь один раз для data и тут же пишешь размер в len? А сам str_buffer выделяется автомагически? А в send() его придётся скармливать по частям?.. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 12:06 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CEMb, обычно ещё используют счётчики использования(refcount) что бы строки не дублировать при копировании объекта ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 12:54 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)CEMb, обычно ещё используют счётчики использования(refcount) что бы строки не дублировать при копировании объектаЯ обошелся без ссылок, если идет присваивание - то исходная строка берется указателем, если нет - аллоцируется буффер. Позволяет использовать как с автодеструктором, так и с сохранением значения. Главное условие - снизить минимизацию работы с памятью. Возможно глупое решение, но на текущий момент меня устраивает вполне. Код: plaintext 1.
Код: plaintext 1.
Код не претендует на оригинальность и возможно сплошной костыль, но я только начал изучать конструкторы копирования и перегрузку операторов присваивания :) Код: plaintext 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43. 44. 45. 46. 47. 48. 49. 50. 51. 52. 53. 54. 55. 56. 57. 58. 59. 60.
... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 16:59 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dimitry SibiryakovА сам str_buffer выделяется автомагически? А в send() его придётся скармливать по частям?.. Ну если на стеке, то автоматически, иначе динамически. В send она должна сама себя скармливать, по-хорошему: наружу из класса торчат методы, которые знают, как работать с данными. Внешний код не должен знать и разбираться с внутренними структурами. Иначе, в случае, если вдруг между len и data добавили smth, то весь внешний код поедет. Для меня сомнительна польза запихать всё в текстовый буфер ради send(). В том же случае, с добавлением smth в "структуру", мы получим больше проблем при обращении к "переменным". ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 05:15 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorЕсть класс, который хранит размер и буфер описывающий в общем случае строку данных. После выделения буфера, сразу прописываю размер буфера, однако vs2017 выдает warning. Я так полагаю из за того, что конкретно для *m_buffer не выделяется память. Как правильно обойти этот варнинг? Код: plaintext 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29.
Ну, говнокод во всей своей красе. Даже смотреть противно. Странно, что только warning и только один... Вот это вообще как у тебя компилируется? Код: plaintext 1. 2. 3. 4.
... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 10:47 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorЕсть класс, который хранит размер и буфер описывающий в общем случае строку данных. После выделения буфера, сразу прописываю размер буфера, однако vs2017 выдает warning. Я так полагаю из за того, что конкретно для *m_buffer не выделяется память. Как правильно обойти этот варнинг? Да и самое главное -- НАФИГА? ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 10:48 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
MasterZivВот это вообще как у тебя компилируется? Код: plaintext 1. 2. 3. 4.
А что не так? Тебя пробел смущает? Разыменовывается указатель на char, конвертируется в int и привет. ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 11:02 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
NekZА что не так? Тут return умножается на m_Buffer ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 11:16 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Anatoly MoskovskyNekZА что не так? Тут return умножается на m_Buffer Спасибо Бьярне за перегруженную семантику * и &. В итоге так и смотрится, вызывая когнитивный диссонанс. ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 11:24 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
>> Странно, что только warning и только один... Ну.. скажем так с нуля писать НЕ говнокод на сях наверное мало кому под силу :) Язык очень верткий, то что делается одним способом оказывается неправильным и делается еще пятью которые правильные. Все в процессе ;) ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 16:48 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
А нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =D ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 16:53 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorА нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =Dкаждый тру-программер должен написать свой CString!!!!!!!! ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 17:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorА нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =D Нафига хранить размер в нескольких последовательных элементах массива char[], а не в отдельном поле? ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 18:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorА нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =D Тут "изобретен" недо std::string, который без зависимостей. ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 18:50 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
MasterZiv Нафига хранить размер в нескольких последовательных элементах массива char[], а не в отдельном поле? Наверно чтобы прога вылетела на таком коде Код: plaintext 1. 2.
а потом баг искать долго и упорно. ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 18:56 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dima TТут "изобретен" недо std::string, который без зависимостей.Зависимость имелась ввиду - разбирать с нуля как устроен мир. Вот вы написали что бред хранить длину строки в одном с ней буфере. Я подумал - действительно, смысла нет. Да, строка пишется в Send() но в составе общего блока команды, поэтому в принципе можно и вынести размер в поле. Как будет окончательный вариант, обязательно выложу :) ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 20:01 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorВот вы написали что бред хранить длину строки в одном с ней буфере. никто такого не писал ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 20:11 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorДа, строка пишется в Send() но в составе общего блока команды, поэтому в принципе можно и вынести размер в поле. Ты попытался создать строку заточенную под сериализацию. Это излишне. Просто сделай функцию отправки строки в поток: сначала размер, потом содержимое. А строку оставь std::string. Если ты думаешь что в TCP быстрее уйдет одни кусок чем два поменьше, то заблуждаешься. В общем случае в итоге они скорее всего уйдут вместе одним IP-пакетом. ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 20:12 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)CiplusorВот вы написали что бред хранить длину строки в одном с ней буфере. никто такого не писал Он буфер выделяет в куче, отсюда грабли 21718321 ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 20:16 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorЗависимость имелась ввиду - разбирать с нуля как устроен мир. Если так, то советую начать с ассемблера, это тот самый ноль. Если потом захочется опуститься ниже нуля, то микросхемотехника. ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 20:22 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dima Tkealon(Ruslan)пропущено... никто такого не писал Он буфер выделяет в куче, отсюда грабли 21718321 он его там вообще не выделяет ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 20:27 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)Dima Tпропущено... Он буфер выделяет в куче, отсюда грабли 21718321 он его там вообще не выделяет Глубоко не вникал, но есть такие строки в коде Код: plaintext 1. 2. 3.
т.е. выделяет, но руки бы оторвать за такое выделение ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 20:33 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
NekZСпасибо Бьярне за перегруженную семантику * и &. В итоге так и смотрится, вызывая когнитивный диссонанс. Пользуясь случаем, позанудствую, что указатели и прочие ссылки надо писать слева, потому что они накладываются справа налево. int*& p - ссылка на указатель на int CiplusorНу.. скажем так с нуля писать НЕ говнокод на сях наверное мало кому под силу :)я думаю: в жизни надо попробовать всё Главное понимать, что ты пишешь и почему именно так ты пишешь. Гораздо хуже, когда человек бездумно учит best practics и перестаёт включать мозги. Да, у него "идеальный код", но в нестандартной ситуации, которой в книжках нет, он быстро садится в лужу. Даже хрен с ней, с лужей - идеальный код не всегда оптимален и безопасен. Лучше один раз пробухаться и осознать гармонию в жизни, чем читать 100 книжек о вреде алкоголя в больших дозах, а потом ещё всех донимать здоровым образом жизни. Dima Tт.е. выделяет, но руки бы оторвать за такое выделениеиногда надо от души самозабвенно поскакать по граблям, чтобы проникнуться ... |
|||
:
Нравится:
Не нравится:
|
|||
30.10.2018, 05:45 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dima Tт.е. выделяет, но руки бы оторвать за такое выделениеЯ правильно понимаю - за то что не используется static_cast? :) ... |
|||
:
Нравится:
Не нравится:
|
|||
30.10.2018, 09:02 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorDima Tт.е. выделяет, но руки бы оторвать за такое выделениеЯ правильно понимаю - за то что не используется static_cast? :) За то что m_Buffer никак не проверяется на NULL. ... |
|||
:
Нравится:
Не нравится:
|
|||
30.10.2018, 09:09 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
http://www.cplusplus.com/reference/cstdlib/malloc/?kw=malloc malloc() If the function failed to allocate the requested block of memory, a null pointer is returned. ... |
|||
:
Нравится:
Не нравится:
|
|||
30.10.2018, 09:10 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
http://www.cplusplus.com/reference/cstdlib/malloc/?kw=malloc If the function failed to allocate the requested block of memory Причём она может failed не только потому что память кончилась. ... |
|||
:
Нравится:
Не нравится:
|
|||
31.10.2018, 05:11 |
|
|
start [/forum/topic.php?all=1&fid=57&tid=2017734]: |
0ms |
get settings: |
11ms |
get forum list: |
13ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
38ms |
get topic data: |
11ms |
get forum data: |
3ms |
get page messages: |
69ms |
get tp. blocked users: |
1ms |
others: | 256ms |
total: | 410ms |
0 / 0 |