[Risolto] [python]verifica condizione non funzionante

[pandaubuntu]

Codice: Seleziona tutto

class fast():
   def __init__(self):
   self.narg = len(sys.argv) - 1
   self.arg = sys.argv
   self.contatore = 1
   def controll(self):
   contatore = 1
   while 1:
      #esecuzione di altro codice che usa la variabile MA NON LA MODIFICA
           self.contatore = self.contatore + 1
      if not(self.narg > self.contatore):
      break
        return 0

in pratica questa classe mi serve per verificare le operazioni che deve svolgere a seconda dei degli argomenti ricevuti dell'utente, ho dovuto usare un ciclo perché c'é la possibilità che vengano inserite più cose ma non funziona l'incremento di self.contatore e di conseguenza genera un loop infinito sul primo argomento.

[Zoff]

Controlla l'indentazione.

[pandaubuntu]

Mi ha copiato male l'ndentazione  adesso direi che è quella giusta

[Zoff]

Se è esattamente quella del file non va ancora bene.

[pandaubuntu]

cosa ha che non va 

[Zoff]

O usi i tab o usi gli spazi, non dovresti usare entrambi.

[pandaubuntu]

Codice: Seleziona tutto

class fast():
   def __init__(self):
      self.narg = len(sys.argv) - 1
      self.arg = sys.argv
      self.contatore = 1
   def controll(self):
      contatore = 1
      while 1:
         #esecuzione funzioni
         self.contatore = self.contatore + 1
         if not(self.narg > self.contatore):
            break
      return 0


ora va bene secondo te, a me và comunque in loop perché non mi incrementa

[Zoff]

A me così funziona.

[Bakuriu]

Codice: Seleziona tutto

class fast():
   def __init__(self):
      self.narg = len(sys.argv) - 1
      self.arg = sys.argv
      self.contatore = 1
   def controll(self):
      contatore = 1
      while 1:
         #esecuzione funzioni
         self.contatore = self.contatore + 1
         if not(self.narg > self.contatore):
            break
      return 0


ora va bene secondo te, a me và comunque in loop perché non mi incrementa

A cosa dovrebbe servire la prima riga? contatore = 1? Il confronto lo fai su self.contatore.

Comunque a mio parere è molto meglio evitare il confronto come lo fai tu... è più complesso da capire.
Fai:

Codice: Seleziona tutto

if self.contatore >= self.narg:
    break

È più chiaro e più efficiente.

Altra piccola cosa: self.contatore = self.contatore + 1 è meglio scritto come self.contatore += 1.

[pandaubuntu]

avevo fatto una boiata nel main, per quello andava in loop, ma il codice và bene

Codice: Seleziona tutto

class fast():
   def __init__(self):
      self.narg = len(sys.argv) - 1
      self.arg = sys.argv
      self.contatore = 1
   def controll(self):
      contatore = 1
      while 1:
         #esecuzione funzioni
         self.contatore = self.contatore + 1
         if not(self.narg > self.contatore):
            break
      return 0


ora va bene secondo te, a me và comunque in loop perché non mi incrementa

A cosa dovrebbe servire la prima riga? contatore = 1? Il confronto lo fai su self.contatore.

Comunque a mio parere è molto meglio evitare il confronto come lo fai tu... è più complesso da capire.
Fai:

Codice: Seleziona tutto

if self.contatore >= self.narg:
    break

È più chiaro e più efficiente.

Altra piccola cosa: self.contatore = self.contatore + 1 è meglio scritto come self.contatore += 1.

Infatti contatore non serviva più e l'ho eliminato.
per il confronto, è necessario usare il ciclo per fare dei vari confronti, sono obbligato a fare un ciclo
per il += grazie della dritta  ;)

[Bakuriu]

In realtà il tutto lo puoi scrivere molto più pythonicamente come un ciclo for(così elimini il confronto direttamente):

Codice: Seleziona tutto

for _ in range(1, self.narg+1):
    #esecuzione funzioni


L'unica differenza tra questo for e il while è che il for potrebbe non essere eseguito, nel caso non ci siano argomenti allo script.
(come se il confronto fosse all'inizio del while e non alla fine). Controlla se ti serve davvero l'equivalente del do..while o se ti basta il for come l'ho scritto io.