Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
BOOL CMarkup::Search(CString OprName, CString strSQL, short nType)
{
...
sCommandLine = sPath.Left(nPos)+ _T("\\Getorder.exe search ")+OprName+_T(" ")+strSQL+_T(" ")+strType;
::wcscpy(cCommandLine, sCommandLine);
BOOL ret = CreateProcess(NULL, cCommandLine, NULL, NULL, FALSE, CREATE_NO_WINDOW, NULL, NULL, &si, &pi);
if (ret) {
CloseHandle(pi.hThread ); //Close the thread handle as soon as it is no longer needed.
WaitForSingleObject(pi.hProcess, INFINITE); //Suspend our execution until the child has terminated.
GetExitCodeProcess(pi.hProcess ,&dwExitCode); //The child process terminated;get its exit code.
//if(dwExitCode == STILL_ACTIVE) TerminateProcess(pi.hProcess, 0);
CloseHandle(pi.hProcess); //Close the process handle as soon as it is no longer needed.
}
CString ss,s;
ss = _T("s.txt");
CStdioFile in;
if(!in.Open (sPath+_T("\\")+ss,CFile::modeRead | CFile::typeBinary))
return TRUE;
while(in.ReadString(s))
{
s.TrimLeft();
s.TrimRight();
if(s != _T("**********")) continue;
ORDERTYPE m_OrderData;
in.ReadString(s);
s.TrimLeft();
s.TrimRight();
m_OrderData.OprName = s;
in.ReadString(s);
s.TrimLeft();
s.TrimRight();
m_OrderData.Language =s;
in.ReadString(s);
s.TrimLeft();
s.TrimRight();
m_OrderData.OrderNo =s;
...
}
in.Close();
DeleteFile(sPath+_T("\\")+ss);
Прислали исходники приложения на C++ с MFC. Там просто один сплошной ГК от начала и до конца, но это место просто убило.
Задача - найти заказ в базе данных MSSQL по SQL-запросу. Для этого они запускают отдельное приложение .NET, которое подключается к базе и дампит запись в текстовый файл, а приложение C++ из него читает.
Вброшу.
Это Unix-way. В лялихе так принято: использовать мелкие программки для решения задач. Если уже есть прожка, выполняющая поиск нужным образом, то именно её и вызываем из нашей. Результат получаем через текстовый файл, всё верно.
Так что писал явно луноход, перешедший на винду. 😀
1. Обращение к MSSQL из C++
2. Вызов .NET DLL через COM Interop, если уж ниасилили 1
3. Ну ладно, не знают про 1 и 2, но про пайпы-то должны знать, это святое!
это говновэй в N слоев
на верхнем слое говновей в том, что результат складывается в текстовый файл, имя которого явно захардкожено как в этом коде, так и в коде дотнетовской отрыжки, и поэтому надо как то следить чтобы никто кроме нас не запустил эту утилиту параллельно, и никто кроме нас не смог почитать и удалить файл - имеем заведомо однопоточность, уверен что с нерешенными конфликтами
на слое пониже - вызывается утилита через командную строку, что крайне медленно и небезопасно (вся безопасность сводится к правам на запуск утилиты пользователем текущего процесса), хотя существует способ засунуть дотнетовский код в длл и вызывать методы из оной, затем приходится тупейшим способом парсить текстовый файл
на слое еще пониже - кто и на каком этапе делает авторизацию к базе? всё в утилите? тоже захардкожено?
на слое еще пониже - тотальный говнокод и низкая культура реализации даже задуманного
ну и "не можем открыть файл с результатами - ну и пох, возвратим TRUE" - умилило
Это Unix-way. В лялихе так принято: использовать мелкие программки для решения задач. Если уже есть прожка, выполняющая поиск нужным образом, то именно её и вызываем из нашей. Результат получаем через текстовый файл, всё верно.
Так что писал явно луноход, перешедший на винду.
😀
1. Обращение к MSSQL из C++
2. Вызов .NET DLL через COM Interop, если уж ниасилили 1
3. Ну ладно, не знают про 1 и 2, но про пайпы-то должны знать, это святое!
на верхнем слое говновей в том, что результат складывается в текстовый файл, имя которого явно захардкожено как в этом коде, так и в коде дотнетовской отрыжки, и поэтому надо как то следить чтобы никто кроме нас не запустил эту утилиту параллельно, и никто кроме нас не смог почитать и удалить файл - имеем заведомо однопоточность, уверен что с нерешенными конфликтами
на слое пониже - вызывается утилита через командную строку, что крайне медленно и небезопасно (вся безопасность сводится к правам на запуск утилиты пользователем текущего процесса), хотя существует способ засунуть дотнетовский код в длл и вызывать методы из оной, затем приходится тупейшим способом парсить текстовый файл
на слое еще пониже - кто и на каком этапе делает авторизацию к базе? всё в утилите? тоже захардкожено?
на слое еще пониже - тотальный говнокод и низкая культура реализации даже задуманного
ну и "не можем открыть файл с результатами - ну и пох, возвратим TRUE" - умилило
> Это Unix-way. В лялихе так принято
Ути-пути
так что писал явно виндомозглый дебил.
PS. Я хотел минусануть но промазал.
И этот же способ обращения к БД в том же cpp-файле методом копипаста распространён ещё на несколько других таблиц...