CodingBat sum67: pourquoi est-ce une solution de mal?

Je suis en train de travailler sur la suite de codingbat problème:

Retour à la somme des nombres dans le tableau, à l'exception ignorer les sections de numéros commençant par un 6 et s'étendant jusqu'à la prochaine 7 (tous les 6 sera suivie par au moins un 7). De retour de 0 pour pas de numéros.
sum67([1, 2, 2]) → 5
sum67([1, 2, 2, 6, 99, 99, 7]) → 5
sum67([1, 1, 6, 7, 2]) → 4

Ma solution est la suivante:

def sum67(nums):
    sum = 0 
    throwaway = 0
    for i in range(len(nums)):
        if throwaway == 0:
            if nums[i] == 6:
                throwaway = 1
        elif throwaway == 1 and i > 0 and nums[i-1] == 7:
            throwaway = 0
        if throwaway == 0:
            sum += nums[i]
    return sum

Je suis totalement d'sais que ce n'est pas la meilleure solution, mais je suis juste curieux de savoir pourquoi c'est mauvais. Pourriez-vous svp m'expliquer pourquoi c'est faux, et dans lequel cas particulier, il donne un mauvais résultat?

Êtes-vous conscient de la bool type?
for i in range(len(nums))? Eek!
totalement [k]maintenant, ce n'est pas la meilleure solution" -- il n'y a que peu de sens de critiquer son style. En dehors de n'être pas très "pythonic", il n'y a rien de mal avec for i in range(len(seq)):
Merci Ferdinand! Facile pour moi, les gars! Cependant, je ne sais for i in range(len(seq)): n'est pas très pythonic, mais je voulais passer à l'élément précédent de la liste à l'aide [i-1]. Évidemment, ce n'est pas nécessaire, mais j'ai juste écrit la première solution qui est venu à mon esprit, sans beaucoup de réflexion (comme vous pouvez le dire...), Le grand Nick aurait pu ajouter visible comme le test d'un Ferdinand suggéré: sum67([6,7,6,7]) -> 0, juste pour prendre mon erreur en compte. Bonne journée les gars!

InformationsquelleAutor Wilco | 2011-12-05

python

5

Bien, votre programme a un bug. Vérifier les résultats de l'suivantes:
```
print sum67([1,2,5])
print sum67([1,2,6,5,7])
print sum67([1,2,6,5,7,6,7])
```
Ce sera d'impression:
```
8
3
16 <-- wrong
```
Si un 7 est suivie par un 6 immédiatement, vous devrez ajouter le 6 et tous les numéros suivants. Je ne suis pas sûr si plus d'une gamme de 6 ... 7 est autorisée dans l'entrée, mais si c'est le cas, vous devez corriger votre algorithme.

Cette simple mise en œuvre ne retour bons numéros:
```
def sum67(nums):
        state=0
        s=0
        for n in nums:
                if state == 0:
                        if n == 6:
                                state=1
                        else:
                                s+=n
                else:
                        if n == 7:
                                state=0
        return s
```
En outre, si vous n'avez pas besoin d'utiliser un index pour quelques obscures raisons, vous pouvez directement effectuer une itération sur les éléments d'une liste ( for element in list: ... ).
- +1: Plusieurs 6-7 plages sont autorisées (il y a des tests pour ça); le plus simple des cas de test qui échoue dans le cas des OP solution serait sum67([6,7,6,7]) -> 0
- Merci man, j'ai trop compliqué sans aucune raison: ma solution serait de travailler si j'ai ajouté " et nums[i]!=6 à la ligne 8, mais, évidemment, le vôtre est beaucoup mieux. Oui, j'ai utilisé un indice parce que je voulais accéder à l'élément précédent de la liste, mais comme vous montrons ce n'est pas nécessaire.
- En fait, vous pouvez faire le state variable dans ma solution un bool (True ou False), mais l'entier solution a l'avantage que vous pouvez avoir plus de deux membres et est, à mon avis, plus pédagogique. Content d'avoir pu aider!
InformationsquelleAutor hochl

Ci-dessous est ma solution pour votre référence:

def sum67(nums):
flag=False
sum=0

for num in nums:
    if(num==6):                  #Turn the flag on if the number is 6
        flag=True
        continue
    if(num==7 and flag is True): #Turn the flag Off when 7 is seen after 6
        flag=False
        continue
    if(flag is False):           #Keep on adding the nums otherwise
       sum+=num
return sum

InformationsquelleAutor Anirudh

public int sum67(int[] nums) {
 int sum=0;
  for(int i=0; i<nums.length ; i++)



{
          if(nums[i]==6)
               for(int k=i+1 ; k<nums.length ; k++ )
                  {if(nums[k]==7)
                                {i=k; break;}
                  }   
           else if(nums[i]==6) 
              sum=sum+nums[i];
          else
              sum=sum+nums[i];


  }
  return sum;
}

InformationsquelleAutor TEERATH KUMAR

1

Juste pour ajouter, si le dernier élément est un 7, vous n'avez pas besoin de parcourir chaque élément. La question des états " ignorer les sections de numéros commençant par un 6 et s'étendant jusqu'à la prochaine 7 (tous les 6 sera suivie par au moins un 7)". Une fois que vous trouver un 6, vous pouvez simplement vérifier si le dernier élément est un 7 et si tout simplement sortir et de retourner la somme; sinon, vous pouvez continuer à passer par les éléments restants. Ceci est utile lorsque vous travaillez avec un grand nombre d'éléments dans un tableau.
ex. [1, 2, 2, 6, 99, 99, 2, 99, 99, 2, 99, 99, 2, 99, 99, 7]

Ma solution:
```
def sum67(arr):
    sum = 0
    tmp = 0
    for n in arr:
        if n == 6:
            tmp = 6
            continue
        if tmp == 6:
            if n == 7:
                tmp = 0
                continue
            elif arr[-1] == 7:
                break
            else:
                continue
        sum += n
    return sum
```
- sum67([1,2,6,5,7,1,6,7]) ne correspond pas à la réponse.
InformationsquelleAutor Victor S

Voici ma solution à ce problème. Comme déjà répondu, le problème est lorsqu'un 6 se produit immédiatement après un 7. J'ai résolu ce problème de manière un peu différente, j'ai donc pensé à la poste.

def sum67(nums):
  total = 0
  i=0  
  while i < len(nums):
    if nums[i] == 6:
      while nums[i] != 7:
        i+=1
      i+=1
    if i<len(nums) and nums[i]!=6:  
      total+=nums[i]
      i+=1
  return total

InformationsquelleAutor TheNeophyte

Ma solution:

def sum67(nums):
    result = 0
    flag = True
    for num in nums:
        if num == 6:
            flag = False
        if flag:
            result += num
        if num == 7:
            flag = True
    return result

InformationsquelleAutor K. Menyah

Je sais que ce n'est pas la meilleure solution, mais la pensée à l'action :

def detect67(nums):
  count = nums.count(6)
  pos6 = 0
  pos7 = 0
  sum1 = []
  for i in range (len(nums)):
    sum1.append(nums[i])  
    # if I do just sum1 = nums, they use SAME memory locations.
  #print ("Count=",count)
  for i in range (count):
    pos6 = sum1.index(6)
    pos7 = sum1.index(7)
  if pos7<pos6:
    sum1[pos7] = 0
    pos7 = sum1.index(7)

    del nums[pos6:pos7+1]
    del sum1[pos6:pos7+1]
    count = nums.count(6)
    if count == 0:
      return (nums)



  return (nums)


def sum67(nums):

 detect67(nums)
 sums=0
 if nums == []:
   return 0
 for i in range (len(nums)):
   sums+=nums[i]
 return sums

InformationsquelleAutor Arun

C'est mon montage de @TheNeophyte code. J'ai aimé cette solution parce que je serais probablement approcher le problème. La seule édition était vraiment de prendre le

if i<len(nums) and nums[i]!=6:

et juste mettre un elif déclaration sans dire

i<len(nums)

puisqu'il n'est pas nécessaire de nouveau après la première boucle while.

Mon montage ci-dessous VVV

def sum67(anArray):
    arrTotal = 0
    n =0
    while n < len(anArray):
        if anArray[n] == 6:
            while anArray[n] != 7:
                n+=1
            n+=1
        elif anArray[n] != 6:
            arrTotal += anArray[n]
            n+=1
    return arrTotal

InformationsquelleAutor michael harris

0

voici mes 6 liner réponse. Espérons que cela aide
```
def summer_67(arr):
s = 0
for x in arr:
    if x not in range(6,7+1):
        s = s + x
return s
```
- La question n'est pas de demander une autre solution au problème, la question est de se demander pourquoi la solution ne fonctionne pas.
InformationsquelleAutor Ryan Darren Morales

Bien, vous pourriez utiliser des boucles imbriquées..

def sum67(nums):
    s=0
    flag=1
    for n in nums:
        while flag:
            if n!=6:
                s+=n
                break
            else:
                flag=0
        while not flag:
            if n!=9:
                break
            else:
                flag=1
    return s

InformationsquelleAutor Pyrosill

Ma solution pour cela, je Sais que c'est assez lisible mais pas le meilleur code:

def sum67(nums):
  offcount = False
  sum = 0 
  for i in range(len(nums)):
    if offcount == False:
      if nums[i] == 6:
        offcount = True
      else:
        sum = sum + nums[i]
    else:
      if nums[i] == 7 :
        offcount = False
  return sum

InformationsquelleAutor Dave Charm Bulaquena

J'espère que cela aide:)

def sum67(nums):
    result = 0
    inside = False
    for i in range(len(nums)):
        if not inside and nums[i] != 6:
            result += nums[i]
        if nums[i] == 6:
            inside = True
        if nums[i] == 7:
            inside = False
    return result

InformationsquelleAutor Madhan

def sum67(nums):
  sum = 0
  sumbrake = False

  for i in nums:
    if i != 6 and sumbrake == False:
      sum += i
    else:
      sumbrake = True

    if i == 7:
      sumbrake= False

  return sum

InformationsquelleAutor SayedRakib

0
```
def sum67(nums):

    while nums.count(6) >= 1:
        list = (nums[0:nums.index(6)]) + (nums[nums.index(7)+1:len(nums)])
        nums = list
    return sum(nums)
```
- Salut à tous, bienvenue sur stackoverflow. Pouvez-vous veuillez donner une explication de ce que votre code est en train de faire et pourquoi il répond à la question d'origine?
InformationsquelleAutor V VISHNUJYOTHI

def sum67(arr):
    for n in range (0, len(arr)):
        if arr[n] == 6:
            Sum1 = sum(arr[0:n])
            print ("Before 6 Sum Numbers:", Sum1)
    for k in range (0, len(arr)):
        if arr[k] ==7:
            Sum2 = sum(arr[(k)+1:])
            print ("After 7 Sum Numbers:", Sum2)
    if 6 and 7 in arr:
        print("Last Result")
        return (Sum1+Sum2)
    else:
        print("Last Result")
        return sum(arr)

14 d'autres réponses à une question posté il y a 7 ans. Pourquoi est votre solution unique ou digne de considération? Sans une explication, il sera très probablement aller ignorés par ceux qui visitent ce post à la recherche de l'aide.

InformationsquelleAutor Saidv2007

Vous devez vous connecter pour publier un commentaire.