Close Topic Options

[Lexus]

У нас есть здесь спецы в c++?
Я тут начал работать По профессии я математик, но програмировать надо больше чем считать. Пока проект не начался, я хотел немножко подготовиться. задача стоит простая:

считать с определённой директории список файлом и записать его в array/list

делал так, но ничего не выдаёт:

CODE

#include "stdafx.h" #include <iostream> #include <windows.h> #include <wtypes.h> #include <string> using namespace std; bool FileExists(const TCHAR *fileName); bool dirExists(const string& dirName_in); int _tmain(int argc, _TCHAR* argv[]) { const string Folder = "C:\\tests\\MUC unverdichtet\\";   HANDLE fHandle; WIN32_FIND_DATA wfd; fHandle=FindFirstFile((LPCWSTR)Folder.c_str(),&wfd);   do {     if (!( (wfd.cFileName[0]=='.') && ( (wfd.cFileName[1]=='.' && wfd.cFileName[2]==0) || wfd.cFileName[1]==0 ) ))     {         if (wfd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)         {             MessageBox(0,wfd.cFileName,(LPCWSTR)"Folgendes Verzeichnis wurde gefunden:",0);         }         else         {             MessageBox(0,wfd.cFileName,(LPCWSTR)"Folgende Datei gefunden:",0);         }     } } while (FindNextFile(fHandle,&wfd)); FindClose(fHandle); return 0; }

как же я скучаю по c# ))

[Lord KiRon]

1. (и главное) Нафига тебе MesssageBox причем в програме которая явно консольная - хрена он что выдаст. поменяй например на cout или printf или поменяй проект на Win32, но первый вариант проще.

2. В фолдере задать маску , типа "*.*" или "." а не только путь.
3. Не забудь проверку если не нашло ничего первый раз
4. Не забывай инициализировать переменные и обнулять структуры. К твоим проблемам это не имеет никакого отношения но это правильная практика.

This post has been edited by Lord KiRon on 20-03-2013, 00:15

[Lexus]

QUOTE (Lord KiRon @ 19-03-2013, 22:15)

1. (и главное) Нафига тебе MesssageBox причем в програме которая явно консольная - хрена он что выдаст. поменяй например на cout или printf или поменяй проект на Win32, но первый вариант проще. 2. В фолдере задать маску , типа "*.*" или "." а не только путь. 3. Не забудь проверку если не нашло ничего первый раз 4. Не забывай инициализировать переменные и обнулять структуры. К твоим проблемам это не имеет никакого отношения но это правильная практика.

завтра попробую. спасибо. я тут не раз ешё буду доставать )

[Lexus]

CODE

const string Folder = "C:\\tests\\MUC unverdichtet\\*.*";   HANDLE fHandle; WIN32_FIND_DATA wfd;   string b = Folder+"*"; fHandle=FindFirstFile((LPCWSTR)Folder.c_str(),&wfd);   do {     if (!( (wfd.cFileName[0]=='.') && ( (wfd.cFileName[1]=='.' && wfd.cFileName[2]==0) || wfd.cFileName[1]==0 ) ))     {         if (wfd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)         {             cout<<wfd.cFileName<<endl;         }         else         { cout<<wfd.cFileName<<endl;;         }     } } while (FindNextFile(fHandle,&wfd)); FindClose(fHandle);

Output: 0013F8DC или другие подобные.

[Lord KiRon]

Хэндл инвалидный , значит или путь неверный или файлов не найдено. Хотя не понятно на каком этапе ты делал скриншот.

Да, и еще чтоб было проще дебагировать - выруби юникод в настройках прокта.


This post has been edited by Lord KiRon on 20-03-2013, 20:01

[Set]

Вспомнил, почему не перевариваю С++

http://msdn.microsoft.com/en-us/library/windows/desktop/aa365200%28v=vs.85%29.aspx

[Lexus]

QUOTE (Lord KiRon @ 20-03-2013, 17:57)

Хэндл инвалидный , значит или путь неверный или файлов не найдено. Хотя не понятно на каком этапе ты делал скриншот. Да, и еще чтоб было проще дебагировать - выруби юникод в настройках прокта.

забил я на это и сделал через dirent

Но не расслабляйтесь, всё только наинается=

[Lord KiRon]

QUOTE (Set @ 21-03-2013, 09:34)

Вспомнил, почему не перевариваю С++

Плохому танцору...

[Lexus]

Продолжимс

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

std::vector<std::string> results
while(getline(ss.str,separator)
{
results.push_back(str);
}

и этот грёбанный пушбек жрёт 90% моего времени. Я пробывал сначала толкать в лист, в нём судя по описанию пушбек добрее, а потом коппировать в вектор. Но что-то всё стало медленнее чем))))

Можно ли как-то это обойти?

[Lord KiRon]

Тебе повезло что ты это не на .Net делаешь.
А вообще при таком подходе жрется до хрена памяти, тут так прямо и не решить, надо в зависимости от ситуации и использования.
Например хранить в вкторе не стринги а позиции запятых.

[Lexus]

QUOTE (Lord KiRon @ 30-03-2013, 23:07)

Тебе повезло что ты это не на .Net делаешь. А вообще при таком подходе жрется до хрена памяти, тут так прямо и не решить, надо в зависимости от ситуации и использования. Например хранить в вкторе не стринги а позиции запятых.

Память меня меньше CPU напрягает)))))

[Set]

Определи число строк в файле, сразу создай вектор нужного размера и пиши в него напрямую, как в массив: results[n] = ...

[Lexus]

QUOTE (Set @ 31-03-2013, 06:45)

Определи число строк в файле, сразу создай вектор нужного размера и пиши в него напрямую, как в массив: results[n] = ...

А возможно определить кол-во не читая всё файло?

[Lord KiRon]

Если ты утверждаешь что память вообще не проблема , то открывай файл черз mapped view (чтоб он при доступе "выглядел" как бафер, а не файл) и сохраняй позиции запятых в файле в отдельный вектор.
Походу можно не пользоватся вектором а хранить в массиве используя doubling, то есть держишь массив созданный через new скажем на 1000 мест и переменную "индекс" последнего свободного места + переменную размера массива.
Если дошел до последнего (1000-го) то создаешь новый массив в два раза большр (то-есть 2000) и копируешь в него старый memcopy , а старый освобождаешь и так далее...
операции подстановки лонгов в массив почти мгновенные, копирование непрерывного блока памяти тоже операция достаточно быстрая и в принципе будет (при правильно выбранном начальном размере) достаточно редкой.
Если память вообще не ограничена можешь создать массив вообще один раз - размером с файл, тогда точно ничего и копировать не придется

This post has been edited by Lord KiRon on 31-03-2013, 17:50

[FiL]

я не понял. ты считываешь весь файл и засовываешь его в вектор. А нахрена? Считал строку, обработал и пошел дальше. Зачем ты хранишь все прочитанные строки?

Ну и таки если уж так хочется засовывать весь файл в память (несколько странно в этом разрезе говорить о гигантских файлах), то ты-бы место в векторе выделял крупными кусками. А то для каждой строки вызывается реаллокация памяти, ибо вектор переполнился. Выделил место по 1000 строк - считал 1000. Потом снова выделил одним разом еще под 1000. И так далее. Ну, или если строк всегда более миллиона, то можно не по 1000, а по 100.000 за итерацию.