C中的分段错误?数组、指针、函数

Segmentation fault in C? Arrays, pointers, functions

嗨,由于我缺乏 C 语言知识(大学二年级)。编译器吃掉了我的代码并构建了应用程序。但是在接受第一个值 - numOfIntegers 后它停止工作并且调试告诉分段失败。 SIGSEGV。 如何解决?

有代码:

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

/* -----------------------------------------
    Program: Question 1
    Author: Maggot #9
    Email: maggot99999@gmail.com
    ID: B00076450
    Date: 16 September 2015
    Purpose: Who knows?
   ------------------------------------------ */

void wait(int);
void controlMenu(int, int[]);
int sumOfIntegers(int, int[]);
int avgOfIntegers(int, int[]);
int prodOfIntegers(int, int[]);
int minInteger(int, int[]);
int maxInteger(int, int[]);


const char * getName (int value)
{

  static char * arrayName[] = {"first","second","third", "fourth",
    "fifth","sixth", "seventh", "eighth", "ninth", "tenth"};

  static char badValue[] = "unknown";

  if (value<10 && value>=0)
    return arrayName[value];
  else
    return badValue;
}

int getValue(int numOfInteger)
{
  int value;
  wait(100);
  printf("Please enter %s the value:", getName(numOfInteger));
  scanf("%d",&value);

  return value;
}


void prepare(int * numOfIntegers)
{
  wait(300);
  printf("Hey again that C stupid lang\n\n");
  wait(200);
  printf("Please enter how many values you want to put: ");
  scanf("%d",numOfIntegers);
  return;
}


void initialize(int numOfIntegers,int* arrayNum[])
{
  int i;
  for(i=0; i<(numOfIntegers); i++)
    arrayNum[i] = getValue(i);

  wait(500);
  printf("\nPlease enter press any button to continue");
  wait(100);
  getch();
  wait(600);
  system("cls");
  wait(200);
  return;
}


int main()
{
  int numOfIntegers;
  prepare(&numOfIntegers);
  int arrayNum[numOfIntegers];
  initialize(numOfIntegers, &arrayNum[numOfIntegers]);
  controlMenu(numOfIntegers, &arrayNum[numOfIntegers]);
  return 0;
}


void controlMenu(int numOfIntegers, int arrayNum[])
{
  int i;
  char chooseNum;
  printf("Please choose any of the following:\n\n1. The integers accepted\n2. The sum of the integers\n3. The average of the integers\n4. The product of the integers\n5. The smallest integer\n6. The largest integer\n0. Exit menu\n");
  while(1)
  {
    chooseNum = getch();
    switch(chooseNum)
    {
      case '0':
        return;
      case '1':
        printf("\n>>> The integers are:");
        for(i=0; i<(numOfIntegers); i++)
          {
            printf("\n>>> The %s is %d", getName((i+1)), arrayNum[i]);
          }
        break;

      case '2':
        printf("\n>>> The sum of integers is: %d", sumOfIntegers(numOfIntegers, &arrayNum[numOfIntegers]));
        break;

      case '3':
        printf("\n>>> The average of integers is: %d", avgOfIntegers(numOfIntegers, &arrayNum[numOfIntegers]));
        break;

      case '4':
        printf("\n>>> The product of integers is: %d", prodOfIntegers(numOfIntegers, &arrayNum[numOfIntegers]));
        break;

      case '5':
        printf("\n>>> The smallest integer is: %d", minInteger(numOfIntegers, &arrayNum[numOfIntegers]));
        break;

      case '6':
        printf("\n>>> The largest integer is: %d", maxInteger(numOfIntegers, &arrayNum[numOfIntegers]));
        break;

      default:
        break;
    }
    printf("\n\n");
  }
}


int sumOfIntegers(int numOfIntegers,int arrayNum[])
{
  int sum=0;

  for(int i=0; i<(numOfIntegers); i++)
    sum += arrayNum[i];

  return sum;
}


int avgOfIntegers(int numOfIntegers, int arrayNum[])
{
  int average=0;

  average = sumOfIntegers(numOfIntegers, arrayNum[numOfIntegers])/numOfIntegers;

  return average;
}


int prodOfIntegers(int numOfIntegers, int arrayNum[])
{
  int i,product=0;

  for(i=0; i<(numOfIntegers); i++)
    product *= arrayNum[i];

  return product;
}


int minInteger(int numOfIntegers, int arrayNum[])
{
  int i,smallest=0;
  smallest = arrayNum[0];

  for(i=1; i<(numOfIntegers); i++)
    {
      if(smallest>arrayNum[i])
        smallest=arrayNum[i];
      else
        continue;
    }

  return smallest;
}


int maxInteger(int numOfIntegers, int arrayNum[])
{
  int i,largest=0;
  largest = arrayNum[0];

  for(i=1; i<(numOfIntegers); i++)
    {
      if(largest<arrayNum[i])
        largest=arrayNum[i];
      else
        continue;
    }

  return largest;
}


void wait(int ms)
{
  Sleep(ms);
  return;
}

我可以在 getName() 中看到这个错误,它将访问超出数组范围的内存

if (value>10 || value<1)
    return arrayName[value];

我相信你用错了测试,试试

if (value <= 10 && value > 0)
    return arrayName[value-1];

假设 value1..10 范围内,正如文本数组所暗示的那样。

2) GetValue 中的错误,您输入 numOfIntegerreturn value 未初始化。

3) 在 prepare 语句中

scanf("%d",&numOfIntegers);

不会将输入值传递回调用者。您应该已经传递了一个指向变量的指针,或者返回了输入的值。

但可能还有很多其他错误。一步一步地构建你的程序,检查并尝试在你进行时破坏它(使用荒谬的输入)。注意编译器警告 - 我列出的第二个错误会产生一个。

EDIT 好吧...让我们检查函数 prepare 去除噪声后是

void prepare(int numOfIntegers)
{
    scanf("%d",&numOfIntegers);
    return;
}

这为传递的函数参数输入了一个值。这是合法的,因为您可以像使用局部变量一样使用函数参数(可能需要 const 限定)。

虽然不是编码错误,但没有任何作用。 1) 您通常传递这样的参数以供函数以某种方式使用,也许在其提示符的限制 and/or 内。 2) 像这样改变参数不会找到返回调用者的方式。

这里有两种方法可以解决这个问题。

A) 函数returns 输入值

int prepare(void)
{
    int value;
    scanf("%d", &value);    // the address of value
    return value;
}

...

int input = prepare();
printf("%d\n", input);

B) 函数接受一个指针参数

void prepare(int *value)
{
    scanf("%d", value);     // value is already a pointer
}

...

int input;
prepare(&input);
printf("%d\n", input);