Discussion :

[GraineDeDev]

Bonjour !

je viens d implementer un code source c++ :

une fonction renvoyant un pointeur sur char, depuis un fichier.
un fichier lui-meme contenant le code source d un shader, afin de fournir le pointeur en question au code opengl afin de le compiler...

Merci de critiquer mon travail (en bien ou en mal, no souci)...
(j apprend de mes erreurs)

Et si ca peut etre utile a quelqu'un, c est coool !
merci.

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
 
#include <iostream>
#include <fstream>
#include <string>
 
char* chargement(std::string nom)
{
    char* adresse = nullptr;
    int taille;
 
    std::ifstream flux(nom, std::ios::in);
    if(!flux){std::cout << "fichier " << nom << " introuvable !\n";}
 
    flux.seekg(0, std::ios::end);
    taille = flux.tellg();
 
    flux.seekg(0, std::ios::beg);
    for (int i = 0; i<taille; i++)
    {
        flux.read(adresse+i, sizeof(char));
    }
    // ajouter \0
    adresse[taille] = '\0';
    // fermer le fichier
    flux.close();
    // renvoyer adresse
    return adresse;
 
}

[GraineDeDev]

Je suis conscient qu il existe deja beaucoup de fonctions faisant la meme chose...
juste pour le plaisir...

merci.

[Bousk]

ton adresse n'est jamais initialisée, et quitte à faire du c++, autant utiliser std::vector.
Et si tu récupères la taille du fichier, lis-le en un seul appel et non un char à la fois.

[GraineDeDev]

oups...

ligne 8 : n est-ce pas une initialisation ?

std::vector ou meme std::string.... est-ce que ces conteneurs seront composés comme des char* ?

merci de vos critiques !!

[GraineDeDev]

Bien vu !!

j ai modifié la ligne 8 :

char* adresse;

merci : nullptr n est pas une initialisation...

[Bousk]

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
 
#include <fstream>
#include <vector>
 
std::vector<char> chargement(std::string nom)
{
    std::ifstream flux(nom);
    if(!flux)
    {
        return {};
    }
 
    flux.seekg(0, std::ios::end);
    const int taille = flux.tellg();
    flux.seekg(0, std::ios::beg);
    std::vector<char> data;
    data.resize(taille + 1);
    flux.read(&data[0], taille * sizeof(char));
    adresse[taille] = '\0';
    flux.close();
    return data;
}