0 / 0 / 0
Регистрация: 31.07.2022
Сообщений: 13
1

Оцените код в плане производительности

20.03.2023, 16:44. Показов 510. Ответов 8
Метки нет (Все метки)

Author24 — интернет-сервис помощи студентам
Решил написать программу, которая записывает и считывает бинарные файлы. Для удобства решил вместо char* использовать vector<char>, чтобы не морочиться с дополнительными переменными. Через долгие мучения получился такой код:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
#include <iostream>
#include <fstream>
#include <vector>
using namespace std;
 
void readFile(string path, vector<char> &info)
{
    ifstream ifs(path, ios::binary);
 
    ifs.seekg(0, ios::end);
    size_t size = ifs.tellg();
    ifs.seekg(0, ios::beg);
 
    info.resize(size);
 
        for (size_t i = 0; i < size; i++)
        {
            ifs.read(&info[i], 1);
        }
 
    ifs.close();
}
 
void writeFile(string path, vector<char> info)
{
    ofstream ofs(path, ios::binary);
 
        for (size_t i = 0; i < info.size(); i++)
        {
            ofs.write(&info[i], 1);
        }
 
    ofs.close();
}
 
int main()
{
    //тест
    vector<char> da;
    da.push_back('H');
    da.push_back('I');
    da.push_back('!');
    writeFile("info.dat",da);
    vector<char> da2;
 
    readFile("info.dat",da2);
 
    for (size_t i = 0; i < da2.size(); i++)
    {
        cout << da2[i];
    }
    return 0;
}
Как по мне, код получился неплохим, но похоже, что из-за циклов он будет медленно работать. Можно ли его как-то оптимизировать?
0
20.03.2023, 16:44
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
20.03.2023, 16:44
Ответы с готовыми решениями:

Оценить сборку для игр в плане производительности в играх
Всем привет, у меня такой вопрос если я поставлю в эту материнскую плату ASUS M2N68AM PLUS эту видеокарту HD7750 1GB DDR5, 6 гб оперативки...

Samsung G910 как улучшить ноутбук в плане производительности
Подскажите как улучшить ноутбук samsung g910 в плане производительности , процессор , видеосистема , куллер , память я уже поменял DDR...

Есть ли какая-нибудь зависимость шины материнки и ЦПУ в плане производительности
есть ли какая нибуть зависимость шини матеренки и ЦПУ? Добавлено через 10 минут в плане производительности.

8
147 / 135 / 50
Регистрация: 14.05.2021
Сообщений: 642
20.03.2023, 16:53 2
C++
1
void writeFile(string path, vector<char> info)
Зачем нам копия вектора нужна?
Цитата Сообщение от Alexey1024 Посмотреть сообщение
ofs.write(&info[i], 1);
Писать по одному char'у в цикле? Это извращение какое-то. Пиши сразу буфер нужного размера. С чтением та же байда.
0
0 / 0 / 0
Регистрация: 31.07.2022
Сообщений: 13
20.03.2023, 16:54  [ТС] 3
Решил ещё немного улучшить код и избавился от циклов:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#include <iostream>
#include <fstream>
#include <vector>
using namespace std;
 
vector<char> readFile(string path)
{
    ifstream ifs(path, ios::binary);
    ifs.seekg(0, ios::end);
    size_t size = ifs.tellg();
    ifs.seekg(0, ios::beg);
    char* data = new char[size];
    ifs.read(data, size);
    ifs.close();
    return vector<char>(data, data+size);
}
 
void writeFile(string path, vector<char> info)
{
    ofstream ofs(path, ios::binary);
    char* data = &info[0];
    ofs.write(data, info.size());
    ofs.close();
}
 
int main()
{
    //тест
    vector<char> da;
    da.push_back('H');
    da.push_back('E');
    da.push_back('L');
    da.push_back('P');
    da.push_back('!');
    writeFile("info.dat",da);
    vector<char> da2;
 
    da2 = readFile("info.dat");
 
    for (size_t i = 0; i < da2.size(); i++)
    {
        cout << da2[i];
    }
    return 0;
}
0
147 / 135 / 50
Регистрация: 14.05.2021
Сообщений: 642
20.03.2023, 16:56 4
Цитата Сообщение от Alexey1024 Посмотреть сообщение
char* data = new char[size];
Зачем? Можно сразу сделать вектор нужного размера. Тогда уйдет копирование.
Цитата Сообщение от Alexey1024 Посмотреть сообщение
void writeFile(string path, vector<char> info)
Опять же: копирование.

Добавлено через 53 секунды
Цитата Сообщение от Alexey1024 Посмотреть сообщение
char* data = &info[0];
Чем не устраивает data()?
0
0 / 0 / 0
Регистрация: 31.07.2022
Сообщений: 13
20.03.2023, 17:05  [ТС] 5
А так норм?
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
#include <iostream>
#include <fstream>
#include <vector>
using namespace std;
 
void readFile(string path, vector<char> &info)
{
    ifstream ifs(path, ios::binary);
    ifs.seekg(0, ios::end);
    size_t size = ifs.tellg();
    ifs.seekg(0, ios::beg);
    info.resize(size);
    ifs.read(&info[0], size);
    ifs.close();
}
void writeFile(string path, vector<char> &info)
{
    ofstream ofs(path, ios::binary);
    ofs.write(&info[0], info.size());
    ofs.close();
}
int main()
{
    string str = "Hello, World!";
    vector<char> v1;
    for (size_t i = 0; i < str.length(); i++)
    {
        v1.push_back(str[i]);
    }
    writeFile("info.dat", v1);
 
    vector<char> v2;
    readFile("info.dat", v2);
    for (size_t i = 0; i < v2.size(); i++)
    {
        cout << v2[i];
    }
    return 0;
}
0
147 / 135 / 50
Регистрация: 14.05.2021
Сообщений: 642
20.03.2023, 17:29 6
Норм. Только что бы я еще изменил:
C++
1
 ofs.write(&info[0], info.size());
Нужно записывать info.size() умноженный на размер типа в векторе, чтобы было универсально. Ну или использовал std::byte вместо char.
Цитата Сообщение от Alexey1024 Посмотреть сообщение
vector<char> v1;
    for (size_t i = 0; i < str.length(); i++)
    {
        v1.push_back(str[i]);
    }
Зачем цикл с пушбэками? Вектор вроде умеет принимать итераторы, если память не изменяет, давно плюсы не трогал.
Ну и там, где нет изменения веткора, костантные ссылки бы. И со строками то же самое. Почему у тебя везде передача по значению?
0
Модератор
Эксперт Java
 Аватар для alecss131
2719 / 1216 / 388
Регистрация: 11.08.2017
Сообщений: 3,772
Записей в блоге: 2
20.03.2023, 23:50 7
Цитата Сообщение от psergee Посмотреть сообщение
Вектор вроде умеет принимать итераторы
ага
C++
1
2
3
    std::string str = "Hello, World!";
    std::vector<char> v1;
    v1.insert(v1.end(), str.begin(), str.end());
Цитата Сообщение от Alexey1024 Посмотреть сообщение
C++
13
ifs.read(&info[0], size);
чем не устраивает?
C++
13
ifs.read(info.data(), size);
в записи тоже самое
0
732 / 692 / 110
Регистрация: 29.05.2015
Сообщений: 4,152
21.03.2023, 02:53 8
А чё его оценивать? Нужно взять любой файл размером (например) 1 Гб, поменять расширение на незарегистрированное, прочитать его, добавить к каждому байту по единице и записать под другим именем. И измерить время.

А потом проделать обратную операцию (вычесть из каждого байта по 1) - и проверить, вернётся ли файл в исходное состояние.

Расширение менять, потому что есть подозрение, что файлы .txt обрабатываются по-другому.
0
19409 / 10028 / 2443
Регистрация: 30.01.2014
Сообщений: 17,678
21.03.2023, 09:23 9
Цитата Сообщение от psergee Посмотреть сообщение
Вектор вроде умеет принимать итераторы
Да, и можно без промежуточного string обойтись:
C++
1
2
    std::string_view str = "Hello, World!";
    std::vector<char> v1(str.begin(), str.end());
Добавлено через 2 минуты
Цитата Сообщение от alexu_007 Посмотреть сообщение
Расширение менять, потому что есть подозрение, что файлы .txt обрабатываются по-другому.
Бинарность файла определяется вторым параметром потока (в данном случае у ТС - ios::binary), расширение библиотеке фиолетово.
0
21.03.2023, 09:23
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
21.03.2023, 09:23
Помогаю со студенческими работами здесь

SQL object VS PL/SQL record в плане производительности
Интересует вопрос как именно ведут себя объявленные на уровне БД sql типы в плане производительности по сравнению с PL/SQL типами. ...

Оцените сборку, может можно удешевить без потерь качества и производительности!
http://www.arsplus.ru/personal/basket.php В магазине Арсенал+. Компьютер нужен для игр, просмотра фильмов.

Графическая станция, бюджет около 50 т.р. Пожалуйста оцените конфигу! Можно ли удешевить без особенных потерь производительности?
Всем доброго времени суток! Широкими познаниями в области железа не могу похвастаться, посему очень нужен совет профессионалов! ...

Как оптимизировать код по памяти и производительности?
Задание 1. Написать метод, принимающий на вход строку и проверяет, является ли данная строка палиндромом. Игнорировать пробелы и знаки...

Возможно ли сократить код и не проиграть в производительности?
if (_keyId % 2 == 0) { var t = new Thread(delegate () { LeftClick(_hitEnd); }); ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
9
Ответ Создать тему

Редактор формул (кликните на картинку в правом углу, чтобы закрыть)
Опции темы

Новые блоги и статьи
Контейнер std::map в C++
bytestream 09.02.2025
Контейнер std::map в C++ - один из наиболее мощных инструментов стандартной библиотеки, предназначенный для хранения пар ключ-значение. Каждый элемент в map состоит из уникального ключа и связанного. . .
Как в Python сделать вывод с print без перевода строки и пробела
hw_wired 09.02.2025
Функция print в Python обеспечивает гибкие возможности для вывода информации в консоль. При стандартном использовании эта функция автоматически добавляет символ перевода строки в конце выводимого. . .
Как в Python проверить, что у объекта есть атрибут
hw_wired 09.02.2025
В Python существует несколько встроенных способов проверки наличия атрибутов у объектов. Наиболее распространенным является использование функции hasattr(), которая позволяет безопасно определить. . .
Как удалить экспортированну­ю переменную окружения в Linux
hw_wired 09.02.2025
В Linux работа с переменными окружения - важная часть системного администрирования и разработки. Экспортированные переменные окружения отличаются от обычных локальных переменных тем, что они доступны. . .
Ошибка Error: error:0308010C:­digital envelope routines::unsup­ported
hw_wired 09.02.2025
Ошибка "error:0308010C:digital envelope routines::unsupported" чаще всего появляется при работе с Node. js приложениями и связана с изменениями в системе безопасности криптографических алгоритмов. . . .
В чем отличие между .prop() и .attr()
hw_wired 09.02.2025
В jQuery методы . prop() и . attr() часто вызывают путаницу, поскольку на первый взгляд предназначены для похожих целей. Однако между ними существуют принципиальные различия в работе с DOM-элементами и. . .
В чем отличие SCSS и SASS
hw_wired 09.02.2025
SCSS и SASS появились как решение проблем, связанных с ограничениями обычного CSS при разработке крупных веб-проектов. Традиционный CSS, несмотря на свою простоту, не предоставлял разработчикам. . .
Как найти дубликаты в таблице базы данных
hw_wired 09.02.2025
Дублирование записей в таблицах баз данных может возникать по разным причинам: ошибки при вводе данных, некорректная работа систем импорта, слияние данных из разных источников или неправильная. . .
Как удалить дубликаты из массива в JavaScript
hw_wired 09.02.2025
Самый простой и современный способ удаления дубликатов в JavaScript - использование структуры данных Set в сочетании с Array. from. Set автоматически хранит только уникальные значения, а Array. from. . .
Go Protobuf: новый Opaque API
hw_wired 09.02.2025
Protocol Buffers (protobuf) давно зарекомендовал себя как эффективный формат сериализации данных, широко используемый в микросервисных архитектурах и распределенных системах. Однако существующая. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru