Форум программистов, компьютерный форум CyberForum.ru

"Ляпы" в программе. Хороший стиль программирования - C++

Войти
Регистрация
Восстановить пароль
Другие темы раздела
C++ Ошибка при запуске скомпилированной программы http://www.cyberforum.ru/cpp/thread1544150.html
при запуске скомпилированной программы выходит ошибка:
C++ Распаковка Docx документа Народ как на C++ или может есть какие нибудь готовые библиотеки. Распаковать вордовский документ с расширением docx что бы можно было получить все xml файлы которые лежат в нем. З.Ы. Работаю в Microsoft Visual Studio поэтому zipQtReader не подходит. Так же наткнулся на 7-zip библиотеку но не понял как ей пользоваться. Помогите очень надо.!!! http://www.cyberforum.ru/cpp/thread1543855.html
C++ По поводу исходников GNU Octave
Всем добрый вечер. Сижу я тут, ковыряюсь в исходниках этой самой программы и наткнулся на такое определение: DEFUN (__version_info__, args, , "-*- texinfo -*-\n\ @deftypefn {Built-in Function} {retval =} __version_info__ (@var{name}, @var{version}, @var{release}, @var{date})\n\ Undocumented internal function.\n\ @end deftypefn")
Работа программистом на C++. Перспектива C++
Живу в городе Львов, Украина. Учу С++. Студент 2-го курса. Столкнулся со следующей проблемой - помимо того, что город является центром IT в Украине, т.е. масса офисов GlobalLogic, SoftServe, <EPAM> и других широко известных компаний, которые не только дают массу вакансий, а и проводят бесплатные курсы для юниор-лвл с дальнейшим трудоустройством, почему-то свернули наборы на С++ почти полностью....
C++ Посчитать количество символов в файле docx http://www.cyberforum.ru/cpp/thread1542554.html
Народ есть файл 123.docx надо на C++ посчитать сколько символов в нем записано. Как это можно сделать?
C++ Прерывания в ОСи: прототип функции-обработчика Как реализовать прерывания на C++. Какой должен быть прототип функции-обработчика. Как делать выход из обработчика. Mожет callback надо делать? Я сделал так: void* handler() { __asm__ __volatile__("pushad"); printf("!!!"); __asm__ __volatile__("popad; leave; iret); } подробнее

Показать сообщение отдельно
DrOffset
6920 / 4113 / 940
Регистрация: 30.01.2014
Сообщений: 6,908
05.10.2015, 18:55     "Ляпы" в программе. Хороший стиль программирования
George Sirin,
Ну или вот тут:
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
54
55
56
57
58
59
60
61
62
63
64
// Destructor.
// virtual.
KeyConfig::~KeyConfig()
{
    // Delete storages.
 
    // Bytes dyn buf.
    if(read_buffer)
        delete [] read_buffer;
 
    // String.
    strLastError.clear();
}
 
// Load config key data from file.
// Returns: true - success, false - error.
bool KeyConfig::loadConfigKeyData(wchar_t *strFilepath)
{
    // Open file.
    ifstream keyConfig(strFilepath, ios::in | ios::binary);
    if( ! keyConfig )
    {
        keyConfig.close();
        strLastError = wstring(L"Can't open file.") + strFilepath;
        return false; // Return error.
    }
 
    // Get whole bytes num in file.
    keyConfig.seekg(0, std::ios_base::end);
    const long int nFileLen = (const long int)(keyConfig.tellg());
    keyConfig.seekg(0, std::ios_base::beg); // Return to back.
 
    // Read whole data.
    read_buffer = new char [nFileLen+4]; // Reserve a piece of data in the end of buf - to put terminating character ('\0').
    keyConfig.read(read_buffer, nFileLen);
    if( ! keyConfig ) // Route read errors.
    {
//!!! Если попали сюда, то буфер освободился, но деструктор об этом не узнает и вызовет delete [] еще раз.
//!!! Ну и проверки на ноль конечно же не нужны. Но это уже безобидный "ляп"
        if(read_buffer) 
            delete [] read_buffer;
/*!! Нужно хотя бы так:
        if(read_buffer) {
            delete [] read_buffer;
            read_buffer = 0;
        }
*/
        keyConfig.close();
        strLastError = L"Can't read data from key config file.";
        return false; // Return error.
    }
 
    // Setup the terminating character ('\0') at and of buf.
    read_buffer[nFileLen+0] = 0x00;
    read_buffer[nFileLen+1] = 0x00;
    read_buffer[nFileLen+2] = 0x00;
    read_buffer[nFileLen+3] = 0x00;
 
    // Close file.
    keyConfig.close();
 
    // Return success.
    return true;
}
Ну и повсюду странное: бросаем строки в качестве исключений. Обработка ошибок неравномерная, то исключения, то last error.
Класс, который привел выше, кроме того, никак не запрещает копирование и присваивания своих объектов. А это, судя по семантике, предполагается.

Ну и всякое такое по мелочи, то конст забудем, то ссылку...
 
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru