ASCII Schriftart einlesen



  • Danke. Sorry, ich hatte wieder was getrunken.
    Ich schau mir im Moment erst mal nur die Reaktionen an. Aber ich bedanke mich schon mal. Das sieht aus, als ob ich das verstehen könnte.

    Bis dann.



  • Ich musste eigentlich nur 2x eine Abfrage, ob string leer ist einfügen. Das umgeht dann noch andere Punkte, wie das es machmal auch 'leere' Zeichen gibt:

    @
    @
    @
    @
    @
    @
    @
    @@
    

    Außerdem brauch ich auch die Höhe nicht ständig mitzählen, sondern kann sie gleich aus fontChar.datanehmen.

    	    	                std::string str;
    	        		std::getline(fstream, headInfo); // read head info
    	        		// read font
    	        		FontChar fontChar;
    	        		while (std::getline(fstream, str))
    	        		{
    	        			if (!str.empty()) // if string is not empty / no  blank line
    	        			{
    	        				str.erase(str.end() - 1); // last char is no font char
    	        				fontChar.width = str.length() - 1; // get width of font char
    	        				if (!str.empty() && str.back() == '@') // last line of font char
    	        				{
    	        					str.erase(str.end() - 1); // last char is no font char
    	        					fontChar.data.push_back(str); // write string in fontChar.data vector
    	        					fontChar.height = std::size(fontChar.data);  // get height of font char
    	        
    	        					characterSet.push_back(fontChar); // write font char in characterSet vector
    	        					//printFontChar(fontChar);
    	        					//_key();
    	        					fontChar.clear();
    	        					// new font char
    	        					continue; // go to while
    	        				}
    	        				fontChar.data.push_back(str);
    	        			}
    	        		}
    	        	
    

    In den Original Fonts gibt es noch Kommentare zwischen dem Head Info und den Font, aber ohne jedes Zeichen, die einen Kommentar einleiten würde. Die editiere ich weg. Das würde zu weit führen, die auch noch einzubeziehen.



  • @zeropage sagte in ASCII Schriftart einlesen:

    In den Original Fonts gibt es noch Kommentare zwischen dem Head Info und den Font, aber ohne jedes Zeichen, die einen Kommentar einleiten würde. Die editiere ich weg. Das würde zu weit führen, die auch noch einzubeziehen.

    Wieso? Die zu ignorieren sollte einfach genug sein.
    Die Schleife kann man dann auch noch etwas vereinfachen:

        while (std::getline(fstream, str)) {
            if (str.empty()) // skip empty lines
                continue;
            if (str.back() != '@') // skip comments
                continue;
    
            str.pop_back(); // remove data line marker
    
            bool const isLastLineOfChar = !str.empty() && str.back() == '@';
            if (isLastLineOfChar)
                str.pop_back(); // remove last-line-of-char marker
    
            fontChar.data.push_back(str);
            if (isLastLineOfChar) {
                fontChar.height = std::size(fontChar.data);
                characterSet.push_back(std::move(fontChar));
                fontChar.clear();
            }
        }
    


  • Mit Kommentar sieht zB so aus:

    flf2a$ 8 7 54 0 12 0 64
    banner.flf version 2 by Ryan Youck (youck@cs.uregina.ca)
    (From a unix program called banner)
    I am not responsible for use of this font  
    Thanks to Glenn Chappell for his help
    Katakana characters by Vinney Thai <ssfiit@eris.cs.umb.edu>
    Cyrillic characters from "koi8x8" BDF font.
    Date: August 11, 1994
    
    Merged by John Cowan <cowan@ccil.org>
    Modified by Paul Burton <solution@earthlink.net> 12/96 to include new parameter
    supported by FIGlet and FIGWin.  May also be slightly modified for better use
    of new full-width/kern/smush alternatives, but default output is NOT changed.
     $ $@
     $ $@
     $ $@
     $ $@
     $ $@
     $ $@
     $ $@
     $ $@@
    

    Manche Fonts schauen so aus:

    flf2a 25 25 45 0 3
    doh.flf by Curtis Wanner (cwanner@acs.bu.edu)
    latest revision - 4/95
    
    
                                     @
                                     @
                   AAA               @
                  A:::A              @
                 A:::::A             @
                A:::::::A            @
               A:::::::::A           @
              A:::::A:::::A          @
             A:::::A A:::::A         @
            A:::::A   A:::::A        @
           A:::::A     A:::::A       @
          A:::::AAAAAAAAA:::::A      @
         A:::::::::::::::::::::A     @
        A:::::AAAAAAAAAAAAA:::::A    @
       A:::::A             A:::::A   @
      A:::::A               A:::::A  @
     A:::::A                 A:::::A @
    AAAAAAA                   AAAAAAA@
                                     @
                                     @
                                     @
                                     @
                                     @
                                     @
                                     @@
                        @
                        @
    BBBBBBBBBBBBBBBBB   @
    B::::::::::::::::B  @
    B::::::BBBBBB:::::B @
    BB:::::B     B:::::B@
      B::::B     B:::::B@
      B::::B     B:::::B@
      B::::BBBBBB:::::B @
      B:::::::::::::BB  @
      B::::BBBBBB:::::B @
      B::::B     B:::::B@
      B::::B     B:::::B@
      B::::B     B:::::B@
    BB:::::BBBBBB::::::B@
    B:::::::::::::::::B @
    B::::::::::::::::B  @
    BBBBBBBBBBBBBBBBB   @
                        @
                        @
                        @
                        @
                        @
                        @
                        @@
    

    Gibts dann auch mit Kleinbuchstaben. Wie soll ich wissen, welches Zeichen zum Kommentar, welches zum Zeichen gehört? Bzw, wann beginnt der Font?

    Danke für die Vereinfachung, hatte ich auch schon begonnen, aber nicht so schick 😉



  • PS:

    @hustbaer sagte in ASCII Schriftart einlesen:

    Wieso? Die zu ignorieren sollte einfach genug sein.

            if (str.back() != '@') // skip comments
                continue;
    

    Wenn ich das jetzt richtig verstehe, wie kann ich mir sicher sein, das ein Kommentar dieses Zeichen nicht als letztes hat? EDIT: Obwohl das natürlich recht unwahrscheinlich sein wird. Vielleicht sollte man erst dann anfangen zu editieren, wenn ein Font nach dem Einlesen offensichtlich kaputt ist?



  • Also ich kenne die Spec von dem Format nicht. Nur... so wie das aussieht wüsste ich nicht wie man Kommentar/nicht-Kommentar sonst unterscheiden sollte. Kann mMn. fast nur das fehlende @ am Zeilenende sein.



  • OK, hab mal gegoogelt: https://github.com/Marak/asciimo/issues/3
    Wie viele Kommentar-Zeilen es gibt steht in der Header. Also noch einfacher zu ignorieren.
    Dafür muss das @ kein @ sein sondern kann ein beliebiges Zeichen sein - es muss nur im ganze File das selbe verwendet werden.
    Spec: http://www.jave.de/docs/figfont.txt



  • Die (besser lesbare) Spezifikation gibt es unter The FIGfont Version 2 FIGfont and FIGdriver Standard.



  • Oh, ich hatte mal versucht, diesen Header selbst verstehen zu wollen, bin aber gescheitert 😉
    Mit Anleitung sieht das schon besser aus. Besten Dank.



  • Ich habe auf einen Schlag 400 mir noch unbekannte Fonts eingelesen und wieder ausgegeben (erst mal nur in reiner ascii-Form). Für mich ist das zu meiner Zufriedenheit gelöst.
    Ich zeige trotzdem die komplette Klasse, falls doch noch eine Baustelle ist.

    #pragma once
    #include <filesystem>
    #include <fstream>
    #include <string>
    #include <vector>
    #include <sstream> 
    
    namespace figlet
    {
    	struct ErrorState
    	{
    		bool state = false;
    		std::string type;
    	} error;
    
    	struct Directory
    	{
    		std::string path;
    		std::vector<std::string> folders;
    		std::vector<std::string> files;
    
    		void clear()
    		{
    			folders.clear();
    			files.clear();
    		}
    	} directory;
    
    	void getDirectory()
    	{
    		directory.clear();
    		for (const auto& entry : std::filesystem::directory_iterator(directory.path))
    		{
    			std::string name = entry.path().filename().string();
    			if (std::filesystem::is_directory(entry))
    				directory.folders.push_back(name);
    
    			else if (std::filesystem::is_regular_file(entry))
    				directory.files.push_back(name);
    		}
    	}
    
    	bool setPath(const std::string& path)
    	{
    		std::filesystem::path fsPath = path;
    		if (!std::filesystem::exists(fsPath))
    		{
    			error = { true, path + ": path not exists" };
    			return false;
    		}
    		directory.path = fsPath.string();
    		getDirectory();
    		return true;
    	}
    
    
    	class Font
    	{
    		struct Info
    		{
    			std::string name;
    			const std::string signature = "flf2a";
    			char lineMarker = '@';
    
    			char hardBlank = '$';
    			int height = 0;
    			int baseLine = 0;
    			int maxLength = 0;
    			int oldLayout = 0;
    			int commentLines = 0;
    			int printDirection = 0;
    			int fullLayout = 0;
    			int codetagCount = 0;
    		};
    
    		struct FontChar
    		{
    			int width = 0;
    			int height = 0;
    			std::vector<std::string> data;
    		};
    
    		Info fontInfo;
    		std::vector<FontChar> characterSet;
    		ErrorState err;
    
    	public:
    		Font() = default;
    
    		Info info() const { return fontInfo; }
    		std::vector<FontChar> font() const { return characterSet; }
    		ErrorState error() const { return err; }
    
    		bool loadFromFile(const std::string& fontName)
    		{
    			if (directory.path.empty())
    				return Error("no path to font set");
    
    			if (fontName.length() < 5)
    				return Error(fontName + ": invalid font name");
    
    			if (fontName.substr(fontName.length() - 4, fontName.length()) != ".flf")
    				return Error(fontName + ": no flf file");
    
    			std::string fileName = directory.path + fontName;
    			std::ifstream fstream(fileName);
    			if (!fstream)
    				return Error(fileName + ": " + strerror(errno));
    
    			if (!readFontInfo(fstream, fontName))
    				return Error(fontInfo.name + ": invalid header");
    
    			if (!readFont(fstream))
    				return Error(fontInfo.name + ": invalid font");
    
    			return true;
    		}
    
    	private:
    		std::vector<std::string> splitString(const std::string& str, const char delim)  const
    		{
    			std::vector<std::string> vec;
    			std::stringstream sstream(str);
    			std::string item;
    
    			while (std::getline(sstream, item, delim))
    				vec.push_back(item);
    
    			return vec;
    		}
    
    		bool readFontInfo(std::ifstream& fstream, const std::string& fontName)
    		{
    			// geht das alles eleganter?
    			fontInfo.name = fontName.substr(0, fontName.length() - 4);
    			std::string header;
    			std::getline(fstream, header);
    			if (header.empty()) return false;
    			std::vector<std::string> head = splitString(header, ' ');
    
    			std::size_t n = 0;
    			std::string signature = head[n];
    			fontInfo.hardBlank = signature.back();
    			signature.pop_back();
    			if (signature != fontInfo.signature) return false;
    
    			if (++n >= std::size(head)) return false;
    			fontInfo.height = std::stoi(head[n]); // height is essential
    
    			if (++n >= std::size(head)) return false;
    			fontInfo.baseLine = std::stoi(head[n]);
    
    			if (++n >= std::size(head)) return false;
    			fontInfo.maxLength = std::stoi(head[n]); // maxLength is essential
    
    			if (++n >= std::size(head)) return true;
    			fontInfo.oldLayout = std::stoi(head[n]);
    
    			if (++n >= std::size(head)) return true;
    			fontInfo.commentLines = std::stoi(head[n]);
    
    			if (++n >= std::size(head)) return true;
    			fontInfo.printDirection = std::stoi(head[n]);
    
    			if (++n >= std::size(head)) return true;
    			fontInfo.fullLayout = std::stoi(head[n]);
    
    			if (++n >= std::size(head)) return true;
    			fontInfo.codetagCount = std::stoi(head[n]);
    
    			return true;
    		}
    
    		bool readFont(std::ifstream& fstream)
    		{
    			std::string str;
    			for (int i = 0; i < fontInfo.commentLines; ++i)
    				std::getline(fstream, str); // skip comments
    
    			FontChar fontChar;
    			while (std::getline(fstream, str))
    			{
    				if (str.empty() || str.back() != fontInfo.lineMarker)
    					continue; // skip invalid lines
    
    				str.pop_back(); // remove data line marker
    				fontChar.width = str.length() - 1;
    				if (fontChar.width > fontInfo.maxLength)
    					return false;
    
    				const bool isLastLineOfChar = !str.empty() && str.back() == fontInfo.lineMarker;
    				if (isLastLineOfChar) str.pop_back(); // remove last-line-of-char marker
    				fontChar.data.push_back(str);
    
    				if (isLastLineOfChar)
    				{
    					fontChar.height = std::size(fontChar.data);
    					if (fontChar.height != fontInfo.height)
    						return false;
    
    					characterSet.push_back(std::move(fontChar));
    				}
    			}
    			return true;
    		}
    
    		bool Error(const std::string& errType)
    		{
    			err = { true, errType };
    			return false;
    		}
    	};
    }
    

    Wen es interessiert, die Fonts habe ich von hier:
    http://www.figlet.org/



  • @zeropage sagte in ASCII Schriftart einlesen:

    bool readFontInfo(std::ifstream& fstream, const std::string& fontName)
    {
    	// geht das alles eleganter?
    	// ...
    	std::vector<std::string> head = splitString(header, ' ');
    	// ...
    }
    

    Warum liest du erst alles in einen std::vector<std::string> ein, um dann wieder jeden einzelnen Zahlenwert mittels std::stoi auszulesen? Benutze doch einfach direkt einen std::stringstreamund lese daraus die einzelnen Werte.



  • Danke, da komme ich aber noch nicht richtig mit der Benutzung klar. Muss ich noch schauen.



  • std::string header;
    std::getline(fstream, header);
    if (header.empty()) return false;
    std::stringstream sstream(header);
    
    std::string signature;
    if (!(sstream << signature << fontInfo.height << fontInfo.baseLine << fontInfo.maxLength))
       return false;
    // hier noch genauere Auswertung für signature (fontInfo.signature + fontInfo.hardblank)...
    
    sstream << fontInfo.oldLayout  /* << ... */;
    
    return true;
    


  • Oh! Besten Dank 🙂 Da habe ich jetzt ja was.



  • Vielen Dank nochmal.

                    bool readFontInfo(std::ifstream& fstream, const std::string& fontFile)
    		{
    			fontInfo.name = fontFile.substr(0, fontFile.length() - 4);
    			std::string header;
    			std::getline(fstream, header);
    			std::stringstream sstream(header);
    
    			std::string signature;
    			if (!(sstream 
    				>> signature 
    				>> fontInfo.height 
    				>> fontInfo.baseLine 
    				>> fontInfo.maxLength 
    				>> fontInfo.oldLayout 
    				>> fontInfo.commentLines))
    				return false;
    				
    			fontInfo.hardBlank = signature.back();
    			signature.pop_back();
    			if (signature != fontInfo.signature) 
    				return false;
    
    			sstream 
    				>> fontInfo.printDirection 
    				>> fontInfo.fullLayout 
    				>> fontInfo.codetagCount; 
    			return true;
    		}
    
    


  • Wtf? Ich muss einen offenen fstream und den Dateinamen übergeben!? Echt jetzt?



  • Ja? Und?

    Das fontFile brauche ich für den Namen. Das ich den fstream brauche ist doch offensichtlich, oder was?

    Wenn das ein Fehler ist, warum kommst Du in der letzten Minute angekleckert? Den "offenen fstream'" habe ich von Anfang an benutzt.

    EDIT: Und was soll überhaupt dies blöde Fragestellung? "WTF Echt jetzt"? Jetzt weiß ich bestimmt, was Du mir mitteilen willst???



  • Ich sehe da jetzt auch kein Problem. Wahrscheinlich hat @Swordfish gemeint, du könntest dann direkt über den Dateinamen einen Stream öffnen, aber diese Funktion ist ja nur eine Teilfunktion der loadFromFile.

    Du könntest allerdings direkt den passenden Font-Namen übergeben, anstatt in der Funktion die Extension mittels substr wegzuschneiden.

    Und dazu noch eine Kleinigkeit bzgl.

    if (!readFontInfo(fstream, fontName))
    	return Error(fontInfo.name + ": invalid header");
    

    Hier greifst du im Fehlerfall auf fontInfo.name zu. Du setzt zwar zuerst in der Funktion readFontInfo diesen Namen und erst danach wird der Fehlerfall geprüft, aber dies ist eher ein Code-Smell.
    Dies würde auch dafür sprechen vor der Funktion den reinen Font-Namen zu ermitteln und diesen dann weiter zu benutzen.



  • Ein paar Dinge fallen mir noch auf. Ein Fehlerhandling hat die Funktion gar nicht.
    Was ist wenn der Stream nicht offen ist? Kann man direkt am Anfang abfangen.
    Was ist wenn fontfile leer ist, oder keine Endung hat?

    Abgesehen davon bin ich bei Swordfish. Das übergeben der Referenzen auf den stream finde ich nur dann tragbar, wenn in der Datei, aus der gelesen wird, noch andere Dinge stehen als nur das was in dieser Funktion rausgelesen wird. Wenn dort nur die Font-Daten drin stehen, ist eine Blackbox viel schöner, die den Dateinamen übernimmt und Daten zurückgibt oder irgendwo speichert bzw. false zurückgibt ( oder ersatzweise eine intensivere Fehlerkommunikation ). Aber das ist ein stückweit auch Geschmackssache.

    Abgesehen davon empfinde ich "WTF? Echt jetzt?" als keine schöne Art der Gegenfrage 😉



  • Sehe ich anders. readFontInfo ist eine private Methode, da kann man als Precondition ruhig festlegen, dass der stream schon geöffnet sein muss. Wenn die Methode als istream& operator>>( ifstream& is, FontInfo& fi ) implementiert wäre muss der stream ja ebenfalls geöffnet sein. Die Übergabe des Dateinamens halte ich allerdings auch für fraglich, das gehört für mich auf die Ebene, in der readFontInfo aufgerufen wird. Schließlich wird der Name ja nicht aus der Datei gelesen.

    Ansonsten:

    • Du gibst in deinem Code überall Kopien zurück (info(), font(), error()). Das geht auch mit Referenzen auf konstante Objekte.
    • Die Methode Error ist groß geschrieben, im Gegensatz zu allen anderen Methoden. Außerdem setzt sie die Variable err, was (für mich) aus dem Namen der Methode nicht ersichtlich ist. Ich würde hier
    if( Fehlerfall )
    {
       setError( ... );
       return false;
    }
    

    bevorzugen, auch wenn´s eine Zeile mehr ist.

    • Ich bin auch kein Fan von internen Klassen/Strukturen, warum Info eine interne Struktur von Font ist verstehe ich nicht.
    • Variablen im namespace figlet erlauben nur die Benutzung eines Font Objekts gleichzeitig. In die Klasse Font gehören sie mMn auch nicht, da könnte man eine Klasse FontReader drumrumbauen. Oder man wirft eine entsprechende Exception.
    • FontChar::clear() ist überflüssig, wenn du die entsprechende Variable zurücksetzen möchtest geht das auch mit var = FontChar()
    • ist characterSet.push_back(std::move(fontChar)) nicht das Gleiche wie characterSet.emplace_back( fontChar );?

Log in to reply