文字列と文字を連結しようとしたときのCのセグメンテーションエラー

Question

文字列テストに文字を追加しようとすると、セグメンテーションエラーが発生します。私は複数の反復を試みたが、なぜエラーが出るのか理解できない。テストを\0に設定しようとしました。私は外部テストにどのようにアクセスしているのか分かりません。

#include <stdio.h> 
#include <cs50.h> 
#include <ctype.h> 
#include <string.h> 

#define _XOPEN_SOURCE 
#include <unistd.h> 
#include <crypt.h> 

//this intializes the argument count and argument vector 
int main(int argc, string argv[]) { 
    //this makes sure it is a string 
    if (argv[1] != NULL) { 
     //this makes sure that the user typed 2 arguments, 
     // the second being the keyword 
     if (argc != 2) { 
      printf("no hash inputed\n"); 
      return 1; 
     } 

     //main program once the keyword has been verified 
     int h_len = strlen(argv[1]); 
     string hash = argv[1]; 
     string test = "A"; 
     int p_len = 0; 
     printf("H len is %i and P len is %i\n", h_len, p_len); 
     printf("%s and test: %s\n", hash, test); 

     //this will iterate through the 1st space and test the charachters 
     for (int j = 0; j < 4; j++) { 

      //iterates through the characters that can be used in the password 
      //for (int i = A; i < z; i++) 
      //for (char ch = 'A' ; ch <= 'z' ; ch == 'Z' ? ch = 'a' : ++ch) 
      for (char i = 'A'; i <= 'z'; i++) { 
       //this skips the ASCII inbetween Z and a 
       if (i > 'Z' && i < 'a') { 
        i = 'a'; 
       } 
       printf("%c", i); 
       //test[j] = test[j] + (char)i; 
       test = strncat(test, &i, 1); 
       printf(" test is %s\n", test); 
       ... 

Answer 1

すべてがここで間違っている

あなたカントstrcatのはchar *と、その文字の後にゼロがある場合は、あなたが知っていることはないと文字のアドレス。文字列が短すぎるため、1つの文字列よりも長い文字列には対応できません。次に、testが文字列リテラルを指しているので、ROメモリを使って何かをしようとします。

購入C

Answer 2

に関する良い本あなたはそれで一つの文字（テスト=「A」）で文字列を持っているので、それが2（A +ヌル文字）の長さを有し、他の文字を追加しようとしていますそれで、それはテストの外にアクセスします。この宣言で

Answer 3

string test = "A"; 

リテラル"A"文字列へのポインタtestが宣言されています。文字列リテラルは変更できません。

C標準（6.4.5文字列リテラル）

7によれば、これらの配列は、それら 要素が適切な値を有する別個設けられているかどうかを指定されていません。 にプログラムの試みは、このような配列を変更した場合、動作は

test = strncat(test, &i,1); 

この声明の中でしかし

未定義であるリテラルは、ポインタ testによって指される文字列を変更しようとする試みがあります。

追加の文字を格納するのに十分なスペースがある文字配列を宣言する必要があります。また

あなたは関数が常に終端ゼロを付加しますので、あなたがn + 1文字のコピー先の配列にメモリを確保する必要があり、ソース文字列からnの文字をコピーする機能strncatを使用している場合。

ある（C規格、7.23.3.2 strncat関数、302脚注）

302）したがって、 配列で終わることができる文字の最大数は、S1によって指さがstrlen関数であります（s1）+ n + 1となる。この恐ろしいCS50で、stringは、char *として定義されていること

Answer 4

注のでstring test = "A";長さ1とNULLの文字列定数にテストポイントを行いますchar *test = "A";と同等です！

何かを連結するのに十分な長さであっても、それは読み取り専用であり、読み取り専用ではないにしても不可能ではありません。