Proste dodawanie spoj

Question

Witam.

Wiem, że pytanie były już podobne pytania, jakkolwiek dręczy mnie dlaczego w moim kodzie nie dziala poprawnie ilość testów. Z góry dziękuję.

#include <iostream>

using namespace std;

int x,n;

int main()

{cin>>x;

for(int i=0;i<x;i++)

{

cin>>n;

float liczby[n],suma=0;

for(i=0;i<n;i++)

{cin>>liczby[i];

}

for(int i=0;i<n;i++)

{

suma=suma+liczby[i];

}

cout<<suma;

}

 

return 0;

}

Answer 1

Zacznijmy od tego, że:

cin>>n;

float liczby[n];

Coś takiego jest niemożliwe i jedynie na codeblocks to zadziala. Tablice musisz alokować dynamicznie.

Po za tym radzę używać typu double, a nie float, ze względu na precyzje.

int n;
cin>>n;

double* liczby = new double[n];

 

Jak już zakończysz działanie programu nie zapomnij zwolnić pamięci

delete [] liczby;

 

A co do głównego pytania. Pętla tak się zachowuje, ponieważ we wnętrzu pętli głównej inkrementujesz zmienną "i", która odpowiada za pętle z ilością testów. Musisz zmienić nazwę zmiennej.

 Wczytywanie i sumowanie danych możesz zrobić w jednej pętli. 

 for(int j=0;j<n;j++)
  {
       cin>>liczby[j];
       suma += liczby[j];
  }

Staraj się pisać kod bardziej przejrzysty.  

Comments

1. VLA działa na dowolnym kompilatorze z rodziny GCC, Clangu chyba też, IDE nie ma tu nic do rzeczy.

2. double i float nie robią dużej żadnej różnicy przy normalnych liczbach a to pierwsze działa znacznie wolniej, po double sięga się kiedy jest potrzebne.

3. Swoim zaleceniem co do zwalniania pamięci sugerujesz błędne podejście do dynamicznej alokacji, powinien użyć std::vector<float> buffer(n) i nawet nie tykać new i delete, w C++ surowych wskaźników używa się w ostateczności.

4. Powinieneś wspomnieć o tym, że nie używa się zmiennych globalnych a część użytych przez niego typów jest zła (np. n powinno być bez znaku).

Popraw to to może cofnę downvote, teraz twoja odpowiedź jest bezużyteczna.

---

Autor pytania jest początkującym w programowaniu, więc zalecenie do tworzenia dynamicznej tablicy przez wskaźnik jest jak najbardziej uzasadnione - powinien umieć sprawnie się nimi posługiwać.

---

Tak było przed C++11, teraz raw pointerów można się uczyć dodatkowo / na wysokim poziomie, nie mają żadnych zastosowań dla początkującego programisty + często ludzie się przez nie poddają uważając za zbyt trudne. W C wskaźniki to podstawy, ale w C++ używa się ręcznie tyle co w Javie.

---

Jak najbardziej się z tym zgadzam, że nie są często używane, ale nie wyklucza to, że nie powinien umieć się nimi posługiwać. Wskaźniki zawsze będą trudne, póki solidnie się ich nie przećwiczy ;)

---

Dokładnie, nie mówię żeby się ich nie uczył, po prostu zalecam odsunięcie tego w czasie aż będzie dobrze rozumiał funkcję i obiektówkę, więc będzie mógł się nauczyć wskaźników raz a porządnie (a nie na raty) + z dużym zapasem wiedzy i doświadczenia łatwiej przyswaja się takie techniczne tematy.