Scrabble Word Counter GUI

[jor177]

Graag zou ik door middel van Swing te gebruiken een GUI voor het bereken van de woordwaarde in Java willen bouwen.

Ik ben al in meerdere richtingen geduwd. De één zegt dat ik enum's moet gebruiken en de ander zegt dat ik beter HashMaps kan gebruiken.

Ik vraag mij af welke van de twee jullie mij adviseren, of is er een betere methode?

Als ik namelijk het woord (string) dat de gebruiker invoert uitlees moet ieder character worden vergeleken. Als ik bijvoorbeeld 'bananen' intyp moet er bijvoorbeeld twee keer de waarde van 'a' worden geregistreerd.

Alvast bedankt!

Jordi

[RobIII]

jor177 schreef op donderdag 18 oktober 2018 @ 12:11:
De één zegt dat ik enum's moet gebruiken en de ander zegt dat ik beter HashMaps kan gebruiken.

Er zijn meer wegen die leiden naar Rome. Soms is 't één niet beter (of juist wél) dan 't ander. En dan maak je zélf een afweging; simpelweg door de "pro's en cons" tegen elkaar te zetten. En soms rol je even je stoel naar achteren, leg je je handen gevouwen op je hoofd, doe je je ogen dicht en komt je zelf met een andere (soms nóg betere) optie. Ik kan er zo nog een handvol bedenken zonder enige moeite (wat dacht je bjiv. van een array met 26 elementen waarin de letterwaarden staan voor een mooie O(1) lookup zonder gedoe met hashes? Of, hell, een joekel van een switch-case... of een database of een webservice ).

Zonder relevante(!) code of enige andere informatie is er niemand die hier iets zinnigs over gaat kunnen zeggen, laat staat "het" antwoord geven (en dat is er toch niet). Daarbij mis ik, per onze Quickstart, behoorlijk je eigen inzet. Wat heb je zélf al geprobeerd, gezocht, gevonden en wat waren je eigen bevindingen of overwegingen voor ofwel het één ofwel het ander?

[Voor 43% gewijzigd door RobIII op 18-10-2018 12:52]

[heuveltje]

jor177 schreef op donderdag 18 oktober 2018 @ 12:11:
Graag zou ik door middel van Swing te gebruiken een GUI voor het bereken van de woordwaarde in Java willen bouwen.

Ik ben al in meerdere richtingen geduwd. De één zegt dat ik enum's moet gebruiken en de ander zegt dat ik beter HashMaps kan gebruiken.

Ik vraag mij af welke van de twee jullie mij adviseren, of is er een betere methode?

Als ik namelijk het woord (string) dat de gebruiker invoert uitlees moet ieder character worden vergeleken. Als ik bijvoorbeeld 'bananen' intyp moet er bijvoorbeeld twee keer de waarde van 'a' worden geregistreerd.

Alvast bedankt!

Jordi

Ben je nu niet gruwelijk aan het overdenken ?
Een gemiddeld scrabble woord is wat ? 6 letters ofzo ?
10 letters vergelijken en optellen duurt .000000000001 ms op elke moderne pc.

Tenzij je dit voor tienduizenden mensen tegelijk wilt doen als webservice, gewoon zo simpel maken als kan

[BranderoS]

Kun je niet gewoon een for-loopje bouwen die over alle letters in de String loopt?

De waarde van een letter kan je ergens wel definiëren en die haal je dan op voor elke letter. In de for-loop creëer je dan het sommetje.

[jor177]

code:

private static final Map<Character, Integer> letterwaardes = new HashMap<>(); 
  
    
    static { 
        letterwaardes.put('a', 1); 
        letterwaardes.put('e', 1); 
        letterwaardes.put('i', 1); 
        letterwaardes.put('o', 1); 
        letterwaardes.put('u', 1); 
        letterwaardes.put('l', 1); 
        letterwaardes.put('n', 1); 
        letterwaardes.put('r', 1); 
        letterwaardes.put('s', 1); 
        letterwaardes.put('t', 1); 
        letterwaardes.put('d', 2); 
        letterwaardes.put('g', 2); 
        letterwaardes.put('b', 3); 
        letterwaardes.put('c', 3); 
        letterwaardes.put('m', 3); 
        letterwaardes.put('p', 3); 
        letterwaardes.put('f', 4); 
        letterwaardes.put('h', 4); 
        letterwaardes.put('v', 4); 
        letterwaardes.put('w', 4); 
        letterwaardes.put('y', 4); 
        letterwaardes.put('k', 5); 
        letterwaardes.put('j', 8); 
        letterwaardes.put('x', 8); 
        letterwaardes.put('q', 10); 
        letterwaardes.put('z', 10); 
    }

Ik neem aan dat de invoer (string) naar een char array moet gaan?
Op deze manier?

code:

String InvoerS = invoer.getText();
        char[] stringnaarCharArray = InvoerS.toCharArray();

Ik zie nog niet helemaal voor me hoe zo'n loop eruit moet zien. Dit is natuurlijk een 2D HashMap, en ik zie alleen voorbeelden van een enkele.

[BranderoS]

Verwijderd,

Aardig mogen zijn binnen GoT is er niet meer bij de laatste tijd.

[Voor 185% gewijzigd door BranderoS op 18-10-2018 19:32]

[RobIII]

Dames, heren, zullen we ophouden het complete antwoord voor te kauwen?

Give a man a fish and feed him for a day. Teach a man how to fish and feed him for a lifetime.

[jor177]

Als ik het mij goed kan herinneren ging het ongeveer zo:

code:

public static int calculateWordValue(String word) 
{ 
int wordValue = 0; 
for (char c : word.toLowerCase().toCharArray()) { 
wordValue += letterwaardes.get(c); } 
return wordValue; 
}

Dit is onderdeel van mijn code:

code:

String word = invoer.getText();

want hij moet natuurlijk weten waar hij de string van 'word' vandaan moet halen.
Nou vraag ik mij af waar ik die methode moet plaatsen, want met Swing lijkt het allemaal nog stroever te gaan dan het al ging. Ik krijg namelijk de volgende foutmeldingen:

code:

<identifier> expected

en ook:

code:

cannot find symbol

bij 'word' in de for-loop.

[RobIII]

jor177 schreef op donderdag 18 oktober 2018 @ 21:06:

code:

<identifier> expected

Dat is gewoon een syntax fout (maar je post geen code waar de error naar verwijst dus daar kunnen we je niet mee helpen op deze manier...).

jor177 schreef op donderdag 18 oktober 2018 @ 21:06:

code:

cannot find symbol

bij 'word' in de for-loop.

En dat is een kwestie van je eens verdiepen in wat een scope van een variabele is.

En beide meldingen zijn prima te googlen. Ik heb het je al eerder gezegd, we verwachten hier wél eigen inzet. Als je te moe bent om foutmeldingen te googlen of te lui om, wanneer je een vraag hebt, er relevante(!!) code bij te plaatsen, hoe kun je dan wél van ons verwachten je te helpen We zijn niet helderziend en we hebben al helemaal geen zin om alles, stukje-bij-beetje, puzzelstukje-voor-puzzelstukje uit je te moeten trekken. JIJ moet ONS helpen JOU te helpen

[Voor 31% gewijzigd door RobIII op 18-10-2018 21:34]

[Kwistnix]

RobIII schreef op donderdag 18 oktober 2018 @ 19:05:
Dames, heren, zullen we ophouden het complete antwoord voor te kauwen?
[...]

Ik prefereer:

"Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life."

[RobIII]

Kwistnix schreef op donderdag 18 oktober 2018 @ 21:41:
[...]


Ik prefereer:

"Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life."

Het is hier geen HK Zullen we ontopic blijven?

[jor177]

Omdat mijn kennis van Swing beperkt is, heb ik uit willekeur de methode maar ondergebracht bij het event dat de bereking triggert, daar gaf het de minste foutmeldingen.

code:

package Teller;

import java.util.Map;
import java.util.HashMap; 


 
    



public class WordWaardeTeller extends javax.swing.JFrame {

    
    
    
     
    private static final Map<Character, Integer> letterwaardes = new HashMap<>(); 
 
    { 
        letterwaardes.put('a', 1); 
        letterwaardes.put('e', 1); 
        letterwaardes.put('i', 1); 
        letterwaardes.put('o', 1); 
        letterwaardes.put('u', 1); 
        letterwaardes.put('l', 1); 
        letterwaardes.put('n', 1); 
        letterwaardes.put('r', 1); 
        letterwaardes.put('s', 1); 
        letterwaardes.put('t', 1); 
        letterwaardes.put('d', 2); 
        letterwaardes.put('g', 2); 
        letterwaardes.put('b', 3); 
        letterwaardes.put('c', 3); 
        letterwaardes.put('m', 3); 
        letterwaardes.put('p', 3); 
        letterwaardes.put('f', 4); 
        letterwaardes.put('h', 4); 
        letterwaardes.put('v', 4); 
        letterwaardes.put('w', 4); 
        letterwaardes.put('y', 4); 
        letterwaardes.put('k', 5); 
        letterwaardes.put('j', 8); 
        letterwaardes.put('x', 8); 
        letterwaardes.put('q', 10); 
        letterwaardes.put('z', 10); 
    } 
    
   
    
    
    
    
    
   
    public WordWaardeTeller() {
        initComponents();
    }

    
    
    
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        invoer = new javax.swing.JTextField();
        bereken = new javax.swing.JButton();
        uitvoer = new javax.swing.JTextField();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        invoer.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                invoerActionPerformed(evt);
            }
        });

        bereken.setText("berkenen");
        bereken.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                berekenActionPerformed(evt);
            }
        });

        uitvoer.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                uitvoerActionPerformed(evt);
            }
        });

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGap(56, 56, 56)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                    .addComponent(bereken, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                    .addComponent(invoer, javax.swing.GroupLayout.PREFERRED_SIZE, 82, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(uitvoer))
                .addContainerGap(260, Short.MAX_VALUE))
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGap(34, 34, 34)
                .addComponent(invoer, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(bereken)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addComponent(uitvoer, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addContainerGap(168, Short.MAX_VALUE))
        );

        pack();
    }// </editor-fold>                        

    
    
    private void invoerActionPerformed(java.awt.event.ActionEvent evt) {                                       
        
               
    }                                      

    private void berekenActionPerformed(java.awt.event.ActionEvent evt) {                                        
        String word = invoer.getText();
        
        public static int calculateWordValue(word) {
    {
      int wordValue = 0;
      for (char c : word.toLowerCase().toCharArray()) { 
       wordValue += letterwaardes.get(c); } 
        return wordValue; 
    }
    }                                       

    private void uitvoerActionPerformed(java.awt.event.ActionEvent evt) {                                        
        
    }                                       

    
 

    
    public static void main(String args[]) {
        
       
        
        
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new WordWaardeTeller().setVisible(true);
            }
            
            
            
        });
    }

    
    
    // Variables declaration - do not modify                     
    private javax.swing.JButton bereken;
    private javax.swing.JTextField invoer;
    private javax.swing.JTextField uitvoer;
    // End of variables declaration                   
}

[RobIII]

jor177 schreef op donderdag 18 oktober 2018 @ 22:58:
Omdat mijn kennis van Swing beperkt is, heb ik uit willekeur de methode maar ondergebracht bij het event dat de bereking triggert, daar gaf het de minste foutmeldingen.

Mja, da's natuurlijk niet de manier Ik zou als ik jou was proberen het probleem te doorgronden (en vwb die foutmeldingen: da's gewoon een kwestie van (begrijpend) lezen en desnoods googlen).

Maar... het werkt nu dus? Of nog niet? Want aan 168 regels brij zien wij zo in één oogopslag natuurlijk ook weinig tot niets. Beperk je daarom alsjeblieft tot het posten van relevante code. Ik schreef dat niet voor niets tot twee keer toe met een uitroepteken(s) erachter

[Voor 27% gewijzigd door RobIII op 18-10-2018 23:04]

[jor177]

code:

private void invoerActionPerformed(java.awt.event.ActionEvent evt) {                                       
        
               
    }                                      

    private void berekenActionPerformed(java.awt.event.ActionEvent evt) {                                        
        char[] alphabet = {'e', 'n', 'a', 'o', 'i', 'd', 'r', 's', 't', 'g', 'k', 'l', 'm', 'b', 'p', 'u', 'f', 'h', 'j', 'v', 'z', 'c', 'w', 'x', 'y', 'q'}; 
        int[] values = {1, 1, 1, 1, 1, 2, 2, 2, 2, 3, 3, 3, 3, 3, 3, 4, 4, 4, 4, 4, 4, 5, 5, 8, 8, 10};
       
        String keuzeS;
        keuzeS = invoer.getText();
        
       int score = 0;
        for(int i = 0; i < keuzeS.length(); i++) {
        for(int n = 0; n < 26; n++) {
    
            if(keuzeS.charAt(i) == alphabet[n])
            {
              score += values[n];
            }
          }
        }
        
        String uitvoerS = Integer.toString(score);
        
        
        uitvoer.setText(uitvoerS);
    
    }                                       

    private void uitvoerActionPerformed(java.awt.event.ActionEvent evt) {                                        
        
    }                                       

    
 

    
    public static void main(String args[]) {
       
        


        
        
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new WordWaardeTeller().setVisible(true);
            }
            
            
            
        });
    }

    
    
    // Variables declaration - do not modify                     
    private javax.swing.JButton bereken;
    private javax.swing.JTextField invoer;
    private javax.swing.JTextField uitvoer;
    // End of variables declaration                   
}

Ik heb het zo maar even opgelost. Misschien dat iemand mij nog kan vertellen welke methode ik moet aanmaken om het stukje code zelf niet onder te hoeven brengen bij de private void berekenActionPerformed (=bereken knop)?

[RobIII]

jor177 schreef op vrijdag 19 oktober 2018 @ 10:02:
Misschien dat iemand mij nog kan vertellen welke methode ik moet aanmaken om het stukje code zelf niet onder te hoeven brengen bij de private void berekenActionPerformed (=bereken knop)?

Wat let je om 't niet in een aparte method (of zelfs aparte class) te zetten dan

[jor177]

RobIII schreef op vrijdag 19 oktober 2018 @ 10:09:
[...]

Wat let je om 't niet in een aparte method (of zelfs aparte class) te zetten dan

Mijn kennis

[RobIII]

Nou, laat dan eens zien wat je geprobeerd hebt. Of leg je redenatie eens uit. Of leg eens uit waarom jij denkt dat je oplossing niet werkt? Met dergelijke posts:

jor177 schreef op vrijdag 19 oktober 2018 @ 12:06:
Mijn kennis

kunnen wij natuurlijk ook niks