Einige Probleme mit einem Projekt

SeppJ

PepTic schrieb:

das mit dem compiler ist so ne Sache, ich nutze Visual Studio 2010 Express, hat das überhaupt nen c-compiler?

Ja, hat es. Wobei du mit dem noch viel mehr Inkompatiblitäten feststellen wirst, als ich genannt habe. C hat im Lauf der Jahre einige Features neu dazu bekommen. Die wichtigsten Meilensteine waren 1989, 1999 und 2011. Darunter auch einige C++-Features, die nach C zurück geflossen sind.
Problem: Der Microsoft C-Compiler kann nur C wie aus dem Jahr 1989 (was aber durchaus nicht heißen soll, dass das schlecht ist). Du hast aber einige der C-Features aus den neueren Sprachversionen benutzt. Mit dem C++-Compiler sind dir diese ebenfalls nicht aufgefallen, da diese Features auch Teil von C++ sind.

phf

Okay, vielen Dank.
Ich frage mich, was der Unterschied zwischen einem const int, einem fest als Arraygröße eingetragenem Zahlenwert und auch z.B. einem 'define VALUE (128)'
wäre.

Edit: Also char bla[VALUE];

SeppJ

Eine const Variable ist in C wie eine normale Variable, bloß mit Compilerfehler, wenn man sie zu ändern versucht. Sie hat ansonsten für den Compiler keine Sonderbedeutung.

Ein define ist nur eine Textersetzung, die stattfindet, bevor der Compiler überhaupt dran kommt.

#define Bla Blupp
Bla Bla

Der Compiler sieht nur:

Blupp Blupp

Eine feste Zahl im Programm ist eine echte Konstante für den Compiler und kann daher als Arraygröße benutzt werden.

phf

thx

PepTic

Also jetzt bekomme ich einige Warnungen angezeigt.

Der Rückgabewert wird ignoriert: "scanf" main.c Zeile 71
Der 2-Parameter im Aufruf von "scanf" muss die Adresse der Zeichenfolge sein. main.c Zeile 71

Ich weiß aber ehrlich gesagt nicht was ich da jetzt ändern muss.

Auch mit den Hilfestellungen von euch kann ich nicht wirklich was anfangen, wie gesagt, hab den code mit mühe und not gerade so hinbekommen wie er ist...

Könnt ihr vllt noch mehr ins Detail gehen?

SeppJ

PepTic schrieb:

Der Rückgabewert wird ignoriert: "scanf" main.c Zeile 71
Der 2-Parameter im Aufruf von "scanf" muss die Adresse der Zeichenfolge sein. main.c Zeile 71

Ein dicker Fehler, eine Designsschwäche:
Fehler: Du willst eine Zeichenkette einlesen. Du gibst aber die Adresse von etwas anderem an. Ich weiß nicht genau, wie dein neuer Code aussieht, aber wahrscheinlich bezieht sich das auf eine Zeile wie Zeile 41 im Eröffnungsbeitrag, wo du die Adresse eines Kontos übergibst, statt die Adresse des gewünschten Attributs dieses Kontos.
Designschwäche: Eingaben können auch fehlschlagen. Wenn du den Rückgabewert der Eingabefunktionen nicht prüfst, dann arbeitest du mit irgendwelchen Müllwerten weiter, ohne es zu merken.

PepTic

danke für deine Hilfe, ich weiß aber wie gesagt nicht, wie ich es umsetzen soll.

PepTic

Ich möchte ja keine Komplettlösung, nur ein paar Codeansätze würden mir sicher helfen.

SeppJ

PepTic schrieb:

Ich möchte ja keine Komplettlösung, nur ein paar Codeansätze würden mir sicher helfen.

Mir ist nicht klar, womit du noch Schwierigkeiten hast.
Wenn dir scanf oder einfache Kontrollflussverzweigungen Probleme machen, dann geh nochmal zurück zu den ersten Kapiteln deines Buches. Dies ist ja kein Problem, irgendwelche komplizierten Ausdrücke mit scanf-Formatstringmagie zu parsen, sondern bloß die absolute Basisfunktionalität, wie man einfache Integer oder Zeichenketten einliest. Das ist Grundwissen.

PepTic

Hab den Code nochmal überarbeitet:

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include "konto.h" 
#include <conio.h>
int main() 
{ 
    int auswahl; 
	int IDZaehler = 100; 
	int kontoID;
	int kontoID2;
    double guthaben; 

	do
    { 
		printf("-----------------------------------KONTOFUEHRUNG--------------------------------\n");
		printf("Herzlich Willkommen \n");
		printf("------------------- \n\n");
        printf("1) Konto erstellen\n"); 
        printf("2) Konto anzeigen\n"); 
        printf("3) Konto loeschen\n"); 
        printf("4) Ueberweisung\n"); 
        printf("0) Beenden\n\n"); 
		printf("------------------- \n");
        printf("Auswahl: "); 
		scanf("%i", &auswahl);

		switch(auswahl) 
        { 
            case 0: 
				speichere_konto(kontoID);
                return 0; 
                break; 
            case 1: 
                erstelle_konto(IDZaehler); 
                IDZaehler++; 
                break; 
            case 2: 
                printf("Geben sie die Kontonummer ein: "); 
				scanf("%d", &kontoID); 
                zeige_konto(kontoID); 
                break; 
            case 3: 
                printf("Geben sie die Kontonummer ein: "); 
				scanf("%d", &kontoID); 
                loesche_konto(kontoID); 
				printf("Konto wurde geloescht\n");
                break; 
            case 4: 
                printf("Geben sie die Kontonummer ein, von der abgebucht werden soll: "); 
				scanf("%d", &kontoID); 
                printf("Geben sie die Kontonummer ein, an die ueberwiesen werden soll: "); 
				scanf("%d", &kontoID2); 
                printf("Geben sie den Betrag ein, der ueberwiesen werden soll: "); 
				scanf("%f", &guthaben); 
                ueberweisung(kontoID, kontoID2, guthaben); 
                break; 
            default: 
                break; 
        } 
		printf("Druecken Sie eine beliebige Taste um vorzufahren");
		getch();
		system("cls");
    } while(auswahl);
		return;
}

void erstelle_konto(int ID) 
{ 

	printf("\n\nVorname:     "); 
	scanf("%s", konten[ID].vorname); 
    printf("Nachname:    "); 
	scanf("%s", konten[ID].nachname); 
    printf("Kontostand:  "); 
	scanf("%f", &konten[ID].kontostand); 
    printf("Kontonummer: %d\n", ID); 
	konten[ID].kontonummer = ID;
    printf("Konto wurde erstellt!\n\n\n"); 

} 

void loesche_konto(int ID) //man kann nichtvorhandene konten löschen
{ 
    strcpy(konten[ID].vorname, ""); 
    strcpy(konten[ID].nachname, ""); 
    konten[ID].kontostand = 0; 
} 

void zeige_konto(int ID) 
{ 
    if(strlen(konten->vorname) == 0 || strlen(konten->nachname) == 0) 
	{ 
        printf("\n\nDieses Konto existiert nicht!\n\n\n"); 
    } 
    else 
    { 
        printf("\n\nVorname:     %s\n", konten[ID].vorname); 
        printf("Nachname:    %s\n", konten[ID].nachname); 
        printf("Kontostand:  %.2f\n", konten[ID].kontostand); 
        printf("Kontonummer: %d\n\n\n", konten[ID].kontonummer); 
    } 
} 

void speichere_konto(int ID) 
{ 

	FILE *datei; 
    datei = fopen("Konten.txt", "a+"); 

	if(datei == NULL)

	{
		printf("\nFEHLER BEIM SPEICHERN"); 
		return;
	}

	else
	{
		fprintf(datei, "Kontonummer: %d\nVorname:     %s\nNachname:    %s\nKontostand:  %.2f\n\n", konten[ID].kontonummer, konten[ID].vorname, konten[ID].nachname, konten[ID].kontostand); 
	}
    fclose(datei); 

} 

void ueberweisung(int vonID, int zuID, double guthaben) 
{ 

    if(konten->kontostand - guthaben < 0) 
    { 
        printf("\n\nNicht genug Guthaben!\n\n"); 
    } 
    else 
    { 
        konten[vonID].kontostand = konten[vonID].kontostand - guthaben; 
        konten[zuID].kontostand = konten[zuID].kontostand + guthaben; 
        printf("\n\nUeberweisung erfolgreich!\n\n"); 
    } 
}

struct konto
{ 
    char vorname[20]; 
    char nachname[20]; 
    double kontostand; 
    int kontonummer; 
}konten[1000];
typedef struct konto konto;

void erstelle_konto(int); 
void loesche_konto(int); 
void zeige_konto(int); 
void speichere_konto(int);
void ueberweisung(int, int, double);

Wenn ich jetzt Konto anzeigen auswähle, sagt er mir es existiert nicht.
in die Datei wird es aber geschrieben, nur der Kontostand ist 0.00

bekomme außerdem diese Compilerfehler:
Nicht initialisierter Speicher "kontoID" wird verwendet.: Lines: 8, 9, 10, 11, 12, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 29, 31, 32

und

Die möglicherweise nicht initialisierte lokale Variable "kontoID" wurde verwendet.

Das Problem mit dem Rückgabewert besteht immer noch bei scanf und getch.

PepTic schrieb:

Nicht initialisierter Speicher "kontoID" wird verwendet.: Lines: 8, 9, 10, 11, 12, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 29, 31, 32

Du hast der Variablen "kontoID" vor der Nutzung noch keinen Wert zugeteilt. Die Variable ist nicht initialisiert.
Lokale Variablen werden nicht automatisch mit 0 Initiolisiert.

Weißt du, was der Rückgabewert einer Funktion ist?

Du hast jetzt ein globales Array mit deinen Kontendaten.
Der Index in dem Array hat eigentlich wenig mit der Kontonummer zu tun.

DirkB schrieb:

float ist ein schlechter Datentyp für Geldbeträge. double auch. Rechne in Cent.
Eine Kontonummer ist keine Ganzzahl.

Jetzt hast du int für die Kontonummer. Da war unsigned ja noch richtig gut gegen.

SeppJ

edit: Dein Compiler hat doch recht . Hatte den case 0 nicht gesehen.

Weitere Fehler:
Hier dran ist falsch (nur Compilerdiagnostik, nicht das was ich sonst noch ändern würde):
- %f ist nicht der scanf-Formatspezifizierer für double, sondern für float. scanf ist nicht das genaue Gegenstück zu printf!
-Du hast in der main ein return ohne Wert, aber main gibt int zurück!

PepTic

Du hast der Variablen "kontoID" vor der Nutzung noch keinen Wert zugeteilt. Die Variable ist nicht initialisiert.
Lokale Variablen werden nicht automatisch mit 0 Initiolisiert.

Weißt du, was der Rückgabewert einer Funktion ist?

Hab nun ne den Wert auf 0 gesetzt, trotzdem kann ich durch Eingabe der Kontonummer nichts einsehen.

- %f ist nicht der scanf-Formatspezifizierer für double, sondern für float. scanf ist nicht das genaue Gegenstück zu printf!

läuft jetzt!

Du hast jetzt ein globales Array mit deinen Kontendaten.
Der Index in dem Array hat eigentlich wenig mit der Kontonummer zu tun.

und wie müsste es aussehen?

Jetzt hast du int für die Kontonummer. Da war unsigned ja noch richtig gut gegen.

die Kontonummer besteht doch nur aus höchstens 4Ziffern, was empfiehlst du denn?

edit:
Jetzt passt soweit alles, nur möchte ich, dass ich zu Beginn des Programms auf den Inhalt der Datei zugreifen und ihn verwenden kann.