Problème et AVC : _wfopen_s

**Laurent_B_** · 24/10/2024, 13h15

Bonsoir,

Attention :
std::vector<std::wstring> m_keyColor

https://www.w3schools.com/tags/ref_colornames.asp
et
https://learn.microsoft.com/fr-fr/wi...inal-sequences

(Couleurs étendues

Certains émulateurs de terminaux virtuels prennent en charge une palette de couleurs supérieure aux 16 couleurs fournies par la console Windows. Pour ces couleurs étendues, la console Windows choisit la couleur appropriée la plus proche dans la table de 16 couleurs existante pour l’affichage. Contrairement aux valeurs SGR classiques ci-dessus, les valeurs étendues consomment des paramètres supplémentaires après l’indicateur initial, conformément au tableau ci-dessous.)

Merci

**bacelar** · 24/10/2024, 20h49

Code :

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

1
2
        duree_str = L' ' + std::to_wstring(heures) + m_keyColor[0] + m_espace1 + (heures <= 1 ? m_h[0] : m_h[0] + m_h[1]) + m_valuesColor + m_espace2 +
            std::to_wstring(minutes) + m_keyColor[0] + m_espace3 + (minutes <= 1 ? m_min[0] : m_min[0] + m_min[1]) + m_valuesColor;

Toute cette usine à gaz (classe X_Person, classe Person, des champs dans des classes avec des noms abscons), juste pour gérer un pluriel sous forme de suffixe fixe uniquement ???

Si la marque d'un pluriel n'est pas un suffixe fixe dans une langue supportée et tout ce bric-à-brac s'effondre.

Actuellement, tout ce bordel de "personnalisation" ne sert à rien de "constructif".

Code :

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

1
2
duree_str = L' ' + std::to_wstring(heures) + m_keyColor[0] + m_espace1 + (heures <= 1 ? m_labelHeureSingulier : m_labelHeurePluriel) + m_valuesColor + m_espace2 +
            std::to_wstring(minutes) + m_keyColor[0] + m_espace3 + (minutes <= 1 ? m_labelMinuteSingulier : m_labelMinutePluriel ) + m_valuesColor;

C'est déjà plus lisible, on n'utilise pas des std::vector n'importe comment, c'est tout autant personnalisable et on peut facilement gérer d'autres formes de pluriels que de suffixe "fixe". (+s ou +x ; ou halves ou wolves en anglais, etc...).

Je ne comprends toujours pas votre obsession de la "personnalisation" si prématurée.
(reste encore ces conneries de vecteur de m_keyColor et autres m_espaceX imbittables)

Faites simple !

https://www.w3schools.com/tags/ref_colornames.asp

Les couleurs du Web n'ont rien à voir avec les couleurs d'une console "caractère".

Couleurs étendues

Certains émulateurs de terminaux virtuels prennent en charge une palette de couleurs supérieure aux 16 couleurs fournies par la console Windows. Pour ces couleurs étendues, la console Windows choisit la couleur appropriée la plus proche dans la table de 16 couleurs existante pour l’affichage. Contrairement aux valeurs SGR classiques ci-dessus, les valeurs étendues consomment des paramètres supplémentaires après l’indicateur initial, conformément au tableau ci-dessous.)

Arrêtez de vous prendre la tête avec le respect des couleurs au niveau colorimétrique !!!
Il y a des choses bien plus fondamentales et moins aléatoires que ces conneries de "respect" des couleurs !!!

**Laurent_B_** · 24/10/2024, 23h23

Bonsoir,

Ah ! Mince ! Tant pis !

https://github.com/laurentbouleau/Exemple : Octobre 16 (24/10/2024)

Code :

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

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
void Film::AffichagePersonnaliser()
{  
    const class AffichagePersonnaliser perso;
    auto& e_1_0 = perso.m_espace1[0], & e_1_1 = perso.m_espace1[1];
    auto& e_2_0 = perso.m_espace2[0], & e_2_1 = perso.m_espace2[1];
    auto& e_3_0 = perso.m_espace3[0], & e_3_1 = perso.m_espace3[1];
 
    auto& h0 = perso.m_h[0], & h1 = perso.m_h[1];
    auto& h2 = perso.m_h[2], & h3 = perso.m_h[3];
    auto& h4 = perso.m_h[4], & h5 = perso.m_h[5];
    auto& h6 = perso.m_h[6], & h7 = perso.m_h[7];
    auto& h8 = perso.m_h[8], & h9 = perso.m_h[9];
 
    auto& min0 = perso.m_min[0], & min1 = perso.m_min[1];
    auto& min2 = perso.m_min[2], & min3 = perso.m_min[3];
    auto& min4 = perso.m_min[4], & min5 = perso.m_min[5];
    auto& min6 = perso.m_min[6], & min7 = perso.m_min[7];
    auto& min8 = perso.m_min[8], & min9 = perso.m_min[9];
    auto& min10 = perso.m_min[10], & min11 = perso.m_min[11];
 
    //
 
    m_espace1 = e_1_1;
 
    m_h = { h4, h5 };
 
    m_espace2 = e_2_1;
    m_espace3 = e_3_1;
 
    m_min = { min6, min7 };
 
    m_keyColor = perso.m_keyColor;
    m_valuesColor = perso.m_valuesColor;
}

Ok ou pas ?

Code :

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

1
2
duree_str = L' ' + std::to_wstring(heures) + m_keyColor[0] + m_espace1 + (heures <= 1 ? m_labelHeureSingulier : m_labelHeurePluriel) + m_valuesColor + m_espace2 +
            std::to_wstring(minutes) + m_keyColor[0] + m_espace3 + (minutes <= 1 ? m_labelMinuteSingulier : m_labelMinutePluriel ) + m_valuesColor;

et

Code :

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

(((([[:digit:]]{4})-([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2}))(_?))+)(\s(.+))?

Après !!!

Merci

**Laurent_B_** · 25/10/2024, 14h19

Bonjour,

https://github.com/laurentbouleau/Exemple : Octobre 17 (25/10/2024)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
std::wstring Film::calcul_Duree_affichage() const
{
    std::wstring duree_str;
    if (affichage_duree_actif)
    {
        long heures = m_duree / (60 * 60);
        long minutes = (m_duree % (60 * 60)) / 60;
        long secondes = m_duree % 60;
        //duree_str = L' ' + std::to_wstring(heures) + m_keyColor[0] + m_espace1 + (heures <= 1 ? m_h[0] : m_h[0] + m_h[1]) + m_valuesColor + m_espace2 +
        //    std::to_wstring(minutes) + m_keyColor[0] + m_espace3 + (minutes <= 1 ? m_min[0] : m_min[0] + m_min[1]) + m_valuesColor;

        duree_str = L' ' + std::to_wstring(heures) + m_keyColor[0] + m_espace1 + (heures <= 1 ? m_labelHeureSingulier : m_labelHeureSingulier + m_labelHeurePluriel) + m_valuesColor + m_espace2 +
            std::to_wstring(minutes) + m_keyColor[0] + m_espace3 + (minutes <= 1 ? m_labelMinuteSingulier : m_labelMinuteSingulier + m_labelMinutePluriel) + m_valuesColor;
    }
    return duree_str;
}

Ok ou ps ?

Merci

**Laurent_B_** · 25/10/2024, 17h30

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
    const std::wregex filename_format_rg = L"(((" + // Noir
        L"([[:digit:]]{4})" + // Rouge
        L"-" +                // Noir
        L"([[:digit:]]{2})" + // Vert
        L"-" +                // Noir
        L"([[:digit:]]{2})" + // Bleu clair
        L"|" +                // Noir
        L"([[:digit:]]{2})" + // Rouge
        L"-" +                // Noir
        L"([[:digit:]]{2})" + // Vert
        L"|" +                // Noir
        L"([[:digit:]]{2})" + // Bleu clair
        L")" +                // Noir 
        L"(_?)" +             // Rouge
        L")+" +               // Noir
        L"(\\s(.+)" +         // Vert
        L")?";

Marche pas !!!

???

**bacelar** · 26/10/2024, 07h41

Post du "24/10/2024, 22h23"

C'est bien d'avoir renommer la classe "Person" en autre chose, mais pas en "AffichagePersonnaliser".

Le fait de mettre des verbes d'action dans les noms de fonction, c'est justement pour que ces verbes n'apparaissent pas dans les noms de classe.

J'ai indiqué le nom "AffichagePersonnalisation" pas "AffichagePersonnaliser".
Pour qu'il n'y est pas d'ambiguïté entre la classe et la fonction.
Dans les conventions de nommage, on indique aussi souvent que les fonctions doivent commencer par des minuscules :
https://www.geeksforgeeks.org/naming-convention-in-c/
Je ne me suis pas astreint à ces conventions dans mes réponses pour garder vos "conventions" sans vous prendre la tête avec LES "vrais" conventions, pour ne pas vous embrouiller.
Mais, là, je pense qu'il faut un minimum de "rigueur". (malgré que vous n'avez toujours pas fait le travail d'expliciter le rôle des classes/fonctions/variables par des noms explicites.)
Si vous vous astreignez à ces conventions, il y aura encore moins d'ambiguïté.

Et on en revient à "tout ça pour ça".
Juste faire une usine à gaz qui ne permet que de mettre des constantes dans une classe.
Classe avec un nom presque explicite, mais avec des noms de champs à la con, des types de champs complètement à l'Ouest, etc...
Pour rendre la classe "Film" impersonnalisable, car le lien avec la classe "AffichagePersonnaliser" est en dure et on ne peut pas changer les valeurs des champs depuis l'extérieur d'un objet "Film".

"Mon" code :

Code :

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

1
2
3
4
5
6
7
8
9
int wmain(int argc, wchar_t* argv[])
{
...
    AffichagePersonnalisation person = getCurrentAffichagePersonnalisation();//fonction à implémenter pour récupérer la configuration de personnalisation de l'utilisateur.
...
        film.AffichagePersonnaliser(person);
        film.Print();
...
}

Si je passe un objet "AffichagePersonnalisation" à la fonction "AffichagePersonnaliser", c'est pas pour rien.
C'est pour rendre l'objet "film" personnalisable de l'extérieur.

Ok ou pas ?

Donc pas du tout "OK".
---------------------

Post du "25/10/2024, 13h19"

Ok ou ps ?

Absolument pas.
Vous n'avez que changé les noms des champs et supprimé un tableau.
Mais la limitation en termes de formulation du pluriel est toujours la même.
Vous avez un code plus complexe et moins polyvalent que le mien.
Pourquoi voulez-vous absolument faire plus compliqué ???

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
const class AffichagePersonnalisation
{
public:
    std::wstring m_espace1{L" "};
    std::wstring m_espace2{L" "};
    std::wstring m_espace3{L" "};
 
    std::pair<std::wstring,std::wstring> m_labelsHeure { L"heure", L"heures", };
    std::pair<std::wstring,std::wstring> m_labelsMinute{ L"minute", L"minutes"};
 
    std::vector<std::wstring> m_keyColor{ L"\x1b[94;1m", L"\x1b[38;2;0;255;0m", L"\x1b[38;2;255;0;0m", L"\x1b[38;2;255;255;0m" };
    std::wstring m_valuesColor{ L"\x1b[38;2;255;255;255m" }; // Blanc
};

Les labels complets, c'est quand même plus explicite que des "s" qui se baladent.
---------------------

Post du "25/10/2024,16h30"

Marche pas !!!

Quand on numérote, c'est avec des numéros, et, en générale, c'est des nombres.

0 (l'ensemble de ce qui a été capturé par l'expression régulière): (((([[:digit:]]{4})-([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2}))(_?))+)(\s(.+))?
1 : ((([[:digit:]]{4})-([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2}))(_?))+)
2 : (([[:digit:]]{4})-([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2}))
3 : ([[:digit:]]{4})-([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2})-([[:digit:]]{2})|([[:digit:]]{2})
4 : [[:digit:]]{4}
5 : [[:digit:]]{2}
6 : [[:digit:]]{2}
7 : [[:digit:]]{2}
8 : [[:digit:]]{2}
9 : [[:digit:]]{2}
10 : _?
...

**Laurent_B_** · 27/10/2024, 13h52

Bonjour,

https://github.com/laurentbouleau/Exemple : Octobre 19 (27/10/2024)

Code :

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

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
??? getCurrentAffichagePersonnalisation()
{
    const std::pair<std::wstring, std::wstring> espace1{ L"", L" " };
    // _
    const std::pair<std::wstring, std::wstring> espace2{ L"", L" " };
    // _
    const std::pair <std::wstring, std::wstring> espace3{ L"", L" " };

    auto& r_e_1_0 = espace1.first, & r_e_1_1 = espace1.second;
    auto& r_e_2_0 = espace2.first, & r_e_2_1 = espace2.second;
    auto& r_e_3_0 = espace3.first, & r_e_3_1 = espace3.second;

    // h
    std::vector<std::pair<std::wstring, std::wstring>> labelsHeure{ { L"h", L"h" },            // 0
                                                                    { L"H", L"h" },            // 1
                                                                    { L"heure", L"heures" },   // 2
                                                                    { L"Heure", L"Heures" },   // 3
                                                                    { L"HEURE", L"HEURES" } }; // 4

    auto& r_lH_0{ labelsHeure[0] };
    auto& r_lH_1{ labelsHeure[1] };
    auto& r_lH_2{ labelsHeure[2] };
    auto& r_lH_3{ labelsHeure[3] };
    auto& r_lH_4{ labelsHeure[4] };

    // min
    std::vector<std::pair<std::wstring, std::wstring>> labelsMinute{ { L"min", L"min" },          // 0
                                                                     { L"Min", L"Min" },          // 1
                                                                     { L"MIN", L"MIN" },          // 2
                                                                     { L"minute", L"minutes" },   // 3
                                                                     { L"Minute", L"Minutes" },   // 4
                                                                     { L"MINUTE", L"MINUTES" } }; // 5

    auto& r_lM_0{ labelsMinute[0] };
    auto& r_lM_1{ labelsMinute[0] };
    auto& r_lM_2{ labelsMinute[0] };
    auto& r_lM_3{ labelsMinute[0] };
    auto& r_lM_4{ labelsMinute[0] };
    auto& r_lM_5{ labelsMinute[0] };

    //
    std::vector<std::wstring> keyColor{ L"\x1b[94;1m", L"\x1b[38;2;0;255;0m", L"\x1b[38;2;255;0;0m", L"\x1b[38;2;255;255;0m" };
    std::wstring valuesColor{ L"\x1b[38;2;255;255;255m" }; // Blanc

    //

    //return { {r_e_1_1}, {r_e_2_1}, {r_e_3_1}, r_lH_2 };
    return ???;
}

Je suis fatigué !!!

Merci

**bacelar** · 28/10/2024, 02h11

const class AffichagePersonnalisation

Pourquoi vous collez toujours ces putains de "const" devant les classes ???

Code :

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

??? getCurrentAffichagePersonnalisation()

Une fonction "getCurrentX" ne devrait pas retourner un X ?
Alors que devrait retourner une fonction "getCurrentAffichagePersonnalisation" ?
Pas un objet AffichagePersonnalisation ?

C'est quoi tu ce bordel dans getCurrentAffichagePersonnalisation ?

Comme vous n'avez toujours pas analysé comment changer de personnalisation :

Code :

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

1
2
3
4
AffichagePersonnalisation getCurrentAffichagePersonnalisation()
{
    return AffichagePersonnalisation();
}

Faites simple !!!

**Laurent_B_** · 28/10/2024, 23h53

https://github.com/laurentbouleau/Exemple : Octobre 20 (28/10/2024)
J'en ai marre !!!

Code :

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

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 Xxx
{
    const std::pair<std::wstring, std::wstring> espace1{ L"", L" " };
    // _
    const std::pair<std::wstring, std::wstring> espace2{ L"", L" " };
    // _
    const std::pair <std::wstring, std::wstring> espace3{ L"", L" " };
    // h
    std::vector<std::pair<std::wstring, std::wstring>> labelsHeure{ { L"h", L"h" },            // 0
                                                                    { L"H", L"h" },            // 1
                                                                    { L"heure", L"heures" },   // 2
                                                                    { L"Heure", L"Heures" },   // 3
                                                                    { L"HEURE", L"HEURES" } }; // 4
 
    // min
    std::vector<std::pair<std::wstring, std::wstring>> labelsMinute{ { L"min", L"min" },          // 0
                                                                     { L"Min", L"Min" },          // 1
                                                                     { L"MIN", L"MIN" },          // 2
                                                                     { L"minute", L"minutes" },   // 3
                                                                     { L"Minute", L"Minutes" },   // 4
                                                                     { L"MINUTE", L"MINUTES" } }; // 5
    //
    std::vector<std::wstring> keyColor{ L"\x1b[94;1m", L"\x1b[38;2;0;255;0m", L"\x1b[38;2;255;0;0m", L"\x1b[38;2;255;255;0m" };
    std::wstring valuesColor{ L"\x1b[38;2;255;255;255m" }; // Blanc
};

https://www.geeksforgeeks.org/naming-convention-in-c/

Le caractère « m » précède également d'autres modificateurs de nom. Par exemple, « p » pour les pointeurs. // ok
Les variables de référence doivent être précédées de « r ». // ok
ect...

**bacelar** · Hier, 13h51

Je ne comprends toujours pas l'utilité de la classe "Xxx", comme "X_Perso" etc...
Pourquoi faire compliquer ?
Elles servent à quoi ? (BORDEL)

Pour rendre compilable votre code, faut "juste" :
- ajouter "class AffichagePersonnalisation;" devant la déclaration de la classe "Film" dans le fichier "film.h" (parce que vous faite des références croisées entre classes, je comprend pas pourquoi vous vous faite chier avec ça)
- Sortir la ligne "AffichagePersonnalisation getCurrentAffichagePersonnalisation();" de l'intérieur de la déclaration de la classe "AffichagePersonnalisation" dans "film_serie.h", pour qu'elle corresponde à la définition de la fonction dans "film_serie.cpp".

Le caractère « m » précède également d'autres modificateurs de nom. Par exemple, « p » pour les pointeurs. // ok
Les variables de référence doivent être précédées de « r ». // ok
ect...

C'est bien 2 règles de codage que je ne suivrais pas. Elles utilisent "abusivement" la notation hongroise.

Vous faites des trucs incompréhensible au niveau de la "personnalisation".
Justifiez votre usine à gaz, SVP.

Problème et AVC : _wfopen_s

C++

Discussions similaires

Partager

Partager