CS50 Pset3 音乐 - sizeof(string) 在做什么？答案

【问题标题】：CS50 Pset3 Music - what is sizeof(string) doing?CS50 Pset3 音乐 - sizeof(string) 在做什么？
【发布时间】：2019-04-22 18:17:04
【问题描述】：

在 CS50 的 pset3 中，我们应该理解 notes.c 文件：

// 打印频率并输出 WAV 文件，所有音符都在一个八度范围内

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

#include "helpers.h"
#include "wav.h"

// Notes in an octave      
const string NOTES[] = {"C", "C#", "D", "D#", "E", "F",
                    "F#", "G", "G#", "A", "A#", "B"
                   };

// Default octave 
#define OCTAVE 4

int main(int argc, string argv[])
{
// Override default octave if specified at command line
int octave = OCTAVE;  
if (argc == 2)
{
    octave = atoi(argv[1]); 
    if (octave < 0 || octave > 8)
    {
        fprintf(stderr, "Invalid octave\n");
        return 1;
    }
}
else if (argc > 2)
{
    fprintf(stderr, "Usage: notes [OCTAVE]\n");
    return 1;
}

// Open file for writing
song s = song_open("notes.wav");

// Add each semitone
for (int i = 0, n = sizeof(NOTES) / sizeof(string); i < n; i++)
{
    // Append octave to note
    char note[4];
    sprintf(note, "%s%i", NOTES[i], octave);

    // Calculate frequency of note
    int f = frequency(note);

    // Print note to screen
    printf("%3s: %i\n", note, f);

    // Write (eighth) note to file
    note_write(s, f, 1);
}

// Close file
song_close(s);
}

我无法理解的部分是： for (int i = 0, n = sizeof(NOTES) / sizeof(string); i

要使sizeof(string) 工作，代码中的某处不需要名为string 的变量吗？例如一个实际上叫做string的字符串？

不太清楚它指的是什么。

【问题讨论】：

鉴于NOTES 被定义为const string NOTES[] = ...，它为您提供了它实际上是一个类型的线索。
在处理 CS50 时，最好去掉对 string 的所有引用，这会混淆基本的 C 类型 - char *。 CS50 试图简化 C 字符串的工作方式的错误尝试就像在不解释数字是什么的情况下教算术一样。它根本不起作用，因为它妨碍了正确使用 C 字符串所需的基本理解。
注意 CS50 中的 string typedef，因为它定义的实际上不是 string，而是指向 char (char *) 的指针。在 C 中，字符串是包含 0 值终止符的字符值序列。字符串存储在char 的数组中。在大多数情况下，“T”类型的表达式被转换为“指向T”的指针，所以大多数时候当我们处理字符串时，我们正在处理char *类型的表达式，但是char * 不是字符串。它提供了一个关于字符串处理在 C 中的实际工作方式的误导性图片。

标签： c computer-science sizeof cs50

【解决方案1】：

sizeof 可用于变量/表达式以及类型。这里，string 是一个类型，但 NOTES 是一个变量（数组）的实例。

问题的根源是由 CS-50 typedef char* string; 引起的。总体而言，该代码是混淆的完美示例。在 typedef 后面隐藏指针被广泛认为是一种糟糕的做法。

代码实际上是这样写的：

const char* NOTES[] = { ...
...
sizeof(NOTES) / sizeof(NOTES[0])

如果它像上面那样写，毫无疑问它做了什么：将数组的大小除以每个成员的大小，得到数组中的项目数。

我建议停止使用 CS-50 教程。

【讨论】：

谢谢，这是有道理的。既然我已经开始了 CS50，我可能会完成它，但出于兴趣，你能特别推荐任何其他 c 课程/书籍吗？

【解决方案2】：

为了让sizeof(string) 工作，代码中的某处不需要名为string 的变量吗？

不，sizeof 运算符可以用于“类型”的操作数，例如 sizeof(int)。引用规范（强调我的）

sizeof 运算符产生其操作数的大小（以字节为单位），可能是表达式或带括号的类型名称。 [....]

你可以在你的代码中看到string的用法：

const string NOTES[] = {"C", "C#", "D", "D#", "E", "F",
                    "F#", "G", "G#", "A", "A#", "B"
                   };

所以，NOTES 是 type 数组，string，string 是这里的类型名称。

表达式

 for (int i = 0, n = sizeof(NOTES) / sizeof(string); i < n; i++)

对数组中的成员进行计数的尝试非常糟糕，它可能被重写为

 for (int i = 0, n = sizeof(NOTES) / sizeof(NOTES[0]); i < n; i++)

为了可读性，它基本上将整个数组的大小除以一个元素的大小，产生成员的计数。

要添加确切的来源，Check the header file <cs50.h>、string 被定义为 type 那里。

确切的定义是：

 /**
 * Our own data type for string variables.
 */
typedef char *string;

【讨论】：