2017-06-10 6 views
-4

私はjavascriptで書いていませんが、私の友人は1つのカタで彼を助けることができるかどうか尋ねました(https://www.codewars.com/kata/ascii-fun-number-1-x-shape/) 私は彼にこのコードを渡しましたが、私が間違っていたことを誰かに教えてもらえますか?D? エラーメッセージ: コードの実行が完了するまでにサーバーがタイムアウトしました。これはサーバーの負荷が高いためです。非効率なコードによっても発生する可能性があります。再度リクエストをお試しください。コードはjavascriptでコンパイルされていません

function x(n) 
{ 
    printlines(n); 
    print("\n"); 
    for(column = 0; column < n; column++) 
    { 
    if(column = 1) 
    { 
     print("■"); 
    } 
    else 
    { 
     print("□"); 
    } 
    } 
    print("\n"); 
    printlines(n); 
} 


function print(n) 
{ 
    console.log(n); 
} 

function printlines(n) 
{ 
    i = 1; 
    for(line = 0; line < 1; line++) 
    { 
    for(column = 0; column < n; column++) 
    { 
     if(column = line) 
     { 
     print("■"); 
     } 
     else if(column = n - 1) 
     { 
     print("■"); 
     } 
     else 
     { 
     print("□"); 
     } 
    } 
    print("\n"); 
    i++; 
    } 
} 
+0

JavaScriptはコンパイルされた言語ではありません。あなたはコードを実行しようとしていて、失敗しているのですか、または正しい答えを出すことができませんか?あなたは直面している問題に関するもう少し情報を提供できますか? –

+0

_「誰かが私が間違っていたことを教えてもらえますか?あなたは自分自身についての最初の手がかりを持たないもので誰かを "助け"ようとしています。それはどうやってやるの? – CBroe

+0

エラーメッセージ:サーバーがコードの実行を完了するのを待ってタイムアウトしました。これはサーバーの負荷が高いためです。非効率なコードによっても発生する可能性があります。 – Baczek

答えて

1

あなたのループforに(iとラインに同じ)、それを宣言するためにvar columnを追加する必要があり、そしてあなたのifにuが(平等をテストするために====の代わり=を記述する必要が

+0

"出力の処理に失敗しました。データが大きすぎるとSTDOUTに書き込まれる可能性があります。" – Baczek

+0

'x(n)'を実行していますか?そして、あなたはどの値をnに与えますか?単純に:どの機能を実行しますか? –

0

I JavaScriptを有効にするのに便利なhttp://jslint.comを使用してください。しかし、バットからすぐに、forループとJavaScriptで比較するために "=="または "==="を使用する反復処理に使用する変数を宣言する必要があることを伝えることができます。

うまくいけば、これはあなたを正しい道に導きます。

関連する問題