Как перевести код из С++ в Си?

Question

[Sector567]

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

(основная проблема заключается в неявном указатели this. Не получается его заменить)
так как код довольно объемный и здесь не проходит, то выложил его сюда. Хотя на функции загрузки и сохранения можно даже не смотреть- проблема именно в изменении разрешения на си
так как код довольно объемный и здесь не проходит, то выложил его сюда. Хотя на функции загрузки и сохранения можно даже не смотреть- проблема именно в изменении разрешения на си
https://pastebin.com/UedJVPPc

Answer 1

[res2001]

Указатель this? В Си классов нет, вам нужно избавится от классов.
Класс переделывайте по следующей схеме:
1.все члены данных класса переходят в аналогичную структуру на Си как есть.
2.все методы класса реализуются в виде обычных функций, первым параметром в функции передавайте структуру из п.1, можете указатель на структуру назвать this, тогда в коде функции не придется this заменять на что-либо другое.

Comments

[Sector567]

Все кроме изменения разрешения изменил..
Вот код https://pastebin.com/UXwj0Ajd

Начиная с 170 по 303 то строку то, что пытался сделать с изменением разрешения, но, к сожалению ничего не получается... Разрешение изменяется, НО картинка вся белая...В чем моя ошибка понять не могу..

[res2001]

Тут вам поможет только сравнение выполнения кода на С++ и Си.
Добавьте в zoom() вывод всего чего только можно, и потом сравнивайте результаты из С++ и Си версии.

[Sector567]

res2001: блин, а вы не поможете? Просто через 40 минут уже работу сдавать, а я еще часа 2 точно буду возится с этим..

[res2001]

На сколько я понял в zoom() lol - это исходное изображение, а olo - масштабированное.
Но я не увидел ни одного присваивания olo. По моему вы изменяете оригинальную структуру.
Так же не понятно зачем объявлять две идентичные структуры: struct TMP и struct BMP (я не сравнивал досконально, может они и разные).

[Sector567]

попробовал, как вы посоветовали- сравнивать результаты из С++ и Си версии
https://pastebin.com/G1Mm13pj

Застрял на *this.
точнее на этих аспектах
BMP *TMP; (*this);
и
TMP.biHeight, TMP.biWidth.
Не знаю как их в си перевести...

void Zoom(struct BMP *lol, int k) // масштабирование
{
	int r, g, b, end; //Объявление указателя для переменной типа int

	if (k > 1)
	{
		BMP *TMP; (*this);// создание временного изображения

		// задание новых размеров и изменение данных о изображении
		lol->biWidth = lol->biWidth * k;
		lol->biHeight = lol->biHeight * k;
		lol->bfSize = lol->biHeight * lol->biWidth * 3 + 54; //Смещение, с которого начинается само изображение равно 54
		lol->biSizeImage = lol->biHeight * lol->biWidth * 3;
		int sz_line = lol->biWidth * 3;
		while (sz_line % 4 != 0) sz_line++;
		// выравнивание до кратно 4
		lol->MBMP = (unsigned char**)malloc(lol->biHeight * sizeof(unsigned char*));
		for (int i = 0; i < lol->biHeight; i++)
		{
			lol->MBMP[i] = (unsigned char*)malloc(sz_line *sizeof(unsigned char));
			for (int j = 0; j < sz_line; j++)lol->MBMP[i][j] = 255;
		}

		// перебор всех пикселей изображения
		for (int i = 0; i < TMP.biHeight; i++)
			for (int j = 0; j < TMP.biWidth; j++)
			{
				TMP.GetPixel(j, i, &r, &g, &b);
				for (int k1 = 0; k1 < k; k1++)
					for (int k2 = 0; k2 < k; k2++)
						PutPixelRGB(j*k + k1, i*k + k2, r, g, b);
			}
		return;
	}

	//if (k < 0)
	//{
	//	BMP TMP(*this);// создание временного изображения
	//	// задание новых размеров и изменение данных о изображении
	//	biWidth = biWidth / -k;
	//	biHeight = biHeight / -k;
	//	bfSize = biHeight * biWidth * 3 + 54;
	//	biSizeImage = biHeight * biWidth * 3;
	//	int sz_line = biWidth * 3;
	//	while (sz_line % 4 != 0) sz_line++;
	//	// выравнивание до СЛОВА (кратно 4)
	//	MBMP = (unsigned char**)malloc(biHeight * sizeof(unsigned char*));
	//	for (int i = 0; i < biHeight; i++)
	//	{
	//		MBMP[i] = (unsigned char*)malloc(sz_line *sizeof(unsigned char));
	//		for (int j = 0; j < sz_line; j++)MBMP[i][j] = 255;
	//	}
	//	// перебор всех пикселей изображения
	//	int sr, sg, sb;
	//	for (int i = 0; i < biHeight; i++)
	//		for (int j = 0; j < biWidth; j++)
	//		{
	//			if (-k % 2 == 0)end = -k / 2;
	//			else end = -k / 2 + 1;
	//			sr = 0; sg = 0; sb = 0;
	//			// обнуление суммы значений цветов пикселей
	//			for (int k1 = k / 2; k1 < end; k1++)
	//				for (int k2 = k / 2; k2 < end; k2++)
	//					if (-j*k + k2 < TMP.biWidth && -i*k + k1 < TMP.biHeight && -i*k + k1 >= 0 && -j*k + k2 >= 0)
	//					{
	//						TMP.GetPixel(-j*k + k2, -i*k + k1, &r, &g, &b);
	//						sr += r; sg += g; sb += b;
	//					}
	//			sr = sr / (k*k);
	//			sg = sg / (k*k);
	//			sb = sb / (k*k);
	//			if (sr > 255)sr = 255;
	//			if (sg > 255)sg = 255;
	//			if (sb > 255)sb = 255;
	//			PutPixelRGB(j, i, sr, sg, sb);
	//			// помещаем новое значение в результирующую матрицу
	//		}
	//	return;
	//}

	free(lol->MBMP);
}

[res2001]

Запись в С++
BMP TMP(*this);
Это создание нового объекта TMP класса BMP с параметром *this, т.е. видимо создается временный объект на базе существующего. Нужно смотреть копирующий конструктор объекта BMP.
Его сигнатура должна быть такая:
BMP(const &BMP <имя>)
У вас вместо TMP - olo.

Пока что в вашем коде я вижу, что вы все присваивания ведете в lol, это, судя по всему исходный объект, таким образом вы не только не создаете новый масштабированный объект, но и портите исходный.
В общем для полноценного анализа не хватает кода на С++.

[res2001]

TMP.biWidth в вашем случае будет olo->biWidth - это просто обращение к полю класса/структуры. Стрелка появилась, потому что у вас указатель на структуру, а в С++ ссылка.

Answer 2

[Mercury13]

Перед нами совершенно необъектный код, на вид тупо написанный на Си средствами Си++. Так что будет несложно.
1. Убрать из структуры private/public.
2. Вынести функции из структуры, явно прописав в них this:

int BMP::Load(char FileNames[])
→
int BMPLoad(struct BMP* this, char FileNames[])

3. Для старых версий Си — поднять определения всех переменных на верх функции.

А что нам с масштабированием? — на вид написан на том же Си и ничего не требует, кроме задания этого this.

Comments

[Sector567]

Все кроме изменения разрешения изменил..
Вот код https://pastebin.com/UXwj0Ajd

Начиная с 170 по 303 то строку то, что пытался сделать с изменением разрешения, но, к сожалению ничего не получается... Разрешение изменяется, НО картинка вся белая...В чем моя ошибка понять не могу..

[Mercury13]

Sector567:
1. Названия lol и olo смешат.
2. tmp тоже должен быть struct BMP, не надо создавать отдельную struct TMP. И вообще он должен быть временной локальной переменной, а не параметром.
3. tmp надо memcpy из исходной.

[Mercury13]

Sector567: Получилось?

Answer 3

[Wexter]

Наймите человека

Comments

[Sector567]

не успею уже