Optimisation du code

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Optimisation du code

lulooo

salut a tous, :jap:

Voila en ce moment la je travaille sur projet qui consiste a :

Vérifiez mémoire vive du système, à partir de là déterminer la taille maximale des fichiers qu'il peut gérer. (suggérer 10-20% de la fréquence max de RAM) Si le fichier est plus grand que la taille de fichier maximale, fractionnés automatiquement en fichiers temporaires et les utiliser.

donc voici la demarche:

Ouvrir le fichier
Vérifier la quantité de lignes définissent comme [lignes]
Crée un tableau [lignes]
Chargez toutes les urls dans un tableau
fermer le fichier

split.c :

Code :

/ *
split.c accueillera les fonctions pour découper les fichiers d'entrée qui sont trop grands.
Cela devrait garder l'utilisation de la mémoire faible.
Taille du fichier(FileSize) max est déterminé par l'entrée d'utilisateur (-m 4096) ou par le programme lui-même s'il n'est pas spécifié
* /
#include <split.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <Windows.h>
int fileSize ( char File[] ); //int pour stocker la taille du fichier
void fileNameExt ( char Name[], char Ext[], char File[] );
int splitmain( int argc, char *argv[] ) //méthode principale
{
FILE *source, *destination; //définir source et la destination des pointeurs de fichiers
int size, parts; //int pour la taille du fichier de sortie et la quantité de fichiers de sortie
char name[100], ext[100], aux[100];
if( argc < 3 ) //vérifier combien de args ont été entrés
{
printf( "Il n'y a pas suffisamment de args!" );
system( "pause" );
return 0;
}
fileNameExt( name, ext, argv[1] );
parts = atoi( argv[2] );
size = fileSize( argv[1] ); //taille variable = filesize de arg1, le fichier d'entrée.
if( (source = fopen(argv[1], "rb" )) == NULL ) //si fopen renvoie NULL le fichier n'est pas présent, les fichiers vides commence par \ n?
{
printf( "Erreur! Fichier source n'a pas pu être ouvert!" );
system( "pause" );
return 0;
}
for( int i = 0; i < parts; i++ ) //jusqu'à int i est le même que les parties int, boucle.
sprintf( aux, "%s%s-%d", name, ext, i );
printf( "Generating %s file! -- ", aux );
if( (destination = fopen(aux, "wb" )) == NULL ) //si le fichier de destination fopen renvoie NULL alors il n'ya pas de sortie
{
printf( "Errer! fichier de destination %s n'a pas pu etre cree!", aux );
system( "pause" );
return 0;
}
char car = getc( source );
for( int j = 0; j < size/parts && !feof(source); j++ )
{
putc( car, destination );
car = getc( source );
}
printf( "OK!
" );
fclose( destination );
}
fclose( source );
system( "pause" );
return 0;
}
int fileSize( char File[] ) //calculer la taille du fichier
{
FILE *f; //initier pointeur du fichier
if( (f = fopen(File,"rb" )) == NULL ) //si fopen renvoie NULL le fichier n'est pas présent, les fichiers vides commence par \n?
{
printf( "Erreur! Fichier source n'a pas pu être ouvert!" );
system( "pause" );
return 0;
}
fseek( f, 0,SEEK_END );
int sz = ftell(f);
fclose( f );
return sz;
}
void fileNameExt( char Name[], char Ext[], char File[] )
{
char *aux;
if( (aux = strrchr(File, '')) == NULL )
{
if( (aux = strrchr( File, '/' )) == NULL)
{
aux = File;
}
}
if( aux != File )
{
aux++;
}
for( int i = 0; *aux != '.'; i++, aux++ )
{
Name[i] = *aux;
Name[i+1] = '\0;';
}
for( int i = 0; *aux != '\0;'; i++, aux++ )
{
Ext[i] = *aux;
Ext[i+1] = '\0';
}
}

split.h :

Code :

#ifndef SPLIT_H_INCLUDED
#define SPLIT_H_INCLUDED
/* Prototypes des fonctions */
/* split main() */
int splitmain( int argc, char *argv[] );
/* calculer la taille de fichier */
int fileSize( char File[] );
/* */
void fileNameExt( char Name[], char Ext[], char File[] );
#endif

defhaze.c :

Code :

#include <conio.h>
#include <dos.h>
#include <process.h>
#include <stdio.h>
#include <split.h>
int main(int argc, char *argv[])
{
printf(" /*****Bienvenu a De(f)haze*****\ \n" );
printf(" | | \n" );
printf(" | Fait par Luloo | \n" );
printf(" | | \n" );
printf(" | | \n" );
printf(" \*****--------------------*****/ \n" );
int suc6 = 0;
loadfile();
return 0;
}
int loadfile() //Méthode pour charger le fichier d'entrée dans un tableau
{
inputfile = ; //attribuer une valeur à fichier d'entrée
FILE *fpinput; //définir pointeur du fichier d'entrée
fpinput=fopen(“&inputfile”, ”r”); //ouvrir le fichier d'entrée avec mode "r", lecture seule(read only)
//fp = fopen("filename.txt", "r" );
printf("Le comptage et la collecte des urls ... \n" );
while((fgets(fp)) != NULL) //tandis que la rangée / ligne ne renvoie pas NULL (espace ou \n !=NULL), urls++
{
urls++;
}
return 0;
}
for(int i=0; i<urls; i++) //tandis que i est inférieur au montant des urls, tourne en boucle
fscanf(fp, "%d %s\n", &UrlArr[i]); //ajouter une ligne au tableau
fclose(fpinput); //fermez le fichier puisque les urls sont déjà dans le tableau
}
/*
Méthode Append ajoutera les injections réussies dans le fichier de sortie si dans le cas d'un crash ou quoi que vous aurez toujours vos injections réussies
*/
int apppend()
{
FILE *fpoutput; //définir pointeur du fichier de sortie
fpoutput = fopen("output.txt", "a" ); //Inclure dans le fichier de sortie
if (suc6 <1)
{
struct date d;
getdate(&d);
fwrite("fpoutput", "De(f)haze started [%d/%d/%d",d.da_day,d.da_mon,d.da_year]\n" );
fwrite("fpoutput", "Codee par Luloo \n" );
}
}

voila mon probleme cest que je trouve que mon code est un peu mal structure et incomplet

donc en gros, jaimerai avoir quelques idees concernant l'optimisation de mon code et si possible corriger quelques fautes :whistle:

Message cité 2 fois

Publicité

gilou

Modérateur
Modosaurus Rex

lulooo a écrit :

Vérifiez mémoire vive du système, à partir de là déterminer la taille maximale des fichiers qu'il peut gérer. (suggérer 10-20% de la fréquence max de RAM) Si le fichier est plus grand que la taille de fichier maximale, fractionnés automatiquement en fichiers temporaires et les utiliser.

Au cas ou ça t'intéresse, sous Windows, on peut procéder ainsi (code pour gcc):

Code :

// voir http://msdn.microsoft.com/en-us/li [...] s.85).aspx
#define _WIN32_WINNT 0x0501 // Donc au moins au niveau de WinXP
#include <windows.h>
#include <stdio.h>
int main()
{
// voir http://msdn.microsoft.com/en-us/li [...] s.85).aspx
MEMORYSTATUSEX statex;
statex.dwLength = sizeof(statex);
GlobalMemoryStatusEx(&statex);
printf("Memoire physique: %llu Bytes\n", statex.ullTotalPhys/1024);
printf("Memoire physique utilisable: %llu Bytes\n", statex.ullAvailPhys/1024);
return 0;
}

et sous Linux, l'info est à lire dans /proc/meminfo (elle est minorée, car le système peut vider ses caches pour fournir un peu plus de mémoire si nécessaire)
A+,

Message édité par gilou le 28-12-2012 à 18:45:45

---------------
There's more than what can be linked! -- Le capitaine qui ne veut pas obéir à la carte finira par obéir aux récifs. -- No jab ? No job ! -- (╯°□°)╯︵ ┻━┻

Sve@r

lulooo a écrit :

voila mon probleme cest que je trouve que mon code est un peu mal structure et incomplet

Bonsoir

Je crois que ton indentation anarchique y est aussi pour beaucoup...

lulooo a écrit :

Code :

void fileNameExt( char Name[], char Ext[], char File[] )
{
char *aux;
if( (aux = strrchr(File, '')) == NULL )
{
if( (aux = strrchr( File, '/' )) == NULL)
{
aux = File;
}
}
int main(int argc, char *argv[])
{
printf(" /*****Bienvenu a De(f)haze*****\ \n" );
printf(" | | \n" );
printf(" | Fait par Luloo | \n" );
printf(" | | \n" );
printf(" | | \n" );
printf(" \*****--------------------*****/ \n" );
int suc6 = 0;
loadfile();
return 0;
}
int loadfile() //Méthode pour charger le fichier d'entrée dans un tableau
{
inputfile = ; //attribuer une valeur à fichier d'entrée
FILE *fpinput; //définir pointeur du fichier d'entrée
fpinput=fopen(“&inputfile”, ”r”); //ouvrir le fichier d'entrée avec mode "r", lecture seule(read only)
//fp = fopen("filename.txt", "r" );
printf("Le comptage et la collecte des urls ... \n" );
while((fgets(fp)) != NULL) //tandis que la rangée / ligne ne renvoie pas NULL (espace ou \n !=NULL), urls++
{
urls++;
}
return 0;
}
for(int i=0; i<urls; i++) //tandis que i est inférieur au montant des urls, tourne en boucle
fscanf(fp, "%d %s\n", &UrlArr[i]); //ajouter une ligne au tableau
fclose(fpinput); //fermez le fichier puisque les urls sont déjà dans le tableau
}

Juste cet exemple. Tu crois qu'on perçoit facilement l'organisation de la fonction fileNameExt avec tout son code écrit en début de ligne. Inversement le main, lui, se trouve bizarrement et inexplicablement décalé. Et enfin ces 3 dernières lignes qui me semblent situées en dehors de toute fonction...

Donc déjà commence par bien écrire proprement ton code, tu y gagneras en lisibilité et nous aussi...

Message édité par Sve@r le 30-12-2012 à 22:35:27

Trap D

@sver >> Il y a un gros problème d'indentation. j'ai copié/collé la semaine dernière un code bien indenté sous VS2010 qui s'est retrouvé écrabouillé ici, je pense que ce sont les tab qui ne sont pas transformé en espaces. Je n'ai pas eu le courage de tout refaire surtout que lorsqu'il est copié l'affichage est correct, c'est dans l'affichage terminal de la page qu'il y a problème

gilou

Modérateur
Modosaurus Rex

Tu formates le tout avec le plugin qui va bien de Code::Blocks, tu colles ici et ça roule.
C'est ce que je fais pour mon code, son indentation sous emacs n'étant pas compatible avec le formatage du forum.
En appliquant ceci au code que tu as posté, on voit qu'il est faux: splitmain fermé trop tôt manifestement.

Code :

/ *
split.c accueillera les fonctions pour découper les fichiers d'entrée qui sont trop grands.
Cela devrait garder l'utilisation de la mémoire faible.
Taille du fichier(FileSize) max est déterminé par l'entrée d'utilisateur (-m 4096) ou par le programme lui-même s'il n'est pas spécifié
* /
#include <split.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <Windows.h>
int fileSize ( char File[] ); //int pour stocker la taille du fichier
void fileNameExt ( char Name[], char Ext[], char File[] );
int splitmain( int argc, char *argv[] ) //méthode principale
{
FILE *source, *destination; //définir source et la destination des pointeurs de fichiers
int size, parts; //int pour la taille du fichier de sortie et la quantité de fichiers de sortie
char name[100], ext[100], aux[100];
if( argc < 3 ) { //vérifier combien de args ont été entrés
printf( "Il n'y a pas suffisamment de args!" );
system( "pause" );
return 0;
}
fileNameExt( name, ext, argv[1] );
parts = atoi( argv[2] );
size = fileSize( argv[1] ); //taille variable = filesize de arg1, le fichier d'entrée.
if( (source = fopen(argv[1], "rb" )) == NULL ) { //si fopen renvoie NULL le fichier n'est pas présent, les fichiers vides commence par \ n?
printf( "Erreur! Fichier source n'a pas pu être ouvert!" );
system( "pause" );
return 0;
}
for( int i = 0; i < parts; i++ ) { //jusqu'à int i est le même que les parties int, boucle.
sprintf( aux, "%s%s-%d", name, ext, i );
}
printf( "Generating %s file! -- ", aux );
if( (destination = fopen(aux, "wb" )) == NULL ) { //si le fichier de destination fopen renvoie NULL alors il n'ya pas de sortie
printf( "Errer! fichier de destination %s n'a pas pu etre cree!", aux );
system( "pause" );
return 0;
}
char car = getc( source );
for( int j = 0; j < size/parts && !feof(source); j++ ) {
putc( car, destination );
car = getc( source );
}
printf( "OK!
" );
fclose( destination );
}
fclose( source );
system( "pause" );
return 0;
}
int fileSize( char File[] ) //calculer la taille du fichier
{
FILE *f; //initier pointeur du fichier
if( (f = fopen(File,"rb" )) == NULL ) { //si fopen renvoie NULL le fichier n'est pas présent, les fichiers vides commence par \n?
printf( "Erreur! Fichier source n'a pas pu être ouvert!" );
system( "pause" );
return 0;
}
fseek( f, 0,SEEK_END );
int sz = ftell(f);
fclose( f );
return sz;
}
void fileNameExt( char Name[], char Ext[], char File[] )
{
char *aux;
if( (aux = strrchr(File, '')) == NULL ) {
if( (aux = strrchr( File, '/' )) == NULL) {
aux = File;
}
}
if( aux != File ) {
aux++;
}
for( int i = 0; *aux != '.'; i++, aux++ ) {
Name[i] = *aux;
Name[i+1] = '\0;';
}
for( int i = 0; *aux != '\0;'; i++, aux++ ) {
Ext[i] = *aux;
Ext[i+1] = '\0';
}
}

Et ici pareil, loadfile est fermée trop tôt manifestement, ou alors il manque un début à ce qui la suit.

Code :

#include <conio.h>
#include <dos.h>
#include <process.h>
#include <stdio.h>
#include <split.h>
int main(int argc, char *argv[])
{
printf(" /*****Bienvenu a De(f)haze*****\ \n" );
printf(" | | \n" );
printf(" | Fait par Luloo | \n" );
printf(" | | \n" );
printf(" | | \n" );
printf(" \*****--------------------*****/ \n" );
int suc6 = 0;
loadfile();
return 0;
}
int loadfile() //Méthode pour charger le fichier d'entrée dans un tableau
{
inputfile = ; //attribuer une valeur Ã fichier d'entrée
fpinput=fopen(“&inputfile”, ”r”); //ouvrir le fichier d'entrée avec mode "r", lecture seule(read only)
//fp = fopen("filename.txt", "r" );
printf("Le comptage et la collecte des urls ... \n" );
while((fgets(fp)) != NULL) { //tandis que la rangée / ligne ne renvoie pas NULL (espace ou \n !=NULL), urls++
urls++;
}
return 0;
}
for(int i=0; i<urls; i++) //tandis que i est inférieur au montant des urls, tourne en boucle
{
fscanf(fp, "%d %s\n", &UrlArr[i]); //ajouter une ligne au tableau
}
fclose(fpinput); //fermez le fichier puisque les urls sont déjà dans le tableau
}
/*
Méthode Append ajoutera les injections réussies dans le fichier de sortie si dans le cas d'un crash ou quoi que vous aurez toujours vos injections réussies
*/
int apppend()
{
FILE *fpoutput; //définir pointeur du fichier de sortie
fpoutput = fopen("output.txt", "a" ); //Inclure dans le fichier de sortie
if (suc6 <1) {
struct date d;
getdate(&d);
fwrite("fpoutput", "De(f)haze started [%d/%d/%d",d.da_day,d.da_mon,d.da_year]\n" );
fwrite("fpoutput", "Codee par Luloo \n" );
}
}

A+,

Message édité par gilou le 01-01-2013 à 13:47:13

---------------
There's more than what can be linked! -- Le capitaine qui ne veut pas obéir à la carte finira par obéir aux récifs. -- No jab ? No job ! -- (╯°□°)╯︵ ┻━┻

FORUM HardWare.fr

Programmation

Optimisation du code

Sujets relatifs
Optimisation code php	[VBA] Optimisation de code
optimisation SSE pour du code C (auto ou inline) [résolu]	optimisation du code c++
Optimisation de code javascript	Code simple, optimisation possible?
Optimisation d'un code VBA pour excel	Optimisation de code
Optimisation de la syntaxe d'un code (pour gain en rapidité)	optimisation du code, tester valeur avant attribution ?
Plus de sujets relatifs à : Optimisation du code

Page générée en 0.206 secondes