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

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

Восстановить пароль Регистрация
Другие темы раздела
C++ Как правильно распределить динамическую память? http://www.cyberforum.ru/cpp/thread1544470.html
При запуске программы и вводе строки, в одном случае программа выполняется, а в другом выдает, что прекращена работа программы. Понимаю, что ошибка в распределении памяти, но не пойму точно как исправить. Подскажите, пожалуйста. Код: //Ввести строку. Вывести слова заданной длины по алфавиту. #include<iostream> #include<cstdio> #include<cstring> using namespace std; int main() {
C++ Не правильное построение задачи 17 номер Задача на C++ #include <stdio.h> #include <math.h> #include <iostream> int main() { double x = 6.35 ; http://www.cyberforum.ru/cpp/thread1544394.html
C++ Генератор неповторяющихся семизначных чисел
Всем здравствуйте! Очень нужна Ваша помощь. При написании программы столкнулась с проблемой в рандоме. Нужно, чтобы на экран выводились семизначные неповторяющиеся числа. У меня они все начинаются с 10, а дальше все нормально. Необходимо, чтобы все они были разные. Задание полностью: Сгенерировать 49 семизначных неповторяющихся чисел (элементов). Вывести их на экран. Мой код на c++: #include...
C++ Ошибка при запуске скомпилированной программы
при запуске скомпилированной программы выходит ошибка:
C++ Распаковка Docx документа http://www.cyberforum.ru/cpp/thread1543855.html
Народ как на C++ или может есть какие нибудь готовые библиотеки. Распаковать вордовский документ с расширением docx что бы можно было получить все xml файлы которые лежат в нем. З.Ы. Работаю в Microsoft Visual Studio поэтому zipQtReader не подходит. Так же наткнулся на 7-zip библиотеку но не понял как ей пользоваться. Помогите очень надо.!!!
 

Показать сообщение отдельно
DrOffset
6785 / 3996 / 917
Регистрация: 30.01.2014
Сообщений: 6,816
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.
Класс, который привел выше, кроме того, никак не запрещает копирование и присваивания своих объектов. А это, судя по семантике, предполагается.

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