c-分割故障错误是由于调用用户在终端中输入的文本的函数而引起的

Question

摘要

我正在尝试传递用户在终端中放置的文本，并将其传递到名为printString（）的函数。我不完全了解C，但我认为我与不在堆中的指针有关。任何帮助都将得到认可！

#include <stdio.h>
void printString();
int main (int argc, char* argv[]) {
        int commandAmount = 1;
        while (commandAmount < argc) {
        printString(commandAmount, &argv);
        }
        return 0;
}
void printString(int commandAmount, char* argv[]) {
        printf("the word is %s," , argv[commandAmount]);
}

./shortExample example

Segmentation fault (core dumped) 

Answer 1

原型void printstring（）;与实际实现不匹配。
应该是：

void printString(int commandAmount, char* argv[]);

您还可以跳过原型，然后在main之前实现该功能。您的循环while（CommandAmount＆lt; argc）似乎没有任何方法可以完成，因为您永远不会增加CommandAmount。这可能导致不确定的行为，因此您的程序可能会崩溃或做任何事情。
我建议为 -loop做一个来解决这个问题。

示例：

#include <stdio.h>

void printString(int commandAmount, char* argv[]) {
    printf("the word is %s,", argv[commandAmount]);
}

int main(int argc, char* argv[]) {
    for(int commandAmount = 1; commandAmount < argc; ++commandAmount) {
        printString(commandAmount, argv);
    }
}

或以您结构的方式：

#include <stdio.h>

void printString(int commandAmount, char* argv[]); // corrected

int main(int argc, char* argv[]) {
    int commandAmount = 1;
    while (commandAmount < argc) {
        printString(commandAmount, argv);
        ++commandAmount;                           // needed
    }
}

void printString(int commandAmount, char* argv[]) {
    printf("the word is %s,", argv[commandAmount]);
}

Answer 2

对于初学者，此功能声明

void printString();

不提供函数原型。 编译器从函数调用确定函数参数的

printString(commandAmount, &argv);

因此，

printString(commandAmount, &argv);

类型标识符的声明argv，

int main (int argc, char* argv[]) {
                    ^^^^^^^^^^^^

但是函数定义中的相应参数printString具有type char **，因为编译器参数调整了将数组类型用于指针到数组元素类型。

也就是说，该函数声明

void printString(int commandAmount, char* argv[]) {

由编译器调整为

void printString(int commandAmount, char** argv) {
                                    ^^^^^^^^^^^

参数表达式和参数的不兼容类型。结果，此呼叫

printf("the word is %s," , argv[commandAmount]);

调用了未定义的行为。

此外，总体而言，此循环通常

    int commandAmount = 1;
    while (commandAmount < argc) {
    printString(commandAmount, &argv);
    }

是无限循环，因为变量CommandAmount在循环中没有更改。

首先，在main之前，您应该提供功能原型，以使您的程序更安全

void printString(int commandAmount, char** argv);

，并将功能调用

printString(commandAmount, argv);
                           ^^^^

当然还需要更改Main中的循环。

请注意这一点，因为参数的值CommandAmount未在功能中输出，因此实际上是多余的。您可以将指针传递给字符串本身的功能。例如

#include <stdio.h>

void printString( const char *s );

int main( int argc, char* argv[] ) 
{
    for ( int commandAmount = 1; commandAmount < argc; commandAmount++ ) 
    {
        printString( argv[commandAmount] );
    }
    putchar( '\n' );

    return 0;
}

void printString( const char *s ) 
{
    printf( "the word is %s, " , s );
}