Показывает старое значение вместо нового (std::stream)?

Question

[noytmant]

Вообщем:
1) Закачиваю текстовой файл
2) Затем открываю его и задаю строке ss значение

std::remove("base.txt"); // Удаляю старый файл, если он имеется
URLDownloadToFile(0, "url/base.txt", "base.txt", 0, 0);
std::ifstream f("base.txt");
std::stringstream ss;
ss << f.rdbuf();
f.close();

Я открыл файл base.txt, прочитал, получил значение (строку) и присвоил строку к ss
Далее происходит проверка

if(ss.str() == "OK")
{
   Load();
}
else
{
   Exit();
}

На первый раз оно скачивает файл видим там OK и вызывает функцию Load();
Далее я закрываю приложение, захожу на хостинг и меняю значение, теперь там не OK, а NO
Запускаю приложение, теперь мы получили значение NO, но в коде стоит проверка на OK, теперь должен произойти вызов функции Exit();
Но не тут то было, оно вызывает функцию Load() потому что значение файла было OK, а не NO
Какое бы значение я не писал в текстовом файле, оно его качает и каким то волшебным образом оставляет там старое значение. Но ведь скачан новый файл, с новым словом, откуда там вдруг появилось старое вместо нового. Может быть я что то не дописал? Или код не верный? Помогите решить эту проблему =(

Comments

[15432]

а где у вас код скачивания?

[noytmant]

15432, там же
URLDownloadToFile(0, "url/base.txt", "base.txt", 0, 0);

[15432]

noytmant, и если вы отдельно по этой ссылке заходите через браузер - там "NO"?

[15432]

мне больше странно, что вы "удаляете старый файл" после того, как скачали новый, он вообще удаляться должен, и что у вас тогда открывается?

[noytmant]

15432, после того как он закачивает файл, файл остаётся. Чтобы не оставался, нужно его удалить. Каждый раз, когда я буду открывать приложение, он будет закачивать файл, перед этим удаляя имеющийся. Я конечно понимаю, что он банально заменить его, но на всякий случай лучше сделать так. Удалил, закачал, открыл новый. Но в новом значение старого -_-

[Александр Ананьев]

std::remove - это не функция удаления файла.

[noytmant]

Александр Ананьев, в любом случае остаётся старое значение

[jcmvbkbc]

std::remove - это не функция удаления файла.

Александр Ананьев, а в книжку заглянуть?

Answer 1

[jcmvbkbc]

Я открыл файл base.txt, прочитал, получил значение (строку) и присвоил строку к ss

URLDownloadToFile(0, "url/base.txt", "base.txt", 0, 0);
std::remove("base.txt"); // Удаляю старый файл, если он имеется
std::ifstream f("base.txt");

А я вот вижу, что ты сначала файл начинаешь скачиваеть, потом удаляешь, а потом открываешь. Как это должно работать?
Второе -- URLDownloadToFile только начинает скачивание. Когда оно закончится ты не проверяешь.

Comments

[noytmant]

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

[Victor Bomberow]

Но в моём коде всё нормально.

Это не так. Либо представлен не тот снимает, который демонстрирует проблему. Но в этом коде:

1) Файл не удаляется.

2) Файл не скачивается, если уже существует base.txt то uridownload не возвращает S_OK, ваш код это не проверяет и снова и снова читает "старый файл".

[noytmant]

Victor Bomberow, Даже после перезапуска он будет снова читать старый файл?

[jcmvbkbc]

1) Файл не удаляется.

Victor Bomberow, да ладно, может и удаляется.

[jcmvbkbc]

Далее я закрываю приложение, захожу на хостинг и меняю значение, теперь там не OK, а NO

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

[noytmant]

jcmvbkbc, Я уже так делал. На первый раз оно нормально работает. После того как я меняю значение файла, оно удаляет старый (я вижу это с открытой папкой), а потом закачивает его заново с хостинга, но вместо нового значения в НОВОМ ФАЙЛЕ, остаётся старое значение. Это вообще как то не логично работает

[Victor Bomberow]

noytmant, я ошибся. Скорее всего в текущем коде определяется попытка удалить не тот файл, а ваш base.txt не удаляется и remove корректно возвращает 0. И дальше уже файл не скачивается.
Если указать полный путь, то удаляется нужный файл. Емнип, в режиме Debug под Visual Studio CWD является не та директория, в которой находится таргет, а директория, где размещен VS project file.
Для корректности, вам нужно при дебаге (#ifdef _DEBUG ) вместо имени файла подставлять remove ("Debug\\base.txt")

[noytmant]

Victor Bomberow, Я когда тестирую программу она удаляет файл (он пропадает в той папке где был), и после идёт закачивание файла, то есть появляется там же новый файл. С этим у меня никаких проблем нету. Всё удаляется, всё скачивается. Так что проблема не в этом, а в чём я даже не знаю

[noytmant]

Victor Bomberow, Попробуйте реализовать этот код у себя, может быть у вас получится

[Victor Bomberow]

noytmant, всё скачивается и работает. Даже urldownload вслепую перетирает файл, чего я не заметил ранее, т.к. тоже по ошибке указал неверный путь.

Но дело в том, что всё работает корректно.

for (size_t i = 0; i < 2; ++i) {
        LPCTSTR link[]{ L"url-to-file-with-OK/base.txt" ,  L"url-to-file-with-NO/base-1.txt" };

        auto res = URLDownloadToFile(NULL, link[i], L"Debug\\base.txt", NULL, NULL);

        if (res == S_OK) {
            std::cout << "result: OK" << std::endl;
        }
        else {
            std::cout << "result: not good" << std::endl;
        }

        std::ifstream f("Debug\\base.txt");
        std::stringstream ss;
        ss << f.rdbuf();
        f.close();

        std::cout << ss.str() << std::endl;
    }

result: OK
OK

result: OK
NO

Проблема не в коде. Проверял с удалением и добавлением файла link[0] с разным текстом. У вас точно удаление/замена файлов на хостинге отрабатывает корректно?

У вас в файле содержание какое:
1) OK\r\n
2) OK\r\n\r\n

Вообще, текстовый файл должен оканчиваться символом пустой строки. Это видно в выводе в моем примере.

if (ss.str() == "OK\n")
            std::cout << "Correct comparison" << std::endl;

result: OK
OK

Correct comparison