copying file to a folder makes a new folder instead

Question

sesilia maidelin 2021-7-14

0
链接

此问题的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/878223-copying-file-to-a-folder-makes-a-new-folder-instead

评论： sesilia maidelin 2021-7-15

hi, so i want my code to create a folder based on a list of file's names, but when i want to move the file to the folder i created, it made a new folder instead with the name of the variable i used. help please

I = the files 
for gg = 1: numel(I)
    bmp = {I.name}; %the files I want to move 
    bmp_n = char(bmp(gg));
    
    for ii = 1: numel(Plist)
     P = dir( fullfile( mainfolder,Plist{ii}, '*.dcm'));
     dcm = {P(~[P.isdir]).name}; 
      for jj = 1 : numel(dcm)
          dicom = char(dcm(jj)); % converts cell to string
          mkdir( parentfolder, dicom); % (dicom is the variable of the name of the new folders, the new   
%folders' name are eg 00004, 00005)
          ptt = digitsPattern + regexprep(dicom, '.dcm$', '') + "_";
          matchedIdx = contains(bmp_n, ptt) % matching the name of the files to the name of the folder
          if matchedIdx == true
              copyfile input dicom  % in this part it makes a new file dicom and move the files there instead of moving it to 00004, 00005, 00006 etc)
          end
      end
end

any help would be greatly appreciated!

1 个评论
显示 -1更早的评论隐藏 -1更早的评论

dpb 2021-7-14

在 MATLAB Online 中打开

copyfile input dicom %

is the command line form for copyfile -- "input" and "dicom" are interpreted as literal strings, not as variables. Also input is the builtin MATLAB function and should not be used as variable -- and it isn't defined in the above code otherwise.

copyfile(YOURWANTEDFILENAMEINVARIABLE,dicom)

where the first argument needs to be a variable that contains the matching file names in the matched pattern. You'll need to iterate over the number of elements that are TRUE in matchedIdx as copyfile isn't vectorized I do not believe.

请先登录，再进行评论。

请先登录，再回答此问题。

Answer 1

Simon Chan 2021-7-14

0
链接

此回答的直接链接

https://ww2.mathworks.cn/matlabcentral/answers/878223-copying-file-to-a-folder-makes-a-new-folder-instead#answer_746633

在 MATLAB Online 中打开

Note that "myFolder" is the current folder storing the DCM files and the destination folders.

If they are in two different paths, then you may need to change the path:

cd(myFolder);
nfiles = dir(myFolder);
folders = {nfiles([nfiles.isdir]).name};
folders(1:2)=[];
files = dir(fullfile(myFolder,'*.dcm'));
filename={files.name};
for i = 1:length(filename)
	matchedIdx = find(cellfun(@(x) contains(filename{i},x),folders));
	copyfile(filename{i}, fullfile(myFolder,'\',folders{matchedIdx}));
end

11 个评论
显示 9更早的评论隐藏 9更早的评论

sesilia maidelin 2021-7-14

Screenshot 2021-07-14 at 17.37.32.png

I'm very sorry to bother you, but somehow my code makes a folder with the name '\' and moves all the files i meant to move there. can you help me spot what i did wrong? it just said unknown error with copy file this time. for context, the file i'm interested is located in /users/.../Daniel/P1 until P20 so i had to use a loop to extract all the dicom files under it. the folder i want to move the dicom files into is the nufolder. in the loop i made a folder for every dicom file name under nufolder, that's where i want to move the files into.

%change file path here :

mainfolder ='/Users/sesiliamaidelin/Downloads/summer project/CL2 Dicom/Daniel';%dicom images

parentfolder = '/Users/sesiliamaidelin/Downloads/summer project/';

mkdir nufolder;

parentfolder = '/Users/sesiliamaidelin/Downloads/summer project/nufolder';

P_f = dir( fullfile( mainfolder, '*'));

%Plist = setdiff({ P_f([P_f.isdir]).name},{'.','..'}); % works fine up to here

Plist = {P_f([P_f.isdir]).name};

Plist(1:2)=[];

bmp_path = '/Users/sesiliamaidelin/Downloads/summer project/input';

I = dir(fullfile(bmp_path,'*.bmp'))

Ilist = {I.name};

%I = dir( fullfile( input, '*'))

%%

for ii = 1: numel(Plist)

P = dir( fullfile( mainfolder,Plist{ii}, '*.dcm'));

dcm = {P(~[P.isdir]).name}; % works great!! gets all the dicom file in the p files

for jj = 1 : numel(dcm)

[filepath, name, ext]= fileparts(dcm(jj)); % converts cell to string

mkdir( parentfolder, name); % makes folders after the dcm names in nufolder

end

matchedIdx = find( cellfun(@(x) contains(name,x),dcm));

copyfile(fullfile(mainfolder, dcm{jj}), fullfile(parentfolder, '\', Plist{matchedIdx}))

end

Simon Chan 2021-7-15

在 MATLAB Online 中打开

Modify the code again and hope it help.

clear; clc;
mainfolder ='C:\SIMON\Machine\Figures\CL2 Dicom\Simon';         % Subfolders contains dicom images
parentfolder = 'C:\SIMON\Machine\Figures';                      % Working directory
mkdir nufolder;                                                 % Create nufolder under working directory
nufolderpath = 'C:\SIMON\Machine\Figures\nufolder';             % Define the path name of nufolder
filesAndFolders = dir([mainfolder '/**']);                      % Get all subfolder and files names
alllist={filesAndFolders.name};                                 % Convert to cell
dcmidx = cellfun(@(x) contains(x,'.dcm'),alllist);              % Index to extract dicom files only
dcmlist = alllist(dcmidx);                                      % List of dicom files only
dcmpath = {filesAndFolders(dcmidx).folder};                     % Extract the path of each dicom files
for jj = 1 : numel(dcmlist)
      	[filepath, name, ext]= fileparts(dcmlist(jj));          % Extract file name 
      	mkdir(nufolderpath, name);                              % makes folders after the dcm names in nufolder
	    copyfile(fullfile(dcmpath{jj},'\',dcmlist{jj}), fullfile(nufolderpath, '\', name))  % Copy file directly and so string comparison is needed
end 

Left one is the folders and files location before running the script.

Right one is the result after running the script.

Walter Roberson 2021-7-15

At some point I found (and posted) a reference indicating that NTFS reads the Locale information of the person who created the NTFS file system, and locks the sorting order appropriate for that Locale into the filesystem.

For example, many many PC users use Windows-1252, which is like a superset of ISO-8896-1 . It has associated sorting rules, and those sorting rules are different than the rules mandated by Unicode . I personally never use Windows-1252: I always go for en-CA (English, Canada) locale, which uses UTF-8 by default. Someone who was using an NTFS filesystem created under Windows-1252 could get a different sorting order than I would get for creating exactly the same files in exactly the same order under en-CA . Furthermore if I change locale to create a second filesystem, then if I were to copy the same files to the two different filesystems, then the sort order could end up different for the two filesystems.

Apple has not been perfect either. The sorting order they used for filesystems before OS-X changed when HFS was put into use. HFS's rules for canonicalization of names were broken, and were changed, with (if I recall correctly) the change going into effect for the HFS+ . Some people consider HFS+'s canonicalization to also be broken, so the newer APFS filesystem has slightly different rules yet, if I understand correctly. On the other hand, it is the case that two different people who create HFS+ filesystems should get the same sort order no matter what locale they were using.

sesilia maidelin 2021-7-15

it works nicely, thank you! :)

请先登录，再进行评论。

copying file to a folder makes a new folder instead

1 个评论
显示 -1更早的评论隐藏 -1更早的评论

采纳的回答

11 个评论
显示 9更早的评论隐藏 9更早的评论

更多回答（0 个）

另请参阅

类别

标签

产品

版本

Community Treasure Hunt

copying file to a folder makes a new folder instead

1 个评论 显示 -1更早的评论隐藏 -1更早的评论

采纳的回答

11 个评论 显示 9更早的评论隐藏 9更早的评论

更多回答（0 个）

另请参阅

类别

标签

产品

版本

Community Treasure Hunt

1 个评论
显示 -1更早的评论隐藏 -1更早的评论

11 个评论
显示 9更早的评论隐藏 9更早的评论