コードを2つの別々の関数に分割して文字列長を返すようにCプログラムをコードしようとしています

Question

私はいくつかの例とビデオを見てきましたが、私が見ていないところで私の部分に基本的な誤解があるように感じます。

#include <stdio.h> 
#include <stdlib.h> 
int main (void) 
{ 
    char String; 
    char *sendString; 
    char getString[100]; 
    int counter; 

    *sendString = String; 

    printf("\nPlease enter string:\n"); 
    fgets(getString, 100, stdin); 

    int calcLength(counter); 

    printf("los1111: %d", counter); 

    return 0; 
} 

int calcLength() 
{ 
    char *sendString[100]; 
    int counter; 
    for(counter = 0; *sendString[counter] != '\0'; counter++); 

    printf("los: %d", counter); 
    return counter; 
} 

コードがどこで壊れているかを確認するためにいくつかのテスト 'los' printf関数がありますが、残念ながら私には何ももたらされません。どんな助けでも大歓迎です。

Answer 1

私はあなたを少し複雑に思っています。このコードブロックで

：

char String; 
char *sendString; 
char getString[100]; 
int counter; 

*sendString = String; 

だけfgetsから文字列の長さを計算する場合は、これらのchar宣言のほとんどを必要としません。

だけ fgetsに文字列バッファ、および、最終的な文字列の長さを計算するために使用される整数宣言 myStrLenを与える必要が

char getstring[100]; 
int myStrLen; 

：

これは単純に減らすことができます。

さらに、calcLength()機能にいくつかの問題があります。 char []パラメーターをこの関数に渡す必要があるため、長さの計算に必要な文字列を知っています。それは本当にこのような単純なする必要があります：このコードを向上させることができるが、それはポイントにあなたを助ける必要があります。

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

#define BUFFSIZE 100 

int calcLength(char sendstring[]); 

int 
main(void) { 
    char getstring[BUFFSIZE]; 
    int myStrLen; 

    printf("\nPlease enter string:\n"); 
    if (fgets(getstring, BUFFSIZE, stdin) != NULL) { 
     getstring[strlen(getstring)-1] = '\0'; // To remove '\n' character from fgets() 
    } 

    myStrLen = calcLength(getstring); 

    printf("String length = %d\n", myStrLen); 

    return 0; 
} 

int 
calcLength(char sendstring[]) { 
    int counter, len = 0; 

    // Every valid character found, increment a counter 
    for (counter = 0; sendstring[counter] != '\0'; counter++) { 
     len++; 
    } 
    return len; 
} 

注：これらの推奨事項のすべてに

int 
calcLength(char sendstring[]) { 
    int counter, len = 0; 

    for (counter = 0; sendstring[counter] != '\0'; counter++) { 
     len++; 
    } 
    return len; 
} 

、あなたのコードは次のようになります。正しい方向に。私は、明確化のためにコードにいくつかのコメントを追加しました。