システムコールでファイルの行数を表示したい

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
タイトルの通りのプログラムを作りたいのですが、以下の警告が出てしまいます。どこが原因なのでしょうか。
またコマンド、./a.out /proc/meminfo　を打つと正しい結果は42になるはずなのですが、自分のコードだと10064になってしまいます。
間違っている部分を教えていただけると幸いです。

plactice.c: In function ‘main’:
plactice.c:15:24: warning: assignment makes integer from pointer without a cast [-Wint-conversion]
if (buf[i] = "\n");
エラーメッセージ

### 該当のソースコード

```ここに言語名を入力
ソースコード
#include <stdio.h>
#include <unistd.h>
#include <fcntl.h>

int main(int argc, char* argv[]){
    int cnt = 0;
    char buf[1024];
    int fd, n, i;

    fd = open(argv[i], O_RDONLY);
    while ((n = read(fd, buf, sizeof(buf)))>0){
        for (i = 0; i < n; i++){
            if (buf[i] = "\n");
                cnt+=1;
        }
    }

    close(fd);

    printf("%d\n", cnt);

    return 0;
}

### 試したこと

ここに問題に対して試したことを記載してください。

### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

for (i = 0; i < n; i++){
if (buf[i] = "\n");
cnt+=1;
}
}

この狭い領域だけで3つの間違いがあります。

1. if((buf[i] = "\n")が代入であること（比較演算は==です）
2. buf[i]と文字列の"\n"を比較しようとしていること（buf[i]はcharですので、文字の'\n'と比較するのが適切です）
3. if((buf[i] = "\n");とセミコロンが入っているので、条件分岐がこのセミコロンまでとなってしまいます。

`

Comments

[mo_chi_o]

とてもわかり易い説明ありがとうございます。

Answer 2

変数は初期化してから使いましょう。

Answer 3

やってみてませんけど
張り付けているソースをコピペすると

if (buf[i] = "\n");

となっていますが

if (buf[i] == '\n')

の間違いではありませんか？

合っているかどうかの判断は 「=」でなく「==」
文字は「"」で囲むのでなく「'」で囲む（「"」で囲むと文字でなく文字列の扱いになる）
if () の後に「;」を書くと 空行を実行してしまう
（if の結果次の行を１行実行したいのであれば ; をつけてはいけない
1行でも {} で囲む方が間違いが少ないです）

Answer 4

if (buf[i] = "\n");

これ、間違ってます

Answer 5

とりあえずエラーのところ。

if (buf[i] = "\n");
↓
if (buf[i] == '\n');