Guten Abend! :-)

JTR666

@JTR666
Aber eigentlich auch nicht, denn mit getline(...) kann der Text wunderbar ausgespuckt werden...

wob

@JTR666
Aber eigentlich auch nicht, denn mit getline(...) kann der Text wunderbar ausgespuckt werden...

Es bringt nichts, wenn du weiter keinen Code zeigst. Wenn ich aus dem Fenster schaue, sehe ich Schnee. Ist genauso relevant.

JTR666

Dieser Beitrag wurde gelöscht!

JTR666

#include <iostream>
#include <vector>
#include <fstream>
#include <string>

int main()
{
std::string text, word;
char letter, c;
std::fstream file;
std::vectorstd::string wordlist, final_wordlist;
std::vector<int> counter;

file.open("Der_verwirrte_Mann.txt");

do
{
    std::cout << (char)file.get() << std::endl;
    /*if(65 <= file.get() <= 90 || 97 <= file.get() <= 122)
    {
        word += file.get();

        if(0 <= file.get() < 65 || 91 <= file.get() < 97)
        {
            wordlist.push_back(word);
        }
    }

    word = ' ';*/
}while(!file.eof());

/*string_compare(wordlist, file, counter);*/

/*for(int i=0; i<wordlist.size(); i++)
{
    std::cout << wordlist[i] << " " << counter[i] << std::endl;
}*/

return 0;

wob

@JTR666 sagte in Guten Abend! :

/*if(65 <= file.get() <= 90 || 97 <= file.get() <= 122)

Das Programm reproduziert den Fehler nicht, weil der fehlerhafte Teil auskommtiert ist.

Hinweis: auch ein .get() innerhalb eines if liest ein Zeichen ein.
Hinweis 2: Mehrfachverkettungen wie 1 < a < 3 gibt es nicht. Daraus wird 1 < a (was ein bool ist), dessem Wert also 0 oder 1, wird dann weiter verglichen.
Hinweis 3: der EOF-Check muss direkt hinter das get(), nicht ans Ende. Siehe mein Beispiel.
Hinweis 4: siehe https://en.cppreference.com/w/cpp/header/cctype und werde deine Zahlen wie 90, 65, ... los.

JTR666

@wob Ich dachte halt die ganze Zeit, dass eof für "end of file" stünde, weswegen ich das n die while-Bedingung gepackt hab.
Was macht die Funktion get denn dann im Einzelnen?
Also sie liest ja immer einen neuen char aus.
Wenn dann eine if-Verknüpfung kommt, interpretiert sie diese ebenfalls als ein Zeichen, weil damit erst mal der filestream unterbrochen wird, oder wie?

wob

@JTR666 sagte in Guten Abend! :

Ich dachte halt die ganze Zeit, dass eof für "end of file" stünde,

Und wie kommst du nun darauf, dass das nicht so ist?!

@JTR666 sagte in Guten Abend! :

weil damit erst mal der filestream unterbrochen wird,

Da wird nichts unterbrochen.

Swordfish

@JTR666 sagte in Guten Abend! :

/* ... */

ftfy

#include <cctype>
#include <string>
#include <map>
#include <algorithm>
#include <fstream>
#include <iostream>

template<typename UnaryPredicate>
void ltrim(std::string &str, UnaryPredicate p)
{
    str.erase(str.begin(), std::find_if(str.begin(), str.end(), p));
}

template<typename UnaryPredicate>
void rtrim(std::string &str, UnaryPredicate p)
{
    str.erase(std::find_if(str.rbegin(), str.rend(), p).base(), str.end());
}

int main()
{
    auto const filename = "foo.txt";
    std::ifstream is{ filename };
    if (!is.is_open()) {
        std::cerr << "Couldn't open \"" << filename << "\" for reading :(\n\n";
        return EXIT_FAILURE;
    }

    std::map<std::string, long long unsigned> words;
    for (std::string word; is >> word;) {
        ltrim(word, [](char unsigned ch) { return std::isalpha(ch); });
        rtrim(word, [](char unsigned ch) { return std::isalpha(ch); });
        std::transform(word.begin(), word.end(), word.begin(), std::tolower);
        ++words[word];
    }

    for (auto const& word : words)
        std::cout << '\"' << word.first << "\":\t" << word.second << '\n';
}

Swordfish

@JTR666 sagte in Guten Abend! :

Was macht die Funktion get denn dann im Einzelnen?
Also sie liest ja immer einen neuen char aus.

Ja. Genau. Jedes mal wenn Du std::basic_istream::get() aufrufst bekommst Du das jeweils nächste Zeichen oder EOF zurück - egal wo Du die funktion Aufrufst.

Alleine hier (ich habe das /* entfernt)

@JTR666 sagte in Guten Abend!:

    std::cout << (char)file.get() << std::endl;
    if(65 <= file.get() <= 90 || 97 <= file.get() <= 122)
    {
        word += file.get();

        if(0 <= file.get() < 65 || 91 <= file.get() < 97)
        {
            wordlist.push_back(word);
        }
    }

liest Du also minimal 2, maximal 6 Zeichen. (Short-circuit evaluation).

Du solltest auch Magic Numbers im Code vermeiden und stattdessen in diesem Fall die Funktionen aus <cctype> verwenden.

Wenn man Zeichen will sollte man auch Character literals verwenden anstatt Integer. Also zB.

bool is_upper(char ch)
{
    return 'A' <= ch && ch <= 'Z';
}

und NICHT

bool is_upper(char ch)
{
    return 65 <= ch && ch <= 90;
}