Segmentation fault

zio_mangrovia

il 22 mag 2018, 21:05

15 risposte

zio_mangrovia

Utente Attivo

Iscritto da: ott, 2015

Messaggi:: 301

Segmentation fault

22 mag 2018, 21:11

#include <iostream>
using namespace std;

const int P=5;
const int S=5;

class Distributore {
   int** armadio;
   int ripiani;
public:
   Distributore::Distributore(const int);
   friend ostream& operator<<(ostream& os, const Distributore&);
}

Distributore::Distributore(const int num)
{
   ripiani=num;
   armadio=new int*[ripiani];
   for(int r=0; r<ripiani; r++)
      armadio[r]=new int[S];
      for(int r=0; r<ripiani; r++)
         for(int c=0; c<S; c++)
            armadio[r][c]=1;
}

ostream& operator<<(ostream& os, const Distributore& dis)
{
   cout << endl;
   for(int x=0; x<dis.ripiani; x++) {
      cout << endl << x+1 << ": ";
      for(int y=0; y<S; y++)
         cout << dis.armadio[x][y] << ' ';
      }
}
[i][/i]
int main()
{
Distributore d(3);
cout << d <<endl;
}

Sono quasi 6 h che cerco di spulciare questo codice ma non capisco perchè ottengo all'esecuzione del programma "segmentation fault".
Questo errore lo elimino solo se sostituisco nel main la riga di codice cout << d <<endl; con cout << d;
Come mai?

15 Risposte

O
oregon
Super Famoso
Iscritto da
nov, 2011
Messaggi:
21552
Re: Segmentation fault
22 mag 2018, 21:34
Ma scusa ... in questa parte di codice
```
stream& operator<<(ostream& os, const Distributore dis)
{
   cout << endl;
   for(int x=0; x<dis.ripiani; x++)
      cout << endl << x+1 << ": ";
      for(int y=0; y<S; y++)
         cout << dis.armadio[x][y] << ' ';
}
```
sei sicuro che sia tutto a posto? Io vedo un problema nella dichiarazione, nel fatto che non restituisce nulla e in quei cicli strani ...
Z
zio_mangrovia
Utente Attivo
Iscritto da
ott, 2015
Messaggi:
301
Re: Segmentation fault
22 mag 2018, 21:48
oregon ha scritto:

stream& operator<<(ostream& os, const Distributore dis) { cout << endl; for(int x=0; x<dis.ripiani; x++) { cout << endl << x+1 << ": "; for(int y=0; y<S; y++) cout << dis.armadio[x][y] << ' '; } }
sei sicuro che sia tutto a posto? Io vedo un problema nella dichiarazione, nel fatto che non restituisce nulla e in quei cicli strani ...
Incredibile, è quasi tutto il giorno e tu in mezzo sec!
Ho corretto la dichiarazione aggiungendo &.
E' vero dovrebbe esserci un 'return os' !

Perchè cicli strani?!
Nippolo
Molto attivo
Iscritto da
mar, 2018
Messaggi:
1241
Re: Segmentation fault
22 mag 2018, 22:03
Guarda che il codice che hai riportato nel post iniziale contiene molti errori, a me infatti non compila.
Z
zio_mangrovia
Utente Attivo
Iscritto da
ott, 2015
Messaggi:
301
Re: Segmentation fault
22 mag 2018, 22:09
Nippolo ha scritto:
Guarda che il codice che hai riportato nel post iniziale contiene molti errori, a me infatti non compila.
l'ho ricopiato a mano, non ho potuto fare copia e incolla del codice.
Presumo che ci sia qualche errore di trascrizione, il compilatore non dava errori.
Ma essenzialmente ciò che avevo dimenticato era il return nella funzione operator responsabile del segmentation fault.

Secondo voi perchè si parla di cicli strani? Ho sbagliato qualcosa?
Nippolo
Molto attivo
Iscritto da
mar, 2018
Messaggi:
1241
Re: Segmentation fault
23 mag 2018, 00:06
Per evitare di parlare di presunti errori che in realtà non esistono, riguardati il codice e, nel caso ci siano errori di trascrizione, correggilo.
Z
zio_mangrovia
Utente Attivo
Iscritto da
ott, 2015
Messaggi:
301
Re: Segmentation fault
23 mag 2018, 06:16
Nippolo ha scritto:
Per evitare di parlare di presunti errori che in realtà non esistono, riguardati il codice e, nel caso ci siano errori di trascrizione, correggilo.
ok
O
oregon
Super Famoso
Iscritto da
nov, 2011
Messaggi:
21552
Re: Segmentation fault
23 mag 2018, 07:17
Le due for sono indipendenti o uno dentro l'altra?
Come mai usi x nella seconda for?
Z
zio_mangrovia
Utente Attivo
Iscritto da
ott, 2015
Messaggi:
301
Re: Segmentation fault
23 mag 2018, 08:24
oregon ha scritto:
Le due for sono indipendenti o uno dentro l'altra?
Come mai usi x nella seconda for?
Mi riscuso per gli errori di trascrizione, vedo di stare più attento nei prossimi thread.
Ho corretto, mancava una parentesi graffa.
I for sono uno dentro l'altro, la x nel secondo for è utilizzata perchè devo stampare una matrice.
Z
zio_mangrovia
Utente Attivo
Iscritto da
ott, 2015
Messaggi:
301
Re: Segmentation fault
23 mag 2018, 08:25
oregon ha scritto:
Ma scusa ... in questa parte di codice

stream& operator<<(ostream& os, const Distributore dis) { cout << endl; for(int x=0; x<dis.ripiani; x++) { cout << endl << x+1 << ": "; for(int y=0; y<S; y++) cout << dis.armadio[x][y] << ' '; } }
sei sicuro che sia tutto a posto? Io vedo un problema nella dichiarazione, nel fatto che non restituisce nulla e in quei cicli strani ...
O
oregon
Super Famoso
Iscritto da
nov, 2011
Messaggi:
21552
Re: Segmentation fault
23 mag 2018, 08:30
L'ultimo argomento non è passato correttamente come ti era stato detto.

Vuoi postare il codice completo e corretto altrimenti non se ne esce. Ma tu lo compili? Copia e incolla il codice che compili, non trascriverlo nel forum.
Z
zio_mangrovia
Utente Attivo
Iscritto da
ott, 2015
Messaggi:
301
Re: Segmentation fault
23 mag 2018, 08:39
Si compilo, ma ho risolto grazie al suggerimento del valore di ritorno che avevo dimenticato nella funzione operator<<
O
oregon
Super Famoso
Iscritto da
nov, 2011
Messaggi:
21552
Re: Segmentation fault
23 mag 2018, 09:02
Il compilatore dobrebbe avvisarti che manca il valore di ritorno.
Z
zio_mangrovia
Utente Attivo
Iscritto da
ott, 2015
Messaggi:
301
Re: Segmentation fault
23 mag 2018, 10:01
oregon ha scritto:
Il compilatore dobrebbe avvisarti che manca il valore di ritorno.
sinceramente non mi ha avvisato, ho notato soltanto che se nel codice inserivo cout << d << endl; ottenevo un errore in esecuzione mentre se lasciavo cout << d non avevo alcun errore.
In entrambi i casi la compilazione non dava alcun errore.
Se lo ritenete interessante vedo di fare il copia incolla del codice.
Nippolo
Molto attivo
Iscritto da
mar, 2018
Messaggi:
1241
Re: Segmentation fault
23 mag 2018, 12:04
Ecco i vari errori/imprecisioni che ho notato:
- nel prototipo del costruttore non c'è bisogno di utilizzare l'operatore di risoluzione di visibilità ::, inoltre se dichiari l'argomento const tanto vale passarlo per riferimento e non per valore;
- nel prototipo dell'overload dell'operatore <<, l'identificatore os è superfluo (soprattutto se per tutti gli altri argomenti hai omesso l'identificatore);
- nella definizione del costruttore mancano delle parentesi graffe e inoltre il secondo for è inutile;
- mancano delle parentesi graffe anche nella definizione dell'overload di <<.

Devi accedere o registrarti per scrivere nel forum

15 risposte