open_memstream() richtig nutzen.

am ende der Ausführung einen "Speicherabzugfehler

Und was sagt der Debugger?

Wutz

open_memstream ist kein C-Sprachstandard,
UB bei %p, da nur für Datenzeiger definiert, nicht für Funktionen.

#include <inttypes.h>
#include <stdbool.h>
#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

/*
 * hat keinen Pointer, deswegen auch keine Ausgabe dazu
 */
void formatMoney2(int64_t cents, FILE* stream) {
	bool isNegative = cents < 0;
	cents = (isNegative ? -cents : cents);
	int64_t euros = cents/100;
	cents -= 100*euros; // cents = 0

	fprintf(stream, "%s%"PRId64".%02"PRId64" euros\n", (isNegative ? "-" : ""), euros, cents); //formatMoney
}

char* formatMoney3(int64_t cents) {
	bool isNegative = cents < 0;
	cents = (isNegative ? -cents : cents);
	int64_t euros = cents/100;
	cents -= 100*euros; // holt sich die cents 0.x

	size_t characterCount;

	char* result;
	FILE* stream = open_memstream(&result, &characterCount);
	formatMoney2(cents, stream);
	fclose(stream);
	return result;
}

void printMoney(uint64_t cents, char* (*formatter)(int64_t cents)) {
	char* string = formatter(cents);
	printf("formatMoney(%"PRId64", %p) = %s\n", cents, formatter, string);
	free(string);
}

int main() {

	formatMoney2(-196720000000000, stdout);
	formatMoney2(-314, stdout);
	formatMoney2(-31, stdout);
	formatMoney2(-3, stdout);
	formatMoney2(0, stdout);
	formatMoney2(2, stdout);
	formatMoney2(27, stdout);
	formatMoney2(271, stdout);

	printMoney(-196720000000000, formatMoney3);
	printMoney(-314, formatMoney3);
	printMoney(-31, formatMoney3);
	printMoney(-3, formatMoney3);
	printMoney(0, formatMoney3);
	printMoney(2, formatMoney3);
	printMoney(27, formatMoney3);
	printMoney(271, formatMoney3);
	
	return 0;
}

https://onlinegdb.com/rk5T2yJCB

pcovc

@Wutz

Soweit ich weiß schon.
https://www.gnu.org/software/libc/manual/html_node/String-Streams.html

@Wutz sagte in open_memstream() richtig nutzen.:

open_memstream ist kein C-Sprachstandard,

Wieso ein UB bei %p. Der gibt mir doch einfach den Pointer der in formatMoney liegt.

UB bei %p, da nur für Datenzeiger definiert, nicht für Funktionen.

Swordfish

@pcovc sagte in open_memstream() richtig nutzen.:

Soweit ich weiß schon.

Was Du verlinkst ist eine Dokumentation zur glibc, das hat nichts mit einem Standard (ANSI/ISO) zu tun.

pcovc

@manni66
nach fclose in Zeile 36 kriege ich:
Program received singal SIGSEGV, Segmentation fault.
in _IO_mem_finish at memstream.c:136

Ich kann damit nicht so viel anfangen. vielleicht kannst du mir dazu was sagen?

pcovc

@Swordfish sagte in open_memstream() richtig nutzen.:

Was Du verlinkst ist eine Dokumentation zur glibc, das hat nichts mit einem Standard (ANSI/ISO) zu tun.

ach ich habe was falsch bei der antwort verstanden.
Aber es sollte doch trotzdem mit open_memstream() lösbar sein oder nicht?
Zumindest sagt es die aufgabenstellung:
Diese Variante soll nicht selbst den String zusammensetzen, sondern auf formatMoney2() zurückgreifen indem sie zunächst einen FILE* mit open_memstream( )erzeugt,dann mit formatMoney2() den String in diesen Stream reinschreibt, und zum Schluss mit fclose() den Stream schließt und den Ergebnisstring zurückgibt.

pcovc

@Wutz
Ich muss mich entschuldigen. Der Ansatz hat mir doch sehr geholfen.
Nun stimmt es auch.

Swordfish

@others (damit man nicht lange suchen braucht) Der Fehler liegt an den falschen Parametern für open_memstream(). Die Funktion erwartet als zweiten Parameter einen Pointer auf size_t. Wahrscheinlich war gewollt *characterCount zu inkrementieren, nicht den Pointer selbst. (Was aber auch keinen sinn hat weil sich open_memstream() nicht um den wert des size_t schert.)

SeppJ

Offtopic: Ja, ja, ja, ich kümmere mich drum. Der Thread war vielleicht 5 Sekunden im falschen Forum und schon sind 2 Meldungen da...

pcovc

@SeppJ
Stimmt. Danke dafür