hallo,

ich habe hier ein Problem mit len=name.length(), ich frag mich warum dies nicht funktioniert? Weil ich die stringlänge 15 schon definiert habe?

char name[15];

Hi, das liegt einfach daran, dass Du keinen std::string benutzt, sondern ein char Array. versuchs mal mit

std::string name;

grüße

- erstmal was soll das ganze eigentlich machen ??
- wenn du std::string benutzen willst musst du es auch inkluden ( #include <string> )
- dein funktionskopf ist irgendwie falsch... mach mal das "=" weg..
- es ist sehr verwirrend, dass du sowohl "name" als parameter(string) hast als auch als globales char-array
- was willst du bei dieser if-abfrage vergleichen ? "char name[15]" oder "(string name)
-> zur zeit wird char name[15] verglichen
-> zum vergleichen des strings machst du

if((strcmp(name.c_str(),"go.txt"))==0)

// oder

if(name == "go.txt")

Gustl schrieb:

#include <math.h>
#include <conio.h> //getch();
#include <cstring>
#include <fstream>
#include <iostream>

using namespace std;

    // warum globale Variablen?
    int a=0;
    fstream rf,wf;
    char name[15];

// Warum gibt es ein Default Paramter und dafür keinen Namen für die Variable?
// Was soll der Rückgabewert für eine Aufgabe erfüllen.
// Es riecht danach, daß bool der bessere Typ wäre.    
int txtcsv(string=name)
{
    int v=0,len;
    //len=std::string::size()
    len=name.length();
    name.erase(len-4,3);
    cout<<name<<endl;
    // Warum strcmp?
    // if ("go.txt" == name) tut es bei einem std::string auch
    if((strcmp(name,"go.txt"))==0)
    {v=1;}
    return v;
    }

Versuchen wir mal das Ganze abzurunden

#include <cmath>
#include <conio.h> //getch();
#include <string>
#include <fstream>
#include <iostream>

using namespace std;

bool txtcsv(std::string name) {
    std::string::size_type len (name.length());

    name.erase(len-4,3);

    cout << name << endl;

    if ("go.txt" == name) {
        return true;
    }

    return false;
}

Hier der Code:

#include <math.h>
#include <conio.h> //getch();
#include <cstring>
#include <fstream>
#include <iostream>

using namespace std;

    int a=0;
    fstream rf,wf;
    std::string name;

int txtcsv(string=name)
{
    int v=0,len;
    //len=std::string::size()
    len=name.length();
    //std::distance(name.begin(), name.end());
    name.erase(len-4,3);
    cout<<name<<endl;
    //if((strcmp(name,"go.txt"))==0)
    {v=1;}
    return v;
    }

void schreiben()  //schreibt die Tablelle in die Datei
{
    wf.open(name,ios::out);
    wf<<"i.;i;i*i;sqrt(f);log(d)"<<endl;   
    wf<<endl;
    for(int i=1;i<=100;i++)
    {
    float f=i;
    double d=i;    
    wf<<i<<".;"<<i<<";"<<i*i<<";"<<sqrt(f)<<";"<<log(d)<<endl;                
    }
    /*wf<<setw(8)<<"i"<<setw(8)<<"i"<<setw(8)<<"i*i"<<setw(11)<<setprecision(3)<<setiosflags(ios::fixed)<<"sqrt(f)"<<setw(11)<<"log(d)"<<endl;   
    wf<<endl;
    for(int i=1;i<=100;i++)
    {
    float f=i;
    double d=i;    
    wf<<setw(7)<<i<<setw(1)<<"."<<setw(8)<<i<<setw(8)<<i*i<<setw(11)<<setprecision(3)<<setiosflags(ios::fixed)<<sqrt(f)<<setw(11)<<log(d)<<endl;                
    }*/
    wf.close();
    return;    
}

int main()
{
    do
    {
    cout<<"Name der Datei?"<<endl;
    cin>>name;
    int gang=0;
    gang=txtcsv(name);
    cout<<"gang="<<gang<<endl;
    rf.open(name,ios::in);
    if(rf)
          {
          cout<<"Die Datei existiert bereits!\n"
              <<"1 - Datei ueberschreiben?\n"
              <<"2 - Datei auslesen?\n"
              <<"3 - Dateinamen aendern?\n"
              <<"4 - Inhalt der Datei loeschen?"<<endl;
          cin>>a;
          switch(a)
                   {
                   case 1:
                        schreiben();
                        cout<<"Datei wurde ueberschrieben..."<<endl;
                        break;
                   case 2:
                        char s[100];
                        while(!rf.eof())                        //bis zum ende der file eof
                        {
                        rf.getline(s, 100);                     //liest Zeilenweise in char s ein
                        cout<<s<<endl;
                        }
                        rf.close();
                        break; 
                   case 3:
                        rf.close();
                        break;
                   case 4:
                        wf.open(name,ios::out|ios::trunc);      //öffnet die Datei zum schreiben und löscht den Inhalt
                        wf.close();
                        rf.close();
                        cout<<"Inhalt der Datei geloescht"<<endl; 
                        break;   
                   }   
          }
    else
    {
    schreiben();
    cout<<"Datei wurde erstellt... / Tabelle wurde einschrieben..."<<endl;
    a=0;
    }
    }while(a==3);
    getch();
    return EXIT_SUCCESS;
}

Ich will das ich einen Dateinamen angeben kann und das Programm dann auf die Endungen csv und txt entsprechend reagiert. in txt soll er eine tabelle mit abständen und Kommastellen etc. schreiben und in csv alles nur mit trennzeichen ;
Das habe ich vor...

@Con@n
Wenn ich std::string name; anstatt char name[15];
einsetze bringt er mir dann beim zugreifen auf dateien fehler...

das mit dem if(go.txt)... sollte nur ein test sein und wird dann abgeändert.

Hab auch schon gegooglet und hier gesucht, aber irgendwie keine antwort auf mein problem gefunden.

#include <cmath>
#include <cstring>
#include <fstream>
#include <ostream>
#include <iostream>

using namespace std;

// globale Variablen entfernt, die braucht man nicht

// richtige Parameter Übergabe eingeführt
// wobei sich die Frage stellt, ob Du das Original verändern willst
// oder nur die lokale Kopie?
int txtcsv(string name) {
	// den richtige Datentyp für len eingesetzt
	std::string::size_type len (name.length());

	int v(0);

	name.erase(len-4,3);
	cout << name << endl;
	//if((strcmp(name,"go.txt"))==0)
	{v=1;}

	return v;
}

void schreiben(std::ostream& wf) {
    wf << "i.;i;i*i;sqrt(f);log(d)\n\n";

    for(int i=1;i<=100;i++) {
		float	f(i);
		double	d(i);    
		wf << i << ".;" << i << ";" << i*i << ";" << sqrt(f) << ";" << log(d) << "\n";                
    }

    /* auskommentiertes gelöscht */

    return;    
}

int main() {
	int a = 0;
	std::string	name;
	std::fstream	rf;

    do {
		cout << "Name der Datei?" << endl;
		cin >> name;

		int gang (txtcsv(name));

		cout << "gang=" << gang << endl;

		rf.open(name.c_str(),std::ios::in);

		if (rf) {
			cout << "Die Datei existiert bereits!\n"
			     << "1 - Datei ueberschreiben?\n"
			     << "2 - Datei auslesen?\n"
			     << "3 - Dateinamen aendern?\n"
			     << "4 - Inhalt der Datei loeschen?" << endl;
			cin >> a;

			switch(a) {
				case 1:
					{
						std::fstream datei (name.c_str(), std::ios::out);
						schreiben(datei);
						cout << "Datei wurde ueberschrieben..." << endl;
					}
					break;
				case 2:
					char s[100];
					while(!rf.eof()) {		// bis zum ende der file eof
						rf.getline(s, 100);	// liest Zeilenweise in char s ein
						cout << s << endl;
					}
					rf.close();
					break;
				case 3:
					rf.close();
					break;
				case 4:
					{
						std::fstream datei (name.c_str(), std::ios::out|std::ios::trunc);
					}
					rf.close();
					cout<<"Inhalt der Datei geloescht"<<endl;
					break;  
			}  
		} else {
			std::fstream datei (name.c_str(), std::ios::out);
			schreiben(datei);
			cout << "Datei wurde erstellt... / Tabelle wurde einschrieben...\n";

			a=0;
		}
	} while(a==3);

    // getch(); // reine Windows Eigenheit
    return EXIT_SUCCESS;
}