- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
void load(char *file)
{
reader = fopen((const char*)file, "r+b");
if(reader)
{
byte *b = (byte*)malloc(sizeof(byte));
fread(b, sizeof(byte), 1, reader);
if(b == 0x0)
{
int *wh_val = (int*)malloc(sizeof(int) * 2);
fread(wh_val, sizeof(int), 2, reader);
width = *wh_val;
height = *(wh_val + 1);
pixels = (Color**)malloc(sizeof(Color*) * width);
for (int i = 0; i < width; ++i)
{
*(pixels + i) = (Color*)malloc(sizeof(Color) * height);
for (int j = 0; j < height; ++j)
{
byte *rgb = (byte*)malloc(sizeof(byte) * 3);
fread(rgb, sizeof(byte), 3, reader);
Color c = Color(0);
c.red = *(rgb) / 255.0;
c.green = *(rgb + 1) / 255.0;
c.blue = *(rgb + 2) / 255.0;
*(*(pixels + i) + j) = c;
}
}
}
}
}
Пишу я код ни о чем не задумываясь, а когда задумался, уже было это.
ЗЫ. эта ф-ция читает картинку из спец. файла.
Follow us!
ЗЫ. Color это клас.
строки 10..13 бездельничают
тем более, что статические массивы - это безусловное зло.
а что за хрень: ведь <b> есть указатель, т.е. если или другими словами malloc() не смог выделить байт памяти, то юзать malloc() опять? хитро
И не понятно что делает эта функция. Вернее что делает понятно, но результаты её так и остаются внутри. Не отдаваясь не через return, и нн неявно. А если изменяется какой-то глобальный метод, то это уже сильно, так как не вижу что функция в классе, а хранить и изменят глобальные переменные это слишком. Ладно если бы изменялись члены класса, но не так же.
+ когда это в cpp был сборщик мусора? Наверно нужно очищать всё что намусорили, а не оставлять все на съедение стёка.
Зачем там в цикле *rgb = malloc() каждый раз делается? Если уж хочется скушать памяти, можно это сделать как-то так:
, а уже циклами по массиву бегать.
чувак, мой тебе искренний совет: пожалей своих будущих пользователей и тех, кто будет потом твой код ковырять: смени профессию
Учиться надо, а не рубить на корню.
Бейсику и Паскалю совсем не учили?
ЗЫ. Меня никто не учил, сам все делал.
Копипастить можно и в пределах одной функции.
>учусь я исключительно на своих ошибках
Лучше всё же на чужих, их больше.
У каждого свое мнение. Я считаю что лучше на своих ошибках учиться.
Сколько раз у Вас написана эта строчка "(*(ibuf + sizeof(int) - 1) << (8 * 3))" ?
И такого там очень-очень много.
Кроме того между открывающей и закрывающей скобкой не должно быть более 20 строк.
Уберите волшебные числа типа 3.
Я на сях давно не писал, и глаз у меня уже не тот, но мне кажется что не всю память, забранную у кучи mallocом Вы ей возвращаете.
В сях нет гарбич коллектора, и рано или поздно память утечет.
Но вот давеча поговорил с тимлидером, он приверженец одной точки выхода.
Но завтра если не забуду, запощу, что у нас есть на эту тему.
лучший рефактроинг тот -- в результате которого файлы только удалились
собственно мой вариант
[spoiler]
[/spoiler]
Эй, Создатель, нужен тег СПОЙЛЕР срочно!
Размеры могут быть и 0-ми в хедере, не советуй больше.