Problème et AVC : _wfopen_s

Version imprimable

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

22/02/2025, 04h15
bacelar

Ok pour votre gestion de "m_valuesColor".
(Mais vous n'avez pas besoin de ça si vous gériez l'affichage autrement.)

Citation:

Citation:

Mais comme j'insiste depuis le début pour ne pas avoir le code d'affichage dans ces classes, je ne suis pas étonné par ces "difficultés".

Pardon ???

C'est à dire ?
Je vous "reproche" de vouloir implémenter l'affichage des classes métier (Serie, Saison, Episode, etc...) dans ces classes plutôt que dans du code à part, depuis des mois.

Citation:

Ni ch, ni tr !!!

Faites un effort avec les noms de variable, SVP !!!

Ce que vous faites avec ces variables n'a aucun sens, donc, avec des noms pareils, j'y comprends rien.

Donc, je ne vois pas ce que vous reprochez à ces variables.

Expliquez-nous, EN FRANçAIS, comment fonctionne l'affichage d'une séquence de visionnage, SVP.
22/02/2025, 12h58
Laurent_B_

Citation:

Je vous "reproche" de vouloir implémenter l'affichage des classes métier (Serie, Saison, Episode, etc...) dans ces classes plutôt que dans du code à part, depuis des mois.

Ok : utils.h ???

Citation:

Faites un effort avec les noms de variable, SVP !!!

Ce que vous faites avec ces variables n'a aucun sens, donc, avec des noms pareils, j'y comprends rien.

Donc, je ne vois pas ce que vous reprochez à ces variables.

Expliquez-nous, EN FRANçAIS, comment fonctionne l'affichage d'une séquence de visionnage, SVP.

Ok !

Exemple Kaleidoscope :

Jaune (47 min) : 03/01/2023
Dans le quartier des diamants, Leo et sa nouvelle équipe cherchent de quoi financer un casse à plusieurs milliards : des bons au porteur gardés dans un coffre à New York.
Jaune [bis] : 27/01/2025
Jaune [bis] : 30/01/2025
Vert (44 min) : 03/01/2022
Un sombre diagnostic incite Leo à s'évader de prison avec l'aide d'un codétenu. Après d'émouvantes retrouvailles une fois dehors, il commence à préparer sa vengeance.
Etc...

et exemple Star Wars Rebels :

Hors saison :
11/08/2014 : La galaxie connaît une période sombre depuis que l'Empire galactique a étendu son emprise. (4)
1 : La Machine dans le Ghost (4 min) : 21/08/2024
Hera, Kanan et Chopper sont poursuivis par des TIE et doivent travailler ensemble pour tenter de leur échapper...
1 : La Machine dans le Ghost [bis] : 22/02/2025
2 : Attaque artisttique (4 min) : 23/08/2024
Sabine s'introduit dans une enceinte Impériale afin de créer une diversion pour ses amis...
Etc...

Saison 1 :
Cette saison commence sur Lothal avec la rencontre entre le jeune voleur, orphelin et sensible à la Force Ezra Bridger avec le groupe de rebelles de Lothal constitués de Kanan Jarrus, Hera Syndulla, Chopper, Garazed Orrelios et Sabine Wren. Ensemble, ils combattirent l'Empire Galactique et libérèrent un groupe d'esclaves Wookiees sur Kessel ; durant ce combat, Kanan dû révéler sa véritable identité de Jedi. L'Agent Alexsandr Kallus du Bureau de la Sécurité Impériale contacta alors Dark Vador qui envoya un Inquisiteur Pau'an les pourchasser.

Par la suite, le groupe du Ghost réalisa une mission sur Garel où ils rencontrèrent R2-D2 et C-3PO, ils furent piégés sur Stygeon Prime par le Grand Inquisiteur, infiltrèrent l'académie impériale de Lothal et partirent sur l'astéroïde PM-1203 pour récupérer de l'équipement pour le mystérieux Fulcrum, indicateur pour Hera. Ezra évolua ainsi au sein de sa "nouvelle famille", notamment avec Kanan qui s'occupa de sa formation de Jedi. Durant le Jour de l'Empire, Ezra découvrit que ses parents avaient été fait prisonniers par l'Empire grâce à son ami Tsoklo qu'il sauva. Il construit par la suite son sabre laser grâce à un cristal kyber trouvé au Temple Jedi de Lothal.

Finalement, les activités de ce groupe de rebelles gênant l'Empire, le Grand Moff Wilhuff Tarkin fut envoyé sur place pour les combattre. Il réussit, avec l'Inquisiteur et Kallus, à capturer Kanan et l'emmena dans le Système Mustafar où les rebelles de Lothal attaquèrent le vaisseau de Tarkin. Il libérèrent Kanan qui combattit et tua l'Inquisiteur, il s'enfuirent ensuite, aidés de la cellule rebelle de Fulcrum qui se révéla être Ahsoka Tano, ancienne Jedi et Padawan d'Anakin Skywalker. (15)
1x01 : Prémices d'une Rébelle (Partie 1) (25 min) : 16/08/2024
Un adolescent orphelin aide un groupe de rebelles.
1x01 : Prémices d'une Rébelle (Partie 1) [bis] : 22/02/2025
Etc...
1x15 : Galaxie en flammes (24 min) : 28/08/2025
L'équipage affronte de nombreux obstacles afin de libérer Kanan...

https://github.com/laurentbouleau/Exemple : Février 10 (22/02/2025)

Voilà !!!
22/02/2025, 21h31
Laurent_B_

Ok ou pas ?
Dites-moi ?

Et :
WordPress (https://www.b-lolo.fr/)===> Matomo : ok !
https://github.com/laurentbouleau/ =/=> Matomo : Comment faire ?

Merci vraiment;)

https://github.com/laurentbouleau/Exemple : Février 11 (23/02/2025)

Ok !

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
struct SequenceVisionnage
{
    SequenceVisionnage(const Episode& episode, const SequenceVisionnage& src) :
        m_episode{ episode }, m_titres{ src.m_titres }, m_streaming{ src.m_streaming },
        m_duree_en_seconde{ src.m_duree_en_seconde }, m_resume{ src.m_resume },
        m_DatesVisionnage{ src.m_DatesVisionnage }/*, m_numero{src.m_numero}*/, m_NumeroEpisode{ src.m_NumeroEpisode },
        m_espace3{ src.m_espace3 }, m_labelMinuteSingulier{ src.m_labelMinuteSingulier }, m_labelMinutePluriel{ src.m_labelMinutePluriel },
        m_keyColor{ src.m_keyColor }, m_valuesColor{ src.m_valuesColor }
    {
    };
    SequenceVisionnage(const Episode& episode, const InfosVisionnage& info_vis) :
        m_episode{ episode }, m_titres{ info_vis.m_titres }, m_streaming{ info_vis.m_streaming },
        m_duree_en_seconde{ info_vis.m_duree_en_seconde }, m_resume{ info_vis.m_resume },
        m_DatesVisionnage{ info_vis.m_DatesVisionnage }/*, m_numero{info_vis.m_numero}*/, m_NumeroEpisode{ info_vis.m_NumeroEpisode },
        //m_espace3{ info_vis.m_espace3 }, m_labelMinuteSingulier{ info_vis.m_labelMinuteSingulier }, m_labelMinutePluriel{ info_vis.m_labelMinutePluriel }, // Pour quoi ?
        m_keyColor{ info_vis.m_keyColor }, m_valuesColor{ info_vis.m_valuesColor }
    {
    };
    boolean operator==(const SequenceVisionnage& rhs) const { return this == &rhs; };

    std::wstring calcul_Duree_affichage(int numero_sequence) const;

    void AffichagePersonnaliser(AffichagePersonnalisation perso);

    void Print(int numero_sequence) const;

    std::wstring Print_Dates_de_visionnage() const;
    long Print_Titre_chiffre(long episode) const;

    std::wstring m_espace1;
    std::wstring m_labelHeureSingulier;
    std::wstring m_labelHeurePluriel;
    std::wstring m_espace2;
    std::wstring m_espace3;
    std::wstring m_labelMinuteSingulier;
    std::wstring m_labelMinutePluriel;
    std::vector<std::wstring>m_keyColor;
    std::wstring m_valuesColor;

    std::wstring m_streaming{ L"" };
    std::vector<std::wstring> m_titres;
    long m_duree_en_seconde{ -1 };
    std::vector<std::wstring> m_resume{};
    std::vector<DateRecord> m_DatesVisionnage{};

    long m_NumeroEpisode{-1};
    long m_duree { -1 };

private:
    const Episode& m_episode;
};

???

Code:

1
2
3
4
5
6
7
8
9
10
11
std::wstring SequenceVisionnage::calcul_Duree_affichage(int numero_sequence) const
{
    std::wstring duree_str;
    if (numero_sequence == 1)
    {
        long minutes = m_duree_en_seconde / 60;
        long secondes = m_duree_en_seconde % 60;
        duree_str += m_keyColor[1] + L" (" + m_valuesColor + std::to_wstring(minutes) + m_keyColor[1] + m_espace3 + (minutes <= 1 ? m_labelMinuteSingulier : m_labelMinutePluriel) + L')' + m_valuesColor;
    }
    return duree_str;
}

Ok !

23/02/2025, 22h41
Laurent_B_

https://github.com/laurentbouleau/Exemple : Février 12 (23/02/2025)

ok : class et struct :
film.h, film_serie.h et serie.h : avant !!!
Maintenant : utils.h !!!

Voilà !!!

Mes suggestions de modification de code sont basées sur la version "Février 10 (22/02/2025)" du dépôt Github.

Citation:

Citation:

Expliquez-nous, EN FRANçAIS, comment fonctionne l'affichage d'une séquence de visionnage, SVP.

Citation:

Exemple Kaleidoscope :
Jaune (47 min) : 03/01/2023
Dans le quartier des diamants, Leo et sa nouvelle équipe cherchent de quoi financer un casse à plusieurs milliards : des bons au porteur gardés dans un coffre à New York.
Jaune [bis] : 27/01/2025
Jaune [bis] : 30/01/2025
Vert (44 min) : 03/01/2022
Un sombre diagnostic incite Leo à s'évader de prison avec l'aide d'un codétenu. Après d'émouvantes retrouvailles une fois dehors, il commence à préparer sa vengeance.
Etc...

et exemple Star Wars Rebels :
Hors saison :
11/08/2014 : La galaxie connaît une période sombre depuis que l'Empire galactique a étendu son emprise. (4)
1 : La Machine dans le Ghost (4 min) : 21/08/2024
Hera, Kanan et Chopper sont poursuivis par des TIE et doivent travailler ensemble pour tenter de leur échapper...
1 : La Machine dans le Ghost [bis] : 22/02/2025
2 : Attaque artisttique (4 min) : 23/08/2024
Sabine s'introduit dans une enceinte Impériale afin de créer une diversion pour ses amis...
Etc...

Saison 1 :
Cette saison commence sur Lothal avec la rencontre entre le jeune voleur, orphelin et sensible à la Force Ezra Bridger avec le groupe de rebelles de Lothal constitués de Kanan Jarrus, Hera Syndulla, Chopper, Garazed Orrelios et Sabine Wren. Ensemble, ils combattirent l'Empire Galactique et libérèrent un groupe d'esclaves Wookiees sur Kessel ; durant ce combat, Kanan dû révéler sa véritable identité de Jedi. L'Agent Alexsandr Kallus du Bureau de la Sécurité Impériale contacta alors Dark Vador qui envoya un Inquisiteur Pau'an les pourchasser.

Par la suite, le groupe du Ghost réalisa une mission sur Garel où ils rencontrèrent R2-D2 et C-3PO, ils furent piégés sur Stygeon Prime par le Grand Inquisiteur, infiltrèrent l'académie impériale de Lothal et partirent sur l'astéroïde PM-1203 pour récupérer de l'équipement pour le mystérieux Fulcrum, indicateur pour Hera. Ezra évolua ainsi au sein de sa "nouvelle famille", notamment avec Kanan qui s'occupa de sa formation de Jedi. Durant le Jour de l'Empire, Ezra découvrit que ses parents avaient été fait prisonniers par l'Empire grâce à son ami Tsoklo qu'il sauva. Il construit par la suite son sabre laser grâce à un cristal kyber trouvé au Temple Jedi de Lothal.

Finalement, les activités de ce groupe de rebelles gênant l'Empire, le Grand Moff Wilhuff Tarkin fut envoyé sur place pour les combattre. Il réussit, avec l'Inquisiteur et Kallus, à capturer Kanan et l'emmena dans le Système Mustafar où les rebelles de Lothal attaquèrent le vaisseau de Tarkin. Il libérèrent Kanan qui combattit et tua l'Inquisiteur, il s'enfuirent ensuite, aidés de la cellule rebelle de Fulcrum qui se révéla être Ahsoka Tano, ancienne Jedi et Padawan d'Anakin Skywalker. (15)
1x01 : Prémices d'une Rébelle (Partie 1) (25 min) : 16/08/2024
Un adolescent orphelin aide un groupe de rebelles.
1x01 : Prémices d'une Rébelle (Partie 1) [bis] : 22/02/2025
Etc...
1x15 : Galaxie en flammes (24 min) : 28/08/2025
L'équipage affronte de nombreux obstacles afin de libérer Kanan...

Ce n'est pas une explication en français.
On va encore se compliquer la vie parce que vous n'expliquez pas votre démarche.

Mais cet exemple ("Star Wars Rebels") montre que le format du texte d'une "saison" pour un "Hors Saison" est très différent d'un format "Saison Classique".
Donc pourquoi tout mettre dans une seule fonction qui doit tout prendre en compte ?
Autant faire 2 fonctions distinctes pour avoir un code simple :
Dans "serie.h" :
Code:

1 2 3 4 5 6 7 8 9 struct Saison { public: ... void Print_Header(); void Print_Header_Hors_Saison(); void Print_Header_Saison(); ... }
et dans "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 void Saison::Print_Header_Hors_Saison() { std::wstring saison_str = m_keyColor[0] + L"Hors saison :" + m_valuesColor + L"\r\n"; wchar_t date_tab[15]; std::wstring date_str{}; if (m_date_diffusee_a_partir_de.first.tm_mday == -1 && m_date_diffusee_a_partir_de.first.tm_mon == -1) { std::wcsftime(date_tab, 15, L"%Y", &m_date_diffusee_a_partir_de.first); std::wstring date_tab_str = date_tab; date_str = date_tab_str; } else if (m_date_diffusee_a_partir_de.first.tm_mday == -1) { std::wcsftime(date_tab, 15, L"%m/%Y", &m_date_diffusee_a_partir_de.first); std::wstring date_tab_str = date_tab; date_str = date_tab_str.substr(0, 2) + m_keyColor[1] + L'/' + m_valuesColor + date_tab_str.substr(3, 4); } else { std::wcsftime(date_tab, 15, L"%d/%m/%Y", &m_date_diffusee_a_partir_de.first); std::wstring date_tab_str = date_tab; date_str = date_tab_str.substr(0, 2) + m_keyColor[1] + L'/' + m_valuesColor + date_tab_str.substr(3, 2) + m_keyColor[1] + L'/' + m_valuesColor + date_tab_str.substr(6, 4); } std::wstring media_str; if (m_date_diffusee_a_partir_de.second != L"") media_str = m_keyColor[0] + m_date_diffusee_a_partir_de.second + m_valuesColor + L' '; std::wstring titre_str; if (m_titres.size() != 0) { titre_str = m_keyColor[0] + m_titres[0] + m_valuesColor; if (m_titres.size() > 1) { titre_str += m_keyColor[1] + m_titres[1] + m_valuesColor + m_keyColor[0] + m_titres[2] + m_valuesColor; } titre_str += m_keyColor[1] + L" : " + m_valuesColor; } std::wstring resume_str; for (auto r : m_resume) resume_str += ((resume_str.size()==0) ? L"" : L"\r\n") + r; std::wstring nombre_episodes_str; if (m_nombre_episodes >= 0) nombre_episodes_str = m_keyColor[1] + L" (" + m_valuesColor + std::to_wstring(m_nombre_episodes) + m_keyColor[1] + L')' + m_valuesColor; std::wcout << saison_str << date_str << media_str << m_keyColor[1] + L" : " + m_valuesColor << titre_str << resume_str << nombre_episodes_str << L"\r\n"; } void Saison::Print_Header_Saison() { std::wstring saison_str = m_keyColor[0] + L"Saison " + m_keyColor[1] + std::to_wstring(m_numero) + m_keyColor[0] + L" :" + m_valuesColor + L"\r\n"; std::wstring titre_str; if (m_titres.size() != 0) { titre_str = m_keyColor[0] + m_titres[0] + m_valuesColor; if (m_titres.size() > 1) { titre_str += m_keyColor[1] + m_titres[1] + m_valuesColor + m_keyColor[0] + m_titres[2] + m_valuesColor; } titre_str += m_keyColor[1] + L" : " + m_valuesColor; } std::wstring resume_str; for (auto r : m_resume) resume_str += ((resume_str.size() == 0) ? L"" : L"\r\n") + r; std::wstring nombre_episodes_str; if (m_nombre_episodes >= 0) nombre_episodes_str = m_keyColor[1] + L" (" + m_valuesColor + std::to_wstring(m_nombre_episodes) + m_keyColor[1] + L')' + m_valuesColor; std::wcout << saison_str << m_valuesColor << titre_str << resume_str << nombre_episodes_str << L"\r\n"; } void Saison::Print_Header() { if(!m_hors_saison) Print_Header_Saison(); else Print_Header_Hors_Saison(); }
Code largement simplifiable mais sans explications en français, on risque de faire de fausses routes.
La présence de ce type de "fourche" de code montre qu'il faudrait 2 classes différentes ; "Saison" et "HorsSaison", pour distinguer correctement les cas.

Cet exemple ("Star Wars Rebels") montre aussi que le format du texte des épisodes pour un "Hors Saison" est très différent d'un format "Saison Classique" qu'on pourra gérer temporairement avec un paramètre supplémentaire à la fonction "Episode:: Print" :
Dans "serie.h" :
Code:

1 2 3 4 5 6 struct Episode { ... void Print(bool hors_saison); ... }
Dans "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 Episode::Print(bool hors_saison) { int numero_sequence = 1; for (const auto& sequence : m_liste_sequence_visionnages_ordonnee_chronologiquement) { sequence.Print(numero_sequence, hors_saison); numero_sequence++; } } ... void Saison::Print() { Print_Header(); for (auto& episode : m_liste_episodes) { episode.Print(m_hors_saison); } ... }
Il faut donc aussi modifier la signature de la fonction "SequenceVisionnage:: Print" pour qu'elle ait en paramètre l'information que c'est un affichage "Hors Saison" ou un affichage "Saison Classique".
Dans "serie.h" :
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 struct SequenceVisionnage { SequenceVisionnage(const Episode& episode, const SequenceVisionnage& src) : m_episode{ episode }, m_titres{ src.m_titres }, m_streaming{ src.m_streaming }, m_duree_en_seconde{ src.m_duree_en_seconde }, m_resume{ src.m_resume }, m_DatesVisionnage{ src.m_DatesVisionnage }/*, m_numero{src.m_numero}*/, m_NumeroEpisode{ src.m_NumeroEpisode }, m_keyColor{ src.m_keyColor }, m_valuesColor{ src.m_valuesColor } , m_labelMinuteSingulier{src.m_labelMinuteSingulier }, m_labelMinutePluriel{src.m_labelMinutePluriel }, m_espace3{src.m_espace3 } { }; SequenceVisionnage(const Episode& episode, const InfosVisionnage& info_vis) : m_episode{ episode }, m_titres{ info_vis.m_titres }, m_streaming{ info_vis.m_streaming }, m_duree_en_seconde{ info_vis.m_duree_en_seconde }, m_resume{ info_vis.m_resume }, m_DatesVisionnage{ info_vis.m_DatesVisionnage }/*, m_numero{info_vis.m_numero}*/, m_NumeroEpisode{ info_vis.m_NumeroEpisode }, m_keyColor{ info_vis.m_keyColor }, m_valuesColor{ info_vis.m_valuesColor } //, m_labelMinuteSingulier{ info_vis.m_labelMinuteSingulier }, m_labelMinutePluriel{ info_vis.m_labelMinutePluriel } { }; boolean operator==(const SequenceVisionnage& rhs) const { return this == &rhs; }; std::wstring calcul_Duree_affichage(int numero_sequence) const; void AffichagePersonnaliser(AffichagePersonnalisation perso); void Print(int numero_sequence, bool hors_saison) const; std::wstring Print_Dates_de_visionnage() const; long Print_Titre_chiffre(long episode) const;//Cette fonction n'a aucun putain de sens. ... }
Dans "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 void SequenceVisionnage::Print(int numero_sequence, bool hors_saison) const { std::wstring numero_str; std::wstring titre_str; std::wstring duree_str; if (hors_saison) { numero_str = std::to_wstring(m_episode.m_numero) + m_keyColor[1] + L" : " + m_valuesColor; } else { numero_str = std::to_wstring(m_episode.m_saison.m_numero) + m_keyColor[1] + L'x' + m_valuesColor + (m_episode.m_numero < 10 ? L"0" : L"") + std::to_wstring(m_episode.m_numero) + m_keyColor[1] + L" : " + m_valuesColor; } assert((m_titres.size() > 0 && m_titres[0].size() > 0)|| numero_sequence > 1); std::vector<std::wstring> titres = m_titres; if (m_titres.size()==0 || (m_titres.size()==1 && m_titres[0].size()==0)) { titres = m_episode.m_liste_sequence_visionnages_ordonnee_chronologiquement[0].m_titres; } std::vector<std::wstring> titre_prefixe_list{ m_keyColor[1] , m_valuesColor }; std::wstring titre_prefixe = titre_prefixe_list[0]; unsigned int titre_prefixe_index = 0; for (auto& titre : titres) { titre_str += titre_prefixe_list[titre_prefixe_index] + titre; titre_prefixe_index = ((titre_prefixe_index + 1) % titre_prefixe_list.size()); } if (titre_str.size() > 0) titre_str += m_valuesColor; duree_str = calcul_Duree_affichage(numero_sequence); std::wstring dates_str = m_keyColor[1] + L" : " + m_valuesColor + Print_Dates_de_visionnage(); std::wstring resume_str; if (numero_sequence == 1) { resume_str += L"\r\n"; for (auto r : m_resume) resume_str += r; } std::wcout << numero_str << titre_str << duree_str << dates_str << resume_str << L"\r\n"; }
On utilise une variable "titres" pour gérer le cas où le fichier d'une séquence de visionnage ne contient pas le(s) titre(s) de l'épisode. (cf. formats de fichier MERDIQUES)
On utilise "intelligemment" une boucle pour générer le formatage du titre.
Et on a simplifié l'utilisation de la fonction "SequenceVisionnage::calcul_Duree_affichage"
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 std::wstring SequenceVisionnage::calcul_Duree_affichage(int numero_sequence) const { std::wstring duree_str; if (numero_sequence == 1) { long minutes = (m_duree_en_seconde/* % (60 * 60)*/) / 60; long secondes = m_duree_en_seconde % 60; duree_str += m_keyColor[1] + L" (" + m_valuesColor + std::to_wstring(minutes) + m_keyColor[1] + L" " + (minutes <= 1 ? m_labelMinuteSingulier : m_labelMinutePluriel) + L')' + m_valuesColor; } else { duree_str += L' ' + m_keyColor[1] + L'[' + m_valuesColor + L"bis" + m_keyColor[1] + L']' + m_valuesColor; } return duree_str; }
J'ai remplacé "m_espace3" par un 'L" "' parce que la gestion de ces conneries de mises en forme graphique dans les classes métier commencent à me courir sur le haricot.

"Exemple Kaleidoscope :" n'est pas vérifiable avec le code actuel dans "Exemple.cpp".
Je vous ai déjà donné plusieurs fois les moyens de tester plusieurs cas dans une exécution.
Vous vous obstinez avec votre bricolage. Je ne vais donc pas m'enquiquiner à faire du bricolage sur votre bricolage pour texter le résultat avec mes modifications sur cet exemple.
Je ne ferais pas d'efforts tant que vous ne faites pas l'effort de suivre nos recommandations.

24/02/2025, 01h59
bacelar

Citation:

Février 11 (23/02/2025)

Ok pour les modifications, qui sont minimes.

Citation:

Février 12 (23/02/2025)

C'est n'importe quoi.

Vous fourrez tout dans "utils.h".
Ça n'a aucune logique.

Vous devez enlever des classes "Serie", "Saison", "Episode", "Film", "InfosVisionnage" et "SequenceVisionnage" tous ce qui est en relation avec de l'affichage.
C'est des classes métier.
Donc laissez les classes dans les anciens fichiers d'en-tête ("serie.h", etc...), c'est pas totalement correcte (on devrait avoir un fichier d'en-tête par classe), mais on va continuer avec.
C'est le code d'affichage que vous devrait transférer des xxx.h/xxx.cpp vers utils.h/utils.cpp, pas les classes.
24/02/2025, 23h59
Laurent_B_

https://github.com/laurentbouleau/Exemple : Février 13 (24/02/2025)
25/02/2025, 11h25
bacelar
Ok, vous êtes "satisfait" du résultat ?

Si oui, il faut maintenant que vous nettoyez vos classes métier des machins qui n'ont rien à y faire, comme l'affichage.

Commencez par ajouter la déclaration d'une fonction libre comme la ligne suivante dans "serie.h" :

Code:

std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out);

Et les lignes suivantes dans "serie.cpp":
Code:

1 2 3 4 5 std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out) { serie.Print(); return out; }
Remplacer dans "Exemple.cpp" la ligne :

Code:

serie.Print();

Par:

Code:

PrintSerieOnOutstream(serie, std::wcout);

Après, il faudra remplacer "serie.Print()" par un code "externe" à la classe.
Puis décommenter les "const" pour que la fonction libre est une signature "correcte".

Et ainsi de suite avec les autres classes métier

1 pièce(s) jointe(s)

https://github.com/laurentbouleau/Exemple : Février 14 (25/02/2025)

Citation:

Si oui, il faut maintenant que vous nettoyez vos classes métier des machins qui n'ont rien à y faire, comme l'affichage.

Commencez par ajouter la déclaration d'une fonction libre comme la ligne suivante dans "serie.h" :

Après !

Mais :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 InfosVisionnage::InfosVisionnage(const Saison& saison, fs::path const& m_cheminFichier) : m_saison{ saison } { ... std::wstring titres = titles_match[2]; // // wchar_t sp = L' '; if (titres[0] == sp) titres = titres.substr(1); // // m_titres = extraire_Titres_Depuis_UneLigne(titres); ... }
Ok ou pas ?

Attention : Exemple : Zero Day.[2025 Netflix].Mini-série
Pièce jointe 664898
et :
https://www.netflix.com/fr/title/81598435 !!!
1x6 : 6 épisodes !!!

Exemple : Cowboy Bebop.[2021 Netflix]
https://www.netflix.com/fr/title/80207033
1x01, 1x02, ..., 1x10 : 10 épisodes !!!

Merci;)

27/02/2025, 00h58
Laurent_B_

1 pièce(s) jointe(s)

Citation:

Gateway Timeout

The gateway did not receive a timely response from the upstream server or application.
Apache/2.4.62 (Debian) Server at www.developpez.net Port 443

Erreur soir !!!

https://github.com/laurentbouleau/Exemple : Février 15 (27/02/2025)

Ni std::wostream& Serie:: PrintSerieOnOutstream(const Serie& serie, std::wostream& out)
Ni std::wostream& PrintSerieOnOutstream(const Serie& serie, std::wostream& out)

Pas compris !!!

Pièce jointe 664921
???

Et :

Citation:

Puis décommenter les "const" pour que la fonction libre est une signature "correcte".

Et ainsi de suite avec les autres classes métier

métier : ou qu'il est ???
Avant, les pages : longtemps !!!

Merci:aie::aie::aie:

Citation:

https://github.com/laurentbouleau/Exemple : Février 14 (25/02/2025)

Citation:

Si oui, il faut maintenant que vous nettoyez vos classes métier des machins qui n'ont rien à y faire, comme l'affichage.

Commencez par ajouter la déclaration d'une fonction libre comme la ligne suivante dans "serie.h" :

Après !

C'est toujours la même histoire.
Vous voulez faire à votre tête en passant avoir assez de recul pour choisir les priorités, moralité, on aborde des trucs d'une effroyable complexité inutilement, car vous ne voulez pas suivre nos conseils.
Heureusement que vous êtes opiniâtre, mais vous ne vous simplifiez pas la vie.
Citation:
Mais :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 InfosVisionnage::InfosVisionnage(const Saison& saison, fs::path const& m_cheminFichier) : m_saison{ saison } { ... std::wstring titres = titles_match[2]; // // wchar_t sp = L' '; if (titres[0] == sp) titres = titres.substr(1); // // m_titres = extraire_Titres_Depuis_UneLigne(titres); ... }
Ok ou pas ?
Pas trop OK, car c'est extrêmement compliqué et bavard pour ce que c'est, pas "générique/généralisable" en plus, et vous avez déjà abordé ce qu'il faut pour faire un code bien plus lisible et expressif.

L'historique des fichiers dans GitHub montre que dans la version "Août 27", vous avez ajouté un bon nombre de fonction dans "utils.h/.cpp", en particulier la fonction "rtrim" (très mal "inséré" au demeurant car utilisation erronée du mot clé "inline").
L'action de supprimer des "espaces" à la fin ou au début d'une chaine de caractères se nomme "Trimmer" la chaine (du verbe anglais 'to trim" : réduire, minimiser).
En ayant déjà cette fonction dans votre base de code, cela montre que vous avez déjà eu le problème, ou un approchant, et que vous y aviez répondu avec une solution "générique".
Regardez toutes les fois où la fonction "rtrim" est appelée dans les fonctions contenues dans le fichier "utils.cpp". (6 fois)

Si on utilise notre ami Google avec la requête "C++ trim string", la première réponse est :
https://stackoverflow.com/questions/...im-a-stdstring
Ce qui donne une très bonne base à l'implémentation de toute une collection de fonctions utilitaires qu'on peut mettre directement dans "utils.h" (on a juste à changer les std::string en std::wstring et les unsigned char en wchar_t):
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 #include <algorithm> #include <cctype> #include <locale> ... // trim from start (in place) inline void ltrim(std::wstring& s) { s.erase(s.begin(), std::find_if(s.begin(), s.end(), [](wchar_t ch) { return !std::isspace(ch); })); } // trim from end (in place) inline void rtrim(std::wstring& s) { s.erase(std::find_if(s.rbegin(), s.rend(), [](wchar_t ch) { return !std::isspace(ch); }).base(), s.end()); } // trim from both ends (in place) inline void trim(std::wstring& s) { rtrim(s); ltrim(s); } // trim from start (copying) inline std::wstring ltrim_copy(std::wstring s) { ltrim(s); return s; } // trim from end (copying) inline std::wstring rtrim_copy(std::wstring s) { rtrim(s); return s; } // trim from both ends (copying) inline std::wstring trim_copy(std::wstring s) { trim(s); return s; }
Après cela, vous remplacez ce pavé illisible :
Citation:
Code:

1 2 3 wchar_t sp = L' '; if (titres[0] == sp) titres = titres.substr(1);
En :

Code:

trim(titres);

(Code plus lisible et qui indique ce qu'il fait)

Citation:

Attention : Exemple : Zero Day.[2025 Netflix].Mini-série
Pièce jointe 664898
et :
https://www.netflix.com/fr/title/81598435 !!!
1x6 : 6 épisodes !!!

Exemple : Cowboy Bebop.[2021 Netflix]
https://www.netflix.com/fr/title/80207033
1x01, 1x02, ..., 1x10 : 10 épisodes !!!

Soyez plus explicite, SVP.
Si je comprends "bien", vous voulez que les 0 en début de numéro d'épisode ne servent que de "padding" pour que tous les numéros d'une saison est la même longueur ?
Normalement, vous devriez être capable de le faire vous-même.
Une solution à l'arrache :

Code:

numero_str = std::to_wstring(m_episode.m_saison.m_numero) + m_keyColor[1] + L'x' + m_valuesColor + ((m_episode.m_numero < 10 && m_episode.m_saison.m_nombre_episodes>10) ? L"0" : L"") + std::to_wstring(m_episode.m_numero) + m_keyColor[1] + L" : " + m_valuesColor;

Vous devriez être capable de faire un code bien plus propre si vous avez une vue plus claire de ce que vous voulez.
std::string et les stream/flux du C++ ont tout ce qu'il faut pour facilement faire du padding avec des 0 à gauche.

Citation:

Ni std::wostream& Serie:: PrintSerieOnOutstream(const Serie& serie, std::wostream& out)
Ni std::wostream& PrintSerieOnOutstream(const Serie& serie, std::wostream& out)

J'ai écrit "ajouter la déclaration d'une fonction libre", il n'y a donc aucune ambiguïté :

Code:

std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out)

Et elle doit être déclarée à l'extérieur de la déclaration de la classe "Serie" (et de toutes les autres classes qui devraient être ailleurs):
Code:

1 2 3 4 5 6 7 class Serie { public: ... }; std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out);
Il faut laisser le "const" en commentaire tant que vous avez ces cochonneries de fonctions d'affichage dans cette classe. (Qui ont une const-correctness à chier, vous devriez déjà être capable de la corrigée, normalement)

Citation:

métier : ou qu'il est ???

Le métier, c'est une couche d'abstraction.
Classiquement, une application est découpée en couches superposées pour facilement pouvoir changer des aspects différents de l'application.
Un découpage classique est :
- Données
- Métier
- IHM (Interface Homme-Machine)
Avec ce découpage, on peut facilement changer comment l'application s'affiche, en ne modifiant que la couche IHM.
On peut aussi facilement changer comment les données sont stockées, en ne changeant que la couche Données.
La couche métier est la partie du code qui n'a pas à changer si on change l'IHM ou comment les données sont stockées.

Actuellement, dans votre code, tout est mélangé.
Le fait que tout est mélangé fait que coder simplement est compliqué.
La chose la plus simple à faire actuellement avec votre code, c'est de séparer la partie IHM du reste (Données + Métier).
Quand j'indique Métier, c'est actuellement "Données + Métier".
(La séparation entre Données et Métier peut encore attendre car la fusion des 2 ne complexifie par encore le code.)
Donc le "Métier", c'est toutes vos classes où vous avez supprimé ce qui ne sert qu'à l'affichage.

Citation:

Avant, les pages : longtemps !!!

????????????

28/02/2025, 22h50
Laurent_B_

Citation:

C'est toujours la même histoire.
Vous voulez faire à votre tête en passant avoir assez de recul pour choisir les priorités, moralité, on aborde des trucs d'une effroyable complexité inutilement, car vous ne voulez pas suivre nos conseils.
Heureusement que vous êtes opiniâtre, mais vous ne vous simplifiez pas la vie.

...
En :
Code : Sélectionner tout - Visualiser dans une fenêtre à part

trim(titres);

Ok !

https://github.com/laurentbouleau/Exemple : Février 16 (28/02/2025)

Merci;)

Citation:
Code:

1 2 3 4 5 6 inline void ltrim(std::wstring& s); inline void rtrim(std::wstring& s); inline void trim(std::wstring& s); inline std::wstring ltrim_copy(std::wstring s); inline std::wstring rtrim_copy(std::wstring s); inline std::wstring trim_copy(std::wstring s);
Attention, vous utilisez très mal le mot clé "inline".

Soit vous utilisez "inline" est le corps des fonctions (définition) doit être aussi dans le .h (cela indique des possibilités d'optimisation au compilateur).(comme le code que je vous ai "donné".)
Soit vous n'utilisez pas "inline", aussi bien dans le .h que dans le .cpp.
Citation:
Code:

1 2 3 4 5 6 7 8 9 10 extern inline void ltrim(std::wstring& s); extern inline void rtrim(std::wstring& s); extern inline void trim(std::wstring& s); //extern std::wostream& Serie::PrintSerieOnOutstream(const Serie& serie, std::wostream& out); extern const std::vector<std::wstring> Genre; extern const std::vector<std::wstring> Sur; extern const std::vector<std::wstring> Sous_Genre; extern const std::vector<std::wstring> Nationalite;
C'est quoi ces lignes avec "extern" ???
Dégagez-moi toutes ces lignes qui n'ont rien à foutre dans "serie.h".

Vous devez juste ajouter un '#include "utils.h"' dans les .cpp où vous utilisez les fonctions déclarées dans "utils.h", par exemple dans serie.cpp.

02/03/2025, 22h28
Laurent_B_
Citation:

Attention, vous utilisez très mal le mot clé "inline".

Ah bon ?

Citation:

Soit vous utilisez "inline" est le corps des fonctions (définition) doit être aussi dans le .h (cela indique des possibilités d'optimisation au compilateur).(comme le code que je vous ai "donné".)

Rien compris !!!

Citation:

Soit vous n'utilisez pas "inline", aussi bien dans le .h que dans le .cpp.

???

Citation:

C'est quoi ces lignes avec "extern" ???
Dégagez-moi toutes ces lignes qui n'ont rien à foutre dans "serie.h".

Vous devez juste ajouter un '#include "utils.h"' dans les .cpp où vous utilisez les fonctions déclarées dans "utils.h", par exemple dans serie.cpp.

series.cpp :
Code:

1 2 3 4 #include "pch.h" #include "film_serie.h" #include "utils.h" ...
Ben oui et...

Citation:

Erreur des outils Éditeur de liens LNK2019

https://github.com/laurentbouleau/Exemple : Mars 01 (02/03/2025)
03/03/2025, 06h32
bacelar

Supprimez tous ces "extern" à la con.
Utilisez correctement 'inline".

Citation:

Supprimez tous ces "extern" à la con.
Utilisez correctement 'inline".

https://github.com/laurentbouleau/Exemple : Mars 02 (03/03/2025)

serie.cpp :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 #include "pch.h" #include "film_serie.h" #include "utils.h" ... using namespace std; namespace fs = std::filesystem; using DateVisionnage = DateRecord; ... // ??? inline void ltrim(std::wstring& s); inline void rtrim(std::wstring& s); inline void trim(std::wstring& s); inline std::wstring ltrim_copy(std::wstring s); inline std::wstring rtrim_copy(std::wstring s); inline std::wstring trim_copy(std::wstring s); ...
Rien fonctionné !!!

Citation:
serie.cpp :
Code:

1 2 3 4 5 6 inline void ltrim(std::wstring& s); inline void rtrim(std::wstring& s); inline void trim(std::wstring& s); inline std::wstring ltrim_copy(std::wstring s); inline std::wstring rtrim_copy(std::wstring s); inline std::wstring trim_copy(std::wstring s);
Cela n'a rien à foutre dans serie.cpp.

Et dans utils.h/cpp :
Soit vous utilisez "inline", alors vous devez tout mettre dans le .h.
Sinon, déclaration dans le .h et définition dans le .cpp.
Quand il y a "inline", vous devez combiné déclaration et définition dans le .h. (RTFM)

03/03/2025, 22h49
Laurent_B_

Citation:

Cela n'a rien à foutre dans serie.cpp.

Ok !

Citation:

Soit vous utilisez "inline", alors vous devez tout mettre dans le .h.

Ok !

Citation:

Quand il y a "inline", vous devez combiné déclaration et définition dans le .h. (RTFM)

RTFM : Comment fonctionne ?

https://github.com/laurentbouleau/Exemple : Mars 03 (03/03/2025)

Merci;)

Après :

Citation:

Soyez plus explicite, SVP.
Si je comprends "bien", vous voulez que les 0 en début de numéro d'épisode ne servent que de "padding" pour que tous les numéros d'une saison est la même longueur ?
Normalement, vous devriez être capable de le faire vous-même.
...
04/03/2025, 01h15
bacelar

Citation:

RTFM : Comment fonctionne ?

RTFM : Read The Fucking Manual.

Vous avez insisté à faire n'importe quoi (inline dans un cpp, faire des extern, etc...) au lieu de prendre un peu de temps à comprendre la documentation sur "inline". Ce n'est pas efficace.

Maintenant, vous pouvez suivre nos conseils ?
Pouvez-vous verbaliser vos problèmes en français, pas en "code" ?
04/03/2025, 02h42
Laurent_B_

Ah ! Très bien ! Mais, je dors !!! Plus tard !

https://github.com/laurentbouleau/Exemple : Mars 04 (04/03/2025)

Citation:

Si je comprends "bien", vous voulez que les 0 en début de numéro d'épisode ne servent que de "padding" pour que tous les numéros d'une saison est la même longueur ?
Normalement, vous devriez être capable de le faire vous-même.
Une solution à l'arrache :

Ok !

Code:

numero_str = std::to_wstring(m_episode.m_saison.m_numero) + m_keyColor[1] + L'x' + m_valuesColor + ((m_episode.m_numero < 10 && m_episode.m_saison.m_nombre_episodes>10) ? L"0" : L"") + std::to_wstring(m_episode.m_numero) + m_keyColor[1] + L" : " + m_valuesColor;

Ok !

Mais : serie.cpp :
Code:

1 2 3 4 5 6 7 8 std::wstring Serie::calcul_Duree_affichage() const { std::wstring duree_str; long minutes = (m_duree % (60 * 60)) / 60; long secondes = m_duree % 60; duree_str = L' ' + std::to_wstring(minutes) + m_keyColor[0] + m_espace3 + (minutes <= 1 ? m_labelMinuteSingulier : m_labelMinutePluriel) + m_valuesColor; return duree_str; }
Kaleidoscope et Love, Death + Robots : 0min !!!
Et : sans épisodes !!! (en mince...:?:roll::aie:)

04/03/2025, 15h50
bacelar

Citation:

std::wstring Serie::calcul_Duree_affichage() const

Vous faites toujours les mêmes erreurs.
Utilisez le débugueur.

Vous aviez le même problème dans la fonction "SequenceVisionnage::calcul_Duree_affichage(int numero_sequence) const".

Comparez les 2 codes.

Vous verrez que ce " % (60 * 60)" n' aucun sens si "m_duree" de serie fonctionne comme "m_duree_en_second" de SequenceVisionnage.
"(60 * 60)", c'est pour convertir des heures en secondes.

La similitude entre les 2 fonctions montre, encore une fois, qu'enlever le code d'affichage de ces classes métier permettrait une grande factorisation du code.
04/03/2025, 22h27
Laurent_B_

https://github.com/laurentbouleau/Exemple : Mars 05 (04/03/2025)
Bon, ils fonctionnent !

Pourquoi :
std::wstring Serie::calcul_Duree_affichage() const : m_duree
et
std::wstring SequenceVisionnage::calcul_Duree_affichage(int numero_sequence) const : m_duree_en_seconde
???

Merci:roll:
05/03/2025, 07h59
bacelar

Citation:

Pourquoi :
std::wstring Serie::calcul_Duree_affichage() const : m_duree
et
std::wstring SequenceVisionnage::calcul_Duree_affichage(int numero_sequence) const : m_duree_en_seconde

Ne voyez-vous pas que c'est quasiment le même code et qu'il serait plus "malin" de factoriser le code ?
Factoriser le code : fusionner les parties pour en faire un seul bloc utilisable à plusieurs endroits, en en faisant une fonction libre, par exemple : "std::wstring miseEnFormeDuree(long dureeEnSeconde, xyz)"

EDIT :
m_duree Versus m_duree_en_seconde ???

Bin c'est vous qui n'avez pas suivi les mêmes règles de nommage dans ces 2 classes.
Il faudrait donc mieux renommer le champ m_duree de la classe Serie en m_duree_en_seconde (utilisez la fonctionnalité de Visual Studio pour le faire, pas à la main, SVP) pour que cela soit plus explicite.
05/03/2025, 15h10
Laurent_B_

https://github.com/laurentbouleau/Exemple : Mars 06 (06/03/2025)

Citation:

Ne voyez-vous pas que c'est quasiment le même code et qu'il serait plus "malin" de factoriser le code ?
Factoriser le code : fusionner les parties pour en faire un seul bloc utilisable à plusieurs endroits, en en faisant une fonction libre, par exemple : "std::wstring miseEnFormeDuree(long dureeEnSeconde, xyz)"

Ok !
05/03/2025, 16h08
bacelar

Le nom et l'ordre des paramètres de la nouvelle fonction sont perfectibles, mais vous avez compris la logique. très bien. ;)

1 pièce(s) jointe(s)

https://github.com/laurentbouleau/Exemple : Mars 07 (06/03/2025)

serie.h

Code:

1
2
3
4
5
6
7
8
struct InfosVisionnage
{
...
    long m_NumeroSaison{};
    long m_NumeroEpisode{};
    bool m_Chiffres_ou_pas{ true };
...
};

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
InfosVisionnage::InfosVisionnage(const Saison& saison, fs::path const& m_cheminFichier) : m_saison{ saison }
{ 
...
    if (file_content.size() > 0)
    {
        const std::wregex numeroPlusTitres_rg{ L"(?:(\\d)+\\.)?(.*)" };
        std::wsmatch titles_match;
        std::regex_match(file_content[0], titles_match, numeroPlusTitres_rg);

        if (titles_match[1].matched)
        {
            int numeroDansFichier = std::stoi(titles_match[1]);
            if (numeroDansFichier != m_NumeroEpisode)
            {
                std::wstring message = L"Le fichier " + nomFichier + L" contient un numéro d'épisode différent de celui dans son nom.";
                OutputDebugStringW(message.c_str());
            }
            m_Chiffres_ou_pas = true;
        }
        else
        {
            m_Chiffres_ou_pas = false;
        }
...
}

serie.h :

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
struct SequenceVisionnage
{
    SequenceVisionnage(const Episode& episode, const SequenceVisionnage& src) :
        m_episode{ episode }, m_titres{ src.m_titres }, m_streaming{ src.m_streaming },
        m_duree_en_seconde{ src.m_duree_en_seconde }, m_resume{ src.m_resume },
        m_DatesVisionnage{ src.m_DatesVisionnage }, /*m_numero{src.m_numero},*/ m_NumeroEpisode{src.m_NumeroEpisode},
        m_keyColor{ src.m_keyColor }, m_valuesColor{ src.m_valuesColor },
        m_labelMinuteSingulier{ src.m_labelMinuteSingulier }, m_labelMinutePluriel{ src.m_labelMinutePluriel }, m_espace3{ src.m_espace3 }
        , m_Chiffres_ou_pas{src.m_Chiffres_ou_pas }
    {
    };
    SequenceVisionnage(const Episode& episode, const InfosVisionnage& info_vis) :
        m_episode{ episode }, m_titres{ info_vis.m_titres }, m_streaming{ info_vis.m_streaming },
        m_duree_en_seconde{ info_vis.m_duree_en_seconde }, m_resume{ info_vis.m_resume },
        m_DatesVisionnage{ info_vis.m_DatesVisionnage }, /*m_numero{info_vis.m_numero}, */m_NumeroEpisode{info_vis.m_NumeroEpisode},
        m_keyColor{ info_vis.m_keyColor }, m_valuesColor{ info_vis.m_valuesColor }
        //, m_labelMinuteSingulier{ info_vis.m_labelMinuteSingulier }, m_labelMinutePluriel{ info_vis.m_labelMinutePluriel }
        , m_Chiffres_ou_pas{ info_vis.m_Chiffres_ou_pas }
    {
    };
...
    long m_NumeroEpisode{ -1 };
    bool m_Chiffres_ou_pas{ true };
...
};

serie.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void SequenceVisionnage::Print(int numero_sequence, bool hors_saison, bool chiffres_ou_pas) const
{
    std::wstring numero_str;
    std::wstring titre_str;
    std::wstring duree_str;

    if (hors_saison /* && chiffres_ou_pas*/) // Je sais pas !
    {
        if(chiffres_ou_pas) // de même !
            numero_str = std::to_wstring(m_episode.m_numero) + m_keyColor[1] + L" : " + m_valuesColor;
    }
    else
    {
        if (chiffres_ou_pas) // de même !
            numero_str = std::to_wstring(m_episode.m_saison.m_numero) + m_keyColor[1] + L'x' + m_valuesColor + ((m_episode.m_numero < 10 && m_episode.m_saison.m_nombre_episodes>10) ? L"0" : L"") + std::to_wstring(m_episode.m_numero) + m_keyColor[1] + L" : " + m_valuesColor;
    }
...
}

serie.cpp :

Code:

1
2
3
4
5
6
7
8
9
10
11
void Episode::Print(bool hors_saison)
{
    int numero_sequence = 1;

    for (const auto& sequence : m_liste_sequence_visionnages_ordonnee_chronologiquement)
    {
        //sequence.Print(numero_sequence, hors_saison);
        sequence.Print(numero_sequence, hors_saison, m_liste_sequence_visionnages_ordonnee_chronologiquement[0].m_Chiffres_ou_pas);
        numero_sequence++;
    }
}

Ok ou pas ! La question ? Pouvez-vous corriger ?

------------------------------------------
Et :

Code:

1
2
3
    std::wstring m_espace1{ L" " };
    std::wstring m_espace2{ L" " };
    std::wstring m_espace3{ L" " };

en
std::vector<std::wstring>space {L" ", L" ", L" "} : ok ou pas ?
------------------------------------------

Et :
Matomo <===> https://www.b-lolo.fr : ok !
Matomo <=/=> https://github.com/laurentbouleau

Pièce jointe 665256
Gratuit : Matomo
Par contre : https://www.b-lolo.fr (WordPress : https://wordpress.com/)

Merci;):?:roll:

07/03/2025, 12h14
bacelar

Je ne suis pas convaincu par votre ajout.

Le nom n'est pas clair, surtout pour un booléen.
Une variable booléenne devrait être un truc de la forme "Estxxx". Votre "Chiffres_ou_pas", c'est "EstChiffre" ou "EstPasChiffre" ?
C'est pas clair.
En plus, c'est encore lié à un machin d'affichage, ce qui n'a rien à foutre dans une classe métier.
S'il y a une logique à garder cette information dans l'objet, il doit avoir un sens bien plus intrinsèque que juste 'j'affiche un truc, ou j'affiche pas un truc".
Une information comme, "le numéro d'épisode est fiable" ou "le numéro d'épisode n'est pas fiable".
Pourquoi ne pas stocker le numéro trouvé dans le nom du fichier ET le numéro trouvé dans le contenu du fichier ?
Ainsi, la classe métier a les informations "nécessaires".
C'est à l'affichage que la manière d'afficher le fait que ces 2 données ne sont pas cohérentes incombe. Mais ne polluez pas votre classe SequenceVisionnage avec des champs aux noms cryptiques.

Et si on décortique la logique implémentée, elle n'a pas de sens "évident".
On affiche le numéro de l'épisode quand on affiche une séquence de visionnage si :
Le contenu du fichier est vide
ou
si la première ligne du fichier respecte L"(?:(\\d)+\\.)?(.*)"
Donc, on affiche le numéro d'épisode même si le numéro dans le nom du fichier et celui dans la première ligne du fichier ne correspondent pas.
C'est pas logique pour moi.
EXPLIQUEZ EN FRANCAIS LA "LOGIQUE" DE VOTRE AFFICHAGE !!!
Pour moi, elle n'a pas de sens.
Et ça encore moins de sens quand, en fait, ce n'est pas dans le cadre d'une séquence de visionnage que la décision est prise, mais dans le cadre de la première séquence de visionnage de l'épisode qui est propagé à l'ensemble des séquences de cet épisode.

C'est peut-être clair dans votre tête mais ce n'est clairement pas le cas dans le code que vous avez pondu.

En résumé, vous complexifiez atrocement votre code pour un bidule abscons qui semble un détail "à la con".
C'est le genre de chose qu'on fait, peut-être, mais à la toute fin du projet.
07/03/2025, 13h32
Laurent_B_

1 pièce(s) jointe(s)

Citation:

Je ne suis pas convaincu par votre ajout.

Le nom n'est pas clair, surtout pour un booléen.
Une variable booléenne devrait être un truc de la forme "Estxxx". Votre "Chiffres_ou_pas", c'est "EstChiffre" ou "EstPasChiffre" ?
...
En résumé, vous complexifiez atrocement votre code pour un bidule abscons qui semble un détail "à la con".
C'est le genre de chose qu'on fait, peut-être, mais à la toute fin du projet.

Ah !!! Ok !!! Après:aie:

Mais :
https://github.com/laurentbouleau/Exemple : Mars 08 (07/03/2025)

Pièce jointe 665275

===> std::vector<std::wstring>m_spaces{ L" ", L" ", L" " };
Comme faire ?

Merci:aie::weird:

Citation:

Citation:

Je ne suis pas convaincu par votre ajout.

Le nom n'est pas clair, surtout pour un booléen.
Une variable booléenne devrait être un truc de la forme "Estxxx". Votre "Chiffres_ou_pas", c'est "EstChiffre" ou "EstPasChiffre" ?
...
En résumé, vous complexifiez atrocement votre code pour un bidule abscons qui semble un détail "à la con".
C'est le genre de chose qu'on fait, peut-être, mais à la toute fin du projet.

Ah !!! Ok !!! Après

Vous ne voyez pas l'absurdité de la situation ?
Vous construisez des usines à gaz sur usine à gaz jusqu'à ce que cela vous pête à la figure.
Non, vous vous plantez complètement de priorité.
Il faut appliquer le principe KISS : Keep It Simple and Stupid.
Si vous ne pouvez pas correctement me justifier cette cochonnerie, dégagez-moi cette "merde", maintenant.

Citation:

std::vector<std::wstring>space {L" ", L" ", L" "} : ok ou pas ?

Très bof.
C'est mieux que 3 champs à la con.
Mais le fait qu'il doit avoir 3 et uniquement 3 machins "spaces", c'est perdu car un std::vector, c'est un nombre arbitraire de valeurs.
Comme utilisé dans le constructeur de "AffichagePersonnalisation", un "std::tuple<std::wstring, std::wstring, std::wstring>", c'est bien plus "safe" et "correct".
Citation:
Code:

1 2 3 4 5 6 7 8 9 class AffichagePersonnalisation { public: AffichagePersonnalisation() {}; /*AffichagePersonnalisation(std::tuple<std::wstring, std::wstring, std::wstring> espaces, std::pair<std::wstring, std::wstring> labelsHeure, std::pair<std::wstring, std::wstring> labelsMinute) : m_espace1{ std::get<0>(espaces) }, m_espace2{ std::get<1>(espaces) }, m_espace3{ std::get<2>(espaces) }, m_labelsHeure{ labelsHeure }, m_labelsMinute{ labelsMinute } { };*/ AffichagePersonnalisation(std::vector<std::wstring>m_spaces, std::pair<std::wstring, std::wstring> labelsHeure, std::pair<std::wstring, std::wstring> labelsMinute); ...
Vous êtes bien gentil avec cette déclaration, mais elle est où l'implémentation ?

Citation:

===> std::vector<std::wstring>m_spaces{ L" ", L" ", L" " };
Comme faire ?

Commencez par enfin m'écouter, dégagez le code d'affichage des classes métiers.
Ainsi, si votre classe "AffichagePersonnalisation" aurait un sens, ce ne serait que comme paramètre de fonctions libres en charge des affichages, donc pas de prise de gueule avec la règle des 5.

07/03/2025, 22h47
Laurent_B_

2 pièce(s) jointe(s)

Citation:

Une information comme, "le numéro d'épisode est fiable" ou "le numéro d'épisode n'est pas fiable".

Séries :
https://www.allocine.fr/series/fiche.../saison-36426/
S03E13 - La cité où tout a commencé
Pièce jointe 665293
et
https://www3.stage.netflix.com/fr/title/70299043
50. La cité où tout a commencé
Pièce jointe 665292

Alors, comment faire ?
08/03/2025, 08h02
bacelar

Le rapport ???
J'ai déjà signalé que le scapping de site Internet est casse-gueule et qu'il faudrait changer de source de données.
Mais je ne vois pas le rapport entre ces pages Web et votre atavisme à mettre des noms abscons avec des concepts peu explicites.

Citation:

Alors, comment faire ?

Comment faire quoi ?

Vous voulez bêtement avoir le même affichage ???

Vous avez des sources de données différentes et qui même dans une seule source peut ne pas respecter une règle quelconque.

Implémentez vos règles et laissez les conneries marketing des autres à l'extérieur de votre projet.

https://github.com/laurentbouleau/Exemple : Mars 09 (08/03/2025)
Avec : L'Attaque des Titans.[2013- Crunchyroll].Manga

Citation:

Pourquoi ne pas stocker le numéro trouvé dans le nom du fichier ET le numéro trouvé dans le contenu du fichier ?
Ainsi, la classe métier a les informations "nécessaires".
C'est à l'affichage que la manière d'afficher le fait que ces 2 données ne sont pas cohérentes incombe. Mais ne polluez pas votre classe SequenceVisionnage avec des champs aux noms cryptiques.

Et si on décortique la logique implémentée, elle n'a pas de sens "évident".
On affiche le numéro de l'épisode quand on affiche une séquence de visionnage si :
Le contenu du fichier est vide
ou
si la première ligne du fichier respecte L"(?\\d)+\\.)?(.*)"
Donc, on affiche le numéro d'épisode même si le numéro dans le nom du fichier et celui dans la première ligne du fichier ne correspondent pas.
C'est pas logique pour moi.
EXPLIQUEZ EN FRANCAIS LA "LOGIQUE" DE VOTRE AFFICHAGE !!!
Pour moi, elle n'a pas de sens.
Et ça encore moins de sens quand, en fait, ce n'est pas dans le cadre d'une séquence de visionnage que la décision est prise, mais dans le cadre de la première séquence de visionnage de l'épisode qui est propagé à l'ensemble des séquences de cet épisode.

C'est peut-être clair dans votre tête mais ce n'est clairement pas le cas dans le code que vous avez pondu.

Comme faire ??? Et le métier : j'en sais rien !!!

Citation:

Commencez par enfin m'écouter, dégagez le code d'affichage des classes métiers.

???

Et : InfosVisionnage::InfosVisionnage(const Saison& saison, fs::path const& m_cheminFichier) : m_saison{ saison }
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 if (file_content.size() > 0) { const std::wregex numeroPlusTitres_rg{ L"(?:(\\d)+\\.)?(.*)" }; std::wsmatch titles_match; std::regex_match(file_content[0], titles_match, numeroPlusTitres_rg); if (titles_match[1].matched) { //int numeroDansFichier = std::stoi(titles_match[1]); long numeroDansFichier = std::stoi(titles_match[1]); if (numeroDansFichier != m_NumeroEpisode) { //std::wstring message = L"Le fichier " + nomFichier + L" contient un numéro d'épisode différent de celui dans son nom."; //OutputDebugStringW(message.c_str()); m_numero = numeroDansFichier; } else { m_numero = m_NumeroEpisode; } } else { m_numero = -1; }
???

Merci:aie::aie::aie:

09/03/2025, 13h05
Laurent_B_

1 pièce(s) jointe(s)
Pièce jointe 665320
Je suis crevé !!!

Alors, comment faire ?
Citation:
Ok, vous êtes "satisfait" du résultat ?

Si oui, il faut maintenant que vous nettoyez vos classes métier des machins qui n'ont rien à y faire, comme l'affichage.

Commencez par ajouter la déclaration d'une fonction libre comme la ligne suivante dans "serie.h" :
Code : Sélectionner tout - Visualiser dans une fenêtre à part

std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out);

Et les lignes suivantes dans "serie.cpp":
Code : Sélectionner tout - Visualiser dans une fenêtre à part

Code:

1 2 3 4 5 std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out) { serie.Print(); return out; }

Remplacer dans "Exemple.cpp" la ligne :
Code : Sélectionner tout - Visualiser dans une fenêtre à part

Code:

serie.Print();

Par:
Code : Sélectionner tout - Visualiser dans une fenêtre à part

Code:

PrintSerieOnOutstream(serie, std::wcout);

Après, il faudra remplacer "serie.Print()" par un code "externe" à la classe.
Puis décommenter les "const" pour que la fonction libre est une signature "correcte".

Et ainsi de suite avec les autres classes métier
Merci beaucoup:aie::weird::?:oops:
09/03/2025, 22h42
Laurent_B_

1 pièce(s) jointe(s)

https://github.com/laurentbouleau/Exemple : Mars 10 (09/03/2025)

Et : L'Attaque des Titans.[2013- Crunchyroll].Manga : Mars 10 (09/03/2025)

Citation:

Ok, vous êtes "satisfait" du résultat ?

Si oui, il faut maintenant que vous nettoyez vos classes métier des machins qui n'ont rien à y faire, comme l'affichage.
...
Par:

Ok !

Citation:

Après, il faudra remplacer "serie.Print()" par un code "externe" à la classe.
Puis décommenter les "const" pour que la fonction libre est une signature "correcte".

Et ainsi de suite avec les autres classes métier

Pièce jointe 665338

Alors ?

Merci vraiment:aie::aie::aie:

2 pièce(s) jointe(s)

Pièce jointe 665342
C'est pourtant clair.
Ligne 33 de serie.h :

Code:

std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out);

Comment le compilateur sait ce que veut dire "Serie" à ce moment de l'analyse du fichier ?
C'est ligne 40 qu'il commencerait à savoir :

Code:

class Serie;

Mais c'est trop tard !!!
Mettez cette ligne "std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, std::wostream& out);" après la ligne 377, pour que cette fonction libre reste à côté de la déclaration de la classe "Serie", mais APRES.
Comprenez et corrigez la première erreur, etc...

Citation:

Et : L'Attaque des Titans.[2013- Crunchyroll].Manga : Mars 10 (09/03/2025)

Vous avez créé 7 versions différentes du dépôt GitHub, juste pour ajouter et supprimer ces fichiers sans jamais chercher à comprendre et corriger le problème.
C'est pas en mettant la tête dans le sol, comme une autruche, que vous allez corriger le problème. :aie:

Votre programme crashe à la ligne 208 de "serie.cpp" : "m_NumeroEpisode = std::stoi(match[filename_numero_episode_index]);" dans le constructeur "InfosVisionnage::InfosVisionnage(const Saison& saison, fs::path const& m_cheminFichier)" parce que la variable "match" initialisée ligne 134 de "serie.cpp" :

Code:

std::regex_match(str, match, filename_format_rg);

est aux fraises.

Sachant que les lignes 136-137 :
Code:

1 2 if (match[filename_numero_saison_index].matched) m_NumeroSaison = std::stoi(match[filename_numero_saison_index]);
Donc le cas où tout ou partie du nom du fichier ne correspond pas au format attendu est déjà pris en compte.
Alors pourquoi croire ligne 208 que le nom du fichier contient toujours le "filename_numero_episode_index" mais pas forcément le "filename_numero_saison_index" ???
Pourquoi ces 2 lignes sont si éloignées d'abord ? Faites des fonctions courtes et simple, pas ces usines à gaz.

Vous vous retrouvez dans cette situation complexe parce que ligne 724 de "serie.cpp" :

Code:

if (std::regex_match(nomFichier, std::wregex{ L"([[:digit:]]{1,2})x(.)+" }))

Pourquoi l'expression régulière ici n'est pas la même que dans le constructeur d'"InfosVisionnage" ?

Résultat de cette incohérence, le fichier ".\L'Attaque des Titans.[2013- Crunchyroll].Manga\2013-04-06\1x02.2017.04-29.txt" passe le test de la ligne 724 mais plante l'initialisation ligne 134.
Soit vous corrigez le nom du fichier, soit vous "corrigez" l'expression régulière :

Code:

const std::wstring date_year_month_day_format = L"([[:digit:]]{4})(?:-|\\.)([[:digit:]]{2})-([[:digit:]]{2})";

(Mais, bon, c'est pas très "logique" comme format de nom de fichier)

Pièce jointe 665341
Ok pour ce diagramme, en début de projet, quand vous n'aviez pas des dizaines et des dizaines de champs pour customiser l'affichage de vos séries et films.
Et la "Console", ici, ce n'est pas forcement "std::wout", pour un affichage complexe, mais une console "technique", pour du débogage, car elle n'a pas le niveau d'abstraction pour "comprendre" ce qu'est un film ou une série.

Vous avez émis le souhait de pouvoir utiliser une interface graphique à la place d'une console et vous avez atrocement complexifié votre affichage, il faut donc séparer l'affichage du classe Film et Serie pour qu'elles restent gérables.

Citation:

Citation:

Citation:

std::vector<std::wstring>space {L" ", L" ", L" "} : ok ou pas ?

Très bof.
C'est mieux que 3 champs à la con.

J'ai quand même dit que c'était mieux que les 3 champs à la con.
Alors pourquoi être revenu au "3 champs à la con" ???

Citation:

S03E13 - La cité où tout a commencé
...
50. La cité où tout a commencé
...

J'avais interprété cela comme un format différent en fonction de la source des données : Allociné ou Netflix.
Mais la nature de la donnée est différente.
13, c'est le numéro de l'épisode dans la saison, 50, c'est le numéro de l'épisode dans la série.
Renommez le champ "m_NumeroEpisode" en "m_NumeroEpisodeDansSaison" et "m_numero" en "m_NumeroEpisodeDansSerie" dans les classes qui vont bien.
Et si le format du texte dans le fichier respect la convention "Netflix" :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 ... if (file_content.size() > 0) { const std::wregex numeroPlusTitres_rg{ L"(?:(\\d)+\\.)?(.*)" }; std::wsmatch titles_match; std::regex_match(file_content[0], titles_match, numeroPlusTitres_rg); if (titles_match[1].matched) { m_NumeroEpisodeDansSerie = std::stoi(titles_match[1]); } std::wstring titres = titles_match[2]; trim(titres); ... }
Nommez correctement les choses, SVP.

Après ces modifications, vous allez vous prendre des craches dans la figure mais vous devriez être capable de voir leurs causes avec le débogueur.

10/03/2025, 23h10
Laurent_B_

1 pièce(s) jointe(s)

https://github.com/laurentbouleau/Exemple : Mars 11 (10/03/2025)

L'Attaque des Titans :

1x02.2017-04-29.txt
et
2x10.2024-12-13.txt
Ok !

Citation:

Vous avez créé 7 versions différentes du dépôt GitHub, juste pour ajouter et supprimer ces fichiers sans jamais chercher à comprendre et corriger le problème.
C'est pas en mettant la tête dans le sol, comme une autruche, que vous allez corriger le problème.

Pourquoi ??? Pouvez-vous expliquer ?

Et problème :
Pièce jointe 665381
J'ai oublié !!!

Merci:aie::aie::aie:

Citation:

Pourquoi ??? Pouvez-vous expliquer ?

C'est juste un constat, vous avez fait 7+2 versions dans GitHub, juste pour ajouter ou supprimer la totalité des fichiers du répertoire contenant les fichiers pour cette série.
Alors que soit la modification de nom d'un seul fichier ou la "correction" d'une expression régulière aurait suffi.

Vous avez choisi de modifier/corriger les noms de 2 fichiers, ok, mais pourquoi supprimer et ajouter tous ces fichiers ?
Comme vous ne commentez pas vos checkin/push, c'est compliqué de comprendre ce qui a été corrigé.

Mes remarques sur l'usage incohérent des expressions régulières reste d'actualité, même si vous n'avez pas à les "corriger/étendre".

Citation:

Et problème :

C'est quoi le problème ?
Les espaces dans la formulation du temps en minutes ?
Vous avez déjà la classe "AffichagePersonnalisation" qui regroupe les informations de mise en forme de l'affichage.
Ajouter un paramètre de ce type à la fonction "PrintSerieOnOutstream", et passer le de proche en proche jusqu'à l'endroit où ces informations sont utiles.

Code:

std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, const AffichagePersonnalisation& perso, std::wostream& out);
Code:

1 2 3 4 5 std::wostream& PrintSerieOnOutstream(/*const*/ Serie& serie, const AffichagePersonnalisation& perso, std::wostream& out) { serie.Print(perso); return out; }
Code:

1 2 void Serie::Print(const AffichagePersonnalisation& perso) {...}
etc...
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 void SequenceVisionnage::Print(int numero_sequence, bool hors_saison, const AffichagePersonnalisation& perso) const { std::wstring numero_str; std::wstring titre_str; std::wstring duree_str; if (hors_saison) { numero_str = std::to_wstring(m_episode.m_numero) + perso.m_keyColor[1] + L" : " + perso.m_valuesColor; } else { numero_str = std::to_wstring(m_episode.m_saison.m_numero) + perso.m_keyColor[1] + L'x' + perso.m_valuesColor + ((m_episode.m_numero < 10 && m_episode.m_saison.m_nombre_episodes>10) ? L"0" : L"") + std::to_wstring(m_episode.m_numero) + perso.m_keyColor[1] + L" : " + perso.m_valuesColor; } assert((m_titres.size() > 0 && m_titres[0].size() > 0) || numero_sequence > 1); std::vector<std::wstring> titres = m_titres; if (m_titres.size() == 0 || (m_titres.size() == 1 && m_titres[0].size() == 0)) { titres = m_episode.m_liste_sequence_visionnages_ordonnee_chronologiquement[0].m_titres; } std::vector<std::wstring> titre_prefixe_list{ perso.m_keyColor[1] , perso.m_valuesColor }; std::wstring titre_prefixe = titre_prefixe_list[0]; unsigned int titre_prefixe_index = 0; for (auto& titre : titres) { titre_str += titre_prefixe_list[titre_prefixe_index] + titre; titre_prefixe_index = ((titre_prefixe_index + 1) % titre_prefixe_list.size()); } if (titre_str.size() > 0) titre_str += perso.m_valuesColor; duree_str = calcul_Duree_affichage(numero_sequence,perso); std::wstring dates_str = perso.m_keyColor[1] + L" : " + perso.m_valuesColor + Print_Dates_de_visionnage(); std::wstring resume_str; if (numero_sequence == 1) { resume_str += L"\r\n"; for (auto r : m_resume) resume_str += r; } std::wcout << numero_str << titre_str << duree_str << dates_str << resume_str << L"\r\n"; }

11/03/2025, 11h30
Laurent_B_

1 pièce(s) jointe(s)

Pardon, mais (attention) :

~~Azertyuiop £££.[2024].Mini-série~~
Azertyuiop £££.[2024-].Mini-série : ok !
L'Attaque des Titans.[2013- Crunchyroll].Manga : Problème !!! Hors saison !!!

Et :
Pièce jointe 665405
blablabla [2024- sur -]

Merci:?:oops::weird:

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