Самописная библиотека по упрощенному использованию сокетов

@Van111 · Регистрация: 03.08.2011

Студворк — интернет-сервис помощи студентам

создал lib(klient.a) который даёт у прощеный доступ к функции отсылки данных программа не будет работать в окне так как там есть функция принтф, но это исправимо вот пример как связаться с сервером и получить отослать от него данные
вот пример

START_WSA
klietn s("127.0.0.1",80);
s.get("hello") отсылка данных
s.give получение данных даные находятся в s.str
s.closesock() закрывает сокет ну можете не закрывать

END_WSA

@kravam · 25.10.2011, 20:54

Это чё за выражение?

C++
1
n=(sock,&stroka[0],sizeof(stroka),0)

@Van111 · 25.10.2011, 20:58 **[ТС]**

Сообщение от ForEveR

((unsigned long *)&ss.sin_addr)[0]=((unsigned long **)hst->h_addr_list)[0][0]; - а ведь есть reinterpret_cas

скажи это Крису Касперскому

Сообщение от ForEveR

WSAStartup(0x202,&wsa); - Опять магическое число.

это нужно для винды

Сообщение от ForEveR

char str[10240]; - магические числа

максимальный размер пакета 4048 но я сделал с запасом это можно

Сообщение от ForEveR

не klient а client.

исправить легко

Сообщение от ForEveR

#include "stdio.h" в коде на С++

можно через iostream

Сообщение от ForEveR

4.

всё тот же Крис тем более это минимальные данные для клиента

Сообщение от ForEveR

while((nsize=recv(s,&str[0],1239,0)) != SOCKET_ERROR) - магические числа.

почитайте про recv, 1239 это размер str без последнего байта он нужен чтоб туда 0 вставить(последний байт). и Опять крис

Сообщение от ForEveR

Использование printf для печати. Печать ошибок в stdout а не stderr.

ну тут виноват потому что не знал об этом

Добавлено через 45 секунд

Сообщение от kravam

Это чё за выражение?

n= количество считанных байт
ну почитайте Криса

silent_1991 · 25.10.2011, 20:59

Van111, магические числа после вашего описания магическими быть не перестают. Чтобы они перестали быть таковыми, надо завести именованные константы на каждое такое число. Имя будет нести информацию, в отличие от непонятного набора цифр.

@Van111 · 25.10.2011, 21:00 **[ТС]**

может просто документацию написать

по моему коду

@fasked · 25.10.2011, 21:06

Сообщение от Van111

скажи это Крису Касперскому

Может быть его код был на Си?

Сообщение от Van111

это нужно для винды

В windows.h вроде бы есть макросы MAKEWORD, которые позволяют сделать это более красиво. В любом случае всегда можно объявить свою константу.

Сообщение от Van111

максимальный размер пакета 4048 но я сделал с запасом это можно

Буфер и максимальный размер пакета не связаны. Вся обработка сырых пакетов происходит внутри сокетов - они для этого и сделаны. Вы думаете, что одним receive нельзя принять больше 4048 байт? Если да, то Вы ошибаетесь.

Сообщение от Van111

исправить легко

Так исправляйте. Ради чего Вы просили критики, чтобы огрызаться в ответ или на самом деле улучшать библиотеку?

Сообщение от Van111

можно через iostream

Нужно. Вы пишете на C++.

Сообщение от Van111

всё тот же Крис тем более это минимальные данные для клиента

Крис Касперски недоученный хакер, а не программист. И его способности в грамотном программировании под большим сомнением.

Сообщение от Van111

почитайте про recv, 1239 это размер str без последнего байта он нужен чтоб туда 0 вставить(последний байт). и Опять крис

Я должен читать код, а не дополнительную литературу, чтобы разобраться в том, как Ваша библиотека работает. Опять же 1239 - то Вы говорите 4048, то 1239... И опять же одним вызовом recv можно принять гораздо больше.

@ForEveR · 25.10.2011, 21:16

Van111, Код С++. Используйте именованные константы. А не магические числа.

@Van111 · 26.10.2011, 13:08 **[ТС]**

Сообщение от fasked

Я должен читать код, а не дополнительную литературу, чтобы разобраться в том, как Ваша библиотека работает. Опять же 1239 - то Вы говорите 4048, то 1239... И опять же одним вызовом recv можно принять гораздо больше.

глупая ошибка там хотел поставить 10239

Добавлено через 48 секунд

Сообщение от fasked

Нужно. Вы пишете на C++.

учту и переделаю

Добавлено через 1 минуту

Сообщение от fasked

Может быть его код был на Си?

ну вы сами сказали что он не мастер а код на с++

Добавлено через 1 минуту

Сообщение от fasked

Буфер и максимальный размер пакета не связаны. Вся обработка сырых пакетов происходит внутри сокетов - они для этого и сделаны. Вы думаете, что одним receive нельзя принять больше 4048 байт? Если да, то Вы ошибаетесь.

сколько по вашему нужно предоставить размер для буфера????

Добавлено через 2 минуты

Сообщение от ForEveR

((unsigned long *)&ss.sin_addr)[0]=((unsigned long **)hst->h_addr_list)[0][0]; - а ведь есть reinterpret_cast.

а вот тупршу поподробней как эту строку переделать?

@ForEveR · 26.10.2011, 13:12

Van111,

C++
1
reinterpret_cast<unsigned long*>(&ss.sin_addr)[0] = reinterpret_cast<unsigned long**>(hsst->h_addr_list)[0][0];

Синтаксический сахар конечно. Но лучше юзать такие преобразования.

@Van111 · 26.10.2011, 13:22 **[ТС]**

кто знает функцию возвращающюю размер пакета который может быть послан функцией send

@fasked · 26.10.2011, 14:03

Сообщение от Van111

глупая ошибка там хотел поставить 10239

Это все равно будет магическое число. Читающему такой код человеку будет трудно понять. А Вам, как разработчику этого кода, будет неудобно в том плане, что если Вы захотите изменить размер буфера, то придется его менять в нескольких местах. Про одно из таких мест можно забыть, что в свою очередь влечет неочевидные ошибки. Поэтому:

C++
1
const size_t packet_buffer_size = 10239;

Ну или как-то так. Но явно не магическими числами.

Сообщение от Van111

сколько по вашему нужно предоставить размер для буфера?

Лично я бы предоставил пользователю библиотеки выбирать размер буфера. Например, если мне надо отправлять небольшие пакеты по несколько байт (не больше 100), то оставшиеся ~10Кб буду жить просто так и никогда не использоваться. Особенно это неприятно на серверной стороне, где память может быть дорогим ресурсом. А другому разработчику может и не хватить 10Кб, он захочет иметь буфер размером побольше, а поменять это значение уже не получится.

@Van111 · 26.10.2011, 14:24 **[ТС]**

fasked, вот фрагмент

C++
1
2
3
4
5
6
7
8
9
10
11
int size_buffer=10240;
int size_send=1024;
 
void size_buf_recv_and_send(int a,int b)
{
    size_buffer=a;
    size_send=b;
        setsockopt(s,SOL_SOCKET,,,) // вот тут дальше с параметрами не могу разобратся 
        setsockopt(s,SOL_SOCKET,,,) // выдаёт ошибку в параметрах должно быть SO_RCVBUF SO_SNDBUF  
 
}

@fasked · 26.10.2011, 14:33

Van111, к сожалению я с конкретными ошибками на windows не могу помочь почти ничем

.

@Van111 · 26.10.2011, 14:54 **[ТС]**

fasked, кстати можно вообще сделать так для всех сокетов устанавливать максимальный буфер для recv 10 мегабайт это абсалютно не портит программу это даст возможность считать любые данные а для send установит 1 килобайт и сделать такой код

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
const   int size_buffer=1000000;
const   int size_send=1024;
void client::give(char* data)
{
int error_size,size=strlen(data);
error_size=size / size_send;
if((size - error_size*size_send)>0)
error_size++;
if((size - error_size*size_send)<0)
error_size--;
for(int i=0;i<error_size;i++)
send(s,&data[0],strlen(&str[0]),0); 
}

Добавлено через 1 минуту
и пользователю не надо печшся о том сколько там данных посылать

Добавлено через 52 секунды
а в client::client как раз добавить те две функции с которыми у меня проблема

@fasked · 26.10.2011, 15:02

Сообщение от Van111

кстати можно вообще сделать так для всех сокетов устанавливать максимальный буфер для recv 10 мегабайт это абсалютно не портит программу это даст возможность считать любые данные а для send установит 1 килобайт

Это не очень красивое архитектурное решение

@Van111 · 26.10.2011, 15:03 **[ТС]**

вот код ещё не до конца сделанный но всеровно посмотрите

Добавлено через 58 секунд

Сообщение от fasked

Это не очень красивое архитектурное решение

хорошо сегодня попозже изменю

Новые блоги и статьи Все статьи Все блоги /
Саморегулирующийся социальный контракт для сервера cross-section. Hrethgir 14.08.2026 С кодом конечно таких глубоких размышлений пока не было, впрочем я уже привык к алгоритмизации. Суть предмета записи: снова в диалоге с нейросетью (я взял пока себе ник для учётки админа - Rector). . . .	Часы электронные Uhbif79 12.08.2026 Выкладываю программу часов. Программа позволяет: 1. Использовать системное время и дату, 2. Есть возможность вводить время и дату вручную. 3. Реализованы 2 будильника: начало и конец рабочего дня. . . .	Часы с будильником на основе класса QLCDNumber Uhbif79 12.08.2026 Всем добрый день, выкладываю программу часов с будильником на основе класса QLCDNumber. Здесь я пробовал самостоятельно создавал классы, впервые столкнулся с видимостью переменной одного класса из. . .	Установка MinGW GCC 16.2 и CMake 8Observer8 10.08.2026 VK Видео: https:/ / vkvideo. ru/ video-240781534_456239017 YouTube: eY5-5PyI9NM Текстовая версия
Неделя из жизни имитационной модели склада: мои кривые руки растут, откуда надо anaschu 10.08.2026 Неделя из жизни имитационной модели склада: как я почти написал неправильную логику и что с этим делать Работаю сейчас над учебно-рабочим проектом: строю в AnyLogic имитационную модель процессов. . .	Калькулятор для расчета родства russiannick 07.08.2026 1. Задача: Создать калькулятор для расчета родства. Родственных связей существует 8 ступеней, такие как: p - отец P - мать q - муж Q - жена b - брат B - сестра s - сын S - дочь	Мир по моей воле kumehtar 07.08.2026 Когда-то кажется, что всё просто. Ты весь такой светлый. Причиняешь добро. Борешься за справедливость в этом тёмном мире. Потом начинаешь замечать одну неприятную вещь. Почти каждый хороший. . .	Кредитный калькулятор Maks 05.08.2026 Решение задачи по прикладной информатике средствами 1С. Задача: Напишите приложение-калькулятор, которое помогает рассчитывать параметры кредита для аннуитетного и дифференцированного видов. . .

silent_1991 5058 / 3118 / 271 Регистрация: 11.11.2009 Сообщений: 7,044
	25.10.2011, 20:59
	Van111, магические числа после вашего описания магическими быть не перестают. Чтобы они перестали быть таковыми, надо завести именованные константы на каждое такое число. Имя будет нести информацию, в отличие от непонятного набора цифр. 0

@Van111 290 / 193 / 23 Регистрация: 03.08.2011 Сообщений: 2,824 Записей в блоге: 12
	25.10.2011, 21:00 [ТС]
	может просто документацию написать по моему коду 0

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	25.10.2011, 21:16
	Van111, Код С++. Используйте именованные константы. А не магические числа. 1

@Van111 290 / 193 / 23 Регистрация: 03.08.2011 Сообщений: 2,824 Записей в блоге: 12
	26.10.2011, 13:22 [ТС]
	кто знает функцию возвращающюю размер пакета который может быть послан функцией send 0

@fasked 5045 / 2624 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 5
	26.10.2011, 14:33
	Van111, к сожалению я с конкретными ошибками на windows не могу помочь почти ничем . 0