• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

SPOJ - SIGSEGV

VPS Starter Arubacloud
0 głosów
183 wizyt
pytanie zadane 17 grudnia 2017 w SPOJ przez niezalogowany
zmienione kategorie 17 grudnia 2017

Witam

Mam dobrze zrobiony program do zadania :

http://pl.spoj.com/problems/FR_02_06/

Jak w temacie, problem jest z błąd wykonania(SIGSEGV). Dowiedziałem się że to oznacza jakieś problemy z używaniem pamięci. Tutaj mój kod:

#include <iostream>
#include <iomanip>

using namespace std;

int E;
float w,m,r[100];
string imie[100],n[100];

int main()
{
    cin>>E;

    for(int i=1; i<=E; i++)
    {
        cin >> imie[i] >> w >> m;
        r[i]=(w/(m*m))*10000;
    }
    for(int i=1; i<=E; i++)
    {
        if(r[i]<=18.5) n[i]="niedowaga";
        if(r[i]>=25) n[i]="nadwaga";
        if (r[i]<25 && r[i]>18.5) n[i]="wartosc_prawidlowa";
    }
    cout << "niedowaga"  << endl;
    for(int i=1; i<=E; i++)
    {
        if(n[i] == "niedowaga")
        {
            cout<<imie[i]  << endl;
        }
    }
    cout << "wartosc prawidlowa"  << endl;
    for(int i=1; i<=E; i++)
    {
        if(n[i] == "wartosc_prawidlowa")
        {
            cout<<imie[i] << endl;
        }
    }
    cout << "nadwaga"  << endl;
    for(int i=1; i<=E; i++)
    {
        if(n[i] == "nadwaga") cout<<imie[i]  << endl;
    }
}#include <iostream>
#include <iomanip>

using namespace std;

int E;
float w,m,r[100];
string imie[100],n[100];

int main()
{
    cin>>E;

    for(int i=1; i<=E; i++)
    {
        cin >> imie[i] >> w >> m;
        r[i]=(w/(m*m))*10000;
    }
    for(int i=1; i<=E; i++)
    {
        if(r[i]<=18.5) n[i]="niedowaga";
        if(r[i]>=25) n[i]="nadwaga";
        if (r[i]<25 && r[i]>18.5) n[i]="wartosc_prawidlowa";
    }
    cout << "niedowaga"  << endl;
    for(int i=1; i<=E; i++)
    {
        if(n[i] == "niedowaga")
        {
            cout<<imie[i]  << endl;
        }
    }
    cout << "wartosc prawidlowa"  << endl;
    for(int i=1; i<=E; i++)
    {
        if(n[i] == "wartosc_prawidlowa")
        {
            cout<<imie[i] << endl;
        }
    }
    cout << "nadwaga"  << endl;
    for(int i=1; i<=E; i++)
    {
        if(n[i] == "nadwaga") cout<<imie[i]  << endl;
    }
}

Jakieś pomysły jak to ugryźć?

 

1 odpowiedź

+1 głos
odpowiedź 17 grudnia 2017 przez Aisekai Nałogowiec (42,190 p.)
1. Te 2 mainy to przez przypadek wkleiles tak?

2. Jeżeli będzie w testach jedna osoba, to po co tworzysz 3x tablice po 100 wartości?

3. Tablice są iterowane od 0, nie od 1.

4. Nie sądzisz, że lepiej użyć else-if niż bawić się w koniunkcje warunków?

5. Po co Ci aż 4 pętle? Nie lepiej było po prostu zrobić trzech, obliczac BMI i sprawdzać czy jest np niedowaga, jeżeli jest to wypisać. Tak zrobić dla 2 pozostałych. Albo zrobić 3 vectory i do nich wrzucać niedowage, nadwagę i wartość poprawna.

Ale głównym problemem może być to, że niepotrzebnie deklarujesz 100 elementowe tablice.

Podobne pytania

0 głosów
1 odpowiedź 116 wizyt
pytanie zadane 28 grudnia 2019 w C i C++ przez amtrax Dyskutant (9,630 p.)
0 głosów
1 odpowiedź 932 wizyt
pytanie zadane 9 marca 2018 w C i C++ przez Lucash Nowicjusz (240 p.)
0 głosów
2 odpowiedzi 331 wizyt

92,454 zapytań

141,262 odpowiedzi

319,089 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...