逆配列C++？私は間違って何をしていますか？

Question

int main(){ 
    int n; 
    cin >> n; 
    vector<int> arr(n); 
    for(int i =0; i < n; i++){ 
     cin >> arr[i]; 
    } 
    for(int i = n-1; n-1 >= 0; i--){ 
     cout << " "<< arr[i] << " "; 
    } 
    return 0; 
} 

私は間違っていますか？私は正しい軌道に乗っているように感じますが、何が間違っているのか分かりません。これはハッカーのランク上の配列DSの問題です。残念ながら私は彼らが答えを明らかにするとは思わない。

Answer 1

i >= 0に条件を変更してください。n - 1 >= 0が無限ループにつながる可能性があります。しかし、どちらの場合でも、必要な結果が得られないため、そうしているかどうかは関係ありません。

for(int i = n-1; i >= 0; i--){ 
    std::cout << " "<< arr[i] << " "; 
} 

またはstd::reverseを使用してください。

std::reverse(arr.begin(), arr.end()); 
for (int i = 0; i < n; i++){ 
    std::cout << arr[i] << " "; 
} 

live hereを参照してください。

Answer 2

for(int i = n-1; n-1 >= 0; i--){ 
    cout << " "<< arr[i] << " "; 
} 

あなたは無限ループにつながるnは、減少しませんnは< = 0

Answer 3

車輪の再発明をしないであれば、標準ライブラリは、ここであなたを助けるでしょう。
使用std::reverse：両方の場合において

int main() 
{ 
    std::vector<int> vec; 
    int n = 0; 
    int temp = 0; 

    std::cin >> n; 
    for (int i = 0; i != n; ++i) { 
     std::cin >> temp; 
     vec.emplace_back(temp); 
    } 

    std::reverse(std::begin(vec), std::end(vec)); 
} 

Answer 4

、必要に応じて（第2のループで）デクリメント/（最初のループで）I = 0変数を初期化し、カウントアップをとっています。 しかし、2番目のループでは、nの値を決して変更しないという事実にもかかわらず、条件をnに設定しました。これは、n-1がプログラム全体にわたって一定である限り、無限ループにつながります。 これを整理するには、以下のコードのように条件をi-1> 0に変更する必要があります。

for(int i = n-1; i-1 >= 0; i--) 
{ 
    cout << " "<< arr[i] << " "; 
} 

このhelps.Cheersの場合は、アップしてください。

Answer 5

for(int i = n-1; n-1 >= 0; i--)

ここで、あなたはあなたのforループの条件で間違いをしました。 n-1にはiを割り当て、デクリメントセクションではi--を使用しました。したがって、ループを停止するか、またはループの終了値がiの条件を使用する必要があります。それでは、次のように条件を変更：[ループ]

for(int i = n-1; i >= 0; i--)