Комментарии 26
это нужно чтобы разобраться как оно работает или есть практическая необходимость?
Код сложно читать. Для начала можно выложить на гитхаб, там хотя бы нормальная подсветка синтаксиса. Потом непонятно почему такая адская смесь си и си++
Тут итераторы а в другом месте уже файлы через FILE.
С ресурсами все плохо ( массивы выделяются, память не освобождается ).
Ну есть странные места
char str_[N + 1];
memset(str_, 0, N);
str_[N] = '\0';Вначале обнулили его первые N-1 элементов, а потом еще и последний тоже.
На мой взгляд главная проблема кода что он очень плотный, и смешанный. В смысле тут и шифрование, и передача данных по сети, создание потоков, и снятие изображения. В идеале если разделить все это на разные куски то можно и тестировать отдельно и чистота кода должна улучшиться.
Вас, возможно, интересует, почему ваша статья заминусована? Я кнопки голосования не трогал, но позволю себе высказать гипотезу.
Формулировка задачи выглядит очень странно. Что такого особенного в передаче файлов по сети? Почему без сторонних библиотек? Зачем шифровать? Почему не взять более традиционные алгоритмы?
Ну и технический уровень реализации тоже не высок. Простите за сравнение, но у меня студенты третьего курса не самого профильного факультета на сокетах посложнее вещи делают. Не говоря уже про то, что они делают с библиотеками.
Это вовсе не означает, что вы не можете написать о своём опыте на хабр, но нужно трезвее оценивать уровень и соответственно менять фокус с того, что вы сделали на то, почему. Может быть, вы учитесь программировать и хотите поделиться своей историей? Или делаете сложную техническую систему, где программирование лишь неважная деталь? При нужном ракурсе вы сможете получить тут поддержку и советы, а не минусы.
Вас, возможно, интересует, почему ваша статья заминусована?
Меня это мало волнует, я накидал текст минут за 20 при уже сыроватом коде.
Формулировка задачи выглядит очень странно.
Да.
Что такого особенного в передаче файлов по сети? Почему без сторонних библиотек? Зачем шифровать? Почему не взять более традиционные алгоритмы?
Особенно то, что можно смотреть рабочий стол. Без сторонних потому что на форуме врятли кто-то будет линковать код со сторонними библиотеками. И без этого помогали два с половиной инвалида. А вы хотите гонять по сети интернет скриншоты рабочего стола не зашифрованными? Приведите пример любого традиционного алгоритма.
Простите за сравнение, но у меня студенты третьего курса не самого профильного факультета на сокетах посложнее вещи делают
А если бы я делал что-то посложнее, во первых не сделал бы, во вторых кода было бы много.
Или делаете сложную техническую систему, где программирование лишь неважная деталь?Программирование такое себе занятие, когда копнешь глубже остальных приходится искать и не всегда находить примеры и решения. Да и по образованию я тоже не спец в этих областях.
Меня это мало волнует, я накидал текст минут за 20 при уже сыроватом коде.
Зачем кидать на хабр сыроватый невнятный текст?
Особенно то, что можно смотреть рабочий стол.
Запакуйте файл в zip c паролем и передавайте. Зачем совмещать передачу информации с шифрованием, тем более что вы совершенно не рубите в теме алгоритмов шифрования, о чем свидетельствует плохо зашифрованный вами файл.
Без сторонних потому что на форуме врятли кто-то будет линковать код со сторонними библиотеками.
На форуме — вряд ли. Так и шлите этот текст на форум. А на Хабр, будьте любезны, подучитесь чему-нибудь и поделитесь полезным а не сырыми недоделками.
Вдобавок «без сторонних библиотек», но вместо программы вы ТУПО ЗАПУСКАЕТЕ FTP клиент. ЗАЧЕМ?
А вы хотите гонять по сети интернет скриншоты рабочего стола не зашифрованными?
Это вы хотите их гонять и гоняете. Кроме того вообще публикуете скриншот незашифрованным.
Приведите пример любого традиционного алгоритма.
Шифр Виженера реализуется в два щелчка.
Программирование такое себе занятие, когда копнешь глубже остальных приходится искать и не всегда находить примеры и решения.
Программирование — это не поиск примеров в Интернете.
Для плюсов есть вещи типа Conan, которые решат проблему со сторонними библиотеками.
Если совсем тяжело — делайте сабмодули, подавляющее большинство библиотек идут в исходниках, собрать их перед сборкой основного «продукта» совершенно не проблема, даже кросс-платформенно, 2021 год на носу, а вы всё с memset балуетесь.
В чем особенность передачи именно картинки? Любой файл прекрасно передается, ведь его можно представить в виде байтового потока, и запихнуть в сокет.
За шифрование — здесь явно смешение абстракций. Отдельный код должен шифровать-дешифровать. И совершенно другой — отправлять. Тогда можно будет использовать разные алгоритмы шифрования, и передавать эти же зашифрованные файлы не только сокетами.
Ну и субъективно — в вашей статье я не вижу уважения к читателю. Вы небрежно формулируете задачу, небрежно описываете решение, и статья тяжелее читается. Ну и судя по вашим ответам на комментарии вы плохо воспринимаете конструктивную критику. Если хотите расти как программист, вам нужно фиксить эти качества.
Recovery Mode? Seriously?
как и весь текст поста код надо целиком на говнокод.ру отправлять.
итого: КГ\АМ
Странная постановка задачи и странное решение. Совсем без библиотек не получится всё равно, можете посмотреть, сколько всего разного прилинковалось к вашему бинарнику.
ЗЫ: писать простыню г-кода только ради убедиться, что он собирается и работает, лично мне было бы лениво.
Больше года прошло с момента вашей первой публикации. Обе статьи в глубоком минусе. В обеих статьях в комментариях открыто отмечается ваша непрофпригодность.
Вы принципиально отказываетесь от самообразования или это некая форма протеста?
Древний костыль на старом костыле