我正在尝试编写一个程序，其中系统需要给定文本的莫尔斯电码。关于将文本转换为莫尔斯电码，我将它们全部写在 main 中(与程序文件本身分开(。现在，我的目标是将其编写为函数，以便在程序的其他函数中使用它。每当我尝试时，它都会给出分段错误。谁能帮我从头开始构建函数本身？
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#include<stdint.h>
#include<ctype.h>
#include <time.h>
char * fileName1 = NULL;
char * fileName2 = NULL;
int main(int argc, char * argv[]) {
int n;
for (n = 0; n < argc; n++) {
// printf("Argument %sn",argv[n]); // prints options delete this in the end,just for debugging
if (strcmp(argv[n], "-text") == 0) {
//text to morsecode
int c, v = 0;
char * str = (char * ) malloc(v);
str = (char * ) realloc(str, (c + strlen(argv[n + 1])));
strcat(str, argv[n + 1]);
strcat(str, " ");
char *alphamorse[]={".-","-...","-.-.","-..",".","..-.","--.","....","..",".---","-.-",".-..","--","-.","---",".--.","--.-",".-.","...","-","..-","...-",".--","-..-","-.--","--.."};
char *nummorse[]={"-----",".----","..---","...--","....-",".....","-....","--...","---..","----."};
int i;
char str1[1000];
i = 0;
while (str[i] != ' ') {
if (str[i] != ' ' && (!isdigit(str[i]))) {
printf("%s ", alphamorse[toupper(str[i]) - 65]);
}
if (str[i] == ' ') {
printf(" ");
}
if (isdigit(str[i]) && str[i] != ' ') {
printf("%s ", nummorse[str[i] - 48]);
}
i++;
}
printf("n");
// end of text to morsecode
}
if (strcmp(argv[n], "-o") == 0) {
//output = concat(output, argv[n + 1]);
n++;
continue;
}
if (strcmp(argv[n], "--") == 0) {
if (n + 1 <= argc) {
fileName1 = argv[++n];
printf(" fileName1=%sn", fileName1);
}
if (n + 1 <= argc) {
fileName2 = argv[++n];
printf(" fileName2=%sn", fileName2);
}
}
}
return 0;
}
您真的不需要也不想复制参数。 做这样的事情似乎最简单：
#include <stdio.h>
#include <ctype.h>
char *alphamorse[] = {
".-", "-...", "-.-.", "-..", ".", "..-.", "--.", "....", "..", ".---", /* A - J */
"-.-", ".-..", "--", "-.", "---", ".--.", "--.-", ".-.", "...", "-", /* K - T */
"..-", "...-", ".--", "-..-", "-.--", "--.." /* W - Z */
};
char *nummorse[]={
"-----", ".----", "..---", "...--", "....-",
".....", "-....", "--...", "---..", "----."
};
void
text_to_morse(char c, FILE *out)
{
if( islower(c) ) {
fputs(alphamorse[c - 'a'], out);
fputc(' ', out);
} else if( isdigit(c) ) {
fputs(nummorse[c - '0'], out);
fputc(' ', out);
} else if( isspace(c) ) {
fputc(c, out);
} else {
fputc(' ', out);
}
}
int
main(void)
{
int c;
while( ( c = tolower(getchar())) != EOF ) {
text_to_morse(c, stdout);
}
return 0;
}
甚至不要为操作输出文件的代码而烦恼。 外壳的存在是有原因的，没有必要重新实现轮子。
我不知道这是否是导致问题的错误，但它是一个错误：
int c, v = 0;
char *str = (char *)malloc(v);
str = (char *)realloc(str, (c + strlen(argv[n+1])));
首先，c未初始化。 它可以是任何值，包括负值。 因此，程序中存在未定义的行为。
此外，不需要 malloc 后跟 realloc 调用。只需分配一次即可完成。
我认为这就是你打算做的
size_t len = strlen(argv[n+1]);
str = (char*)malloc(len + 1 + 1); // +1 for space char to be appended, +1 again for null char
strcpy(str, argv[n+1]); // copy string
strcat(str, " "); // append a space
但还有一个更简单的解决方案。 您甚至不需要将argv[n+1]复制到str.只需将str声明为指针并直接引用argv[n+1]即可。
const char* str = argv[n+1];
现在str和argv[n+1]引用相同的字符串。str对整个程序都有效。 程序的其余部分保持不变。
这看起来很可疑：
i = 0;
while (str[i] != ' ') {
if (str[i] != ' ' && (!isdigit(str[i]))) {
printf("%s ", alphamorse[toupper(str[i]) - 65]);
}
if (str[i] == ' ') {
printf(" ");
}
if (isdigit(str[i]) && str[i] != ' ') {
printf("%s ", nummorse[str[i] - 48]);
}
i++;
}
您正在冗余地调用isdigit并进行评估以确保str[i]不是空格。如果您已经知道它是一个数字，那么检查它是否是一个空格是没有意义的。要么是数字、字母，要么是无法转换的东西。 您的代码将错误地将标点标记视为要在alphamorse中查找的值。 以下内容将跳过标点符号，仅将这些字符视为空格。
i = 0;
while (str[i] != ' ') {
if ((str[i] >= 'A' && str[i] <= 'Z') || (str[i] >= 'a' && str[i] <= 'z')) {
printf("%s ", alphamorse[toupper(str[i]) - 'A']);
}
else if (isdigit(str[i])) {
printf("%s ", nummorse[str[i] - '0']);
}
else {
printf(" ");
}
i++;
}
之后的一切，我不知道它是干什么用的。一般建议是先解析argv[]中的参数。 然后在循环访问命令行参数的循环之外执行文本转换。然后执行保存到文件代码。