diff --git a/DOCS/Алексей/Board.md b/DOCS/Алексей/Board.md index 102b856..b0760b5 100644 --- a/DOCS/Алексей/Board.md +++ b/DOCS/Алексей/Board.md @@ -6,12 +6,11 @@ kanban-plugin: board ## backLog -- [ ] выписать все варианты взаимодействия между всеми клиентами и сервером -- [ ] sendSystem::sendXmlAnswer новый вариант отпарвки XML пакетов ## bugs +- [ ] QT сервер Найти причину двойного вызова проверки при логине инструктором ## feature client Unity @@ -30,18 +29,24 @@ kanban-plugin: board ## feature server -- [ ] Прибраться в Server -- [ ] рефакторинг - [ ] добавить генерацию пустых файлов, если shared не найден +- [ ] добавить подключение без DB +- [ ] ПЕРЕВЕСТИ все действия под операции и формировать процент из них ## NOW -- [ ] добавить автоматическое выключение после создания копии ## Complete +- [ ] добавить автоматическое выключение после создания копии +- [ ] sendSystem::sendXmlAnswer новый вариант отпарвки XML пакетов +- [ ] выписать все варианты взаимодействия между всеми клиентами и сервером +- [ ] QT клиента: Не показывает блокировка, если версия удаляется +- [ ] рефакторинг +- [ ] Прибраться в Server +- [ ] не работает восстановление файлов - [ ] запрет на удаление base - [ ] None hash - [ ] сверстать окно создания новой версии @@ -265,6 +270,10 @@ kanban-plugin: board ## BUGFIX Complete +- [ ] QT клиент: device not open после прерывания загрузки +- [ ] QT сервер При изменении версии правильный списке с файлами прилетает со второго раза +- [ ] QT клиент, если обновление в режиме инструктора доступно, кнопку запуск отключать +- [ ] QT сервер замерзает после выхода пользователя во время скачивания - [ ] Клиент QT выводить инфу если клиент не подключен - [ ] Клиент QT не подключатся повторно, если соединение уже установлено - [ ] Клиент QT Неправильное отображение количества загруженных файлов diff --git a/DOCS/Алексей/ТЕСТ-КЕЙСЫ.md b/DOCS/Алексей/ТЕСТ-КЕЙСЫ.md index 2443d6c..e0bf81b 100644 --- a/DOCS/Алексей/ТЕСТ-КЕЙСЫ.md +++ b/DOCS/Алексей/ТЕСТ-КЕЙСЫ.md @@ -29,4 +29,14 @@ 5. Обновление с клиента на сервер - ЕСТЬ 6. Проверка с перезапуском клиента - ХЭШ КОРРЕКТНЫЙ 7. Восстановление версии с сервера на клиент -8. Проверка с перезапуском клиента \ No newline at end of file +8. Проверка с перезапуском клиента + +Работа с версиями +1. QT клиент: Копирование версии +2. QT клиент: Копирование версии с указанием существующего имени -> ошибка +3. QT клиент: Удаление версии +4. QT клиент: Удаление базовой версии -> ошибка +5. QT клиент: Удаление версии которая выбрана -> ошибка +6. QT клиент: переключении версии +7. Unity: отправка в базовую версию +8. \ No newline at end of file diff --git a/ServerLMS/Systems/commonclienthandler.cpp b/ServerLMS/Systems/commonclienthandler.cpp index 6012e3e..e7e769d 100644 --- a/ServerLMS/Systems/commonclienthandler.cpp +++ b/ServerLMS/Systems/commonclienthandler.cpp @@ -11,6 +11,8 @@ void CommonClientHandler::initialize(QMap *clientsMap, Pro this->processingSystem = processingSystem; this->dataParser = dataParser; this->logger = logger; + + connect(this,&CommonClientHandler::sigSendToLogger,logger,&Logger::addTextToLogger,Qt::AutoConnection); } void CommonClientHandler::sendNewVersionListToAllClient() @@ -85,7 +87,8 @@ void CommonClientHandler::slot_msgToClientFromGUI(QString login, QString text) QString str = "Msg To Client [" + peerAddress + ":" + peerPort + "] : " + textMsg; - logger->addTextToLogger(str); + emit sigSendToLogger(str); + break; } } @@ -111,7 +114,7 @@ void CommonClientHandler::slot_msgToGUIfromClient(QString login, QString text) QString str = "Msg From Client [" + peerAddress + ":" + peerPort + "] : " + textMsg; - logger->addTextToLogger(str); + emit sigSendToLogger(str); break; } } @@ -134,7 +137,7 @@ void CommonClientHandler::slot_sendTaskToClient(QString fullNameClient,QString t QString peerPort = QString::number(handler->getSocket()->peerPort()); QString str = "Task To Client [" + peerAddress + ":" + peerPort + "] : " + textTask; - logger->addTextToLogger(str); + emit sigSendToLogger(str); } } } diff --git a/ServerLMS/Systems/commonclienthandler.h b/ServerLMS/Systems/commonclienthandler.h index a6e471b..1abf5b8 100644 --- a/ServerLMS/Systems/commonclienthandler.h +++ b/ServerLMS/Systems/commonclienthandler.h @@ -27,7 +27,7 @@ public: void slot_msgToGUIfromClient(QString login, QString text); void slot_sendTaskToClient(QString fullNameClient, QString textTask); signals: - + void sigSendToLogger(QString text); private: QMap *clientsMap; ProcessingSystem *processingSystem; diff --git a/ServerLMS/Systems/recognizesystem.cpp b/ServerLMS/Systems/recognizesystem.cpp index 8e30805..2389ca6 100644 --- a/ServerLMS/Systems/recognizesystem.cpp +++ b/ServerLMS/Systems/recognizesystem.cpp @@ -90,13 +90,23 @@ void RecognizeSystem::recognize() if (command == commandUpdateFilesClient) //запускает процесс оновления { sendSystem->updateFiles(updateController->getFileSendList(), - updateController->getClientDataList()); + updateController->getFileDeleteList()); qDebug()<< "Call update"; packetType = PacketType::TYPE_NONE; command = ""; } + if(command == "check") + { + command = ""; + QFile checkFile(clientHash); + checkFile.open(QIODevice::ReadOnly); + updateController->compareFiles(clientHandler,checkFile.readAll()); + checkFile.close(); + + } + if (packetType == PacketType::TYPE_XMLANSWER) { @@ -280,14 +290,6 @@ void RecognizeSystem::recognize() file.close(); - if(command == "check") - { - QFile checkFile(filePath); - checkFile.open(QIODevice::ReadOnly); - updateController->compareFiles(clientHandler,checkFile.readAll()); - checkFile.close(); - } - filePath.clear(); fileSize = 0; tmpBlock.clear(); diff --git a/ServerLMS/Systems/sendsystem.cpp b/ServerLMS/Systems/sendsystem.cpp index 29b2324..65caacd 100644 --- a/ServerLMS/Systems/sendsystem.cpp +++ b/ServerLMS/Systems/sendsystem.cpp @@ -35,14 +35,22 @@ void SendSystem::sendMessageBlock(QString message) void SendSystem::sendFileBlock(QString path) { - qDebug() << "sendFileBlock thread: " << QThread::currentThreadId(); + QFile file(path); + QFileInfo fileInfo(file); + + if(isSendStopped) + { //Поведение на случай отключения клиента + + file.close(); + return; + } + QDataStream stream(socket); stream.setVersion(QDataStream::Qt_DefaultCompiledVersion); - QFile file(path); - QFileInfo fileInfo(file); + fileSize = fileInfo.size(); if(fileSize == 0){ @@ -55,20 +63,19 @@ void SendSystem::sendFileBlock(QString path) stream << path << fileSize; - if(isSendStopped) { //Поведение на случай отключения клиента - - file.close(); - return; - } - socket->waitForBytesWritten(); + socket->waitForBytesWritten(10); - if(file.open(QFile::ReadOnly)){ - while(!file.atEnd()){ + if(file.open(QFile::ReadOnly)) + { + while(!file.atEnd()) + { QByteArray data = file.read(1025*250); stream << data; - socket->waitForBytesWritten(); + socket->waitForBytesWritten(10); + + if(socket->state() == QAbstractSocket::UnconnectedState) break; countSend++; } @@ -77,7 +84,7 @@ void SendSystem::sendFileBlock(QString path) file.close(); countSend = 0; - socket->waitForBytesWritten(); + socket->waitForBytesWritten(10); socket->waitForReadyRead(100); } @@ -237,11 +244,9 @@ void SendSystem::updateFiles(QList fileSendList, QList delet socket->waitForReadyRead(100); } - if(getIsSendStopped()) return; + if(isSendStopped) return; } - - emit sigLoadHash(); sendPacketType(PacketType::TYPE_FINISH); diff --git a/ServerLMS/Systems/tools.h b/ServerLMS/Systems/tools.h index f78fff9..e896664 100644 --- a/ServerLMS/Systems/tools.h +++ b/ServerLMS/Systems/tools.h @@ -22,6 +22,7 @@ static const QString versionListFile = staticDataFolderName + "/versionList.xml" static const QString hashFileName = staticDataFolderName + "/serverHash.xml"; static const QString buildHashName = staticDataFolderName + "/buildHash.xml"; static const QString buildDataPath = "/Application/RRJLoader/RRJ_Data/"; +static const QString clientHash = staticDataFolderName + "/clientHash.xml"; static const QString baseNameVersion = "base";//может вынести комманды куда нибудь? diff --git a/ServerLMS/Systems/updatecontroller.cpp b/ServerLMS/Systems/updatecontroller.cpp index 7a548c1..7b0247a 100644 --- a/ServerLMS/Systems/updatecontroller.cpp +++ b/ServerLMS/Systems/updatecontroller.cpp @@ -50,7 +50,9 @@ void UpdateController::createCopyVersion(QString versionName,QString newVersionN void UpdateController::deleteAssetVersion(QString versionName) { + commonClientHandler->slot_sendPacketToAllClients(PacketType::BUSY); assetManager->deleteVersion(versionName); + commonClientHandler->slot_sendPacketToAllClients(PacketType::FREE); } void UpdateController::compareFiles(ClientHandler* handler, QByteArray array) @@ -59,7 +61,6 @@ void UpdateController::compareFiles(ClientHandler* handler, QByteArray array) loadHash(); clientDataList.clear(); xmlFileDataParse(array); - clientDataList.append(*datas); checkNeedUpdate(handler); mutex->unlock(); } @@ -182,8 +183,6 @@ QString UpdateController::getCommands() } - - void UpdateController::setUpCurrentServerHash() { QList *fileList = new QList; @@ -296,61 +295,69 @@ QList* UpdateController::calculateHash(QString path) { serverDataList.clear(); - QDirIterator iterator(path,QDirIterator::Subdirectories); if(!QDir(path).exists()){ QDir().mkdir(path); } - QDir dir(path); - dir.setFilter(QDir::NoDotAndDotDot); QString hashString; + QStringList filter; + filter << "*"; QList *files = new QList; - while (iterator.hasNext()) + QDirIterator dirIterator(path,filter, QDir::AllEntries | QDir::NoDotAndDotDot, QDirIterator::Subdirectories); + + while (dirIterator.hasNext()) { - iterator.next(); - QFileInfo fileInfo = iterator.fileInfo(); + QFileInfo fileInfo(dirIterator.next()); FileData currentFile; - QFile file(fileInfo.absoluteFilePath()); - - quint64 fileSize = file.size(); - const quint64 bufferSize = 10240; - - if (fileInfo.isHidden()) continue; - - if(fileInfo.isFile() && file.open(QIODevice::ReadOnly) && !fileInfo.fileName().contains(".meta")) + if(fileInfo.isDir() && !fileInfo.fileName().startsWith(".") && fileInfo.fileName() != "RRJLoader") { - char buffer[bufferSize]; - int bytesRead; - int readSize = qMin(fileSize,bufferSize); - - QCryptographicHash hash(QCryptographicHash::Md5); - while(readSize > 0 && (bytesRead = file.read(buffer,readSize)) > 0){ - fileSize -= bytesRead; - hash.addData(buffer,bytesRead); - readSize = qMin(fileSize,bufferSize); - } - - hashString = QString(hash.result().toHex()); - currentFile.hash = hashString; currentFile.path = Tools::createLocalPath(fileInfo.absoluteFilePath()); - - files->push_back(currentFile); - file.close(); - - } - else if(fileInfo.isDir() && fileInfo.fileName() != ".." && !fileInfo.isRoot()) - { currentFile.hash = "FOLDER"; - currentFile.path = Tools::createLocalPath(fileInfo.path()); - if(!files->contains(currentFile)){ + if(!files->contains(currentFile)) + { files->push_back(currentFile); } } } - std::sort(files->begin(),files->end()); + QDirIterator fileIterator(path,filter,QDir::Files | QDir::NoDotAndDotDot,QDirIterator::Subdirectories); + + while (fileIterator.hasNext()) + { + fileIterator.next(); + QFileInfo fileInfo = fileIterator.fileInfo(); + FileData currentFile; + QFile file(fileInfo.absoluteFilePath()); + + quint64 fileSize = file.size(); //буффер для хэширования крупных файлов + const quint64 bufferSize = 10240; + + if(fileInfo.isHidden()) continue; + + if(fileInfo.isFile() && file.open(QIODevice::ReadOnly) && !fileInfo.fileName().contains(".meta")) + { + char buffer[bufferSize]; + int bytesRead; + int readSize = qMin(fileSize,bufferSize); + + QCryptographicHash hash(QCryptographicHash::Md5); + + while(readSize > 0 && (bytesRead = file.read(buffer,readSize)) > 0) + { + fileSize -= bytesRead; + hash.addData(buffer,bytesRead); + readSize = qMin(fileSize,bufferSize); + } + + hashString = QString(hash.result().toHex()); + currentFile.path = Tools::createLocalPath(fileInfo.absoluteFilePath()); + currentFile.hash = hashString; + files->push_back(currentFile); + file.close(); + } + } serverDataList.append(*files); return files; } @@ -398,6 +405,12 @@ void UpdateController::CalculateSizeToSend(QList diffList) void UpdateController::calculateSharedHash() { QDir sharedDir(sharedDataPath); + + if(!QDir(sharedDataPath).exists()) + { + QDir().mkdir(sharedDataPath); + } + QDirIterator dirIterator(sharedDir); QList *fileList = new QList; QList *versionList = new QList; @@ -499,7 +512,6 @@ bool UpdateController::checkDuplicate(QString versionName) void UpdateController::xmlFileDataParse(QByteArray array) { QXmlStreamReader xmlReader(array); - datas = new QList; xmlReader.readNext(); //Крутимся в цикле до тех пор, пока не достигнем конца документа @@ -523,7 +535,7 @@ void UpdateController::xmlFileDataParse(QByteArray array) data.hash = value; } - datas->append(data); + clientDataList.append(data); } } @@ -565,11 +577,6 @@ DataInfo *UpdateController::getCurrentDataInfo() return dataInfo; } -QList *UpdateController::getDatas() const -{ - return datas; -} - void UpdateController::clearCurrentDataInfo() { delete dataInfo; diff --git a/ServerLMS/Systems/updatecontroller.h b/ServerLMS/Systems/updatecontroller.h index 611ef2f..efe3aee 100644 --- a/ServerLMS/Systems/updatecontroller.h +++ b/ServerLMS/Systems/updatecontroller.h @@ -51,7 +51,6 @@ public: DataInfo *getCurrentDataInfo(); void clearCurrentDataInfo(); - QList *getDatas() const; void createVersionListXmlAnswer(QList version); void saveVersionToFile(StreamingVersionData *streamingVersion); void xmlFileDataParse(QByteArray array); @@ -71,7 +70,6 @@ private: QList fileSendList; QList fileDeleteList; - QList *datas; DataInfo *dataInfo; QString buildPath; diff --git a/ServerLMS/clienthandler.cpp b/ServerLMS/clienthandler.cpp index 2ab918e..30c8448 100644 --- a/ServerLMS/clienthandler.cpp +++ b/ServerLMS/clienthandler.cpp @@ -54,6 +54,7 @@ void ClientHandler::initialize(int descriptor,ServerLMSWidget *serverWidget, connect(this,&ClientHandler::sigSocketClose,sendSystem,&SendSystem::socketClose,Qt::AutoConnection); connect(this,&ClientHandler::sigSocketFlush,sendSystem,&SendSystem::socketFlush,Qt::AutoConnection); connect(this,&ClientHandler::sigSendPacketType,sendSystem,&SendSystem::sendPacketType,Qt::AutoConnection); + connect(this,&ClientHandler::sigSendStop,sendSystem,&SendSystem::sendStop,Qt::DirectConnection); connect(socket,&QTcpSocket::readyRead,recognizeSystem,&RecognizeSystem::recognize,Qt::AutoConnection); connect(socket, &QTcpSocket::disconnected, this, &ClientHandler::sendDisable,Qt::AutoConnection); @@ -129,8 +130,10 @@ void ClientHandler::sendNeedUpdate(bool flag, quint64 size, quint64 fileCount,qu void ClientHandler::sendDisable() { - thread->exit(); - thread->wait(); +// thread->exit(); +// thread->wait(); + thread->quit(); + emit sigSendStop(); emit sigClientDisconnected(client->getAddress(),client->getPort()); } diff --git a/ServerLMS/clienthandler.h b/ServerLMS/clienthandler.h index 7e035b8..7f840db 100644 --- a/ServerLMS/clienthandler.h +++ b/ServerLMS/clienthandler.h @@ -67,6 +67,7 @@ signals: bool sigSocketFlush(); void sigSendVersion(); void sigSendPacketType(PacketType packetType); + void sigSendStop(); public : QThread *thread; diff --git a/ServerLMS/multithreadserver.cpp b/ServerLMS/multithreadserver.cpp index 1ac463d..50a0217 100644 --- a/ServerLMS/multithreadserver.cpp +++ b/ServerLMS/multithreadserver.cpp @@ -8,7 +8,7 @@ MultiThreadServer::MultiThreadServer(ServerLMSWidget *widget,UpdateController *u dataParser(dataParser), logger(logger) { - + connect(this,&MultiThreadServer::sigSendToLogger,logger,&Logger::addTextToLogger); } void MultiThreadServer::incomingConnection(qintptr socketDesriptor) @@ -16,14 +16,14 @@ void MultiThreadServer::incomingConnection(qintptr socketDesriptor) ClientHandler* newClient = new ClientHandler; connect(this,&MultiThreadServer::sigInitClient,newClient,&ClientHandler::initialize,Qt::AutoConnection); - connect(newClient,&ClientHandler::sigClientDisconnected,this,&MultiThreadServer::slotDisconnectClient,Qt::QueuedConnection); + connect(newClient,&ClientHandler::sigClientDisconnected,this,&MultiThreadServer::slotDisconnectClient,Qt::AutoConnection); emit sigInitClient(socketDesriptor,serverLmsWidget,updateController,dataParser,logger); disconnect(this,&MultiThreadServer::sigInitClient,newClient,&ClientHandler::initialize); serverLmsWidget->addClient(socketDesriptor,newClient); - logger->addTextToLogger("To Client: " + QString(SERVER_HELLO)); + emit sigSendToLogger("To Client: " + QString(SERVER_HELLO)); } void MultiThreadServer::slotDisconnectClient(QString peerAddress, QString peerPort) @@ -53,8 +53,7 @@ void MultiThreadServer::slotDisconnectClient(QString peerAddress, QString peerPo } emit signalStopSendFile(); - - logger->addTextToLogger("SERVER: Client " + login + " disconnected"); + emit sigSendToLogger("SERVER: Client " + login + " disconnected"); serverLmsWidget->slotUpdateListClients(); serverLmsWidget->autorizationHandler(login); diff --git a/ServerLMS/multithreadserver.h b/ServerLMS/multithreadserver.h index 13a4c50..eb12f80 100644 --- a/ServerLMS/multithreadserver.h +++ b/ServerLMS/multithreadserver.h @@ -18,6 +18,7 @@ signals: void sigInitClient(int descriptor, ServerLMSWidget *serverWidget, UpdateController *updateController, DataParser *dataParser,Logger *logger); void signalStopSendFile(); + void sigSendToLogger(QString text); public slots: void slotDisconnectClient(QString peerAddress, QString peerPort); diff --git a/ServerLMS/serverlmswidget.cpp b/ServerLMS/serverlmswidget.cpp index f52eb00..b90a0c3 100644 --- a/ServerLMS/serverlmswidget.cpp +++ b/ServerLMS/serverlmswidget.cpp @@ -14,7 +14,7 @@ ServerLMSWidget::ServerLMSWidget(QWidget *parent) : tcpServer(nullptr), hostPort(6000), stateServer(stoped), - stateBlockAutorization(blocked), + stateBlockAutorization(unblocked), updateThread(nullptr), loggerThread(nullptr), dataParser(nullptr), @@ -44,11 +44,13 @@ ServerLMSWidget::ServerLMSWidget(QWidget *parent) : loggerThread = new QThread; providerDBLMS = new ProviderDBLMS(this); - connect(providerDBLMS, &ProviderDBLMS::signal_BlockAutorization, this, &ServerLMSWidget::slot_BlockAutorization); providerDBLMS->ConnectionToDB(); logger = new Logger(ui->listWidgetLogger); - connect(logger,&Logger::sigSendTextToLogger,this,&ServerLMSWidget::slotAddToLog,Qt::QueuedConnection); + + connect(providerDBLMS, &ProviderDBLMS::signal_BlockAutorization, this, &ServerLMSWidget::slot_BlockAutorization); + connect(logger,&Logger::sigSendTextToLogger,this,&ServerLMSWidget::slotAddToLog,Qt::AutoConnection); + logger->moveToThread(loggerThread); assetsManager = new AssetsManager; @@ -194,7 +196,7 @@ void ServerLMSWidget::slotUpdateListClients() } int countClients = clientsMap.count(); - logger->addTextToLogger("SERVER: countClients = " + QString::number(countClients)); + emit sigLog("SERVER: countClients = " + QString::number(countClients)); } void ServerLMSWidget::slot_BlockAutorization(bool block)