Stringmanipulationen beim Dateinamen

[berndoa]

Hallo,
ich versage gerade total beim Umbenennen von Dateien....
Ich habe einen Ordner mit Dateien (relativer Pfad ./Pictures).
Manche davon sind jpg, die einen Namen wie "input1920x1080.jpg" oder "input1920x1080Login.jpg" haben
Also input, dann eine Auflösung mit x dazwischen, dann vielleicht noch ein "Login" oder auch nicht.

Nun will ich simpel hingehen, in jeder Datei mit einem Dateinamen im obigen Format das "input" durch "output" ersetzen und vor dem ".jpg" ein "MitRaster" einfügen.Aus obigem beispiel würde also "output1920x1080MitRaster.jpg" und "output1920x1080LoginMitRaster.jpg".

Nur shceitere ich gerade grandios am Rumwerkeln mit Substrings, replace, etc.

Hoffe, dass mir Jemand auf die Sprünge helfen kann wie ich es einfahc und easy schreiben kann.

Hier der relelvante Teil-Code:

        File folder = new File("./Pictures");
        File[] listOfFiles = folder.listFiles();

        for (int i = 0; i < listOfFiles.length; i++) {
            String filename=listOfFiles[i].getName();
            System.out.println("filename"+filename);
            
            if(!(filename.contains("input"))){continue;}
            
            String fileend=(filename.contains("Login.jpg")?"Login.jpg":".jpg");
                
            filename=filename.replace(fileend, "");
            
            fileend=(filename.contains("Login.jpg")?"Login":"");
            
            
            int xpos=filename.indexOf("x");
            
            rewidth=Integer.parseInt(filename.substring(5, xpos));
            reheight=Integer.parseInt(filename.substring(xpos+1));

          //hier steht dann Code der fürs Problem unwichtig ist

     }

Randbemerkung:
Es sind auch noch andere Dateien im ordner, daurm bearbeite ich nur sachen die "input" im Namen haben.

Und die Auflösung aufgeteilt in Breite und Höhe brauche ich für andere Zwecke später im Programm.

 

[thecain]

Hast du dir den Code schonmal mit dem Debugger angeschaut? Das wäre hier mein erster Schritt.

 

[Neumi5694]

Verwende einen Regex, der deinen Dateinamen beschreibt, damit kriegst du dann auch alle Teile des Namens als getrennte Strings raus.

 

[Jw456]

Warum eigentlich so kompliziert zwei replace sollte doch reichen.

File folder = new File("./Pictures");
File[] listOfFiles = folder.listFiles();

        for (File file : listOfFiles) {
            String filename = file.getName();

            if (!(filename.contains("input"))) {
                continue;
            }

            String newfilename = filename.replace("input", "output");
            newfilename = newfilename.replace(".jpg", "MitRaster.jpg");

            file.renameTo(new File(folder.getName(), newfilename));

          }

 

[Neumi5694]

Warum eigentlich so kompliziert zwei replace sollte doch reichen.

File folder = new File("./Pictures");
File[] listOfFiles = folder.listFiles();

        for (File file : listOfFiles) {
            String filename = file.getName();

            if (!(filename.contains("input"))) {
                continue;
            }

            String newfilename = filename.replace("input", "output");
            newfilename = newfilename.replace(".jpg", "MitRaster.jpg");

            file.renameTo(new File(folder.getName(), newfilename));

          }

Sofern es keine Vatiationen in Groß- Kleinschreibung geben kann, ".jpg" im restlichen Namen nicht vorkommen kann und input nur einmal forkommen kann, passt das, stimmt.

 

[Meniskusschaden]

Warum eigentlich so kompliziert zwei replace sollte doch reichen.

Ich würde trotzdem dem Vorschlag von @Neumi5694 folgen. Neben der Umbenennung war ja auch die Extraktion von Breite und Höhe gefordert. Außerdem ist es damit einfacher, passgenau zu filtern. Da muss man keine Konsequenzen fürchten, falls mal eine Datei input1024x768_CAUTION_renaming_permitted_by_DEATH.jpg im Eingabeverzeichnis liegt.

Der Code ist auch nicht so viel aufwändiger:

import java.io.File;
import java.util.regex.Matcher;
import java.util.regex.Pattern;

public class FileProcessor {

    public static void main(String[] args) {
        File folder = new File("./Pictures");
        File[] listOfFiles = folder.listFiles();
        Pattern pattern = Pattern.compile("input((\\d+)x(\\d+)(Login)?)(\\.jpg)");

        for (File file : listOfFiles) {
            String input = file.getName();
            Matcher matcher = pattern.matcher(input);
            if (!matcher.find()) {
                continue;
            }
            String output = "output" + matcher.group(1) + "MitRaster" + matcher.group(5);
            int width  = Integer.valueOf(matcher.group(2));
            int height  = Integer.valueOf(matcher.group(3));

            System.out.println(String.format("input:  %s\noutput: %s\nformat: %dx%d\n", input, output, width, height));
        }
    }
}

 

[berndoa]

Warum eigentlich so kompliziert zwei replace sollte doch reichen.

File folder = new File("./Pictures");
File[] listOfFiles = folder.listFiles();

        for (File file : listOfFiles) {
            String filename = file.getName();

            if (!(filename.contains("input"))) {
                continue;
            }

            String newfilename = filename.replace("input", "output");
            newfilename = newfilename.replace(".jpg", "MitRaster.jpg");

            file.renameTo(new File(folder.getName(), newfilename));

          }

Uff :O
Ich wusste doch dass es eigentlich super easy geht und ich nur voll kompliziert denke :-D

Ich würde trotzdem dem Vorschlag von @Neumi5694 folgen. Neben der Umbenennung war ja auch die Extraktion von Breite und Höhe gefordert. Außerdem ist es damit einfacher, passgenau zu filtern. Da muss man keine Konsequenzen fürchten, falls mal eine Datei input1024x768_CAUTION_renaming_permitted_by_DEATH.jpg im Eingabeverzeichnis liegt.

Der Code ist auch nicht so viel aufwändiger:

import java.io.File;
import java.util.regex.Matcher;
import java.util.regex.Pattern;

public class FileProcessor {

    public static void main(String[] args) {
        File folder = new File("./Pictures");
        File[] listOfFiles = folder.listFiles();
        Pattern pattern = Pattern.compile("input((\\d+)x(\\d+)(Login)?)(\\.jpg)");

        for (File file : listOfFiles) {
            String input = file.getName();
            Matcher matcher = pattern.matcher(input);
            if (!matcher.find()) {
                continue;
            }
            String output = "output" + matcher.group(1) + "MitRaster" + matcher.group(5);
            int width  = Integer.valueOf(matcher.group(2));
            int height  = Integer.valueOf(matcher.group(3));

            System.out.println(String.format("input:  %s\noutput: %s\nformat: %dx%d\n", input, output, width, height));
        }
    }
}

Hm, könntest du mir vielleicht kurz erklären was da passiert?
Das Pattern kann ich mir noch halbwegs denken,
eben ein "input", gefolgt von einigen ziffern, einem "x", wieder ziffern, optionalem "Login" und ein ".jpg" am Ende.
Aber was genau macht die Methoden des Matcher Objekts da, insbesondere das find() und dieses group Teil (Array?)

 

[Jw456]

dann schaue dir Regex an.
find prüft ob der String (input) dem Regex entspricht oder nicht.

Reguläre Ausdrücke

www.straub.as

 

[Meniskusschaden]

Aber was genau macht die Methoden des Matcher Objekts da, insbesondere das find() und dieses group Teil (Array?)

find() hat @Jw456 ja schon erklärt. Mit group() kann man auf die Gruppen des Ausdrucks zugreifen, die man mit den Klammern bildet. Gruppe 0 ist der gesamte Ausdruck, die übrigen Gruppen werden in der Reihenfolge der öffnenden Klammern durchnummeriert.