Problème et AVC : _wfopen_s

Version imprimable

Voir 40 message(s) de cette discussion en une page

https://github.com/laurentbouleau/Exemple : Août 16 (23/08/2024)

Elfy : Compagnon décédé !
Alors, Escitalopram. Trop dormis ou pas peux dormis !

Mais :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
Film::Film(std::filesystem::path racine)
{
    this->racine = racine;
    auto nomDossier = racine.filename().wstring();
    assert(nomDossier.length() > 0 && L"Nom de dossier vide");
 
    std::wregex filename_pattern{ L"(.+?)(?:\\.\\((\\d{4}\\-\\d{2}\\-\\d{2}\\s?)?([^\\)]*)\\))?(?:\\.(.+))?$" };
    std::wsmatch match;
    if (std::regex_match(nomDossier, match, filename_pattern))
    {
        std::wstring titres = match[1];
        m_titres = Dossier_Titres(titres);
 
        if (match[2].matched)
        {
            std::wstring wstr = match[2].str();
            m_date_du_film.tm_year = stoi(wstr) - 1900;
            wstr = wstr.substr(5);
            m_date_du_film.tm_mon = stoi(wstr) - 1;
            wstr = wstr.substr(3);
            m_date_du_film.tm_mday = stoi(wstr);
        }
        m_sur = (match[3].matched) ? match[3].str() : L"";
        m_sous_genre = (match[4].matched) ? match[4].str() : L"";
    }
    else
    {
        assert(false == true && "Le nom du répertoire n'est pas un nom valide.");
    }
}

Voilà !!!

Après :

Code:

m_titres = Dossier_Titres(titres);

Code:

const std::vector<std::wstring> t = ::extraire_Titres_Depuis_UneLigne(titres);

Code:

abc_Titres(m_titres, titres);

C'est tout !!!

Bonsoir,

https://github.com/laurentbouleau/Exemple : Août 17 (23/08/2024)

film.cpp :

Code:

m_titres = Dossier_Titres(titres);

par :

Code:

m_titres = initialiser_Titres(titres);

Et film_serie.cpp :

Code:

1
2
3
4
5
6
std::vector<std::wstring> initialiser_Titres(std::wstring& titres)
{
    assert(titres.length() > 0 && L"Nom de titres vide"); // ??? pour Mot de... ?
    const std::vector<std::wstring> t = ::extraire_Titres_Depuis_UneLigne(titres);
    return t;
}

Film.cpp : ok !
Par contre (serie.cpp) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
Serie::Serie(std::filesystem::path racine)
{
    this->racine = racine;
...
    //std::wregex filename_pattern{ L"(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})?([^\\]]*)\\])?(?:\\.(.+))?" }; // Marches pas !!!
//    std::wregex filename_pattern{ L"(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})?([^\\]]*)\\])?(?:\\.(.+))?" };
    std::wregex filename_pattern{ L"^(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})([^\\]]*)\\])(?:\\.(.+))?$|^(.+)(?:\\.(.+))$" };
    std::wsmatch match;
    if (std::regex_match(nomDossier, match, filename_pattern))
    {
        std::wstring titres = match[1];
        m_titres = initialiser_Titres(titres);
...
        m_sur = (match[3].matched) ? match[3].str() : L"";
        m_sous_genre = (match[4].matched) ? match[4].str() : L"";
    }
    else
    {
        assert(false == true && "Le nom du répertoire n'est pas un nom valide.");
    }
}

Marches pas !!! std::wregex filename_pattern et m_sous_genre ! https://regex101.com/r/dzh1TD/1 ! Tant pis !!! Demain ou plus tard !!!

Pas fini : extraire_Titres_Depuis_UneLigne et abc_Titre !

Bonne nuit:aie::aie::aie:

24/08/2024, 00h10
bacelar

Citation:

Code:

Film::Film(std::filesystem::path racine){...}

Je suis assez "OK" avec le code.
On a juste tendance à faire en sorte que les lignes de code soient assez indépendantes les unes des autres.
Avec votre code, si on modifie un peu le contenu de la variable "filename_pattern", par exemple, en ajoutant des séparateurs optionnels, etc..., il faudra à modifier votre code de décodage de la date.

Le code que j'ai fourni a aussi des dépendances avec la variable "filename_pattern", mais c'était une base.
Je pense qu'il est plus facile de rendre le code de décodage de la date indépendant de "filename_pattern" à partir de mon code, qui utilise plus de captures, que du vôtre qui utilise plus d'informations "en dur".

Dans la classe Film, changement de nom du champ :
"m_date" => "m_date_du_film"
Franchement, c'est tout aussi "mauvais".
On n'a pas plus d'information sur "l'utilité" de ce champ.
Si c'est ni la date de sortie ("m_date_de_sortie") ni la date de reprise ("m_date_de_reprise"), c'est la date de quoi ("du film") ???

La variable "date_du_film_str " dans la fonction "Print_Header", Ok, mais on sait toujours pas ce qu'est cette "m_date_du_film".

"Dossier_Titres" => "initialiser_Titres"
C'est vrai que "Dossier_Titres", c'est vraiment pas top comme nom, je l'avais déjà indiqué.
Mais "initialiser_xxx" était réservé aux fonctions qui initialisaient le champ "m_xxx".
Donc "initialiser_Titres", c'est pas top, car elle n'initialisera pas directement "m_Titres".
J'avais déjà proposé "extraire_xxx_Depuis_yyyy" pour les fonctions qui récupèrent des informations depuis un "endroit", cf. la fonction "extraire_Titres_Depuis_UneLigne(std::wstring& ligne)".
Donc, moi, je ne créerai pas une fonction "initialiser_Titres", mais une fonction "extraire_Titres_Depuis_NomDeFichierOuDeRepertoire((std::wstring& nom)".
Et logiquement "extraire_Titres_Depuis_NomDeFichierOuDeRepertoire" N'utilisera PAS "extraire_Titres_Depuis_UneLigne", car le format est DIFFERANT.
Récupérez donc l'ancien code de "Film::Dossier_Titres" qui utilisait des "std::wstrinf::find" et pas "extraire_Titres_Depuis_UneLigne" pour implémenter la fonction "extraire_Titres_Depuis_NomDeFichierOuDeRepertoire".
Et faites en sorte que "extraire_Titres_Depuis_NomDeFichierOuDeRepertoire" ne modifie pas le champ "m_titres", mais renvoye un résultat (la fonction extrait les données, elle n'initialise pas un champ).

Citation:

Code:

abc_Titres(m_titres, titres);

Franchement, c'est un nom complètement abscons.
Et j'ai déjà fait une proposition de nom et de signature dans un post précédent :

Code:

std::vector<std::wstring> fusionner_Titres(std::vector<std::wstring>& vieux_titres, std::vector<std::wstring>& nouveaux_titres)...

Et, donc, au lieu d'avoir un code cryptique :

Citation:

Code:

abc_Titres(m_titres, titres);

On aurait un code bien plus transparent sur ses "intentions" :

Code:

m_titres = fusionner_Titres(m_titres,nouveaux_titres);
24/08/2024, 00h35
Laurent_B_

https://github.com/laurentbouleau/Exemple : Août 18 (24/08/2024)

https://github.com/laurentbouleau/Ex...20Disney%2B%5D : Marches pas !

Mince !!!

Bonsoir,

https://github.com/laurentbouleau/Exemple : Août 18 (24/08/2024)

film_serie.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
...
std::vector<std::wstring> fusionner_Titres(std::vector<std::wstring>& vieux_titres, std::vector<std::wstring>& nouveaux_titres)
{
    assert(nouveaux_titres.size() < 4 && L"???");
    assert(vieux_titres.size() < 4 && L"???");
    bool found = false;
    if (nouveaux_titres == vieux_titres)
        found = true;
    else
    {
        if (vieux_titres.size() == nouveaux_titres.size())
        {
            //double titres_ration{ 80.0 };
            if (vieux_titres.size() == 1 && nouveaux_titres.size() == 1)
            {
                nouveaux_titres = vieux_titres;
                found = true;
            }
            if (vieux_titres.size() == 3 && nouveaux_titres.size() == 3 && vieux_titres[0] == nouveaux_titres[0] && vieux_titres[1] != nouveaux_titres[1] && vieux_titres[2] == nouveaux_titres[2])
            {
                nouveaux_titres[1] = vieux_titres[1];
                found = true;
            }
            if (vieux_titres.size() == 3 && nouveaux_titres.size() == 3 && (vieux_titres[0] != nouveaux_titres[0] || vieux_titres[1] != nouveaux_titres[1] || vieux_titres[2] != nouveaux_titres[2]))
            {
                nouveaux_titres = vieux_titres;
                found = true;
            }
        }
    }
    return nouveaux_titres;
}
...
std::vector<std::wstring> extraire_Titres_Depuis_UneLigne(std::wstring& titre)
{ // Titre 
    assert(titre.length() > 0 && L"Nom de titres vide"); // ??? pour Mot de... ?
    std::wstring& t = titre;
 
    std::vector<std::wstring> titres;
    std::size_t pos = 0;
    bool found = false;
    const std::wstring d_p = L" : ";
    pos = t.find(d_p);
    if (!found && pos != std::wstring::npos)
    {
        titres.push_back(t.substr(0, pos));
        titres.push_back(d_p);
        titres.push_back(t.substr(pos + 3));
        found = true;
    }
    const std::wstring d_p2 = L": ";
    pos = titre.find(d_p2);
    if (!found && pos != std::wstring::npos)
    {
        titres.push_back(t.substr(0, pos));
        titres.push_back(d_p2);
        titres.push_back(t.substr(pos + 2));
        found = true;
    }
    const std::wstring d_p3 = L"/";
    pos = t.find(d_p3);
    if (!found && pos != std::wstring::npos)
    {
        titres.push_back(t.substr(0, pos));
        titres.push_back(d_p3);
        titres.push_back(t.substr(pos + 1));
        found = true;
    }
    const std::wstring d_p4 = L" - ";
    pos = t.find(d_p4);
    if (!found && pos != std::wstring::npos)
    {
        titres.push_back(t.substr(0, pos));
        titres.push_back(d_p4);
        titres.push_back(t.substr(pos + 3));
        found = true;
    }
    const std::wstring d_p5 = L"- ";
    pos = t.find(d_p5);
    if (!found && pos != std::wstring::npos)
    {
        titres.push_back(t.substr(0, pos));
        titres.push_back(d_p5);
        titres.push_back(t.substr(pos + 2));
        found = true;
    }
    if (!found)
    {
        titres.push_back(t);
        found = true;
    }
    return titres;
}
...

Et film.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
...
void Film::initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& m_titre)
{ // Titre
    auto nomFichier = cheminFichier.wstring();
    assert(nomFichier.length() > 0 && L"Nom de fichier vide");
    std::vector<std::wstring> contenu = lire_fichierTxt(cheminFichier.wstring(), { L"\n" });
    assert((contenu.size() != 0));
 
    std::vector<std::wstring>nouveaux_titres = extraire_Titres_Depuis_UneLigne(contenu[0]);
 
    m_titres = fusionner_Titres(m_titres, nouveaux_titres);
 
    contenu.erase(contenu.begin());
    if (contenu.size() > 0)
    {
        m_duree = initialiser_Duree(contenu[0]);
        contenu.erase(contenu.begin());
        if (contenu.size() > 0)
            m_resume = contenu;
    }
}
...
const void Film::Print_Header()
{
    if (affichage_titres_actif)
    {
        std::wstring titres_str;
        std::wstring date_en_salle_ou_sur_str;
        std::wstring x_sj_str;
        std::wstring sj_str;
        std::wstring duree_str;
        std::wstring note_str;
 
        titres_str = m_keyColor[0] + L"Titre : " + m_valuesColor + m_titres[0];
        if (m_titres.size() > 1)
            titres_str += m_keyColor[1] + m_titres[1] + m_valuesColor + m_titres[2];
        // Date du film
        if (affichage_date_en_salle_ou_sur_actif)
        {
            wchar_t date_string[15];
            wcsftime(date_string, 15, L"%d/%m/%Y", &m_date_en_salle_ou_sur);
            date_en_salle_ou_sur_str = date_string;
            date_en_salle_ou_sur_str = m_keyColor[0] + L" (" + m_valuesColor +
                date_en_salle_ou_sur_str.substr(0, 2) + m_keyColor[0] + L'/' + m_valuesColor +
                date_en_salle_ou_sur_str.substr(3, 2) + m_keyColor[0] + L'/' + m_valuesColor +
                date_en_salle_ou_sur_str.substr(6, 4); //+ m_valuesColor;
            if (affichage_sur_actif && m_sur == L"")
            {
                date_en_salle_ou_sur_str += m_keyColor[1] + L" en salle" + m_valuesColor;
            }
            else if (affichage_sur_actif && (m_sur == L"Disney+" || m_sur == L"Netflix"))
            {
                date_en_salle_ou_sur_str += m_keyColor[1] + L" sur " + m_valuesColor;
                if (m_sur == L"Disney+")
                {
                    date_en_salle_ou_sur_str += L"Disney+ " + m_keyColor[1] + L": " + m_valuesColor + m_disney_sj;
                }
                else
                {
                    date_en_salle_ou_sur_str += L"Netflix " + m_keyColor[1] + L": " + m_valuesColor + m_netflix_sj;
                }
            }
            else 
            {
                date_en_salle_ou_sur_str += m_sur;
            }
            date_en_salle_ou_sur_str += m_keyColor[0] + L')' + m_valuesColor;
        }
        // x signalétique jeunesse
        if (affichage_x_sj_actif && m_sur == L"")
        {
            // Disney+ SJ
            if (affichage_disney_sj_actif && m_disney_sj.length() != 0)
                x_sj_str += m_keyColor[0] + L" (" + m_valuesColor + L"Disney+" + m_keyColor[1] + L" : " + m_valuesColor + m_disney_sj + m_keyColor[0] + L')' + m_valuesColor;
            // Netflix SJ
            if (affichage_netflix_sj_actif && m_netflix_sj.length() != 0)
                x_sj_str += m_keyColor[0] + L" (" + m_valuesColor + L"Netflix" + m_keyColor[1] + L" : " + m_valuesColor + m_netflix_sj + m_keyColor[0] + L')' + m_valuesColor;
        }
        // La signalétique jeunesse
        if (affichage_sj_actif && m_sj.length() != 0)
            sj_str += m_keyColor[0] + L" (" + m_valuesColor + L"SJ" + m_keyColor[1] + L" : " + m_valuesColor + m_sj + m_keyColor[0] + L')' + m_valuesColor;
        // Durée
        if (affichage_duree_actif)
        {
            duree_str = L' ' + std::to_wstring(m_duree / (60 * 60)) + m_keyColor[0] + L"h" + m_valuesColor + L' ' + std::to_wstring((m_duree - ((m_duree / (60 * 60)) * 60 * 60)) / 60) + m_keyColor[0] + L"min " + m_valuesColor;
        }
        // Note
        if (affichage_note_actif)
            note_str += Print_Note();
 
        std::wcout << titres_str << date_en_salle_ou_sur_str << x_sj_str << sj_str << duree_str << note_str << std::endl;
    }
}
...

Citation:

Avec votre code, si on modifie un peu le contenu de la variable "filename_pattern", par exemple, en ajoutant des séparateurs optionnels, etc..., l faudra à modifier votre code de décodage de la date.

Expliquez-moi !

Citation:

Le code que j'ai fourni a aussi des dépendances avec la variable "filename_pattern", mais c'était une base.
Je pense qu'il est plus facile de rendre le code de décodage de la date indépendant de "filename_pattern" à partir de mon code, qui utilise plus de captures, que du vôtre qui utilise plus d'informations "en dur".

Que faire ?

serie.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
Serie::Serie(std::filesystem::path racine)
{
    this->racine = racine;
    auto nomDossier = racine.filename().wstring();
    assert(nomDossier.length() > 0 && L"Nom de dossier vide");

    //std::wregex filename_pattern{ L"(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})?([^\\]]*)\\])?(?:\\.(.+))?" }; // Marches pas !!!
//    std::wregex filename_pattern{ L"(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})?([^\\]]*)\\])?(?:\\.(.+))?" };
    std::wregex filename_pattern{ L"^(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})([^\\]]*)\\])(?:\\.(.+))?$|^(.+)(?:\\.(.+))$" };
    std::wsmatch match;
    if (std::regex_match(nomDossier, match, filename_pattern))
    {
        std::wstring titres = match[1];
        m_titres = initialiser_Titres(titres);
        if (match[2].matched)
        {
            std::wstring annees_str = match[2].str();
            std::wsmatch dummy;
            if (std::regex_match(annees_str, dummy, std::wregex(L"\\d{4}\\-\\d{4}\\s?")))
            {
                m_f_anneesProduction.first = stoi(annees_str);
                m_f_anneesProduction.second = stoi(annees_str.substr(5));
            }
            else
            {
                m_f_anneesProduction.first = stoi(annees_str);
            }
        }

        m_sur = (match[3].matched) ? match[3].str() : L"";
        m_sous_genre = (match[4].matched) ? match[4].str() : L"";
    }
    else
    {
        assert(false == true && "Le nom du répertoire n'est pas un nom valide.");
    }
}

Etc..
Marches pas !!!

Je suis crevé !

Merci vraiment:aie::aie::aie:

25/08/2024, 00h45
Laurent_B_

https://github.com/laurentbouleau/Exemple : Août 19 (25/08/2024)

Et :
https://github.com/laurentbouleau/Ex...7%5D.Animation : Star Wars Rebels.[2014-2017].Animation : Hors saison (11 août 2014 : exemple 0x1.2024-08-21.txt)

Merci:aie::aie::aie:

Bonjour,

Peux dormi ! Je suis fatigué !

https://github.com/laurentbouleau/Exemple : Août 20 (25/08/2024)

serie.cpp :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 ... void Serie::initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& m_titre) { // Titre auto nomFichier = cheminFichier.wstring(); assert(nomFichier.length() > 0 && L"Nom de fichier vide"); std::vector<std::wstring> contenu = lire_fichierTxt(cheminFichier.wstring(), { L"\n" }); assert((contenu.size() != 0)); std::vector<std::wstring>nouveaux_titres = extraire_Titres_Depuis_UneLigne(contenu[0]); m_titres = fusionner_Titres(m_titres, nouveaux_titres); contenu.erase(contenu.begin()); if (contenu.size() > 0) { initialiser_Duree(contenu[0]); contenu.erase(contenu.begin()); if (contenu.size() > 0) m_resume = contenu; } } ...
Ok !

film.cpp et serie.cpp :
Pas de extern ...
Ok ou pas ok ?

Mais :

Kaleidoscope.[2023 Netflix]
Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga
SupraCell.[2024 Netflix].Mini-série
Azertyuiop £££.[2024].Mini-série
Marche pas !

Et :

Star Wars- The Clone wars.[2008-2020].Animation
Seven Deadly Sins- Four Knights of the Apocalypse.[2023].Manga
Dr. STONE.[2019-2022 Crunchyroll].Manga
Andor.[2022- Disney+]
Star Wars Rebels.[2014-2017].Animation
Ok !

Peux être :

Code:

m_sous_genre = (match[4].matched) ? match[4].str() : L"";

???
et/ou :

Code:

std::wregex filename_pattern{ L"^(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})([^\\]]*)\\])(?:\\.(.+))?$|^(.+)(?:\\.(.+))$" };

???

Par contre :
Code:

1 2 3 4 5 6 7 8 9 10 11 std::wstring annees_str = match[2].str(); std::wsmatch dummy; if (std::regex_match(annees_str, dummy, std::wregex(L"\\d{4}\\-\\d{4}\\s?"))) { m_f_anneesProduction.first = stoi(annees_str); m_f_anneesProduction.second = stoi(annees_str.substr(5)); } else { m_f_anneesProduction.first = stoi(annees_str); }
???

Exemple serie.cpp :

abc.[2008]
ou
abc.[2008-]
ou
abc.[2008-2010]
Meilleur !!! Ok ou pas !

Aller : le film : Star Wars - Episode VII - Le réveil de la force.(2015-12-16)
Après : dormir ! Crevé !

Merci d'avance:aie::aie::aie:

https://github.com/laurentbouleau/Exemple : Août 21 (25/08/2024)

serie.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
...
const void Serie::Print_Header()
{
    if (affichage_titres_actif)
    {
        std::wstring titres_str;
        std::wstring annees_str;
        std::wstring sur_str;
        std::wstring x_sj_str;
        std::wstring sj_str;
        std::wstring duree_str;
        std::wstring note_str;

        titres_str = m_keyColor[0] + L"Titre : " + m_valuesColor + m_titres[0];
        if (m_titres.size() > 1)
            titres_str += m_keyColor[1] + m_titres[1] + m_valuesColor + m_titres[2];
        // Année(s)
        if (affichage_annees_actif)
        {
            annees_str = format_Annees(); // L" [" et L']' ???
        }
        // sur
        if (affichage_sur_actif)
        {
            if (m_sur == L"Disney+" || m_sur == L"Netflix")
            {
                sur_str += m_keyColor[0] + L" [" + m_valuesColor + m_keyColor[1] + L"sur " + m_valuesColor;
                if (m_sur == L"Disney+")
                {
                    sur_str += L"Disney+ " + m_keyColor[1] + L": " + m_valuesColor + m_disney_sj;
                }
                else
                {
                    sur_str += L"Netflix " + m_keyColor[1] + L": " + m_valuesColor + m_netflix_sj;
                }
                sur_str += m_keyColor[0] + L']' + m_valuesColor;
            }
            else
            {
                sur_str += m_keyColor[0] + L" [" + m_valuesColor + m_keyColor[1] + L"sur " + m_valuesColor + m_sur + m_keyColor[0] + L']' + m_valuesColor;
            }
        }
        // x signalétique jeunesse
        if (affichage_x_sj_actif && m_sur == L"")
        {
            // Disney+ SJ
            if (affichage_disney_sj_actif && m_disney_sj.length() != 0)
                x_sj_str += m_keyColor[0] + L" (" + m_valuesColor + L"Disney+" + m_keyColor[1] + L" : " + m_valuesColor + m_disney_sj + m_keyColor[0] + L')' + m_valuesColor;
            // Netflix SJ
            if (affichage_netflix_sj_actif && m_netflix_sj.length() != 0)
                x_sj_str += m_keyColor[0] + L" (" + m_valuesColor + L"Netflix" + m_keyColor[1] + L" : " + m_valuesColor + m_netflix_sj + m_keyColor[0] + L')' + m_valuesColor;
        }
        // La signalétique jeunesse
        if (affichage_sj_actif && m_sj.length() != 0)
            sj_str += m_keyColor[0] + L" (" + m_valuesColor + L"SJ" + m_keyColor[1] + L" : " + m_valuesColor + m_sj + m_keyColor[0] + L')' + m_valuesColor;
        // Durée
        if (affichage_duree_actif)
        {
            duree_str = L' ' + std::to_wstring(m_duree/60) + m_keyColor[0] + L"min " + m_valuesColor;
        }
        // Note
        if (affichage_note_actif)
            note_str += calcul_Note_Affichage();

        std::wcout << titres_str << annees_str << sur_str << x_sj_str << sj_str << duree_str << note_str << std::endl;
    }
}
...

Expliquez-moi !

Et (J'ai oublié !) serie.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
...
            if (std::regex_match(annees_str, dummy, std::wregex(L"\\d{4}\\-\\d{4}\\s?")))
            {
                m_f_anneesProduction.first = stoi(annees_str);
                m_f_anneesProduction.second = stoi(annees_str.substr(5));
            }
            else
            {
                m_f_anneesProduction.first = stoi(annees_str);
            }
...

Voilà !!!

Merci:aie::aie::aie:

1 pièce(s) jointe(s)

Citation:

Envoyé par Laurent_B_

https://github.com/laurentbouleau/Exemple : Août 18 (24/08/2024)

https://github.com/laurentbouleau/Ex...20Disney%2B%5D : Marches pas !

Mince !!!

Les expressions régulières pour le décodage des informations dans les noms de fichiers et de répertoire fonctionnent très bien avec "Andor.[2022- Disney+]".

Pièce jointe 658596
EDIT:
Pièce jointe 658674
Comme le montre le débugueur, c'est le contenu du fichier "./Andor.[2022- Disney+]\2022-09-21\1x06.2024-06-27.txt", et en particulier, sa 2 ligne qui pose problème:

Citation:

6. L'œil
54 min____
Profitant d'un incroyable festival local, la mission sur Aldhani arrive à son point de non-retour.

Il y a une tabulation horizontale (\t) après le "min" de la 2ème ligne.
Il faut juste un peu modifier l'expression régulière pour qu'elle gère le "nouveau" format :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 void InfosVisionnage::initialiser_Duree(std::wstring& m) { const std::wregex duree_format_rg{ L"([[:digit:]]+)\\s?(min|MIN|Min)\\s*" }; std::wsmatch match; if (std::regex_match(m, match, duree_format_rg)) { auto duree_en_minute = std::stoi(match[1]); m_duree = duree_en_minute * 60; } else { throw std::invalid_argument("'" + std::string{ m.begin(),m.end() } + "' n'est pas un format de durée valide."); } }
Mais vous devriez penser à étendre cette fonction pour gère les formats mentionnant des heures, en plus des minutes (voire les secondes et les frames).

Citation:

Code:

std::vector<std::wstring> fusionner_Titres(std::vector<std::wstring>& vieux_titres, std::vector<std::wstring>& nouveaux_titres){...}

Dans votre implémentation, vous modifiez le contenu de "nouveaux_titres" alors que le nom de la fonction ne l'indique pas : c'est un effet de bord.
Il faut au maximum éviter les effets de bord pour que votre code reste lisible et maintenable.
Pour renforcer la "sécurité", je change légèrement le type des paramètres pour que mal implémenter cette fonction soit plus compliqué. (rendre les paramètres "const").
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 std::vector<std::wstring> fusionner_Titres(const std::vector<std::wstring>& vieux_titres, const std::vector<std::wstring>& nouveaux_titres) { assert(nouveaux_titres.size() < 4 && L"???"); assert(vieux_titres.size() < 4 && L"???"); std::vector<std::wstring> resultat = nouveaux_titres; if (vieux_titres.size() == nouveaux_titres.size()) { //double titres_ration{ 80.0 }; if (vieux_titres.size() == 1 && nouveaux_titres.size() == 1) { resultat = vieux_titres; } if (vieux_titres.size() == 3 && nouveaux_titres.size() == 3 && vieux_titres[0] == nouveaux_titres[0] && vieux_titres[1] != nouveaux_titres[1] && vieux_titres[2] == nouveaux_titres[2]) { resultat[1] = vieux_titres[1]; } if (vieux_titres.size() == 3 && nouveaux_titres.size() == 3 && (vieux_titres[0] != nouveaux_titres[0] || vieux_titres[1] != nouveaux_titres[1] || vieux_titres[2] != nouveaux_titres[2])) { resultat = vieux_titres; } } return resultat; }
Citation:

Code:

std::vector<std::wstring> extraire_Titres_Depuis_UneLigne(std::wstring& titre){...}

Le nom du paramètre est vraiment pas terrible.
"ligne", ça serait bien plus clair.

Citation:

Code:

void Film::initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& m_titre){...}

A quoi sert le paramètre "m_titre" (en plus d'avoir ce putain de "m_" en préfixe :furieux:) ?
On dégage ce paramètre, SVP.

Citation:

Code:

const void Film::Print_Header(){...}

Il sert à quoi ce const devant ???
C'est quoi un "const void" ?
C'est pas plutôt :

Code:

void Film::Print() const

???
Il faudra aussi corriger de la même manière toutes les fonctions appelées par ces fonction "const" pour qu'elles deviennent elles aussi "const".

Citation:

~~const~~ void Film::Print_Header() const
{...
m_date_en_salle_ou_sur
...
}

"m_date_en_salle_ou_sur", c'est mieux que "m_date".
Mais j'ai du mal a voir la différence entre le champ "m_date_en_salle_ou_sur" et le champ "m_date_de_sortie".

Citation:

Citation:

Avec votre code, si on modifie un peu le contenu de la variable "filename_pattern", par exemple, en ajoutant des séparateurs optionnels, etc..., il faudra à modifier votre code de décodage de la date.

Expliquez-moi !

Citation:

Le code que j'ai fourni a aussi des dépendances avec la variable "filename_pattern", mais c'était une base.
Je pense qu'il est plus facile de rendre le code de décodage de la date indépendant de "filename_pattern" à partir de mon code, qui utilise plus de captures, que du vôtre qui utilise plus d'informations "en dur".

Que faire ?

J'ai l'habitude de faire le code le plus générique possible tant que cela est "gratuit", en termes de complexité et de lisibilité de code.
Si ce n'est pas "gratuit", on peut tomber dans ce qu'on appelle souvent l'"over-engineering" : faire un code très complexe pour gérer des cas qui n'arriveront pas "dans la vraie vie".
"Que faire ?" : bin, utiliser le code que j'ai posté à la place du vôtre, non ???
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 Film::Film(std::filesystem::path racine) { this->racine = racine; auto nomDossier = racine.filename().wstring(); assert(nomDossier.length() > 0 && L"Nom de dossier vide"); std::wregex filename_pattern{ L"(.+?)(?:\\.\$(?:(\\d{4})\\-(\\d{2})\\-(\\d{2})\$))?(?:\\.(.+))?$" }; std::wsmatch match; if (std::regex_match(nomDossier, match, filename_pattern)) { std::wstring part_1 = match[1]; m_titres = extraire_Titres_Depuis_NomDeFichierOuDeRepertoire(part_1); if (match[2].matched) { std::wstring annees_str = match[2].str(); std::wstring mois_str = match[3].str(); std::wstring jours_str = match[4].str(); m_date.tm_year = stoi(annees_str) - 1900; //beurk!! m_date.tm_mon = stoi(mois_str) - 1; //beurk!! m_date.tm_mday = stoi(jours_str); } //m_sur = (match[3].matched) ? match[3].str() : L""; m_sous_genre = (match[5].matched) ? match[5].str() : L""; } else { assert(false == true && "Le nom du répertoire n'est pas un nom valide."); } }
Citation:

Code:

Serie::Serie(std::filesystem::path racine){...}

Etc..
Marches pas !!!

Heu, avec quoi comme série, ça marche pas, SVP ???

Avec :
"Star Wars- The Clone wars.[2008-2020].Animation"
"Seven Deadly Sins- Four Knights of the Apocalypse.[2023].Manga"
"Andor.[2022- Disney+]"

Ca fonctionne.

Citation:

https://github.com/laurentbouleau/Ex...7%5D.Animation : Star Wars Rebels.[2014-2017].Animation : Hors saison (11 août 2014 : exemple 0x1.2024-08-21.txt)

Quel est le problème ?
A part que vos méthodes "Episode:: PrintFirstSequenceVisionnage" et "Episode:: PrintSequenceVisionnage", ce qui explique pourquoi seul les informations sur la saison sont affichées.

Pourquoi les informations sur la saison sont foireuses :
Code:

1 2 3 4 5 6 if (int j = std::stoi(nomFichier)) { m_numero = j; initialiser_Resume(cheminFichier); return; }
Pouvez-vous m'expliquer la logique de l'extrait de code précédent, qui fait partie de la fonction "Saison::initialiser_Fichier" ???
Et "Saison::initialiser_Fichier" a encore un nom foireux car cela ne correspond en rien à ce que la fonction fait réellement.
Avec ce code, votre saison "Hors Saison" a un champ "m_numero" égale à 4 et votre saison "1" a un champ "m_numero" égale à 15.
C'est quoi ce raisonnement ?

Citation:

Code:

void Serie::initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& m_titre){...}

Même connerie que dans "Film::initialiser_Titre", dégagez le paramètre "m_titre"
"Serie::initialiser_Titre" et "Film::initialiser_Titre" devraient être pratiquement identiques. Pensez à fusionner ce code.

Citation:

film.cpp et serie.cpp :
Pas de extern ...
Ok ou pas ok ?

Vous devez comprendre à quoi servent les fichiers d'en-tête (.h).

On va prendre la ligne suivante :
"extern const std::wstring lire_fichierTxt(std::wstring const& nomFichier);"
Vous m'avez collé cette ligne dans :
film.cpp
serie.cpp
film_serie.cpp

Alors que vous n'avez qu'à la mettre UNE fois (sans l'"extern") dans "film_serie.h".

Code:

const std::wstring lire_fichierTxt(std::wstring const& nomFichier);

Vous faites la même chose pour toutes les fonctions qui sont définies dans "film_serie.cpp".

Et vous remplacez ces dizaines de lignes à la con dans les 3 fichier .cpp par un simple :

Code:

#include "film_serie.h"

Citation:

Mais :
Kaleidoscope.[2023 Netflix]
Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga
SupraCell.[2024 Netflix].Mini-série
Azertyuiop £££.[2024].Mini-série
Marche pas !

"Kaleidoscope.[2023 Netflix]", ça plante sur le traitement du fichier ".\Kaleidoscope.[2023 Netflix]\2023-01-01\1x8.2024-03-03.txt" qui est vide.
Dans votre fonction "lire_fichierTxt" :
Code:

1 2 3 4 5 if (contenuFichier == u8"") { //throw std::runtime_error("Le fichier '" + wstr_to_u8(nomFichier) + "' est vide."); return { L"" }; }
Donc, pour un fichier vide, vous renvoyez un std::vector de taille 1 dont le premier et seule élément est une chaine vide.

Mais dans votre fonction "InfosVisionnage::InfosVisionnage":
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 std::vector<std::wstring> file_content = lire_fichierTxt(m_cheminFichier.wstring(), { L"\n" }, false); m_NumeroEpisode = std::stoi(match[filename_numero_episode_index]); if (file_content.size() > 0 /* && file_content[0] != L""*/) { std::size_t pos; if (m_NumeroEpisode != 0) { pos = file_content[0].find(L". "); file_content[0] = file_content[0].substr(pos + 2); } //std::size_t pos = file_content[0].find(c_separateur_numero_titre_dans_ligne_de_fichier); //auto ligne_titres = (pos != std::wstring::npos) ? file_content[0].substr(pos + c_separateur_numero_titre_dans_ligne_de_fichier.length()) : L""; //m_titres = ::extraire_Titres_Depuis_UneLigne(ligne_titres); m_titres = ::extraire_Titres_Depuis_UneLigne(file_content[0]); }
Dans ce cas, l'expression "file_content.size() > 0" est vraie.
L'expression 'file_content[0] != L""' est en commentaire.
"m_NumeroEpisode " est égale à 8, c'est correct.
"pos = file_content[0].find(L". ");" => pos est égale à std::npos (-1).
"pos + 2" est égale à 1.
"file_content[0].substr(1)" renvoie une erreur car "file_content[0]" n'est "long" que de 0 caractère, donc 1, c'est trop loin.

Vous gérez donc très mal les valeurs de retour des fonctions "find" et/ou "lire_fichierTxt".

Pour "Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga", même problème que pour "Kaleidoscope.[2023 Netflix]", le fichier ".\Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga\2021-11-02\1x1.2024-03-22_.txt"

Pour "SupraCell.[2024 Netflix].Mini-série", où est le problème ? (C'est aussi pourri que pour "Star Wars Rebels.[2014-2017].Animation")

Pour "Azertyuiop £££.[2024].Mini-série", où est le problème ? (C'est aussi pourri que pour "Star Wars Rebels.[2014-2017].Animation")

Citation:

Code:

m_sous_genre = (match[4].matched) ? match[4].str() : L"";

???

Quelle est la question ? C'est quoi le rapport ???

Citation:

Code:

std::wregex filename_pattern{ L"^(.+?)(?:\\.\\[(\\d{4}\\-\\d{4}\\s|\\d{4}\\-\\d{4}|\\d{4}\\-\\s|\\d{4}\\s|\\d{4})([^\\]]*)\\])(?:\\.(.+))?$|^(.+)(?:\\.(.+))$" };

???

Quelle est la question ? C'est quoi le rapport ???
Citation:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 std::wstring annees_str = match[2].str(); std::wsmatch dummy; if (std::regex_match(annees_str, dummy, std::wregex(L"\\d{4}\\-\\d{4}\\s?"))) { m_f_anneesProduction.first = stoi(annees_str); m_f_anneesProduction.second = stoi(annees_str.substr(5)); } else { m_f_anneesProduction.first = stoi(annees_str); }
???
Quelle est la question ?

Citation:

Exemple serie.cpp :
abc.[2008]
ou
abc.[2008-]
ou
abc.[2008-2010]
Meilleur !!! Ok ou pas !

Quelle est la question ?

Citation:

Aller : le film : Star Wars - Episode VII - Le réveil de la force.(2015-12-16)

Les fichiers correspondants ne sont pas dans le dépôt Github.

Citation:

Code:

annees_str = format_Annees(); // L" [" et L']' ???

Je comprends pas trop votre problème.
On demande juste à une fonction, ici "Serie::format_Annees", de générer une chaine de caractères, ici la fonction retourne les Années (soit les années de production ou les années de diffusion en fonction des données disponibles) avec le formatage qui y correspond : couleur + les [].

26/08/2024, 23h09
Laurent_B_

https://github.com/laurentbouleau/Exemple : Août 22 (26/08/2024)

Citation:

Les expressions régulières pour le décodage des informations dans les noms de fichiers et de répertoire fonctionnent très bien avec "Andor.[2022- Disney+]".

cf : film_serie.cpp
Code:

1 2 3 4 5 6 7 8 9 10 11 const std::vector<std::wstring> Sur { L"Apple TV+", L"Amazon Prime Video", L"Crunchyroll", L"Disney+", L"DVD", L"Netflix", L"SⱯLTO", L"VOD" };
Pourquoi ?

Citation:

Pièce jointe 658596

Marches pas !

Citation:

Comme le montre le débugueur, c'est le contenu du fichier "./Andor.[2022- Disney+]\2022-09-21\1x06.2024-06-27.txt", et en particulier, sa 2 ligne qui pose problème:

Ok ! Merci !

Citation:

Il faut juste un peu modifier l'expression régulière pour qu'elle gère le "nouveau" format :

Pas compris !!!

Citation:

Mais vous devriez penser à étendre cette fonction pour gère les formats mentionnant des heures, en plus des minutes (voire les secondes et les frames).

Ah ! Bon ! Mince !

Citation:

Dans votre implémentation, vous modifiez le contenu de "nouveaux_titres" alors que le nom de la fonction ne l'indique pas : c'est un effet de bord.
Il faut au maximum éviter les effets de bord pour que votre code reste lisible et maintenable.
Pour renforcer la "sécurité", je change légèrement le type des paramètres pour que mal implémenter cette fonction soit plus compliqué. (rendre les paramètres "const").

Ok !

Citation:

Le nom du paramètre est vraiment pas terrible.
"ligne", ça serait bien plus clair.

Ok !

Citation:

A quoi sert le paramètre "m_titre" (en plus d'avoir ce putain de "m_" en préfixe ) ?
On dégage ce paramètre, SVP.

Ok !

Citation:

Il faudra aussi corriger de la même manière toutes les fonctions appelées par ces fonction "const" pour qu'elles deviennent elles aussi "const".

Ok !

Citation:

J'ai l'habitude de faire le code le plus générique possible tant que cela est "gratuit", en termes de complexité et de lisibilité de code.
Si ce n'est pas "gratuit", on peut tomber dans ce qu'on appelle souvent l'"over-engineering" : faire un code très complexe pour gérer des cas qui n'arriveront pas "dans la vraie vie".

Ok !

Citation:

"Que faire ?" : bin, utiliser le code que j'ai posté à la place du vôtre, non ???

Attention : m_sur !!!
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 Film::Film(std::filesystem::path racine) { this->racine = racine; auto nomDossier = racine.filename().wstring(); assert(nomDossier.length() > 0 && L"Nom de dossier vide"); std::wregex filename_pattern{ L"(.+?)(?:\\.\$(?:(\\d{4})\\-(\\d{2})\\-(\\d{2})\$))?(?:\\.(.+))?$" }; std::wsmatch match; if (std::regex_match(nomDossier, match, filename_pattern)) { std::wstring part_1 = match[1]; m_titres = extraire_Titres_Depuis_NomDeFichierOuDeRepertoire(part_1); if (match[2].matched) { std::wstring annees_str = match[2].str(); std::wstring mois_str = match[3].str(); std::wstring jours_str = match[4].str(); m_date.tm_year = stoi(annees_str) - 1900; //beurk!! m_date.tm_mon = stoi(mois_str) - 1; //beurk!! m_date.tm_mday = stoi(jours_str); } m_sur = (match[5].matched) ? match[5].str() : L""; m_sous_genre = (match[6].matched) ? match[6].str() : L""; } else { assert(false == true && "Le nom du répertoire n'est pas un nom valide."); } }
Avant : cf film_serie.cpp
Code:

1 2 3 4 const std::vector<std::wstring> Sur { ... };
Citation:

Heu, avec quoi comme série, ça marche pas, SVP ???

Avec :
"Star Wars- The Clone wars.[2008-2020].Animation"
"Seven Deadly Sins- Four Knights of the Apocalypse.[2023].Manga"
"Andor.[2022- Disney+]"

Ca fonctionne.

Pardon ?

Citation:

Quel est le problème ?
A part que vos méthodes "Episode:rintFirstSequenceVisionnage" et "PrintSequenceVisionnage", ce qui explique pourquoi seul les informations sur la saison sont affichées.

Pas compris !

Citation:

Pourquoi les informations sur la saison sont foireuses :

J'ai oublié !!!

Citation:

Pouvez-vous m'expliquer la logique de l'extrait de code précédent, qui fait partie de la fonction "Saison::initialiser_Fichier" ???
Et "Saison::initialiser_Fichier" a encore un nom foireux car cela ne correspond en rien à ce que la fonction fait réellement.
Avec ce code, votre saison "Hors Saison" a un champ "m_numero" égale à 4 et votre saison "1" a un champ "m_numero" égale à 15.
C'est quoi ce raisonnement ?

Ben...

Citation:

Même connerie que dans "Film::initialiser_Titre", dégagez le paramètre "m_titre"
"Serie::initialiser_Titre" et "Film::initialiser_Titre" devraient être pratiquement identiques. Pensez à fusionner ce code.

Demain ou plus tard !

Citation:

Vous devez comprendre à quoi servent les fichiers d'en-tête (.h).

On va prendre la ligne suivante :
"extern const std::wstring lire_fichierTxt(std::wstring const& nomFichier);"
Vous m'avez collé cette ligne dans :
film.cpp
serie.cpp
film_serie.cpp

Alors que vous n'avez qu'à la mettre UNE fois (sans l'"extern") dans "film_serie.h".

Je suis fatigué !!!
Bon nuit :aie::aie::aie:

Ah oui :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
void Serie::corriger_Annee_Debut() const
{
    assert((m_f_anneesProduction.first || (saisons.size() > 0 && saisons[0].m_f_anneesDiffusion)) && L"Il faut au moins une date de début.");

    if (!m_f_anneesProduction.first || (saisons.size() > 0 && saisons[0].m_f_anneesDiffusion && m_f_anneesProduction.first > saisons[0].m_f_anneesDiffusion))
        m_f_anneesProduction.first = saisons[0].m_f_anneesDiffusion;
}
...
void Serie::corriger_Annee_Fin() const
{
    assert((m_f_anneesProduction.first || (saisons.size() > 0 && saisons.back().m_f_anneesDiffusion)) && L"Il faut au moins une date de fin.");

    if (!m_f_anneesProduction.first || (saisons.size() > 0 && saisons.back().m_f_anneesDiffusion && m_f_anneesProduction.first > saisons.back().m_f_anneesDiffusion))
        m_f_anneesProduction.first = saisons[0].m_f_anneesDiffusion;
}

Marches pas !!!

Merci

Bonjour,

Citation:

Quel est le problème ?
A part que vos méthodes "Episode:rintFirstSequenceVisionnage" et "PrintSequenceVisionnage", ce qui explique pourquoi seul les informations sur la saison sont affichées.

Pourquoi les informations sur la saison sont foireuses :

J'en sais rien !!!
Code:

1 2 3 4 5 6 if (int j = std::stoi(nomFichier)) { m_numero = j; initialiser_Resume(cheminFichier); return; }
Pas compris !

Citation:

Pouvez-vous m'expliquer la logique de l'extrait de code précédent, qui fait partie de la fonction "Saison::initialiser_Fichier" ???
Et "Saison::initialiser_Fichier" a encore un nom foireux car cela ne correspond en rien à ce que la fonction fait réellement.
Avec ce code, votre saison "Hors Saison" a un champ "m_numero" égale à 4 et votre saison "1" a un champ "m_numero" égale à 15.
C'est quoi ce raisonnement ?

J'en sais rien ! J'ai oublié !!!

Citation:

"Serie::initialiser_Titre" et "Film::initialiser_Titre" devraient être pratiquement identiques. Pensez à fusionner ce code.

Comment faire ? ===> film_serie.cpp) : void initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& titre) ???
A Propos :
Code:

1 2 3 4 5 6 7 8 9 void initialiser_Titre_Original(fs::path const& cheminFichier, std::vector<std::wstring>& m_titre_original) { // Titre Original auto nomFichier = cheminFichier.wstring(); assert(nomFichier.length() > 0 && L"Nom de fichier vide"); std::wstring titre = lire_fichierTxt(cheminFichier.wstring()); assert((titre.size() != 0)); m_titre_original = ::extraire_Titres_Depuis_UneLigne(titre); }
Par :
std::vector<std:wstring> initialiser_Titre_Original(fs::path const& cheminFichier, std::vector<std::wstring>& titre_original)
{ // Titre Original
auto nomFichier = cheminFichier.wstring();
assert(nomFichier.length() > 0 && L"Nom de fichier vide");
std::wstring titre = lire_fichierTxt(cheminFichier.wstring());
assert((titre.size() != 0));

return titre_original = ::extraire_Titres_Depuis_UneLigne(titre);
}[/CODE]
???

Citation:

Vous devez comprendre à quoi servent les fichiers d'en-tête (.h).

On va prendre la ligne suivante :
"extern const std::wstring lire_fichierTxt(std::wstring const& nomFichier);"
Vous m'avez collé cette ligne dans :
film.cpp
serie.cpp
film_serie.cpp

Alors que vous n'avez qu'à la mettre UNE fois (sans l'"extern") dans "film_serie.h".
...
Vous faites la même chose pour toutes les fonctions qui sont définies dans "film_serie.cpp".

Et vous remplacez ces dizaines de lignes à la con dans les 3 fichier .cpp par un simple :

Comment faire ???

---------------------------------

Citation:

"Kaleidoscope.[2023 Netflix]", ça plante sur le traitement du fichier ".\Kaleidoscope.[2023 Netflix]\2023-01-01\1x8.2024-03-03.txt" qui est vide.
Dans votre fonction "lire_fichierTxt" :
...

Donc, pour un fichier vide, vous renvoyez un std::vector de taille 1 dont le premier et seule élément est une chaine vide.

Mais dans votre fonction "InfosVisionnage::InfosVisionnage":
...
Dans ce cas, l'expression "file_content.size() > 0" est vraie.
L'expression 'file_content[0] != L""' est en commentaire.
"m_NumeroEpisode " est égale à 8, c'est correct.
"pos = file_content[0].find(L". ");" => pos est égale à std::npos (-1).
"pos + 2" est égale à 1.
"file_content[0].substr(1)" renvoie une erreur car "file_content[0]" n'est "long" que de 0 caractère, donc 1, c'est trop loin.

Vous gérez donc très mal les valeurs de retour des fonctions "find" et/ou "lire_fichierTxt".

Pour "Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga", même problème que pour "Kaleidoscope.[2023 Netflix]", le fichier ".\Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga\2021-11-02\1x1.2024-03-22_.txt"

Pour "SupraCell.[2024 Netflix].Mini-série", où est le problème ? (C'est aussi pourri que pour "Star Wars Rebels.[2014-2017].Animation")

Pour "Azertyuiop £££.[2024].Mini-série", où est le problème ? (C'est aussi pourri que pour "Star Wars Rebels.[2014-2017].Animation")

Ah ! Mince ! Alors, expliquez-moi ???

----------------------------
Code:

1 2 3 4 5 6 7 std::wstring annees_str = match[2].str(); std::wsmatch dummy; if (std::regex_match(annees_str, dummy, std::wregex(L"\\d{4}\\-\\d{4}\\s?"))) { m_f_anneesProduction.first = stoi(annees_str); m_f_anneesProduction.second = stoi(annees_str.substr(5)); }
et

Citation:

Exemple serie.cpp :
abc.[2008]
ou
abc.[2008-]
ou
abc.[2008-2010]
Meilleur !!! Ok ou pas !

===> oui : vraiment !!!
Donc : abc.[2008] ou abc.[2008-] ou abc.[2008-2010]
Exemple :
Titre : [2022-] (sur Disney : 12+) 45min 4,2/5

--------------------------------

Citation:

Les fichiers correspondants ne sont pas dans le dépôt Github.

Ok !
https://github.com/laurentbouleau/Exemple : Août 23 (27/08/2024)

----------------------------------

Citation:

Je comprends pas trop votre problème.
On demande juste à une fonction, ici "Serie::format_Annees", de générer une chaine de caractères, ici la fonction retourne les Années (soit les années de production ou les années de diffusion en fonction des données disponibles) avec le formage qui y correspond : couleur + les [].

===> -

Reposé !!! Fatigué !!!

27/08/2024, 19h28
Laurent_B_
serie.cpp :

Code:

const std::wregex duree_format_rg{ L"([[:digit:]]+)\\s?(min|MIN|Min)\\s*" };

Ok !
Code:

1 2 3 const void Serie::corriger_Annee_Debut() ... const void Serie::corriger_Annee_Fin()
Oui : j'ai oublié !!!
Exemple :

Code:

void Serie::corriger_Annee_Debut() const

https://github.com/laurentbouleau/Exemple : Août 23 (27/08/2024)

https://github.com/laurentbouleau/Exemple : Août 24 (27/08/2024)

film_serie.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
std::vector<std::wstring> fusionner_Titres(const std::vector<std::wstring>& nouveaux_titres, const std::vector<std::wstring>& vieux_titres)
{
    assert(nouveaux_titres.size() < 4 && L"???");
    assert(vieux_titres.size() < 4 && L"???");

    std::vector<std::wstring> resultat = nouveaux_titres;

    if (vieux_titres.size() == nouveaux_titres.size())
    {
        //double titres_ration{ 80.0 };
        if (vieux_titres.size() == 1 && nouveaux_titres.size() == 1)
        {
            resultat = vieux_titres;
        }
        if (vieux_titres.size() == 3 && nouveaux_titres.size() == 3 && vieux_titres[0] == nouveaux_titres[0] && vieux_titres[1] != nouveaux_titres[1] && vieux_titres[2] == nouveaux_titres[2])
        {
            resultat[1] = vieux_titres[1];
        }
        if (vieux_titres.size() == 3 && nouveaux_titres.size() == 3 && (vieux_titres[0] != nouveaux_titres[0] || vieux_titres[1] != nouveaux_titres[1] || vieux_titres[2] != nouveaux_titres[2]))
        {
            resultat = vieux_titres;
        }
    }

    return resultat;
}

et fim.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
void Film::initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& ligne)
{ // Titre
    auto nomFichier = cheminFichier.wstring();
    assert(nomFichier.length() > 0 && L"Nom de fichier vide");
    std::vector<std::wstring> contenu = lire_fichierTxt(cheminFichier.wstring(), { L"\n" });
    assert((contenu.size() != 0));

    //std::vector<std::wstring>nouveaux_titres = extraire_Titres_Depuis_UneLigne(contenu[0]);
    std::vector<std::wstring>titres = extraire_Titres_Depuis_UneLigne(contenu[0]);

    m_titres = fusionner_Titres(m_titres, titres);

    contenu.erase(contenu.begin());
    if (contenu.size() > 0)
    {
        m_duree = initialiser_Duree(contenu[0]);
        contenu.erase(contenu.begin());
        if (contenu.size() > 0)
            m_resume = contenu;
    }
}

Bizarre !!!

Citation:

On va prendre la ligne suivante :
"extern const std::wstring lire_fichierTxt(std::wstring const& nomFichier);"
Vous m'avez collé cette ligne dans :
...
Je comprends pas trop votre problème.
On demande juste à une fonction, ici "Serie::format_Annees", de générer une chaine de caractères, ici la fonction retourne les Années (soit les années de production ou les années de diffusion en fonction des données disponibles) avec le formage qui y correspond : couleur + les [].

Je suis crevé !!! Demain ou plus tard !

Bonsoir,

https://github.com/laurentbouleau/Exemple : Août 25 (28/08/2024)
Code:

1 2 3 4 5 6 if (int j = std::stoi(nomFichier)) { m_numero = j; initialiser_Resume(cheminFichier); return; }
Que faire ???

Citation:

Pouvez-vous m'expliquer la logique de l'extrait de code précédent, qui fait partie de la fonction "Saison::initialiser_Fichier" ???
Et "Saison::initialiser_Fichier" a encore un nom foireux car cela ne correspond en rien à ce que la fonction fait réellement.
Avec ce code, votre saison "Hors Saison" a un champ "m_numero" égale à 4 et votre saison "1" a un champ "m_numero" égale à 15.
C'est quoi ce raisonnement ?

Pouvez-vous expliquer ???

Code:

const std::wstring racine = t + L"Star Wars- The Clone wars.[2008-2020].Animation";

Marches pas !!!

Attention : Star Wars Rebels.[2014-2017 Disney+].Animation

0x1.2024-08-21.txt !!!
Etc...
Hors saison.txt !!!
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 std::wstring Serie::format_Annees() const { if (m_f_anneesProduction.first && m_f_anneesProduction.second) { return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(m_f_anneesProduction.first.value()) + m_keyColor[1] + L'-' + m_valuesColor + std::to_wstring(m_f_anneesProduction.second.value()) + m_keyColor[0] + L']' + m_valuesColor; } else if (m_f_anneesProduction.first) { return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(m_f_anneesProduction.first.value()) + m_keyColor[0] + L']' + m_valuesColor; } else { std::pair<int, int> anneesDiffusion = calculer_Annees_Diffusion(); return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(anneesDiffusion.first) + m_keyColor[1] + L'-' + m_valuesColor + std::to_wstring(anneesDiffusion.second) + m_keyColor[0] + L']' + m_valuesColor; } }
Expliquez-moi !!!

Merci vraiment:aie::D;)

Remarques sur les versions entre "Août 18 (24/08/2024)" et "Août 23 (27/08/2024)" du dépôt GitHub :

"Août 20 (25/08/2024)" :
Vous enlevez des "extern" devant quelques déclarations de fonction au début des fichiers "film.cpp" et "serie.cpp".
Vous pouvez supprimer toutes ces lignes (pour film.cpp : de la ligne 45 à la ligne 90 (et encore "Soundtrack" ne sert à rien); pour serie.cpp : de la ligne 44 à la ligne 90).
Si cela ne compile plus, c'est que vous avez oublié de faire le #include "qui va bien" (et/ou que vous n'avez pas correctement rempli les .h).

"Août 22 (26/08/2024)" :
Vous avez modifié le fichier "./Andor.[2022- Disney+]\2022-09-21\1x06.2024-06-27.txt" pour qu'il n'y ait plus de "\t" en ligne 2.
Mais, moi, je pense que le problème n'est pas dans le fichier de données en entré, mais dans le code de la fonction "InfosVisionnage::initialiser_Duree" qui gère mal ce cas :
En ajoutant juste "\\s*" à la fin de "duree_format_rg", on gère facilement un nombre arbitraire d'espaces ou de tabulations horizontales derrière la chaine de caractère "min".
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 void InfosVisionnage::initialiser_Duree(std::wstring& m) { const std::wregex duree_format_rg{ L"([[:digit:]]+)\\s?(min|MIN|Min)\\s*" }; std::wsmatch match; if (std::regex_match(m, match, duree_format_rg)) { auto duree_en_minute = std::stoi(match[1]); m_duree = duree_en_minute * 60; } else { throw std::invalid_argument("'" + std::string{ m.begin(),m.end() } + "' n'est pas un format de durée valide."); } }
(Ce code a déjà été donné dans ma précédente réponse.)

"Août 22 (26/08/2024)" (2ème fois, faudrait s'arranger pour mettre des numéro de version unique dans le dépôt GitHub, SVP !!!):
Vous ajoutez "const" au paramètre "vieux_titres" de la fonction "fusionner_Titres", mais pourquoi pas aussi sur le paramètre "nouveaux_titres", comme je l'ai fait dans ma réponse ???
"m_date_en_salle_ou_sur", c'est pas terrible mais c'est quand même largement mieux que "m_date" !!!
Sur la fonction "initialiser_Titre", vous renommez le paramètre de "m_titre" à "titre", alors qu'il faut complètement le supprimer, vous aimez vous rajouter du travail.:aie:
Rendre "const" les fonction "Serie::corriger_Annee_Debut" et "Serie::corriger_Annee_Fin", c'est pas très logique, vu qu'on modifie les valeurs dans des champs de l'objet.
Citation:
cf : film_serie.cpp
Code:

1 2 3 4 5 6 7 8 9 10 11 const std::vector<std::wstring> Sur { L"Apple TV+", L"Amazon Prime Video", L"Crunchyroll", L"Disney+", L"DVD", L"Netflix", L"SⱯLTO", L"VOD" };
Pourquoi ?
Quelle est la question ???

Citation:

Citation:

Il faut juste un peu modifier l'expression régulière pour qu'elle gère le "nouveau" format :

Pas compris !!!

cf. la modification de la fonction "InfosVisionnage::initialiser_Duree" que j'ai proposé au début de ce message.

Code:

const std::wregex duree_format_rg{ L"([[:digit:]]+)\\s?(min|MIN|Min)\\s*" };

Citation:

Citation:

"Que faire ?" : bin, utiliser le code que j'ai posté à la place du vôtre, non ???

Attention : m_sur !!!

Ok, j'avais pas vu qu'il y a un format de la forme "Prey.(2022-08-05 Disney+)" pour les films.
Il peut avoir une information après la date de sortie mais à l'intérieur des parenthèses ().
Pas de problème, mais il fait adapter l'expression régulière pour qu'elle capture cette information :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 Film::Film(std::filesystem::path racine) { this->racine = racine; auto nomDossier = racine.filename().wstring(); assert(nomDossier.length() > 0 && L"Nom de dossier vide"); std::wregex filename_pattern{ L"(.+?)(?:\\.\$(?:(\\d{4})\\-(\\d{2})\\-(\\d{2})\\s*([^\$]*))\\))?(?:\\.(.+))?$" }; std::wsmatch match; if (std::regex_match(nomDossier, match, filename_pattern)) { std::wstring part_1 = match[1]; m_titres = extraire_Titres_Depuis_NomDeFichierOuDeRepertoire(part_1); if (match[2].matched) { std::wstring annees_str = match[2].str(); std::wstring mois_str = match[3].str(); std::wstring jours_str = match[4].str(); m_date.tm_year = stoi(annees_str) - 1900; //beurk!! m_date.tm_mon = stoi(mois_str) - 1; //beurk!! m_date.tm_mday = stoi(jours_str); } m_sur = (match[5].matched) ? match[5].str() : L""; m_sous_genre = (match[6].matched) ? match[6].str() : L""; } else { assert(false == true && "Le nom du répertoire n'est pas un nom valide."); } }
Citation:

Citation:

Heu, avec quoi comme série, ça marche pas, SVP ???

Avec :
"Star Wars- The Clone wars.[2008-2020].Animation"
"Seven Deadly Sins- Four Knights of the Apocalypse.[2023].Manga"
"Andor.[2022- Disney+]"

Ca fonctionne.

Pardon ?

Vous écrivez que cela ne fonctionne pas, mais quand je teste, les informations sont bien extraites, donc qu'est-ce qui ne fonctionne pas dans ces cas ?
Car, pour moi, cela fonctionne.

Citation:

Citation:

Quel est le problème ?
A part que vos méthodes "Episode:: PrintFirstSequenceVisionnage" et "Episode:: PrintSequenceVisionnage", ce qui explique pourquoi seul les informations sur la saison sont affichées.

Pas compris !

Vous semblez indiquer que le "non affichage" des informations d'un épisode n'est pas "normal", mais aucun code dans la classe Episode n'affiche d'informations. En particulier, les 2 fonctions citées qui sont vides, donc n'affiche rien, bizarre pour des fonction "Printxxx".
Citation:
Citation:
Pourquoi les informations sur la saison sont foireuses :
Code:

1 2 3 4 5 6 if (int j = std::stoi(nomFichier)) { m_numero = j; initialiser_Resume(cheminFichier); return; }
J'ai oublié !!!
Si vous ne comprenez pas VOTRE code, personne ne le peut à votre place. Dégagez-le, alors.

Citation:

void Serie::corriger_Annee_Debut() const
{
...
}
...
void Serie::corriger_Annee_Fin() const
{
...
}
Marches pas !!!

Vous pouvez m'expliquer ces "const" ??? 8O:aie::weird:

Citation:

Comment faire ? ===> film_serie.cpp) : void initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& titre) ???

Bin, en faisant simple et générique (type de données un peu complexe car vous découpez très très mal votre code en fonctions INDEPENDANTES):
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 std::optional<long> extraire_Duree_DepuisUneLigneDUnFichier(const std::wstring& ligne) { const std::wregex duree_format_rg{ L"^(?:(\\d+)\\s?(?:h|H)\\s?)?(?:(\\d+)\\s?(?:min|MIN)\\s?)?$" }; std::wsmatch match; std::optional<long> duree; if (std::regex_match(ligne, match, duree_format_rg)) { duree = (match[1].matched ? std::stoi(match[1]) : 0) * 60 * 60 + (match[2].matched ? std::stoi(match[2]) : 0) * 60; } else { throw std::invalid_argument("'" + std::string{ ligne.begin(),ligne.end() } + "' n'est pas un format de durée valide."); } return duree; } std::tuple<std::vector<std::wstring>, std::optional<long>, std::vector<std::wstring>> extraire_Informations_DepuisLeContenuDUnFichier(std::filesystem::path const& cheminFichier) { auto nomFichier = cheminFichier.wstring(); assert(nomFichier.length() > 0 && L"Nom de fichier vide"); std::vector<std::wstring> contenu = lire_fichierTxt(cheminFichier.wstring(), { L"\n" }); assert((contenu.size() != 0)); std::vector<std::wstring>nouveaux_titres = extraire_Titres_Depuis_UneLigne(contenu[0]); std::optional<long> duree; contenu.erase(contenu.begin()); if (contenu.size() > 0) { duree = extraire_Duree_DepuisUnLigneDUneFichier(contenu[0]); contenu.erase(contenu.begin()); } return { nouveaux_titres , duree , contenu }; }
En utilisant encore vos fonctions "Serie::initialiser_Titre" et "Film::initialiser_Titre" qui ont encore des noms TOUT POURRIS !!!
Code:

1 2 3 4 5 6 7 8 9 void Serie::initialiser_Titre(fs::path const& cheminFichier) { auto res = extraire_Informations_DepuisLeContenuDUnFichier(cheminFichier); m_titres = fusionner_Titres(m_titres, std::get<0>(res)); m_duree = std::get<1>(res) ? std::get<1>(res).value() : -1; m_resume = std::get<2>(res); }
Même code pour " Film::initialiser_Titre".
Mais ces fonctions sont clairement des conneries ; car elles font beaucoup trop de choses à la fois.
Donc ne pas perdre du temps à les refactorer ; mais changer le code qui les appels pour qu'il soit plus "logique".

Citation:

Code:

void initialiser_Titre_Original(fs::path const& cheminFichier, std::vector<std::wstring>& m_titre_original){...}

Elle sert à quoi maintenant ???

Faut juste remplacer :
Code:

1 2 3 4 if (nomFichier == L"Titre original.txt") { initialiser_Titre_Original(cheminFichier, m_titres_originaux); }
Par
Code:

1 2 3 4 if (nomFichier == L"Titre original.txt") { m_titres_originaux = extraire_Titres_Depuis_UneLigne(lire_fichierTxt(cheminFichier)); }
Pensez à changer la signature de la fonction "extraire_Titres_Depuis_UneLigne" :

Code:

std::vector<std::wstring> extraire_Titres_Depuis_UneLigne(const std::wstring& ligne){...}

Citation:

Citation:

Vous devez comprendre à quoi servent les fichiers d'en-tête (.h).

On va prendre la ligne suivante :
"extern const std::wstring lire_fichierTxt(std::wstring const& nomFichier);"
Vous m'avez collé cette ligne dans :
film.cpp
serie.cpp
film_serie.cpp

Alors que vous n'avez qu'à la mettre UNE fois (sans l'"extern") dans "film_serie.h".
...
Vous faites la même chose pour toutes les fonctions qui sont définies dans "film_serie.cpp".

Et vous remplacez ces dizaines de lignes à la con dans les 3 fichier .cpp par un simple :

Comment faire ???

Vous devez comprendre à quoi servent les fichiers d'en-tête (.h).

Citation:

Citation:

"Kaleidoscope.[2023 Netflix]", ça plante sur le traitement du fichier ".\Kaleidoscope.[2023 Netflix]\2023-01-01\1x8.2024-03-03.txt" qui est vide.
Dans votre fonction "lire_fichierTxt" :
...

Donc, pour un fichier vide, vous renvoyez un std::vector de taille 1 dont le premier et seule élément est une chaine vide.

Mais dans votre fonction "InfosVisionnage::InfosVisionnage":
...
Dans ce cas, l'expression "file_content.size() > 0" est vraie.
L'expression 'file_content[0] != L""' est en commentaire.
"m_NumeroEpisode " est égale à 8, c'est correct.
"pos = file_content[0].find(L". ");" => pos est égale à std::npos (-1).
"pos + 2" est égale à 1.
"file_content[0].substr(1)" renvoie une erreur car "file_content[0]" n'est "long" que de 0 caractère, donc 1, c'est trop loin.

Vous gérez donc très mal les valeurs de retour des fonctions "find" et/ou "lire_fichierTxt".

Pour "Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga", même problème que pour "Kaleidoscope.[2023 Netflix]", le fichier ".\Azertyuiop - qsdfghjklm.[2021-2023 Netflix].Manga\2021-11-02\1x1.2024-03-22_.txt"

Pour "SupraCell.[2024 Netflix].Mini-série", où est le problème ? (C'est aussi pourri que pour "Star Wars Rebels.[2014-2017].Animation")

Pour "Azertyuiop £££.[2024].Mini-série", où est le problème ? (C'est aussi pourri que pour "Star Wars Rebels.[2014-2017].Animation")

Ah ! Mince ! Alors, expliquez-moi ???

Je viens de donner une explication détaillée de ce qui se passe. Que voulez-vous de plus ?
Qu'est-ce qui n'est pas clair dans l'explication ?
C'est peut-être pas clair PARCE QUE VOTRE code n'est pas clair, non ?:aie:

Citation:

Citation:

Exemple serie.cpp :
abc.[2008]
ou
abc.[2008-]
ou
abc.[2008-2010]
Meilleur !!! Ok ou pas !

===> oui : vraiment !!!
Donc : abc.[2008] ou abc.[2008-] ou abc.[2008-2010]
Exemple :
Titre : [2022-] (sur Disney : 12+) 45min 4,2/5

C'est vraiment pas clair.
C'est juste ajouter un "-" dans les dates quand on n'a pas une date de fin ???
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 std::wstring Serie::format_Annees() const { if (m_f_anneesProduction.first && m_f_anneesProduction.second) { return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(m_f_anneesProduction.first.value()) + m_keyColor[1] + L'-' + m_valuesColor + std::to_wstring(m_f_anneesProduction.second.value()) + m_keyColor[0] + L']' + m_valuesColor; } else if (m_f_anneesProduction.first) { return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(m_f_anneesProduction.first.value()) + m_keyColor[1] + L'-' + m_keyColor[0] + L']' + m_valuesColor; } else { std::pair<int, int> anneesDiffusion = calculer_Annees_Diffusion(); return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(anneesDiffusion.first) + m_keyColor[1] + L'-' + m_valuesColor + std::to_wstring(anneesDiffusion.second) + m_keyColor[0] + L']' + m_valuesColor; } }

Remarques sur les versions entre "Août 23 (27/08/2024)" (2ème fois, faudrait s'arranger pour mettre des numéro de version unique dans le dépôt GitHub, SVP !!!)(BIS) et "Août 25 (28/08/2024)" du dépôt GitHub :

"Août 23 (27/08/2024)" (2ème fois:furieux:)
initialiser_Titre_Original change de signature, mais à quoi elle sert vraiment ? (cf. mon post précédent)
Changement du nom du paramètre de la fonction "extraire_Titres_Depuis_UneLigne", cool !!!
Changement de la "constness" des fonctions "Serie::corriger_Annee_Debut" et "Serie::corriger_Annee_Debut" mal fait. (cf. mon post précédent = plus de "const")

"Août 24 (27/08/2024)"
Vous permutez les noms des paramètres de la fonction "fusionner_Titres".
Je ne comprends absolument pas ce code.
Pouvez-vous l'expliquez pour avoir de meilleurs noms de fonctions et de paramètres ?
(par explication, c'est comment il doit fonctionner, pas ce qu'il fait, SVP)

"Août 25 (28/08/2024)"
Vous supprimer '#include "film_serie.h"',des fichier "film.cpp" et "serie.cpp" montrant que vous n'avez toujours pas compris l'utilité des fichiers d'en-tête.
"Ca fonctionne" parce que vous avez toujours près de 50 lignes A LA CON au début de ces fichiers au lieu de garder cet #include.
Mais vous devez aussi correctement remplir les .h avec ce qu'il faut, et pas des trucs "au pif".
Dans "film_serie.h", vous ne devez déclarer QUE les fonctions définies dans "film_serie.cpp".
Idem dans "utils.h", etc...
Les .h ne doivent inclure (via #include) que le strict minimum, donc rien ou presque.

Dans "serie.cpp" "ajout" de :
Code:

1 2 if (file_content.size() > 0 && file_content[0] != L"") {...}
Ok, fragile mais fonctionnel.

-------------------------
Citation:
film_serie.cpp :
Code:

1 2 std::vector<std::wstring> fusionner_Titres(const std::vector<std::wstring>& nouveaux_titres, const std::vector<std::wstring>& vieux_titres) {...}
et fim.cpp :

Citation:

void Film::initialiser_Titre(fs::path const& cheminFichier, std::vector<std::wstring>& ligne)
{...}

Bizarre !!!
cf. "Août 24 (27/08/2024)"
Vos noms ne sont pas clairs !!!
Si vous n'arrivez pas à comprendre VOTRE logique, imaginez les autres. :aie:

-------------------------
Citation:
Citation:
Code:

1 2 3 4 5 6 if (int j = std::stoi(nomFichier)) { m_numero = j; initialiser_Resume(cheminFichier); return; }
Que faire ???
Bin, c'est vous qui avez ajouté ce code pour lire et prendre en compte les fichiers "4.txt" et autres "11.txt", etc...
C'est vous qui devriez savoir à quoi ils servent et donc savoir comment les utiliser.
Si vous ne savez pas, dégagez ce code.

Citation:

Citation:

Pouvez-vous m'expliquer la logique de l'extrait de code précédent, qui fait partie de la fonction "Saison::initialiser_Fichier" ???
Et "Saison::initialiser_Fichier" a encore un nom foireux car cela ne correspond en rien à ce que la fonction fait réellement.
Avec ce code, votre saison "Hors Saison" a un champ "m_numero" égale à 4 et votre saison "1" a un champ "m_numero" égale à 15.
C'est quoi ce raisonnement ?

Pouvez-vous expliquer ???

C'est moi qui vous demande d'expliquer le code que VOUS avez écrit.

Citation:

Code:

const std::wstring racine = t + L"Star Wars- The Clone wars.[2008-2020].Animation";

Marches pas !!!

J'ai pas les fichiers qui correspondent à ce cas dans le dépôt GitHub.

Citation:

Attention : Star Wars Rebels.[2014-2017 Disney+].Animation
0x1.2024-08-21.txt !!!
Etc...
Hors saison.txt !!!

C'est quoi la question ???
Citation:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 std::wstring Serie::format_Annees() const { if (m_f_anneesProduction.first && m_f_anneesProduction.second) { return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(m_f_anneesProduction.first.value()) + m_keyColor[1] + L'-' + m_valuesColor + std::to_wstring(m_f_anneesProduction.second.value()) + m_keyColor[0] + L']' + m_valuesColor; } else if (m_f_anneesProduction.first) { return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(m_f_anneesProduction.first.value()) + m_keyColor[0] + L']' + m_valuesColor; } else { std::pair<int, int> anneesDiffusion = calculer_Annees_Diffusion(); return m_keyColor[0] + L" [" + m_valuesColor + std::to_wstring(anneesDiffusion.first) + m_keyColor[1] + L'-' + m_valuesColor + std::to_wstring(anneesDiffusion.second) + m_keyColor[0] + L']' + m_valuesColor; } }
Expliquez-moi !!!
Faut juste lire le code.
"std::wstring Serie::format_Annees()" : c'est une fonction qui renvoie une chaine de caractères ("std::wstring"), qui correspond aux années formatées "Serie::format_Annees()" pour l'affichage dans la console. (Années, c'est flou, mais vous ne nous avez toujours pas donné la logique corresponde (années de diffusion ? années de production ? etc ...))
"const", cette fonction n'a pas le droit de modifier les champs de l'objet (assez logique pour une fonction qui ne fait renvoyer une chaine en fonction des valeurs des champs de l'objet).
Après, on vérifie le contenu du champ "m_f_anneesProduction", correspondant aux années de production, qui est de type "std::pair<std::optional<int>, std::optional<int>>".
Donc, "m_f_anneesProduction" est une paire (std::pair) de 2 entiers (int) mais potentiellement non renseigné (std::optional).
Si l'année de début de production : "m_f_anneesProduction.first", est renseigné ET si l'année de fin de production : "m_f_anneesProduction.second" est aussi renseigné, la fonction retourne une chaine de caractère de la forme "[xxxx-yyyy]", avec xxxx l'année de début de production et yyyy l'année de fin de production . (plus tous les machins pour les afficher de la "bonne" couleur).
Sinon, si l'année de début de production : "m_f_anneesProduction.first", est renseigné (mais implicitement l'année de fin de production ne l'est pas), la fonction retourne une chaine de caractère de la forme "[xxxx]", avec xxxx l'année de début de production (format légèrement différent avec le code posté dans mon précédent message). (plus tous les machins pour les afficher de la "bonne" couleur).
Sinon on demande à la fonction "calculer_Annees_Diffusion" de renvoyer les années de diffusion de la série, et la fonction retourne une chaine de caractère de la forme "[xxxx-yyyy]", avec xxxx l'année de début de diffusion et yyyy l'année de fin de diffusion. (plus tous les machins pour les afficher de la "bonne" couleur).

P.S.:
Les fichiers "4.txt" et autres "11.txt" sont peut-être des fichiers pour stocker les informations sur des episodes avant que vous ayez commencé à les visionner, non ?

29/08/2024, 23h04
Laurent_B_

1 pièce(s) jointe(s)

https://github.com/laurentbouleau/Exemple : Août 26 (29/08/2024)

Mais :
Donc : abc.[2008] ou abc.[2008-] ou abc.[2008-2010]
Exemple :

Titre : Andor [2022-] (sur Disney : 12+) 45min 4,2/5
et pas :

Titre : Andor [2022] (sur Disney : 12+) 45min 4,2/5
Ok !

Pièce jointe 658750
et :

Star Wars : Episode VII - Le réveil de la force
et pas :

Star Wars - Episode VII - Le réveil de la force
Voilà !!!
Demain plus tard !

Fatigué !

Bonjour,

https://github.com/laurentbouleau/Exemple : Août 27 (30/08/2024)
film_serie.cpp :

Code:

1
2
3
4
5
...
std::optional<long> extraire_Duree_DepuisUneLigneDUnFichier(const std::wstring& ligne)
...
std::tuple<std::vector<std::wstring>, std::optional<long>, std::vector<std::wstring>> extraire_Informations_DepuisLeContenuDUnFichier(std::filesystem::path const& cheminFichier)
...

Mais : (fim.cpp)

Code:

1
2
3
4
        if (nomFichier == L"Making-of.txt")
        {
            //initialiser_Making_of(cheminFichier);
        }

Marches pas !!!

Merci:?

Voir 40 message(s) de cette discussion en une page