Zjednodušení kódu – C / C++ – Fórum – Programujte.com

Vymýšlel jsem program, který by měl vstupní řetězec string, v kterém je uložen název nějakého souboru rozpoznat, zda-li se jedná o obrázek, nebo ne.
Výsledný kód mi ovšem příjde příliš složitý a zdlouhoavý a příjde mi, že tam nepoužívám vůbec principy objektového programování.
Věřím, že je zde plno možností jak tento kód různě zkrátit a ačkoliv mi vzhledem k jeho funkčnosti nesejde na jeho zkracování, zajímalo by mě, co tam mám navíc nebo zbytečně.
Snažil jsem se postupovat logicky, tak jak rozpoznávám jestli je soubor obrázkem např. já:
1) hledám v řetězci znak tečky, který je předzvěstí, že následující znaky budou příponou (nemusí být ovšem pravda, ale já předpokládám se soubory názvu např. P89753734.jpg )
2) pak čtu následující znaky a porovnávám je se známými připonami obrázky
3) na základě shody rozhoduju, jestli je soubor obrázek, nebo ne.

#include <set>

#include <string>

#include <iostream>



using namespace std;



int main () {

set <char> tecka;

set <char> ::iterator iter2;



tecka.insert('.');



string text = "posrany obrazek.jpg";

cout << text << endl;





static char pripona[5] = { 'x' , 'x' , 'x' , 'x', '\0' };  //pole pripona ma 5 znaku a posledni je null .abc\0

bool bylatecka = false;  //prepinac zda-li byla nactena tecka a jeho vychozi hodnota

for (int i = 0; i < text.size(); i++) {

	iter2 = tecka.find (text.at(i));

		if (iter2 == tecka.end() && bylatecka == false)

			cout << text.at(i);

		else {

			bylatecka = true; //v retezci se objevila tecka tzn.. dalsi tri pismena jsou pripona souboru 

//(jak se soubor bude jmenovat .torrent, nebo kaja.gott.jpg tak jsem v pyci)

			cout << text.at(i);



			static int poc_pripony = 0; 

			if (poc_pripony <= 3) {  //postupne zpaisujem aktualni čtené pismenko do pole pripona (mělo by začínat znakem ".")

				pripona[poc_pripony] = text.at(i) ;	

				poc_pripony++;

				} //if (poc pripony)

					} //else (byla tecka)



} //for (int i)



for (static int poc_pripony1 = 0; poc_pripony1 <= 3; poc_pripony1++) { //jen vypise nazev pripony

				cout << pripona[poc_pripony1];	

			} //for (int poc pripony)



char pripona_jpg[] = ".jpg"; //zde je seznam siroce uzivanych pripon obrazkovych souboru

char pripona_bmp[] = ".bmp";

char pripona_pgn[] = ".pgn";

char pripona_gif[] = ".gif";

char pripona_tga[] = ".tga";

char pripona_TGA[] = ".TGA";

char pripona_JPG[] = ".JPG";

char pripona_BMP[] = ".BMP";

char pripona_PGN[] = ".PGN";

char pripona_GIF[] = ".GIF";

if (!strcmp(pripona, pripona_jpg) || !strcmp(pripona, pripona_JPG) || !strcmp(pripona, pripona_gif) || !strcmp(pripona, pripona_GIF) || !strcmp(pripona, pripona_pgn) || !strcmp(pripona, pripona_PGN) || !strcmp(pripona, pripona_tga) || !strcmp(pripona, pripona_TGA))

cout << endl << "pripona" << pripona << " JE obrazek";

else 

cout << endl << "pripona" << pripona << " NENI obrazek";



cout << endl;

return 0;

} //int main

Nevím, jakým způsobem funguje x-scroll-bar u funkce [ code ] , ale nějak nepříjemně se mi to roztáhlo.... :D

Já bych to udělal, takhle prošel bych si ten řetězec od konce a postupně si ukládal písmena do jinýho stringu. Jakmile bych narazil na tečku mám celou příponu. Teď musím ten string otočit jelikož se mi ta přípona uložila obráceně. No a teď když jí mám obrácenou jí jen porovnám a hotovo.

To Sergei : Problém s tečkou uvnitř názvu bych řešil tak, že bych jako příponu vzal poslední 3 písmena řetězce a uložil bych si je do nějaké proměnné. A porovnání přípon bych řešil tak, že bych si udělal buď dvourozměrné pole char nebo pole string a poté bych porovnával koncovku s jednotlivými indexy pole.

zjednodušit by se to dalo i tím, že si přípony uložíš do pole stringů, a to každou jednou a malými písmeny, pak je porovnáš se získanou příponou, na kterou uplatníš funkci tolower. A porovnání budeš provádot v cyklu.

Sergei
STL může sice stát za starou bačkoru, ale proč ji nevyužít a nevyužít správně, když tu je:

#include <iostream>

#include <string>

#include <set>



const char *allowed_exts[] = {"jpg", "gif", "png", NULL}; // na posledním místě musí být NULL



int main(int argc, char **argv)

{

    // zkontrolujem argumenty

    if (argc < 2) {

        std::cerr << "usage: " << argv[0] << " <file>" << std::endl;

        return -1;

    }



    // vytvoříme set s příponami

    std::set<std::string> exts;

    for (int i = 0; allowed_exts[i] != NULL; ++i) 

        exts.insert(std::string(allowed_exts[i]));



    // zjistíme příponu

    std::string filename = argv[1];



    size_t period = filename.rfind('.'); // cokoli za poslední tečkou je přípona

    if (period == std::string::npos) {

        std::cerr << "žádná tečka :-*" << std::endl;

        return -1;

    }



    std::string ext(filename, period + 1);



    // ověříme, jestli je v množině povolených

    if (exts.count(ext) > 0) {

        std::cout << "jo, tak tohle vypadá jako obrázek" << std::endl;

        return 0;

    } else {

        std::cout << "ne, tohle asi obrázek nebude" << std::endl;

        return 1;

    }

}