前提・実現したいこと

write関数を使って正の値はNそれ以外はPを表示する。

発生している問題・エラーメッセージ


04.c:5:10: error: redefinition of 'n'
            int n = 1;
                ^
04.c:3:28: note: previous definition is here
void    ft_is_negative(int n)
                           ^
04.c:11:3: error: expected expression
                else
                ^
04.c:17:17: error: too few arguments to function call, single argument 'n' was not
      specified
        ft_is_negative();
        ~~~~~~~~~~~~~~ ^
04.c:3:1: note: 'ft_is_negative' declared here
void    ft_is_negative(int n)

該当のソースコード

void ft_is_negative(int n)
{
int n = 1;
char input;

if(n < 0)
input = 'N';
write(1, &input, 1);
else
input = 'P'
write(1, &input, 1)
}
int main(void){

ft_is_negative();
return 0;
}

C言語

試したこと

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

行動規範の内容に同意します

回答1件

ベストアンサー

コンパイルエラー修正

error: redefinition of 'n'

すでに n が定義されているのにまた定義している、というエラーです。
これは引数として int n を受け取っていることに起因します。

したがって、該当する文を削除します。

error: expected expression

添付されたコードは文法が誤っています。
波括弧の数が誤っていたので修正します。

error: too few arguments to function call, single argument 'n' was not

関数 ft_is_negative は引数 n を受け取るはずが、ここではそれを渡していません。
したがって、引数に数値を渡すよう変更します。

論理エラー修正

c
1if(n < 0) {
2    input = 'N';
3    write(1, &input, 1);
4}

Nを出力するのは n が正数のときです。ゆえに、条件式が誤っています。

その他

こちらの環境ではコンパイル時に以下のような警告が発生しました。

c
1Main.c:8:9: warning: implicit declaration of function 'write' is invalid in C99 [-Wimplicit-function-declaration]
2        write(1, &input, 1);

unistd.h をインクルードして修正します。

コード

c
1#include <unistd.h>
2
3void ft_is_negative(int n) {
4    char input;
5    
6    if(n > 0) {
7        input = 'N';
8        write(1, &input, 1);
9    } else {
10        input = 'P';
11        write(1, &input, 1);
12    }
13}
14    
15int main(void){
16    int number = 0;
17    ft_is_negative(number);
18    return 0;
19}