switchとメモリ取得位置
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#define TRUE 1
#define FALSE -1
char* get(char **p_str);
char *get_line(char buf[]);
int comp_rtn(const void *p1, const void *p2);
typedef struct {
int number;
char class_type[10];
char name[15];
char subject[10];
int ten;
} my;
my *data;
void myswap(my *p, my *q);
int main(int argc, char* argv[])
{
FILE *fp;
int field = 0, line = 0;
char buf[1000], *str;
char *bufFormat;
char *bufG;
bufG = (char *)malloc(1000);
if(bufG == NULL){
printf("メモリ不足");
free(bufG);
}
int line2 = 0;
if((fp=fopen("jjj.txt","r"))==NULL){
printf("ファイルが開けません");
}
while(fgets(buf, 1000, fp) != NULL){
line2++;
}
fclose(fp);
if((fp=fopen("jjj.txt","r"))==NULL){
printf("ファイルが開けません");
}
data = (my *)malloc(sizeof(my) * line2);
if(data == NULL){
printf("メモリ不足");
free(data);
}
while(fgets(buf,1000,fp) != NULL){
bufFormat =(char *)malloc(strlen(buf) + 1);
if(bufFormat == NULL){
printf("メモリ不足");
free(data);
}
bufFormat = get_line(buf);
str = bufFormat;
while(*str != '\0'){
bufG = get(&str);
switch(field){
case 0:
data[line].number = atoi(bufG); break
case 1:
strcpy(data[line].class_type, bufG);
break;
case 2:
strcpy(data[line].name, bufG);
break;
case 3:
strcpy(data[line].subject, bufG);
break;
case 4:
data[line].ten = atoi(bufG);
break;
}
str++;
field++;
}
line++;
field = 0;
}
fclose(fp);
qsort(data,line,sizeof(my),comp_rtn);
for(int m = 0; m < line; m++){
printf("%d\n", data[m].number);
printf("%s\n", data[m].class_type);
printf("%s\n", data[m].name);
printf("%s\n", data[m].subject);
printf("%d\n", data[m].ten);
printf("\n");
}
free(data);
return 0;
}
void myswap(my *p, my *q)
{
my temp;
temp = *p;
*p = *q;
*q = temp;
}
char *get(char **p_str)
{
int i;
char *str;
str = *p_str;
static char bufG[1000];
for(i = 0; *str != ',' && *str != '\0' ; i++){
if(*str == '\n'){
bufG[i] = '\0';
}
else if(*str == '\\'){
str++;
if(*str == 'c'){
bufG[i] = ',';
}
else if(*str == '"'){
bufG[i] = '"';
}
}
else{
bufG[i] = *str;
}
str++;
}
bufG[i] = '\0';
*p_str = str;
return bufG;
}
char *get_line(char buf[])
{
int in_quotation = FALSE, i = 0;
char* str = buf;
static char bufG[1000];
while(*str != '\0'){
if(*str=='"'){
if(in_quotation == TRUE){
str++;
if(*str == '"'){
bufG[i] = '\\';
i++;
bufG[i] = '"';
i++;
}
else{
in_quotation = FALSE;
bufG[i] = *str;
i++;
}
}
else{
in_quotation = TRUE;
}
}
else{
switch(*str){
case '\n':
if(in_quotation == TRUE){
bufG[i] = '\\';
i++;
bufG[i] = 'n';
i++;
}
else{
bufG[i] = *str;
i++;
}
break;
case ',':
if(in_quotation == TRUE){
bufG[i] = '\\';
i++;
bufG[i] = 'c';
i++;
}
else{
bufG[i] = *str;
i++;
}
break;
default:
bufG[i] = *str;
i++;
}
}
str++;
}
bufG[i] = '\0';
return bufG;
}
構造体メンバをポインタで宣言する方法はもうできていて
配列の方も組んでいるのですが
比較関数など所々省いてますがこのプログラムに対し以下のことをいわれました
1,switch文のfieldの値を数字じゃなく分かりやすいのに変えよ
2,mainのすぐしたの bufG = (char * )malloc(1000)が
なかなか使用されないのにここでメモリを取るのはおかしい
3,これを使うまでの間にエラーが発生したときのfreeがない
と言われました。
1ですが確かCではswitch文のcase式は整数型定数でなければならない
とあるので無理な気もするのですが、方法ありますか？
2に関してはよくわかりません。効率がよくないのでしょうか？
どの場所がいいのでしょうか
3に関してはどういうことなのかもわかりません。
この３点について教えて下さい。
お礼
ありがとうございました。 サンプル見ながらやってみます。