Questione : Aiutarlo a capire perché un codice è migliore dell'altro .....

Sia il codice #1 che il codice #2 raggiunge lo stesso risultato.  Ho bisogno della comprensione di aiuto perché una può essere migliore del other. class= > " libero " del
> " del codeSnippet " del class= del
class= " lineNumbers " del
class= del 
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
53:
54:
55:
56:
57:
58:
59:
60:
61:
62:
63:
64:
65:
66:
67:
68:
69:
70:
71:
72:
73:
74:
75:
76:
77:
78:
79:
80:
81:
82:
83:
84:
85:
86:
87:
88:
89:
90:
91:
92:
93:
94:
95:
96:
97:
98:
99:
100:
101:
102:
103:
104:
105:
106:
107:
108:
109:
110:
111:
112:
113:
114:
115:
116:
117:
118:
119:
120:
121:
122:
123:
124:
125:
126:
127:
128:
129:
130:
131:
132:
133:
134:
135:
136:
137:
138:
139:
140:
141:
142:
143:
144:
145:
146:
147:
148:
149:
150:
151:
152:
153:
154:
155:
156:
157:
158:
159:
160:
161:
162:
163:
164:
165:
166:
167:
168:
169:
170:
171:
172:
173:
174:
>CODE notpretty #1 " del class= " del id= " codeSnippet835862 del 
 del #include
 del #include
 del #include
 del #include
 del #include
 del #include
 del #include

using il namespace std;

struct StudentAttempt
{
   distanza di int;
   studentName della stringa;
};

cmpAttempts di bool (StudentAttempt lasciato, destra di StudentAttempt) {
   se (left.distance < right.distance) {
      di ritorno allineare;
   }
   altrimenti se (== right.distance di left.distance) {
      se (left.studentName < right.studentName) {
         di ritorno allineare;
      }
      altrimenti
         falso di ritorno;
   }
   altrimenti
      falso di ritorno;
}

bool isDistEqual (StudentAttempt lasciato, destra di StudentAttempt) {
   rinviare (== right.distance di left.distance
      && left.studentName! = right.studentName);
}

size_t printAllEqual (vector& v, StudentAttempt& sa, stringstream& ss del const del const) {

   vector:: const_iterator i = v.begin ();
  size_t la SZ = 0;
  il bFirst di bool = allinea;

  mentre (i! = v.end ()) {

    se (isDistEqual (*i, sa)) {
    
      se (! bFirst) ss << “,„;
      ss << i->studentName;
      ++sz;
    }

    ++i; 
    bFirst = falso;
  }

  la SZ di ritorno;
}
 

conduttura di int () {

   throwDist di StudentAttempt [] = {
      {50, “A„}, {22, “A„}, {16, “B„}, {44, “C„},
      {33, “D„}, {34, “E„}, {22, “F„}, {21, “G„},
      {49, “A„}, {5, “B„}, {2, “C„}, {22, “A„},
      {33, “B„}, {22, “C„}, {22, “D„}, {44, “E„},
   };

   set trattato;

   int len = sizeof)/sizeof (throwDist [0] (di throwDist);
   dist del vector (throwDist, throwDist + len);
   vector:: iterator esso = dist.begin ();

   specie (dist.begin (), dist.end (), cmpAttempts);
   //unique (dist.begin (), dist.end (), cmpDist);


   per (; esso! = dist.end (); it++) {

      stringstream ss;

      pair:: iterator, bool> p = handled.insert (it->distance);

      se (! p.second) continuano; // già trattato

      se (0 < printAllEqual (dist, *it, ss)) 
        cout << setw (2) << (*it) .distance 
           << “piedi:  „ << (*it) .studentName << ss.str () << endl;
   }
}

CODICE #2:
 del #include
 del #include
 del #include
 del #include
 del #include
 del #include
 del #include

using il namespace std;

struct StudentAttempt
{
   distanza di int;
   studentName della stringa;
};

 cmpAttempts di bool (StudentAttempt lasciato, destra di StudentAttempt) {
   se (left.distance < right.distance) 
   {
      di ritorno allineare;
   }
   altrimenti se (== right.distance di left.distance) 
   {
      se (left.studentName < right.studentName) 
	  {
         di ritorno allineare;
      }
      altrimenti
         falso di ritorno;
   }
   altrimenti
      falso di ritorno;
}

conduttura di int () {

   throwDist di StudentAttempt [] = {
      {50, “A„}, {22, “A„}, {16, “B„}, {44, “C„},
      {33, “D„}, {34, “E„}, {22, “F„}, {21, “G„},
      {49, “A„}, {5, “B„}, {2, “C„}, {22, “A„},
      {33, “B„}, {22, “C„}, {22, “D„}, {44, “E„},
   };

   set trattato;

   int len = sizeof)/sizeof (throwDist [0] (di throwDist);
   dist del vector (throwDist, throwDist + len);
   vector:: iterator esso = dist.begin ();

   specie (dist.begin (), dist.end (), cmpAttempts);
   //unique (dist.begin (), dist.end (), cmpDist);

   int x;
  per (x=0; x < dist.size (); x++) 
   {
      stringstream ss;
      pair:: iterator, bool> p = handled.insert (dist [x] .distance);
	  int z = 0;

      se (p.second)  
	  {
		  int y;

     	  per (y=0; y < dist.size (); y++) 
		  {
			  se (dist del && di dist del == di dist [x] .distance [y] .distance [x] .studentName! = dist [y] .studentName)
				{
 					  ss << “,„;
					  ss << dist [y] .studentName;
					  z += 1;
				}
		  }
	 }
	se (z > 0)
	  {
	   cout << setw (2) << dist [x] .distance << “piedi: „ << dist [x] .studentName << ss.str () << endl;
	   z = 0;
	  }
   }
}
class= del

Risposta : Aiutarlo a capire perché un codice è migliore dell'altro .....

Per i iterators… Può esaminare non così la prima volta, ma i iterators sono un genere di più naturale.  Quando risolvete un problema, tendete a pensare nei termini “procedate tutti gli elementi da…„, o matematicamente “per tutti gli elementi in…„.  L'unico senso convenzionale fare che è di usare un ciclo.  Il ciclo di istante nelle lingue della C-famiglia è troppo generale; avete fate alcuni punti sui vostri propri.  Per il ciclo nelle lingue della C-famiglia è un piccolo miglioramento sintattico dell'istante.  In altri linguaggi di programmazione, per il ciclo è un ciclo contato.  Ma questo inoltre significa che dovete trasformare un indice all'elemento spostato ad incrementi.  A causa di quello, tendete spesso a mettere gli elementi in una certa struttura spostata ad incrementi -- ad un allineamento o ad un vettore.

Se pensate di più a questo proposito, che cosa il vostro realmente bisogno è un genere di ciclo generalizzato quel cicli attraverso tutti gli elementi di un contenitore senza necessariamente preparare un certo indice numerico.  Se personificate il processo, l'operaio dice a altro “mi dà quello seguente„ e non “darmi quello 3254th„.  Ciò è esattamente che cosa i iterators sono buoni per.  Il iterator è in qualche modo tipo-limita al contenitore.  Allora potete essere non interessato in dettaglio interni (se è spostato ad incrementi all'interno, o se è una struttura collegata).  Cioè i iterators portano la più astrazione.

Purtroppo, i iterators sono più nuovi di il centro di C++ e quindi là è ciclo e sintassi generalizzati per fare sintatticamente tutti di questo più apparente.  Una volta che abituate il migliore sytax del ciclo generalizzato (in altre lingue, come nel pitone in cui i iterators funzionano quasi magico, nascosto), realmente mancate quella sintassi in C++.  Il nuovo campione di C++ sta andando migliorarlo una punta.
Altre soluzioni  
 
programming4us programming4us