0

Уже теряюсь в догадках. Не открывается файл, имя которого передано через сокет от клиента серверу. Передается все правильно, если только добавляется какой-нибудь перевод строки. Может надо переделать из char[] в строку?

  char buffer[1024];

  ....(тут recv  через сокет), в буфере при отладке "С:/6.pro"
  FILE *in = fopen(buffer, "r");
  if(in == NULL) printf("Error opening file");
  else printf("File was successfully opened");
8
  • в задании так 2. Клиент посылает серверу запрос, содержащий путь искомого файла и размер блока на который должен разбиваться файл. 3. Сервер ищет файл и поблочно пересылает файл, записываемый клиентом (в случае отсутствия искомого файла выдается сообщение об ошибке). 15 ноя 2012 в 15:18
  • 1
    А ну пусть поблочно, для успокоения нервов преподавателя. Цикл fread -- send. Блоками (хотя само соединение -- труба, точнее две трубы -- туда и обратно, и никаких разделителей блоков там изначально нет).
    – alexlz
    15 ноя 2012 в 15:35
  • @АннаХатико, один маленький совет по отладке. Обычно все наглядно при примерно такой печати строк, содержимое которых "сомнительно" printf ("buf: [%s]\n",buf); пробелы и т.п. в начале или в конце буфера сразу бросаются в глаза. И еще проверяйте результаты вызова и при ошибке всегда печатайте их, вызывайте perror(), а не ограничивайтесь сообщением типа "не могу открыть файл". Кстати, еще одно замечание. Не стоит в send() посылать весь буфер ( BUF_SIZE). Правильней ограничится только байтами, относящимися к делу. -- (еще один совет не влезает. см продолжение)
    – avp
    15 ноя 2012 в 16:46
  • (продолжение) И учтите, в recv() Вы можете получить только часть посланных за один send() сообщения. Да, TCP это поток байт, в нем нет никаких "отметок" от каждого из send(). Реально recv() вернет min из количества запрошенных и находящихся в буфере ОС данных. Для localhost или быстрой и незагруженной сети получение части посланных данных нехарактерно. Поэтому на этот момент редко обращают внимание в демо-программах, но его обязательно надо иметь в виду. В реальных протоколах либо в начале порции данных передают ее длину, либо используют маркер конца. У Вас это м.б. '\\n' или '\\0'.
    – avp
    15 ноя 2012 в 17:01
  • @avp Читать, пока результатом recv() не будет 0. Если приём блокирующий (по умолчанию), то значит больше ничего и не будет.
    – alexlz
    15 ноя 2012 в 17:09

3 ответа 3

1

Хм. Примерно так

 fgets(buff, BUF_SIZE, stdin);
 n = strlen(buff);
 if (buff[n-1] == '\n') --n;
 send(my_socket, buff, n, 0);

Передавать не весь buff, а только нужные символы.

18
  • теперь возник такой вопрос: когда пересылается путь к файлу - для некоторых файлов приклеивается к имени кракозябры рандомные, поэтому они не открываются.. как этого избежать? это вообще происходит для файлов, которые нельзя просчитать.. 15 ноя 2012 в 17:31
  • buffer "D:\data2.txtДю^UХJw" вот такое например..( 15 ноя 2012 в 17:43
  • вы строку завершайте нуль символом. 15 ноя 2012 в 17:52
  • на сервер приходит в таком виде 15 ноя 2012 в 18:06
  • 1
    Насколько я понимаю (по метке vc++) речь идет о винде. В ней fgets() считает в буфер строку, завершающуюся не одним '\\n', а двумя символами '\\r''\\n'. Это надо учитывать. @АннаХотико, для отладки печатайте и в клиенте и в сервере результат, возвращаемый всеми send() и recv() и содержимое буфера (естественно только отправленные/принятые байты).
    – avp
    15 ноя 2012 в 20:09
1
  • Протестируйте ваш код с fopen("С:/6.pro", "r"). Что получилось?

  • Может быть, в вашем буфере оказывается не нуль-терминированная строка?

  • Потенциально (хотя очень вряд ли) может не быть соответствующих permission'ов для файла.

  • fopen такие вещи прожевывает, но все-таки у вас неправильный слэш в пути "C:/6.pro".

  • Это оффтоп, но если вы ставите тэг C++, то используйте std::ifstream или std::istreambuf_iterator.

1

Проверьте, закрываете ли вы этот файл до того, как открыли снова, посмотрите что в переменной errno после данного вызова:

char *str;
...

if(in == NULL) {
 str = strerrno(errno);
 printf("Error opening file - %s",str);
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.