我的function或主要问题是什么？

除了我的“播放”function之外，我的所有代码都按照我希望的方式工作。 我正在尝试让程序读取“记录”function创建的.txt文件，然后回放说明。 不幸的是，当用户按下P以播放他们记录的音符时，它只是不断弹出相同的菜单而不是前进到程序的下一步。 任何帮助高度赞赏，它可能是一些小的，但我一直在努力，我正在努力解决自己的问题。

#include "aservelibs/aservelib.h" #include  #include  #include  float mtof(int note, float frequency); FILE play(void); FILE record(void); FILE record2(void); int main() { FILE *textFilePointer; FILE *textFilePointer2; int counter = 0; char user; do { printf("Press A to Record 1st Melody (A), B to Record 2nd Melody (B)\nP to Play Melodies (P):"); scanf(" %c", &user); if (user == 'a' || user == 'A') { textFilePointer = fopen("/Users/Luke/Desktop/midinotes1.txt", "w"); *textFilePointer = record(); counter = 0; } else if (user == 'b' || user == 'B') { textFilePointer2 = fopen("/Users/Luke/Desktop/midinotes2.txt", "w"); *textFilePointer2 = record2(); counter = 0; } else if (user == 'p' || user == 'P') { textFilePointer = fopen("/Users/Luke/Desktop/midinotes1.txt", "r"); textFilePointer2 = fopen("/Users/Luke/Desktop/midinotes2.txt", "r"); counter = 0; } } while(counter  0) { fprintf(file, "%d, %d\n", note, velocity); counter++; } } while (counter  0) { fprintf(file, "%d, %d\n", note, velocity); counter++; } } while (counter < 16); fclose(file); return *file; } 

Answer 1

  fclose(file); return *file; 

如果您关闭它，则无法再使用它。 如果您希望以后仍然使用它（请执行此操作），请不要关闭file 。

Answer 2

你永远不会像以下一样调用播放function：

 else if (user == 'p' || user == 'P') { ... play (); } 

Answer 3

你第二次打电话给fopen后可能是这样的：

  file = fopen("/Users/Luke/Desktop/midinotes1.txt", "r"); file = fopen("/Users/Luke/Desktop/midinotes1.txt", "r"); 

FILE * file无效。 至少，它们是多余的。
您应该删除其中一行。

此外，在使用之前检查该file是否有效：

 file = fopen("/Users/Luke/Desktop/midinotes1.txt", "r"); if(file) //if fopen fails, file == NULL { do { ... 

Answer 4

我在您的程序中看到的问题

1. 当用户输入A或a ，执行：

     textFilePointer = fopen("/Users/Luke/Desktop/midinotes1.txt", "w"); *textFilePointer = record(); counter = 0; 

   录制到"/Users/Luke/Desktop/midinotes1.txt"的代码已经在record()了硬编码。 这里不需要在同一个文件上使用fopen() 。

   record()不需要返回FILE 。 另外，使用FILE作为返回类型似乎很奇怪。 我不认为该标准甚至支持使用FILE 。 我所看到的是使用FILE*作为参数和返回值。

   同样的事情适用于record2() 。

2. 当用户输入p或P ，执行：

     textFilePointer = fopen("/Users/Luke/Desktop/midinotes1.txt", "r"); textFilePointer2 = fopen("/Users/Luke/Desktop/midinotes2.txt", "r"); counter = 0; 

   这里没有打电话play() 。 也许在你抄写代码发布时丢失了。 即使添加一行

     play(); 

   在那里，没有必要在这里使用fopen() 。 play()已经打开了要播放的文件。 此外，您将多次打开文件，而不是使用它们或关闭它们。

3. 您在main中的所有三个if块中将counter设置为0 。 counter值何时为16才能满足do-while循环的退出标准？ 这是疏忽吗？

4. 在play()的实现中，你有：

    file = fopen("/Users/Luke/Desktop/midinotes1.txt", "r"); file = fopen("/Users/Luke/Desktop/midinotes1.txt", "r"); 

   这看起来不对。 你在同一个文件上调用fopen两次。 当函数返回时，您将保留未使用且打开的FILE* 。

   第二个文件永远不会播放。

   我将建议创建一个辅助函数playFile() ，它需要播放给定文件的内容。 然后，从play()调用playFile()两次。 这是伪代码：

    void playFile(char const* file) { FILE* file = fopen(file, "r); if ( file == NULL ) { // deal with error. } else { // Your code to play the contents. // Close the file fclose(file); } } void play() { playFile("/Users/Luke/Desktop/midinotes1.txt"); playFile("/Users/Luke/Desktop/midinotes2.txt"); } 

遵循DRY原则

我将建议创建一个函数recordToFile() ，它具有公共代码record()和record2() ，然后使用正确的参数从record()和record2()调用它。

 void recordToFile(char const* file) { int counter; FILE* file = fopen(file, "w"); if ( file == NULL ) { // Deal with error } else { // Your code to record to the file. // Close the file fclose(file); } } void record(void) { recordToFile("/Users/Luke/Desktop/midinotes1.txt"); } void record2(void) { recordToFile("/Users/Luke/Desktop/midinotes2.txt"); }