また、お世話になります。よろしくおねがいします。最近、プログラムを書いていて
よく思うのですが、私はすごくグローバル変数をつかうような気がするのです。
グローバル変数を使うと簡単に行ける場合が多いからだと思うのですが、このような
コーディングスタイルはよくないのでしょうか?よくWEBをみているとグローバル変数
を多用するのはよくないみたいなことが書かれているのですが、理由がわかりません。
すごく便利だと思うのですが。。多分私がまだビギナーなのでその理由も見えないのだと
思うのですが、わかるかた御意見をください。
あと、以下にちょっとしたプログラムを示します。quitと入力されるまで入力された文字を
ソートして出力するものなんですが、今回二つのファイルに書いてみたところ
このグローバル変数のことを考えました。このメインで使っているstore_into_list
という関数に入力されていく文字をわたしていっているのですが、
わたされるたびにこの関数がよばれるためにstore_into_list()のなかで構造体の配列を
宣言することができませんでした。このような場合に構造体の配列をグローバルにせずに
受け取った文字をいれていくことはできるのでしょうか?

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
//メインファイルです。
#include <stdio.h>
#include <string.h>
#include <stdlib.h>

void show_result(int counter);
void store_into_list(char *str,int counter);

int main(void)
{
  char str[100];
  int quitting=0;
  int counter=0;

  while(! quitting){
    scanf("%s",str);
   
    if(strcmp(str,"quit")==0){
      quitting =1;
    }
    else{
      store_into_list(str,counter);
      counter++;
    } 
  }

  show_result(counter);

  return 0;
}
ーーーーーーーーーーーーーーーーーーーーーーーーーー
//オブジェクトファイルです。
struct Word{
  char word[100];
};

int num=0;

struct Word words[100];

int cmp(const void *x, const void *y)
{
  return strcmp(((struct Word *)x)->word, ((struct Word *)y)->word);
}


void store_into_list(char *str,int counter)
{ 
 
  strcpy(words[counter].word,str);
}

void show_result(int counter)
{
  int i; 

  printf("\nBefore sorted:\n");
  for(i=0; i<counter; i++){
  printf("%s\n",words[i].word);
  }

  qsort(words,counter,sizeof(struct Word),cmp);
  printf("\nsorted:\n");
  for(i=0; i<counter; i++){
    printf("%s\n",words[i].word);
  }
}
ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー

YUOさん、アドバイスありがとうございます。グローバル変数を多用す
ることがあまりよくないことが少しわかってきました。ですが、どうしても
このプログラムでグローバルで宣言せずに文字列をわたす方法がわかりませ
ん。いろいろやってみたのですが、アイディアがわきません。どのように関
数の仕様をかえるといいのでしょうか?

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

void show_result(int counter);
void store_into_list(char *str,int counter);

int main(void)
{
  char str[100];
  int quitting=0;
  int counter=0;

  while(! quitting){
    scanf("%s",str);
   
    if(strcmp(str,"quit")==0){
      quitting =1;
    }
    else{
      store_into_list(str,counter);
      counter++;
    } 
  }

  show_result(counter);

  return 0;
}

//オブジェクトファイル

struct Word{
  char word[100];
};

int num=0;

struct Word words[100];

int cmp(const void *x, const void *y)
{
  return strcmp(((struct Word *)x)->word, ((struct Word *)y)->word);
}


void store_into_list(char *str,int counter)
{ 
 
  strcpy(words[counter].word,str);

}

void show_result(int counter)
{
  int i; 

  printf("\nBefore sorted:\n");
  for(i=0; i<counter; i++){
  printf("%s\n",words[i].word);
  }

  qsort(words,counter,sizeof(struct Word),cmp);
  printf("\nsorted:\n");
  for(i=0; i<counter; i++){
    printf("%s\n",words[i].word);
  }
  

}

// word.h

struct Word
{
    char word[100];
};

void show( struct Word* words, int n );

// word.c
#include "word.h"
#include <stdio.h>

void show( struct Word* words, int n )
{
    int i;
    for( i = 0; i < n; ++i )
        printf( "%s\n", words[i].word );
}

// main.c
#include "word.h"
#include <stdio.h>
#include <string.h>

int main()
{
    struct Word words[100];
    char str[100];
    int counter=0;

    while( 1 )
    {
        scanf( "%s", str );   
        if( strcmp( str, "quit" ) == 0 )
            break;
        else
            strcpy( words[counter++].word, str );
    }
    show( words, counter );

    return 0;
}

// word.h
#ifndef _WORD_H_
#define _WORD_H_

struct Word
{
    char word[100];
};

void show( struct Word* words, int n );

#endif

// -------- mani.c --------
#include <stdio.h>
#include <string.h>
#include "word.h"

int main(void)
{
    Word *word;  char str[100];

    word = create(50);
    while (scanf("%s", str) == 1 && strcmp(str, "quit"))
        add(word, str);
    puts("\n--- before sort ---");
    show(word);
    sort(word);
    puts("\n--- after sort ---");
    show(word);
    destroy(word);
    return 0;
}

// -------- word.h --------
#ifndef WORD_H
#define WORD_H

typedef struct Word Word;

Word *create(int capa);
void destroy(Word *);
void add(Word *, const char *);
void show(Word *);
void sort(Word *);
int  size(Word *);

#endif


// -------- word.c --------
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "word.h"

struct Word {
    int capa;
    int size;
    char **word;
};

Word *create(int capa)
{
    Word *wp = malloc(sizeof(Word));
    wp->word = malloc(sizeof(char *) * capa);
    wp->capa = capa;
    wp->size = 0;
    return wp;
}

void destroy(Word *wp)
{
    free(wp->word);
    free(wp);
}

void add(Word *wp, const char *str)
{
    wp->word[wp->size] = strdup(str);
    wp->size++;
}

void show(Word *wp)
{
    int i;
    for (i = 0; i < wp->size; i++) puts(wp->word[i]);
}

static int compare(const void *x, const void *y)
{
    return strcmp(*(char **)x, *(char **)y);
}

void sort(Word *wp)
{
    qsort(wp->word, wp->size, sizeof(char *), compare);
}

int size(Word *wp)
{
    return wp->size;
}

void destroy(Word *wp)
{
    int i;
    for (i = 0; i < wp->size; i++) free(wp->word[i]);
    free(wp->word);
    free(wp);
}