geekcomputers / geekcomputers/Python
Multiple issues with the CountMillionCharacters family of programs.
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 35.4k
- Fork
- 12.9k
- Merge medio
- 2h 37m
- PR unite (30g)
- 1
Descrizione
The README states:
CountMillionCharacter.py And CountMillionCharacter2.0.py - Gets character count of a text file.
Yet CountMillionCharacter2.0.py seems to be counting words, and says as much.
There is also a folder CountMillionCharacters-Variations which contains a third version of the program, named variation1.py. It would obviously be good to decide on what the CountMillionCharacters programs are meant to do, and the versions/variations to create.
For now, I think there are some changes to be made in at least two of the versions:
- For one, dropping the use of
.upper(), or at least replacing it with.casefold(). - The use of
.get()invariation1.pycould be replaced with adefaultdict. variation.pycontains a camelCase variable name.CountMillionCharacter.pyusesexit()for seemingly no good reason.CountMillionCharacter.py[callsmain()] instead of exiting the program or using a loop.- ...and more.
I will try to tweak the code and create a pull request (or a few).
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia dalle voci del README per CountMillionCharacter.py e CountMillionCharacter2.0.py, quindi confronta questi file con CountMillionCharacters-Variations/variation1.py. Decidi e documenta il comportamento previsto e la struttura delle versioni prima di modificare la gestione delle maiuscole e minuscole indicata, l’uso di defaultdict, la denominazione delle variabili e il flusso di uscita del programma; il lavoro è completato quando i programmi e il README sono coerenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- tooling
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100