Освобождение памяти из под динамического двумерного массива

[Croessmah]

Вообще, что такое n и зачем оно нужно?
Если это максимальная длина слова в файле,
то не забудьте, что еще должен быть '\0' в конце,
т.е. резервировать нужно n + 1 байт на слово.

И работайте с заранее подготовленным и проверенным файлом,
чтобы исключить ошибки связанные с генерацией данных.

[Haric_110]

Функцию забыл убрать просто... это не решает проблему, но если убрать всё, кроме выделения и освобождения памяти, то ок.
Значит, ошибка в заполнении массива. Хорошо. В чём может быть проблема? Может, в считывании

input >> strings[ind]

?
не литералом '\0' всё завершается? Смотрю - вроде, нет...

[Croessmah]

Сообщение от Haric_110

В чём может быть проблема?

1) Ошибка в данных в файле
2) n не учитывает '\0'.

[Haric_110]

Сообщение от Croessmah

Вообще, что такое n и зачем оно нужно?
Если это максимальная длина слова в файле,
то не забудьте, что еще должен быть '\0' в конце,
т.е. резервировать нужно n + 1 байт на слово.

Да, Вы правы, проблема заключалась именно в этом... (пример файла данных я прикладывал к теме)

Спасибо! Говорил же, что ошибка очень проста)

[Croessmah]

Сообщение от Haric_110

Говорил же, что ошибка очень проста

Только ошибка оказалась не в коде, а в данных.
Хотя и в коде тоже, как выяснилось, есть.

for (int ind_2 = 0; ind_2 < n; ++ind_2) strings[ind_1][ind_2] -= 'a';

Если слово по длине будет меньше n,
то '\0' также перепишется.
Как вариант:

char *wcur = strings[ind_1];
while (*wcur) {
   *wcur -= 'a';
   ++wcur;
}

[Croessmah]

Ну и используйте std::string, зачем Вам возня с char*?
многие присутствующие оптимизации в std::string,
вполне возможно, сделают код быстрее и эффективнее (а может и нет)))).
Например, gcc (емнип), сравнивает строки не по символам,
а более большими кусками (int'ами, вроде).
А при коротких строках (например, до 15 символов, включительно),
может быть применена оптимизация SSO,
которая исключит динамическую аллокацию памяти под строку.

[Haric_110]

Но это меня удивило.

Сообщение от Croessmah

Берем код с той же ошибкой:

#include <iostream>

int main()
{
    const int n = 3;
    int arr[n] = {555, 555, 555};
    int i = 0;
    while (i < n) {
        arr[i] = i++;
    }
    for(auto &e: arr) {
        std::cout << e << ", ";
    }
}

Смотрим на одном компиляторе (gcc): http://rextester.com/IAKLQC88748
Цитата:
555, 0, 1,
Смотрим на другом (CL): http://rextester.com/MSRRVL5172
Цитата:
0, 1, 2,
Оба поведения законны и ничего не нарушают в части языка,
но при выполнении будут проблемы,
мы же выходим за пределы массива.

Я бы не сказал, что поведение gcc законнно с точки зрения языка, он не различает преинкремент и постинкремент... или не совсем...
Как он вообще работает?
в вычислении индекса он инкрементирует i, а в присваивании — нет...

[Haric_110]

Сообщение от Croessmah

Ну и используйте std::string, зачем Вам возня с char*?

Мне нужно преобразовать строки в массив чисел и с ними работать, а не с символами. Библиотеки плохо знаю — мой главный недостаток.

Сообщение от Croessmah

А при коротких строках (например, до 15 символов, включительно),
может быть применена оптимизация SSO,
которая исключит динамическую аллокацию памяти под строку.

Строки длиной до миллиона будут, так что, скорее всего, это не сработает... но я учту на будущее.

[Croessmah]

Сообщение от Haric_110

Я бы не сказал, что поведение gcc законнно с точки зрения языка, он не различает прединкремент и постинкремент

Абсолютно законно.
Это ошибка программиста, он не знает правила C++.
А вот компиляторы эти правила знают.

Ошибка в том, что одна и та же переменная используется и изменяется в одном выражении.
И вот порядок этих изменений и использования в данном случае не определен.
Распишем то, как компилятор может переделать это выражение:

//arr[i] = i++;
//i = 0
//1:
temp_i = i;//temp_i = 0
++i; //i = 1
arr[temp_i] = temp_i;//arr[0] = 0;
//2
temp_i = i;//temp_i = 0
++i;//i = 1
arr[i] = temp_i;//arr[1] = 0;
//т.е. какое значение i нужно использовать в квадратных скобках?
//то, которое было до инкремента или значение после инкремента?
//правильный ответ - любое, в зависимости от хотелок компилятора

Можете почитать это: http://alenacpp.blogspot.ru/2005/11/...ce-points.html
Хоть точки следования несколько устаревшее понятие, но смысл тот же.

[Haric_110]

Первый вариант мне кажется логичным, я думал, так все и работают, а второй — нет.
Надеюсь, мне не придётся работать с компилятором, который будет думать не так, как думаю я)
Чёрт, я разочарован...