kleines verwaltungsprogramm für wareneingang

b_wise

danke für die schnelle antwort!

also das abspeichern in der Textdatei hab ich absichtlich so strukturiert, damit man die Möglichkeit hat, am Ende des Tages die Wareneingänge auszudrucken.

aber ich probier jetzt mal dein Lösungsvorschlag aus und speicher die zusammengehörenden Werte Zeilenweise ab.

noch zum Verständnis:

Kugellager;SKF;100;19.21;Hans Wurst
Kugellager;FAG;15;23.43;Hans Wurst

müsste ich in diesem Fall auf token4 zugreifen, um meine Preise auszulesen?

Hab hier mal deine Idee versucht auf die schnelle umzusetzten, habs aber noch nicht ausprobiert:

//globale Variable: 

float gesamtkosten = 0;

//lokal:

char string [255];
char trennzeichen [255] = ";";
char *wort;
float preis = 0;

while (!feof(fp)) 
{
  fgets(string,sizeof(string),fp);
  int i = 1;
  wort = strtok(string, trennzeichen);
  while(wort != NULL) 
  {
    if (wort == 4);
     {
      preis = atof(wort);
      gesamtkosten = gesamtkosten + preis;
     }
    wort = strtok(NULL, trennzeichen); 
  }

}

hab mich nach http://de.wikibooks.org/wiki/C-Programmierung:_Strings#strtok gerichtet.

bin ich damit auf dem richtigen weg, oder tapp ich immer noch im dunkeln? ^^

strtok ist doch voll der schrott. mach das doch mit fscanf und fprintf

b_wise

hm wüsste nicht, wie ich das mit fscanf und fprintf machen könnte...
wenn du mir einen konkreteren Lösungsvorschlag geben könntest, wär ich dir dankbar

TRENN DIE WERTE DURCH LEERZEICHEN UND LIES DANN EINFACH MIT fscanf ein und schreib die datei entsrpechend mit fprintf. solche strtok und fwrite frickeleien und parsereien gehen meist in die h0se

sroyy for caps

mngbd

bin ich damit auf dem richtigen weg, oder tapp ich immer noch im dunkeln?

So hatte ich mir das vorgestellt. (Hab's aber nicht getestet)
Falls du in Zeile 19 i statt wort gemeint hast.

also das abspeichern in der Textdatei hab ich absichtlich so strukturiert, damit man die Möglichkeit hat, am Ende des Tages die Wareneingänge auszudrucken.

Das kriegst du auch so hin.
ZB mit Excel, dafür musst du die Datei dann nur mit der Endung .csv versehen.
CSV ist auch außerhalb von Office weit verbreitet.

b_wise

hi Leute,

hab jetzt wieder etwas, woran ich mich aufgehangen habe.
Und zwar will ich beim Einlesen von dem Stückpreis alle möglichen falschen Eingaben abfangen..und den Anwender solang was eingeben lassen, bis der Stückpreis die richtige Form hat.

Zunächst will ich den Anwender solange was eingeben lassen, bis er eine zahl eingibt:

do
{
  printf("Stueckpreis:");
  gets(pr); 

  i=0; a=pr[0];
  while (a!= '\0') 
  {
    if ((a>='0') && (a<='9'))
    {
    c = 1;  
    }     
    else
    {
    c = 0;
    }
    a=pr[++i];
   }

}while ( c == 0);

Mit dieser Schleife kann ich folgende Falscheingaben abfangen:

fünfzig
12asdfkl

Das Problem an der schleife ist jedoch, das Eingaben wie z.B. "alöksdf123.30" (weil am ende eine Zahl steht) oder "12,50" nicht abgefangen werden können!

Oder jedes mal, wenn ich "12,50" statt "12.50" eingebe, wird mir der Stückpreis mit "12.00" in der Textdatei abgespeichert!

Was kann ich dagegen tun ? Gibt es einen Standartbefehle, mit denen ich die Fehler abfangen kann ? Hoffe es kann mir jemand bei den zwei Sachen helfen.

danke schonmal !

Zunächst will ich den Anwender solange was eingeben lassen, bis er eine zahl eingibt:

Der Stückpreis ist wahrscheinlich keine ganze Zahl, also willst du eine Gleitkommazahl abfragen.
Du könntest man: strtof und man: strtold verwenden:

#include <stdio.h>
#include <stdlib.h>

int main (void)
{
  char *start = "3.14159Xasdf", *last = NULL;
  double zahl = 0.0;

  zahl = strtold(start, &last);

  printf("Zahl: %f\nnaechstes Zeichen: %x (%c)\n", zahl, *last, *last);

  return 0;
}

Wenn das nächste Zeichen nicht Null ist, dann ist nach der Zahl noch Rauschen (in dem Fall 'X').

b_wise schrieb:

do
{
  printf("Stueckpreis:");
  gets(pr);

Über man: gets(3) aber:

gets(3) schrieb:

Never use gets(). Because it is impossible to tell without knowing the
data in advance how many characters gets() will read, and because
gets() will continue to store characters past the end of the buffer, it
is extremely dangerous to use. It has been used to break computer
security. Use fgets() instead.

Also wenn du pr im Kontext definiert hast

fgets(pr, sizeof(pr), stdin)
// statt
gets(pr);

Sonst musst du dir noch die Länge merken.

b_wise schrieb:

if ((a>='0') && (a<='9'))
{
c = 1;  
}    
else
{
c = 0;
}

c = (a>='0') && (a<='9');

b_wise

hm das hilft mir alles nicht weiter

Und warum soll ich raten?
Wo bist du denn steckengeblieben?

b_wise

servus,

ich will nur reelle Zahlen einlesen, aber irgendwie kann ich mit den zwei funktionen nur ganze zahlen einlesen...weiss einfach nicht mehr wie ich das lösen soll

void einlesen_stueckpreis()
{

    printf("Stueckpreis: ");
    gets(stueckpr);
    kontrolle_eingabe_stueckpreis(stueckpr);

}

int kontrolle_eingabe_stueckpreis(char stueckpreis[])

{    

char a;
int i = 0, c = 0;

do 
 {

  a = stueckpreis[0];
  while (a!= '\0') 
  {
    if ((a>='0') && (a<='9') && '.')
    {
    c = 1;  
    }     
    else
    {
    printf("\nEs ist nur die Einganbe von Reellen Zahalen gueltig!\n\n");
    einlesen_stueckpreis();
    }
    a=stueckpreis[++i];
   }

 }while ( c == 0); 

return 0;

}

danke schonmal für lösungsvorschläge

Dein Code sieht eher wenig vertrauenswürdig aus:

einlesen_stueckpreis ruft kontrolle_eingabe_stueckpreis auf, und umgekehrt, bist du sicher,
daß da nicht noch eine Endlosschleife rauskommt?

((a>='0') && (a<='9') && '.')

'.' ist immer wahr. Wahrscheinlich meinst du

((a>='0') && (a<='9') || a=='.'))

In rellen Zahlen sind BTW noch mehr Zeichen als Ziffern und der Punkt zulässig.
Zb + und -, aber auch E und e. Etwas vergessen? Egal, denn:

~mngbd schrieb:

Du könntest man: strtof und man: strtold verwenden

Das hatte ich gesagt, und alles, was dir dazu eingefallen ist, war:

hm das hilft mir alles nicht weiter

Also mein Vorschlag für Gleitkommazahlen statt ganzen Zahlen:

int kontrolle_eingabe_stueckpreis(char stueckpreis[])
{
    char *last = NULL;

    if (strtold(stueckpreis, &last) == 0.0 && stueckpreis == last)
        return 1;  // stueckpreis ist keine gültige Zahl
    else
        return 0;  // stueckpreis ist gültig
}

Wenn du mir weiterhin nicht sagst, warum dir das nicht weiterhilft, wird mir die Lust vergehen, dir zu helfen.